X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C885A1.616625B6@onstor-exch02.onstor.net>; Fri, 14 Mar 2008 00:02:35 -0700
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: PERFORCE change 28351 for review
Date: Fri, 14 Mar 2008 00:02:34 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E08DF381F@onstor-exch02.onstor.net>
In-Reply-To: <20080313235044.2869a39b@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 28351 for review
Thread-Index: AciFn7q8IB+POiLvQfub3s/4tvQ3rQAAY5gw
References: <WEBMAILQ3FrV8TajXg1000007e3@mail.onstor.com> <20080313235044.2869a39b@ripper.onstor.net>
From: "Narain Ramadass" <narain.ramadass@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

Sure Andy. Missed that - thanx for pointing that out. Will keep that in
mind next time.=20

-----Original Message-----
From: Andy Sharp=20
Sent: Thursday, March 13, 2008 11:51 PM
To: Narain Ramadass
Subject: Re: PERFORCE change 28351 for review

Hi Narain,

Don't forget in your comment to tell the bug headline as well as the bug
number.  I usually paste the bug headline from CQ unless it's crap, then
I just write my own short one liner.

Thanks!

a

On 13 Mar 2008 22:55:01 -0700 Narain Ramadass
<narain.ramadass@onstor.com> wrote:

> Change 28351 by narainr@narainr_dev on 2008/03/13 22:52:53
>=20
> 	Partial fix for TED21375. Do not hold access lock across call to

> dump_map as there are no FS operations performed by dump_map - it only

> writes stuff from the mgmt volume to the tape.
> =09
> 	       Added stats gathering for number of hardlinks
encountered.
> 	       Reviewed by: jobia
> =09
> 	       Regression focus: Run dump on a FS with large number of
inodes=20
> and simultaneously load the filesystem with NFS/CIFS traffic as well=20
> as snapshot create/delete operations that require the exclusive FS=20
> access lock.
>=20
> Affected files ...
>=20
> ... //depot/dev/nfx-tree/code/sm-fs/dump/main.c#7 edit ...=20
> //depot/dev/nfx-tree/code/sm-fs/dump/traverse.c#23 edit ...=20
> //depot/dev/nfx-tree/code/sm-fs/fs-dump.h#10 edit
>=20
>=20
> http://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/28351?ac=3D10
