Re: [Xen-devel] [PATCH for-xen-4.5 v3 00/16] xen: Break multiboot (v1) dependency and add multiboot2 support

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 10.10.14 at 13:07, <daniel.kiper@xxxxxxxxxx> wrote:
> On Fri, Oct 10, 2014 at 09:23:55AM +0100, Jan Beulich wrote:
>> >>> On 08.10.14 at 19:52, <daniel.kiper@xxxxxxxxxx> wrote:
>> > Patch #13 reveals a bug which probably was introduced between commit
>> > 3e2331d271cc0882e4013c8f20398c46c35f90a1 (VT-d: suppress UR signaling for
>> > further desktop chipsets) and 61fdda7acf3de11f3d50d50e5b4f4ecfac7e0d04
>> > (x86/HVM: properly bound x2APIC MSR range). Xen crashes at video_endboot()
>> > because earlier scrub process wipes vga_console_info data (sic!). So,
>> > it means that at least page containing this structure was freed mistakenly
>> > somewhere. Interestingly this issue appears on legacy BIOS machines only.
>> > EFI platforms work as usual. It is possible to workaround this bug by
>> > passing no-bootscrub to xen.gz.
>>
>> Giving a little more detail on how exactly things crash would be rather
>> useful to make suggestions. Or if you intend to figure this out all by
>> yourself, then I think you would have better done so before posting
>> this series.
> 
> I am attaching console log. Xen crashes at 
> xen/drivers/video/vga.c:video_endboot():175.
> 
>     switch ( boot_info->vga_console_info.video_type )

Without your changes this line is

    switch ( vga_console_info.video_type )

with the variable being a global, non-_initdata one. That way it just
can't go away unless we have some sort of memory corruption.
Therefore I would rather think that the problem is inside your series.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel