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