On Fri, Dec 01, 2006 at 03:08:53PM +0900, Horms wrote: > On Tue, Nov 21, 2006 at 07:13:56AM +0800, Zou Nan hai wrote: > > This patch make normal "kexec -l" first try physical address suggested > > by vmlinux. > > > > If there is no enough memory, kexec tools will search /proc/iomem and > > find a place to put the new kernel. > > > > This is necessary for "kexec -l" to work on SN platform. > > > > Signed-off-by: Zou Nan hai <nanhai.zou@xxxxxxxxx> > > > > Hi Nan hai, > > sorry for not responding sooner. > > I have take a look at your patch with an eye to including it into > kexec-tools-testing. It does seem like a good feature. > > Below is an updated version of the patch that cleans up your > implementation a little. In particular. > > * update_loaded_segments() returns -1 on error, 0 on success > - amongst other things the previous incarnation could return nothing > at all in once case. > * else { if () { ; } } -> else if () { ; } > * < 80 columns wide > * removed trailing whitespace > > Is it ok to apply this incarntation of the patch? > I can make my changes a second patch if you prefer. > > Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx> Hi, I have applied your patch, and then my changes as a second patch to kexec-tools-testing. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |