Re: [PATCH openbmc v6 00/18] Update flash update to be useable at runtime.

Re: [PATCH openbmc v6 00/18] Update flash update to be useable at runtime.

[Milton Miller II]

[-- Attachment #1: Type: text/html, Size: 2417 bytes --]

Re: [PATCH openbmc v6 00/18] Update flash update to be useable at runtime.

[Andrew Jeffery]

[-- Attachment #1: Type: text/plain, Size: 2120 bytes --]

Hi Milton,

On Wed, 2016-06-22 at 19:30 -0500, OpenBMC Patches wrote:
> The underlying storage for a file system can not be updated when it is in use.  The initial solution was to update the flash as the system was being shutdown, with output to the console.
> 
> Updates were made to the init script that allows the flash to not be used for a given boot but the update script was not enhanced to make this user friendly, and no documentation was provided to prepare the system for this mode.
> 
> This series partially addresses #293 by enhancing the update script to
> 1. provide more fine grained checks on update conflicts allowing use at runtime
> 2. provide additional checks that would cause failure to flash
> 3. provide user help and consistent tagging of error output
> 4. provide an error return code if the checks fail
> 
> In addition the series provides
> * A fix to properly assemble the flash image, filling with `0xff` characters as intended.
> * A cleanup to the build recipe and naming.
> * A diagnostic message to shutdown if the update did not remove a file indicating it was successful.
> * A method to build `init-options-base` into the initramfs providing options to `init` while allowing additional sources of options like the kernel command line. 
> * A self containted image `initramfs-netboot.cpio.u-boot` for loading over the network.
> 
> These last two are intended to ease creating images for alternate network based deployment and testing, easing the use of the features and options already in the `init` script.
> 
> 
> ---
> This change is [https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/openbmc/openbmc/297)
> 
> 
> 
> https://github.com/openbmc/openbmc/pull/297
> 
> Brad Bishop (3):
>   Add pflash parallel build fix patch
>   Remove unnecessary skeleton dependencies
>   Removed unused host-ipmid-bt application
> 
> Milton D. Miller II (9):
...
> Milton Miller (6):

Nit pick: Probably worth reconfiguring git to use a consistent user.name.

Cheers,

Andrew

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[PATCH openbmc v6 00/18] Update flash update to be useable at runtime.

[OpenBMC Patches]

The underlying storage for a file system can not be updated when it is in use.  The initial solution was to update the flash as the system was being shutdown, with output to the console.

Updates were made to the init script that allows the flash to not be used for a given boot but the update script was not enhanced to make this user friendly, and no documentation was provided to prepare the system for this mode.

This series partially addresses #293 by enhancing the update script to
1. provide more fine grained checks on update conflicts allowing use at runtime
2. provide additional checks that would cause failure to flash
3. provide user help and consistent tagging of error output
4. provide an error return code if the checks fail

In addition the series provides
* A fix to properly assemble the flash image, filling with `0xff` characters as intended.
* A cleanup to the build recipe and naming.
* A diagnostic message to shutdown if the update did not remove a file indicating it was successful.
* A method to build `init-options-base` into the initramfs providing options to `init` while allowing additional sources of options like the kernel command line. 
* A self containted image `initramfs-netboot.cpio.u-boot` for loading over the network.

These last two are intended to ease creating images for alternate network based deployment and testing, easing the use of the features and options already in the `init` script.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/openbmc/openbmc/297)
<!-- Reviewable:end -->


https://github.com/openbmc/openbmc/pull/297

Brad Bishop (3):
  Add pflash parallel build fix patch
  Remove unnecessary skeleton dependencies
  Removed unused host-ipmid-bt application

Milton D. Miller II (9):
  image_types_uboot: Don't truncate when assembling flash
  initfs: Fix recipe, remove unnecessary class
  initfs: shutdown: comment on update success
  initfs: update: Don't exec sh or sulogin on error just exit 1
  initfs: update: Do not cause an error if no files were saved
  initfs: update: add option to perform checks but skip actual update
  initfs: update: check individual mtd partitions for mounts
  initfs: update: add check for image size vs mtd size
  initfs: init: add file for base options

Milton Miller (6):
  initfs: update: Consistently add ERROR: and print to stderr
  initfs: update: add --help message
  image-types_uboot: Package a self contained netboot image
  initfs: update: handle no images to update
  initfs: update: Sanitze whitelist directory entries
  initfs: whitelist: Sanitize obmc events whitelist entry