<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><br><br><font color="#990099">-----Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>> wrote: -----</font><br><br>>To: OpenBMC Patches <<a target="_blank" href="mailto:openbmc-patches@stwcx.xyz">openbmc-patches@stwcx.xyz</a>>,<br>><a target="_blank" href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><br>>From: Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>><br>>Date: 02/25/2016 04:43AM<br>>Cc: Milton Miller II/Austin/IBM@IBMUS<br>>Subject: Re: [PATCH openbmc 8/9] initfs: Add flash clear and update<br>>during init<br>><br>>On Wed, 2016-02-24 at 12:40 -0600, OpenBMC Patches wrote:<br>>> From: Milton Miller <<a target="_blank" href="mailto:miltonm@us.ibm.com">miltonm@us.ibm.com</a>><br>>> <br>>> Add code to clear the rwfs (preserving files from the whitelist)<br>>> during init, by a command line option.<br>>> <br>>> Also allow it to be triggered by the file system not being a jffs2.<br>><br>>So this helps avoid a two-stage upgrade process right? We write a new<br>>(fixed wrt spi-nor) image-kernel and an updated image-initrd, reboot,<br>>and then have obmc-init.sh switch the rwfs partition to JFFS2. Is<br>>this<br>>worth mentioning?<br>><br><br>Well, I gave oral instructions at our morning scrum.  Its a one-time<br>conversion, and can be acoomplished by leaving out image-rwfs<br>when updating.<br><br>We could also leave a zero-length image-rwfs.<br><br>Also, we have a u-boot update to go with this kernel (to see serial<br>ouput when rebooting).<br><br><br>>A cleaning function seems useful; did you give any thought to a<br>>factory<br>>-reset function (i.e. not preserving the whitelist files)? I don't<br>>think this is covered either here or in obmc-update.sh? Avoiding<br>>pflash<br>>would be nice but it's probably a separate problem.<br><br>Norm has a setting to remove the whitelist when doing the update on<br>shutdown version.  There is a pending request (that I need to rebase) <br>to split the whitelist into seperate files by component.  It still needs to <br>distribute the files to various recipes that are included.<br><br>I don't support pflash for bmc update, and bmc_update.py<br>in skeleton has been updated to copy images since Jan 30.<br><br>><br>>Otherwise:<br>><br>>Reviewed-by: Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>><br>><br><br>Thanks.<br>milton<br><br></font><BR>