Messages in this thread | | | Date | Wed, 16 Jan 2013 10:20:53 -0800 | Subject | Re: [PATCH v7u1 00/31] x86, boot, 64bit: Add support for loading ramdisk and bzImage above 4G | From | Yinghai Lu <> |
| |
On Wed, Jan 16, 2013 at 9:38 AM, H. Peter Anvin <hpa@zytor.com> wrote: > On 01/16/2013 09:31 AM, Yinghai Lu wrote: >> >> > > The first sentence here doesn't parse, and this description doesn't give any > hint to anyone who is researching this code in say, five years, what > problems this caused. > > I can't really figure it out, either; from looking at the thread and the > patch I'm assuming the problem is somehow that the code failed to use the > last page in the brk buffer, which somehow lead to a Xen boot failure... but > the bits from here to there are totally unclear.
ok, let me try again. tip:x86/mm2 does not fail on xen testing from Stefano and Konrad.
Stefano found booting failure with PV xen with for-x86-boot.
for-x86-boot has one patch that is trying to get back an wasted page in pre-allocated BRK and that cause xen pv fails with 2G setup. that patch just uncover one bug in x86/mm2.
So questions: 1. do we need to rebase x86/mm2 to fold the two patches in. ? 2. or just put fixes at beginning for-x86-boot ?
Thanks
Yinghai
| |