AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20080307174504.1c98e8b4@ripper.onstor.net>
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<kumarv@onstor.com>,<tim.gardner@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/andys@onstor.net@onstor-exch02.onstor.net/INBOX	0	BB375AF679D4A34E9CA8DFA650E2B04E08C0F3DC@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Fri, 7 Mar 2008 18:43:40 -0800
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Kumar Vakacharla (HCL)" <kumarv@onstor.com>
Cc: "Tim Gardner" <tim.gardner@onstor.com>
Subject: Re: Review request : Defect  TED00020613
Message-ID: <20080307184340.303392b8@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E08C0F3DC@onstor-exch02.onstor.net>
References: <BB375AF679D4A34E9CA8DFA650E2B04E08C0F3DC@onstor-exch02.onstor.net>
Organization: Onstor
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Fri, 7 Mar 2008 14:35:08 -0800 "Kumar Vakacharla (HCL)"
<kumarv@onstor.com> wrote:

> Andy, 
> 
>  
> 
>             Can you please review this fix. 
> 
>             
> 
> Thanks,
> 
> Kumar.
> 
>  
> 
> ________________________________
> 
> From: Tim Gardner 
> Sent: Friday, March 07, 2008 2:31 PM
> To: Kumar Vakacharla (HCL)
> Subject: RE: Review request : Defect TED00020613
> 
>  
> 
> Please have Andy Sharp do the code review.
> 
> Thanks for your effort on this.
> 
> 	 
> 
> 	
> ________________________________
> 
> 
> 	From: Kumar Vakacharla (HCL) 
> 	Sent: Friday, March 07, 2008 2:10 PM
> 	To: Tim Gardner
> 	Subject: Review request : Defect TED00020613
> 
> 	Tim,
> 
> 	 
> 
> 	I have done a fix for the defect TED00020613 and it needs to
> be reviewed.  
> 
> 	Please let me know whom can I send it for review.
> 
> 	 
> 
> 	Here are the details about the fix:
> 
> 	 
> 
> 	 
> 
> 	Defect Description: 
> 
> 	 
> 
> 	can't create two system get alls without renaming the first
> 
> 	 
> 
> 	 
> 
> 	Cause: 
> 
> 	 
> 
> 	This is happening only in Cougar machines.  Generally <.N> is
> appended if there are more than one directories output by 
> 
> 	support.sh in one day. But this behavior is failing in Linux
> because of the following two reasons. 
> 
> 	 
> 
> 	1.       In Linux, echo command doesn't interpret backslash
> escapes unless "-e" option is provided.  In the support.sh while
> creating new directory (in function setup_new_target_dir) 
> 
> 	      "echo" has been used (without "-e" option) to pass the
> arguments to the binary filesh.   This is working fine in bobcat and
> cheetah because this feature is enabled by default (irrespective of
> "-e" option) in 
> 
> 	      shell "sh" used by openbsd. 
> 
> 	 
> 
> 	2.       Having resolved the above issue, there is one more
> issue in Linux which is related to the alignment of the buffer
> containing directory entry records.  
> 
> 	In function cmd_readdir (in sm-file/filesh.c) we use a global
> character buffer readBuf which is used to hold the directory entry
> records. This buffer has been populated with the record data 
> 
> 	using memcpy in the function file_req_read_dir in
> sm-file/file-server.c. 
> 
> 	 
> 
> 	Each directory record will be aligned to 8
> bytes(sizeof(uint64)) internally. We use this rule to traverse to the
> next directory record in the buffer. 
> 
> 	But in Linux I found that starting address of readBuf is not 8
> byte aligned but only 4 byte aligned and this has triggered the
> problem.
> 
> 
> 	Because of this it is traversing to an offset which may not be
> the beginning of the next record and hence reading the directory
> entries fail eventually. 
> 
> 	 
> 
> 	Files Modified: 
> 
> 	 
> 
> 	
> /homes/kumarv/work/dev/nfx-tree/code/ssc-support/support.sh

Looks like someone already beat you to the punch on this one, but
perhaps your change is better.  Have you verified that "echo -e" works
as expected on OpenBSD as well as Linux?

Line 1964, you changed a variable name from TEMPFILE2 to TEMPFILE2_BASE
which I don't think you want to do.

 	
> /homes/kumarv/work/dev/nfx-tree/code/sm-file/filesh.c 

looks ok


> 	 
> 
> 	 
> 
> 	P4CLIENT=kumarv-DEV
> 
> 	Location: /homes/kumarv/work/dev            
> 
> 	 
> 
> 	Please let me know if you need any clarifications.

With just two files, it wasn't that big a deal, but in the future,
capture your changes into a changelist and just send me the changelist
number.  My usual convention is to also send the output of the command
"p4 describe <cl#>" so the reviewer can see the list of files and the
checkin comment that will accompany the change.  If you are unfamiliar
how to do this, see the perforce command "p4 change" or do it using the
GUI.  In the GUI, when you select some files and click on [submit] it
gives you the option to save it to a changelist number.

Cheers,

a


