X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C883E5.CDC7F012@onstor-exch02.onstor.net>; Tue, 11 Mar 2008 19:07:20 -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: Prom upgrade runtime changes
Date: Tue, 11 Mar 2008 19:07:19 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E08D2A210@onstor-exch02.onstor.net>
In-Reply-To: <20080311181729.64c51a74@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: Prom upgrade runtime changes
Thread-Index: AciD3tgmWf7cXLwnRZ+kzs97iiaLgwAANvGA
References: <BB375AF679D4A34E9CA8DFA650E2B04E08D29F3A@onstor-exch02.onstor.net><BB375AF679D4A34E9CA8DFA650E2B04E056C9468@onstor-exch02.onstor.net><BB375AF679D4A34E9CA8DFA650E2B04E08D2A198@onstor-exch02.onstor.net> <20080311181729.64c51a74@ripper.onstor.net>
From: "Warren Gale" <warren.gale@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>
Cc: "Larry Scheer" <larry.scheer@onstor.com>


Ok, Cool... Thanks.. I'll start working on it..
Please see below ..  <wg>  :)



-----Original Message-----
From: Andy Sharp=20
Sent: Tuesday, March 11, 2008 6:17 PM
To: Warren Gale
Cc: Larry Scheer
Subject: Re: Prom upgrade runtime changes

My comments:

nfx-tree/code/sm-prom-upgrade/from-upgrade.c

---------
<wg>  actually I don't know who did it, I just get to update it...
      I agree that it's ugly, but it is used for Bobcat and Cheetah=20
      as well.  Since the partitions of the PROM have changed for=20
      Cougar, and their sizes, this seemed to work best without having
      to make lots of changes other code
--------- =20

     this is your code, but this is damn ugly.  shouldn't you set
     PART_DESC_SIBYTE_FP_LAYOUT to be the right value for cougar
     instead of adding yet another nasty ifdef?  i leave it up to you.
     do the right thing or your fired.  i kid.


nfx-tree/code/sm-prom-upgrade/prom-part-api.h


     again, i would do a big ifdef COUGAR define these two sibyte
     macros, or define them the previous way.


nfx-tree/code/ssc-nfxsh/cmd_promupgrade.c


     looks ok.  if you can stand variables named 'puout'

---------
<wg>
    Ok Changed  ifdefs here on 503 and friend.. (Like that one)..
    Changed 625 and got rid of 1023... Sorry I missed that one.
---------

     line 503 and friend.  these ifdef COUGARs should have an else
     for the other code that doesn't apply when it's a cougar.

     line 625 don't do this.  put whole, complete assignments inside
     each ifdef block.

     line 1023 is bogus

     rest is ok.  some formatting changes a welcome refreshment.

nfx-tree/code/ssc-nfxsh/menu.c


     looks good


nfx-tree/code/ssc-prom-upgrade/prom-upgrade.c

-------------
<wg>
http://wiki.onstor.net/wiki/Onstor_Coding_Style_Guide
Whitespace and formatting:
I guess number 1 and number 2 is just plain wrong.
Ok, I can put it back. Your call.. :)
(That's what I get for trying to take ownership of a file :(  stung :) )
-----------

     holy cauliflower! what's with all the unneccessary whitespace
     changes?  can you please redo this, and delete all the unnecessary
     whitespace changes so I can see the substantive changes?  if there
     is some trailing whitespace that needs cleaning, you can do bits
     of that, but not the whole dang file.  And especially since you
     went the wrong way: converting tabs to spaces.  my lord.





On Tue, 11 Mar 2008 17:51:59 -0700 "Warren Gale"
<warren.gale@onstor.com> wrote:

> Larry,
>=20
> =20
>=20
>     Ok, Cool.  I changed all the strcpy  's  to   strncpy  's  in this
> file.
>=20
>     It tested ok also.
>=20
> Thanks,
>=20
> Warren
>=20
> =20
>=20
> Andy,
>=20
>    If you ignore white space on this file there are very few changes..
>=20
>  Ie tkdiff -db =20
>=20
> =20
>=20
> Thanks,
>=20
> Warren
>=20
> ________________________________
>=20
> From: Larry Scheer=20
> Sent: Tuesday, March 11, 2008 5:15 PM
> To: Warren Gale
> Cc: Andy Sharp
> Subject: RE: Prom upgrade runtime changes
>=20
> =20
>=20
> nfx-tree/code/sm-prom-upgrade/from-upgrade.c
>=20
> =20
>=20
>      looks good
>=20
> =20
>=20
> nfx-tree/code/sm-prom-upgrade/prom-part-api.h
>=20
> =20
>=20
>      looks good
>=20
> =20
>=20
> nfx-tree/code/ssc-nfxsh/cmd_promupgrade.c
>=20
> =20
>=20
>      line 127 you might get future grief for using strcpy versus
>=20
>      strncpy.  Perhaps you should change this to keep Coverity from
>=20
>      flagging it as a defect and then you won't have to worry about
>=20
>      some HCL guy bollixing up your code.
>=20
> =20
>=20
> nfx-tree/code/ssc-nfxsh/menu.c
>=20
> =20
>=20
>      looks good
>=20
> =20
>=20
> nfx-tree/code/ssc-prom-upgrade/prom-upgrade.c
>=20
> =20
>=20
>      Not reviewed. Check with Andy
>=20
> =20
>=20
> ________________________________
>=20
> From: Warren Gale=20
> Sent: Tuesday, March 11, 2008 11:59 AM
> To: Larry Scheer
> Subject: FW: Prom upgrade runtime changes
>=20
> =20
>=20
> Larry,
>=20
>   I'd like to get this in on the next build.
>=20
> I sent this to Andy S.   Thought I'd also send you a copy.
>=20
> =20
>=20
> Thanks, and for the p4 help too.
>=20
> Warren
>=20
> =20
>=20
> ________________________________
>=20
> From: Warren Gale=20
> Sent: Tuesday, March 11, 2008 11:38 AM
> To: Andy Sharp
> Subject: Prom upgrade runtime changes
>=20
> =20
>=20
> Andy,
>=20
>   Please review.
> Thanks,
>=20
> Warren
>=20
> =20
>=20
> Change 28170 by warreng@warreng-cougar on 2008/03/04 18:05:31
> *pending*
>=20
> =20
>=20
>         Fix to /onstor/bin/prom-upgrade command.
>=20
>         Fix was to get prom-upgrade app Id and use it as 1st param to
> sendAgileMsg.
>=20
>         Add support for cougar prom partitions.
>=20
>         fix nfxsh prom upgrade command.  (for Cougar, only FP and TXRX
> are upgraded)
>=20
>         fix to nfxsh prom upgrade files command.
>=20
>         Now able to get a tar.gz file from upgrade server.
>=20
> =20
>=20
> Affected files ...
>=20
> =20
>=20
> ... //depot/dev/nfx-tree/code/sm-prom-upgrade/from-upgrade.c#2 edit
>=20
> ... //depot/dev/nfx-tree/code/sm-prom-upgrade/prom-part-api.h#1 edit
>=20
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_promupgrade.c#11 edit
>=20
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/menu.c#10 edit
>=20
> ... //depot/dev/nfx-tree/code/ssc-prom-upgrade/prom-upgrade.c#5 edit
>=20
> =20
>=20
