AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20080820173511.74531688@ripper.onstor.net>
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<bill.nadzam@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	BB375AF679D4A34E9CA8DFA650E2B04E08FC5C72@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Wed, 20 Aug 2008 17:36:18 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Bill Nadzam" <bill.nadzam@onstor.com>
Subject: Re: PERFORCE change 30456 for review
Message-ID: <20080820173618.1c988868@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E08FC5C72@onstor-exch02.onstor.net>
References: <20080820170132.004ac31a@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E08FC5C72@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

Understood.  It already went into dev branch as 30455, jah?  And it
looks like you put it into r330rel, so that's all good.  What we have
going on is a branching discussion about how to not-stupidly do these
patch branches.  That discussion is still ongoing, so just hold off for
reverting anything for now, in case you have been itching to do the
revert ~:^)

Cheers,

a

On Wed, 20 Aug 2008 17:13:46 -0700 "Bill Nadzam"
<bill.nadzam@onstor.com> wrote:

> Its broken in 4.0 as well. 
> 
> -----Original Message-----
> From: Andy Sharp 
> Sent: Wednesday, August 20, 2008 5:02 PM
> To: Bill Nadzam
> Subject: Re: PERFORCE change 30456 for review
> 
> Bill, this is the wrong branch.  Please revert this and put it in the
> r330rel branch.  At least I think that is the name.  Double check with
> Larry.
> 
> Thanks,
> 
> a
> 
> 
> On 20 Aug 2008 15:55:01 -0700 billn <bill.nadzam@onstor.com> wrote:
> 
> > Change 30456 by billn@billn-dev on 2008/08/20 15:53:29
> > 
> > 	
> > 	Change 30455 by billn@billn-dev on 2008/08/20 15:51:03
> > 	
> > 	Fix TED00025151, Found during a customer upgrade from
> > 3.2.0.4 to the 3.3 GA code. Customer was not able to access his 
> > 	Clariion storage array. Found Logic in device path detection
> > 	had been inverted. Since ONStor does not have a Clariion
> > 	array in house the attachment was never tested before
> > release. 
> > 	reviewed by jong
> > 
> > Affected files ...
> > 
> > ... //depot/r400rel/nfx-tree/code/sm-sdm/sdm-init-disk.c#6 edit
> > 
> > 
> > http://liszt:1818/@md=d&cd=//depot/$c=G35@/30456?ac=10
