X-Git-Url: https://mattmccutchen.net/rsync/rsync-patches.git/blobdiff_plain/f20eb4505bcf8baec9b636c1ea3464e9676834cd..a75921111fdf7354d07296084231945f01b3c644:/early-checksum.diff diff --git a/early-checksum.diff b/early-checksum.diff index 3c8108e..ccdada2 100644 --- a/early-checksum.diff +++ b/early-checksum.diff @@ -8,9 +8,15 @@ to its checksum pass during its normal find-the-different-files pass. I have benchmarked this a little, and it appears to slow things down for a local copy, so the old algorithm is used for local copies. ---- orig/flist.c 2005-11-10 16:58:36 -+++ flist.c 2005-09-16 16:41:25 -@@ -36,6 +36,7 @@ extern int am_daemon; +To use this patch, run these commands for a successful build: + + patch -p1 flags |= FLAG_SUM_DIFFERS; + } + } } - if (!preserve_perms) { ---- orig/generator.c 2005-12-15 23:00:49 -+++ generator.c 2005-12-15 23:16:48 -@@ -69,6 +69,7 @@ extern int ignore_timeout; + return file; +--- old/generator.c ++++ new/generator.c +@@ -70,6 +70,7 @@ extern int ignore_timeout; extern int protocol_version; extern int fuzzy_basis; extern int always_checksum; @@ -45,7 +50,7 @@ for a local copy, so the old algorithm is used for local copies. extern int checksum_len; extern char *partial_dir; extern char *basis_dir[]; -@@ -376,7 +377,8 @@ void itemize(struct file_struct *file, i +@@ -577,7 +578,8 @@ void itemize(struct file_struct *file, i /* Perform our quick-check heuristic for determining if a file is unchanged. */ @@ -53,18 +58,18 @@ for a local copy, so the old algorithm is used for local copies. +int unchanged_file(char *fn, int fnamecmp_type, struct file_struct *file, + STRUCT_STAT *st) { - if (st->st_size != file->length) + if (st->st_size != F_LENGTH(file)) return 0; -@@ -385,6 +387,8 @@ int unchanged_file(char *fn, struct file +@@ -586,6 +588,8 @@ int unchanged_file(char *fn, struct file of the file time to determine whether to sync */ if (always_checksum && S_ISREG(st->st_mode)) { char sum[MD4_SUM_LENGTH]; + if (pre_checksum && fnamecmp_type == FNAMECMP_FNAME) + return !(file->flags & FLAG_SUM_DIFFERS); file_checksum(fn, sum, st->st_size); - return memcmp(sum, file->u.sum, checksum_len) == 0; + return memcmp(sum, F_SUM(file), checksum_len) == 0; } -@@ -622,7 +626,7 @@ static int try_dests_reg(struct file_str +@@ -825,7 +829,7 @@ static int try_dests_reg(struct file_str match_level = 1; /* FALL THROUGH */ case 1: @@ -73,7 +78,7 @@ for a local copy, so the old algorithm is used for local copies. continue; best_match = j; match_level = 2; -@@ -1160,7 +1164,7 @@ static void recv_generator(char *fname, +@@ -1511,7 +1515,7 @@ static void recv_generator(char *fname, ; else if (fnamecmp_type == FNAMECMP_FUZZY) ; @@ -82,39 +87,38 @@ for a local copy, so the old algorithm is used for local copies. if (partialptr) { do_unlink(partialptr); handle_partial_dir(partialptr, PDIR_DELETE); ---- orig/hlink.c 2005-12-15 23:00:49 -+++ hlink.c 2005-12-15 23:17:31 -@@ -210,7 +210,7 @@ int hard_link_check(struct file_struct * - itemizing = code = 0; - break; - } -- if (!unchanged_file(cmpbuf, file, &st3)) -+ if (!unchanged_file(cmpbuf, 0, file, &st3)) - continue; - statret = 1; - st = &st3; ---- orig/main.c 2005-12-15 07:55:39 -+++ main.c 2005-11-07 05:00:49 -@@ -45,6 +45,7 @@ extern int copy_links; +--- old/hlink.c ++++ new/hlink.c +@@ -448,7 +448,7 @@ int hard_link_check(struct file_struct * + } + break; + } +- if (!unchanged_file(cmpbuf, file, &alt_st)) ++ if (!unchanged_file(cmpbuf, 0, file, &alt_st)) + continue; + statret = 1; + *stp = alt_st; +--- old/main.c ++++ new/main.c +@@ -47,6 +47,7 @@ extern int copy_dirlinks; extern int keep_dirlinks; extern int preserve_hard_links; extern int protocol_version; +extern int always_checksum; extern int recurse; extern int relative_paths; - extern int rsync_port; -@@ -60,8 +61,10 @@ extern char *filesfrom_host; - extern char *rsync_path; - extern char *shell_cmd; - extern char *batch_name; -+extern char curr_dir[MAXPATHLEN]; + extern int sanitize_paths; +@@ -71,6 +72,9 @@ extern char *batch_name; + extern char curr_dir[MAXPATHLEN]; + extern struct filter_list_struct server_filter_list; - int local_server = 0; ++extern char curr_dir[MAXPATHLEN]; ++ +int pre_checksum = 0; - struct file_list *the_file_list; - - /* There's probably never more than at most 2 outstanding child processes, -@@ -659,6 +662,7 @@ static void do_server_recv(int f_in, int + int local_server = 0; + int new_root_dir = 0; + mode_t orig_umask = 0; +@@ -788,6 +792,7 @@ static void do_server_recv(int f_in, int struct file_list *flist; char *local_name = NULL; char *dir = NULL; @@ -122,18 +126,18 @@ for a local copy, so the old algorithm is used for local copies. int save_verbose = verbose; if (filesfrom_fd >= 0) { -@@ -702,6 +706,10 @@ static void do_server_recv(int f_in, int +@@ -831,6 +836,10 @@ static void do_server_recv(int f_in, int filesfrom_fd = -1; } + strlcpy(olddir, curr_dir, sizeof olddir); + if (always_checksum && !local_server && argc > 0) -+ pre_checksum = push_dir(argv[0]); ++ pre_checksum = push_dir(argv[0], 0); + flist = recv_file_list(f_in); verbose = save_verbose; if (!flist) { -@@ -710,6 +718,9 @@ static void do_server_recv(int f_in, int +@@ -839,6 +848,9 @@ static void do_server_recv(int f_in, int } the_file_list = flist; @@ -143,7 +147,7 @@ for a local copy, so the old algorithm is used for local copies. if (argc > 0) local_name = get_local_name(flist,argv[0]); -@@ -758,6 +769,7 @@ int client_run(int f_in, int f_out, pid_ +@@ -918,6 +930,7 @@ int client_run(int f_in, int f_out, pid_ { struct file_list *flist = NULL; int exit_code = 0, exit_code2 = 0; @@ -151,13 +155,13 @@ for a local copy, so the old algorithm is used for local copies. char *local_name = NULL; cleanup_child_pid = pid; -@@ -829,11 +841,18 @@ int client_run(int f_in, int f_out, pid_ +@@ -992,11 +1005,18 @@ int client_run(int f_in, int f_out, pid_ filesfrom_fd = -1; } + strlcpy(olddir, curr_dir, sizeof olddir); + if (always_checksum && !local_server) -+ pre_checksum = push_dir(argv[0]); ++ pre_checksum = push_dir(argv[0], 0); + if (write_batch && !am_server) start_write_batch(f_in); @@ -170,13 +174,13 @@ for a local copy, so the old algorithm is used for local copies. if (flist && flist->count > 0) { local_name = get_local_name(flist, argv[0]); ---- orig/rsync.h 2005-12-15 23:00:49 -+++ rsync.h 2005-09-16 16:41:26 -@@ -64,6 +64,7 @@ - #define FLAG_DEL_HERE (1<<3) /* receiver/generator */ - #define FLAG_SENT (1<<3) /* sender */ - #define FLAG_HLINK_TOL (1<<4) /* receiver/generator */ -+#define FLAG_SUM_DIFFERS (1<<5) /* receiver/generator */ +--- old/rsync.h ++++ new/rsync.h +@@ -68,6 +68,7 @@ + #define FLAG_HLINK_LAST (1<<7) /* receiver/generator */ + #define FLAG_HLINK_DONE (1<<8) /* receiver/generator */ + #define FLAG_LENGTH64 (1<<9) /* sender/receiver/generator */ ++#define FLAG_SUM_DIFFERS (1<<10)/* receiver/generator */ - /* update this if you make incompatible changes */ - #define PROTOCOL_VERSION 29 + #define BITS_SET(val,bits) (((val) & (bits)) == (bits)) + #define BITS_SETnUNSET(val,onbits,offbits) (((val) & ((onbits)|(offbits))) == (onbits))