[Skiboot] [PATCH v2 00/10] Bring gard tool inline with pflash

Stewart Smith stewart at linux.vnet.ibm.com
Mon Nov 16 15:41:44 AEDT 2015


Cyril Bur <cyril.bur at au1.ibm.com> writes:
> Note: the first two patches in this series are fixes for memory leaks found
> during this work.
>
> Quite a bit of work went into pflash to abstract out arch and reduce code
> duplication when accessing system flash (or binary images of the flash).
>
> This series updates the gard tool to do all that.
>
>
> Cyril Bur (10):
>   external/common: Don't leak temporary filename
>   external/gard: Fix memory leak
>   external/gard: Clean after check to avoid leaving bad .o files around
>   external/common: Don't just expect `sh` to be in $PATH
>   external/pflash: Fixup LDFLAGS
>   external/gard: Update gard building to use external/common files
>   external/tests: Add ability to strip version string from result
>   external/gard: Add version information
>   external/gard: Make use of the common/ flash reading code
>   external/gard: Add make dist
>
>  external/common/arch_flash_powerpc.c    |   3 +-
>  external/common/get_arch.sh             |   2 +-
>  external/common/rules.mk                |   2 +-
>  external/gard/Makefile                  |  65 ++++++-----
>  external/gard/Makefile.dist             |  10 ++
>  external/gard/gard.c                    | 184 +++++---------------------------
>  external/gard/rules.mk                  |  35 ++++++
>  external/gard/test/Makefile.check       |   8 +-
>  external/gard/test/results/02-usage.out |   1 +
>  external/gard/test/tests/02-usage       |   2 +
>  external/pflash/rules.mk                |   5 +-
>  external/test/test.sh                   |   6 ++
>  12 files changed, 134 insertions(+), 189 deletions(-)
>  create mode 100644 external/gard/Makefile.dist
>  create mode 100644 external/gard/rules.mk
>

Thanks, series applied as of 168b714, along with a patch of mine to
build all the gard/pflash deps in their respective directories arther
than the common ones, as this fixes my make check and travis.



More information about the Skiboot mailing list