From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?Nguy=E1=BB=85n=20Th=C3=A1i=20Ng=E1=BB=8Dc=20Duy?= Subject: [PATCH 3/4] fsck: avoid reading every object twice Date: Fri, 4 Nov 2011 22:47:49 +0700 Message-ID: <1320421670-518-4-git-send-email-pclouds@gmail.com> References: <1320421670-518-1-git-send-email-pclouds@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: =?UTF-8?q?Nguy=E1=BB=85n=20Th=C3=A1i=20Ng=E1=BB=8Dc=20Duy?= To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Fri Nov 04 16:49:46 2011 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1RMM1Z-0003Ig-92 for gcvg-git-2@lo.gmane.org; Fri, 04 Nov 2011 16:49:45 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932615Ab1KDPtl convert rfc822-to-quoted-printable (ORCPT ); Fri, 4 Nov 2011 11:49:41 -0400 Received: from mail-iy0-f174.google.com ([209.85.210.174]:36170 "EHLO mail-iy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932551Ab1KDPtj (ORCPT ); Fri, 4 Nov 2011 11:49:39 -0400 Received: by mail-iy0-f174.google.com with SMTP id e36so2682080iag.19 for ; Fri, 04 Nov 2011 08:49:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references :mime-version:content-type:content-transfer-encoding; bh=29P4p8wKOcNb6JUhJ079vYnHUzEyFDlstEhTlubgosE=; b=OonaEDIFn1tW4HPLZHb/eCRglDreQd2pphDZn0To6JNnDCbpwOEVo5sHiyU8ONd2T8 be7OuRFP6dDDhJecXuuZ4LgGFg+gy9Ify3962s02M/3HL4J7zwmoDr0SacTnBWWzoZkk ipGDhgb4pLzu4mm2+5QHTH+UsczMGoe9XvhWs= Received: by 10.231.0.221 with SMTP id 29mr3807258ibc.75.1320421779592; Fri, 04 Nov 2011 08:49:39 -0700 (PDT) Received: from pclouds@gmail.com ([115.73.210.255]) by mx.google.com with ESMTPS id l28sm22127692ibc.3.2011.11.04.08.49.36 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 04 Nov 2011 08:49:38 -0700 (PDT) Received: by pclouds@gmail.com (sSMTP sendmail emulation); Fri, 04 Nov 2011 22:48:20 +0700 X-Mailer: git-send-email 1.7.4.74.g639db In-Reply-To: <1320421670-518-1-git-send-email-pclouds@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: During verify_pack() all objects are read for SHA-1 check. Then fsck_sha1() is called on every object, which read the object again (fsck_sha1 -> parse_object -> read_sha1_file). Avoid reading an object twice, do fsck_sha1 while we have an object uncompressed data in verify_pack. On git.git, with this patch I got: $ /usr/bin/time ./git fsck >/dev/null 98.97user 0.90system 1:40.01elapsed 99%CPU (0avgtext+0avgdata 616624max= resident)k 0inputs+0outputs (0major+194186minor)pagefaults 0swaps Without it: $ /usr/bin/time ./git fsck >/dev/null 231.23user 2.35system 3:53.82elapsed 99%CPU (0avgtext+0avgdata 636688ma= xresident)k 0inputs+0outputs (0major+461629minor)pagefaults 0swaps Signed-off-by: Nguy=E1=BB=85n Th=C3=A1i Ng=E1=BB=8Dc Duy --- builtin/fsck.c | 42 +++++++++++++++++++++++++----------------- pack-check.c | 13 ++++++++++--- pack.h | 5 ++++- 3 files changed, 39 insertions(+), 21 deletions(-) diff --git a/builtin/fsck.c b/builtin/fsck.c index 4ead98d..0603f64 100644 --- a/builtin/fsck.c +++ b/builtin/fsck.c @@ -282,14 +282,8 @@ static void check_connectivity(void) } } =20 -static int fsck_sha1(const unsigned char *sha1) +static int fsck_obj(struct object *obj) { - struct object *obj =3D parse_object(sha1); - if (!obj) { - errors_found |=3D ERROR_OBJECT; - return error("%s: object corrupt or missing", - sha1_to_hex(sha1)); - } if (obj->flags & SEEN) return 0; obj->flags |=3D SEEN; @@ -332,6 +326,29 @@ static int fsck_sha1(const unsigned char *sha1) return 0; } =20 +static int fsck_sha1(const unsigned char *sha1) +{ + struct object *obj =3D parse_object(sha1); + if (!obj) { + errors_found |=3D ERROR_OBJECT; + return error("%s: object corrupt or missing", + sha1_to_hex(sha1)); + } + return fsck_obj(obj); +} + +static int fsck_obj_buffer(const unsigned char *sha1, enum object_type= type, + unsigned long size, void *buffer, int *eaten) +{ + struct object *obj; + obj =3D parse_object_buffer(sha1, type, size, buffer, eaten); + if (!obj) { + errors_found |=3D ERROR_OBJECT; + return error("%s: object corrupt or missing", sha1_to_hex(sha1)); + } + return fsck_obj(obj); +} + /* * This is the sorting chunk size: make it reasonably * big so that we can sort well.. @@ -627,17 +644,8 @@ int cmd_fsck(int argc, const char **argv, const ch= ar *prefix) prepare_packed_git(); for (p =3D packed_git; p; p =3D p->next) /* verify gives error messages itself */ - if (verify_pack(p)) + if (verify_pack(p, fsck_obj_buffer)) errors_found |=3D ERROR_PACK; - - for (p =3D packed_git; p; p =3D p->next) { - uint32_t j, num; - if (open_pack_index(p)) - continue; - num =3D p->num_objects; - for (j =3D 0; j < num; j++) - fsck_sha1(nth_packed_object_sha1(p, j)); - } } =20 heads =3D 0; diff --git a/pack-check.c b/pack-check.c index e33ea79..372d6b2 100644 --- a/pack-check.c +++ b/pack-check.c @@ -42,7 +42,8 @@ int check_pack_crc(struct packed_git *p, struct pack_= window **w_curs, } =20 static int verify_packfile(struct packed_git *p, - struct pack_window **w_curs) + struct pack_window **w_curs, + verify_fn fn) { off_t index_size =3D p->index_size; const unsigned char *index_base =3D p->index_data; @@ -129,6 +130,12 @@ static int verify_packfile(struct packed_git *p, free(data); break; } + if (fn) { + int eaten =3D 0; + fn(entries[i].sha1, type, size, data, &eaten); + if (eaten) + data =3D NULL; + } free(data); } free(entries); @@ -159,7 +166,7 @@ int verify_pack_index(struct packed_git *p) return err; } =20 -int verify_pack(struct packed_git *p) +int verify_pack(struct packed_git *p, verify_fn fn) { int err =3D 0; struct pack_window *w_curs =3D NULL; @@ -168,7 +175,7 @@ int verify_pack(struct packed_git *p) if (!p->index_data) return -1; =20 - err |=3D verify_packfile(p, &w_curs); + err |=3D verify_packfile(p, &w_curs, fn); unuse_pack(&w_curs); =20 return err; diff --git a/pack.h b/pack.h index 722a54e..70f3c29 100644 --- a/pack.h +++ b/pack.h @@ -70,10 +70,13 @@ struct pack_idx_entry { off_t offset; }; =20 + +typedef int (*verify_fn)(const unsigned char*, enum object_type, unsig= ned long, void*, int*); + extern const char *write_idx_file(const char *index_name, struct pack_= idx_entry **objects, int nr_objects, const struct pack_idx_option *, un= signed char *sha1); extern int check_pack_crc(struct packed_git *p, struct pack_window **w= _curs, off_t offset, off_t len, unsigned int nr); extern int verify_pack_index(struct packed_git *); -extern int verify_pack(struct packed_git *); +extern int verify_pack(struct packed_git *, verify_fn fn); extern void fixup_pack_header_footer(int, unsigned char *, const char = *, uint32_t, unsigned char *, off_t); extern char *index_pack_lockfile(int fd); extern int encode_in_pack_object_header(enum object_type, uintmax_t, u= nsigned char *); --=20 1.7.4.74.g639db