AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20090117020028.0661692d@ripper.onstor.net>
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:exch1.onstor.net
NSV:
SSH:
R:<larry.scheer@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/andys@onstor.net@exch1.onstor.net/INBOX	0	2779531E7C760D4491C96305019FEEB51851A850F7@exch1.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Sat, 17 Jan 2009 02:00:37 -0800
From: Andrew Sharp <andy.sharp@onstor.com>
To: Larry Scheer <larry.scheer@onstor.com>
Subject: Re: PERFORCE change 31696 for review
Message-ID: <20090117020037.378789af@ripper.onstor.net>
In-Reply-To: <2779531E7C760D4491C96305019FEEB51851A850F7@exch1.onstor.net>
References: <WEBMAIL3PmTRKBIG0kM00000091@mail.onstor.com>
	<2779531E7C760D4491C96305019FEEB51851A850F7@exch1.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

All that still works.  It only applies to the "autoload" function or
command.  The load command, the prom upgrade command, all still do tftp
just fine.  And you can get the old autoload functionality by using the
autoload_old command.  But I predict no one will ever use it.

Cheers,

a


On Fri, 16 Jan 2009 19:46:48 -0800 Larry Scheer
<larry.scheer@onstor.com> wrote:

> I must have overlooked the comment about no longer loading the image
> via TFTP when I first review this change. TFTP load is how we are
> currently upgrading proms and the FPGA. How will we upgrade the proms
> and FPGA in the field? Warren's upgrade prom program via the nfxsh
> was flakey at best. There was no nfxsh program to upgrade the FPGA. I
> think we still need the TFTP image load capability if for nothing
> else but updating the prom and FPGA.
> 
> -----Original Message-----
> From: Andy Sharp 
> Sent: Friday, January 16, 2009 4:20 PM
> To: Anurag Agarwal; Amit Bothra; Anand Mitra; Andy Sharp; Bill
> Fisher; Chris Vandever; Deepak Veliath; Ian Brown; Jan Seidel; Jobi
> Ariyamannil; Jonathan Goldick; Larry Scheer; Maxim Kozlovsky;
> Prasanna Kumar; Ram Upendran; Sandrine Boulanger; Sripal Surendiran
> (HCL) Subject: PERFORCE change 31696 for review
> 
> Change 31696 by andys@ripper on 2009/01/16 16:17:14
> 
> 	Change the default image loading algorithm from the PROM
> loading all the images at a snail's pace, to loading just the Linux
> 	kernel on the SSC, and then the TXRX and FP images are loaded
> by new userspace Linux program at 5x the speed.  Results in sub 1
> 	minute boot times on production builds.  User space load
> program is compatible with old proms and will not try to load an
> already loaded image.
> 	
> 	This only affects the "autoload" behavior on the SSC.  The old
> 	behavior is still available via the new command
> "old_autoload". 
> 	The image loading program does not do TFTP image loading;
> 	however, this hardly matters, as a developer can have a remote
> 	directory NFS mounted and have the boot image loaded that way.
> 	In other words, TFTP isn't needed.
> 	
> 	Other general PROM fixing goodness:
> 	
> 	Fix annoyance where bare "diskload" command loaded the kernel
> 	from the first compact flash regardless of what the boot_dev
> 	environment variable was set to.
> 	
> 	Fix annoyance where the diskload command required you to also
> 	specify the load address if you wanted to specify the file
> name to load.  Load address should be able to use a default just like
> 	everything else in this world.
> 	
> 	Add reboot command for TXRX and FP.  Unlike the reboot command
> 	issued from the SSC prom which reboots the entire machine,
> 	only the respective 1480 is rebooted.
> 	
> 	Remove a tiny bit of the massive amount of dead code.
> 	
> 	reviewed by rendellf brians larrys
> 
> Affected files ...
> 
> ... //depot/dev/linux/Makefile#2 edit
> ... //depot/dev/linux/rootfs/Makefile#42 edit
> ... //depot/dev/linux/rootfs/etc/init.d/rcS#10 edit
> ... //depot/dev/linux/tools/Makefile#1 add
> ... //depot/dev/linux/tools/load_vmlinux.c#1 add
> ... //depot/dev/nfx-tree/linux.mk#5 edit
> ... //depot/dev/prom/cg/code/prom-decompress/makerom.c#4 edit
> ... //depot/dev/prom/cg/code/prom-pmon/cmdtable.c#14 edit
> ... //depot/dev/prom/cg/code/prom-pmon/commands.c#9 edit
> ... //depot/dev/prom/cg/code/prom-pmon/main.c#14 edit
> ... //depot/dev/prom/cg/code/sm-chassis/cm-api.c#2 edit
> ... //depot/dev/prom/cg/code/sm-loader/load-auto.c#7 edit
> ... //depot/dev/prom/cg/code/sm-loader/load-cougar-diag-util.c#13 edit
> ... //depot/dev/prom/cg/code/sm-loader/load-cougar.c#12 edit
> ... //depot/dev/prom/cg/code/sm-loader/load-cougar.h#3 edit
> ... //depot/dev/prom/cg/code/sm-pmondisk/Makefile#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/dinode.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/dir.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/disk.c#3 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/disk.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/diskio.c#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/diskio.h#3 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/disklabel.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/dkbad.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/exec_elf.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/filesystem.c#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/fs.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/i82365reg.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/lseek.c#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/mips_disklabel.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/open.c#3 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/pcmcia.c#3 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/pcmciareg.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/pcmciavar.h#3 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/pmondisk.c#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/queue.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/read.c#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/ufs.c#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/ufs.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmondisk/wdcreg.h#2 delete
> ... //depot/dev/prom/cg/code/sm-pmonext2fs/disk.c#4 edit
> ... //depot/dev/prom/cg/code/sm-ui/cli.c#2 edit
> 
> 
> http://liszt:1818/@md=d&cd=//depot/$c=G35@/31696?ac=10
