[PATCH 0/9] kexec userspace and purgatory cleanups

[Simon Horman]

On Wed, Apr 23, 2008 at 06:56:47AM -0700, Eric W. Biederman wrote:
> Jamey Sharp <jamey at thetovacompany.com> writes:
> 
> > GIT: Please enter your email below.
> > GIT: Lines beginning in "GIT: " will be removed.
> > GIT: Consider including an overall diffstat or table of contents
> > GIT: for the patch you are writing.
> >
> > This patch series fixes issues I encountered while porting kexec to run
> > on Windows. (I'll be happy to say more about that insane, but
> > surprisingly successful, plan on another occasion.) 
> 
> Sounds fun.  I like the idea of /sbin/kexec running on other operating
> systems, as long as it isn't too much maintenance overhead.
> 
> Portability tends to reveal all kinds of things, and increase
> the user/developer base.
> 
> Now if we only had the code to boot windows...
> 
> > These patches,
> > however, are only the ones that affect the kexec userspace tools on
> > Linux. They fall into three categories.
> 
> I just took a quick skim through the patches and they generally
> look good.

I have done likewise and they seem fine to me.
I am going to go ahead and merge them.

> > Bug fixes, primarily in the i386 purgatory source:
> > - [1/9] Fix undefined symbol errors on readw/writew.
> > - [2/9] Fix copy-paste bug: entry16 does not start at entry16_debug.
> > - [3/9] Conform more closely to Documentation/i386/boot.txt.
> > - [9/9] Die on early EOF in slurp_file, instead of infinite-looping.
> >
> > Compiler and assembler warning fixes:
> > - [4/9] Fix "Warning: indirect jmp without `*'".
> > - [5/9] Fix all gcc warnings for ARCH=i386 builds.

Thanks, I've had less successful attempts at that myself.

> > Cleanups or simplifications:
> > - [6/9] <sys/mman.h> is not needed by any i386 kexec/ source.

I have been able to confrim that its not needed by other arches either -
well, more specifically I've confirmed that if you take it out all
the architectures still compile. I'll post a patch that removes
<sys/mman.h> for non-i386 to complete your work here.

> > - [7/9] Replace weak definitions with source filename overriding.
>   The other benefit of this is that it slightly reduces the binary
>   size which is advantageous for embedded development.
> 
> > - [8/9] Simplify initialization of argument list for `shutdown`.
> 
> I lifted this out sysvinit where that structure is needed because
> processing of optional processing of the -t <when> option.  Since
> we aren't doing that your structure is better.

-- 
Horms