Re: seabios 0.6.1 regression

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/16/2010 04:17 PM, Alexander Graf wrote:
On 16.11.2010, at 15:15, Avi Kivity wrote:

>  On 11/16/2010 03:19 PM, Alexander Graf wrote:
>>  >>>
>>  >>
>>  >>   Rewriting it to use inb / stos works (jecxz ; insb; loop doesn't) so it looks like a kernel bug in insb emulation.
>>  >>
>>  >
>>  >   Turns out is was a subtle bug in the tpr optimization we do for Windows XP.  The problem happens when we load the vapic option rom from the firmware config interface.  With inb / movb, writing the vapic area happens in guest context, which the kernel is prepared to handle.

^^

>>  With insb, the write happens from kvm, which is then undone on the next entry, leading to the tpr being set to a high value.
>>
>>  Shouldn't the vapic area be mapped in on demand? Then we could map it on option rom init time and everyone's happy.
>
>  Mapped in? It's an option rom.

According to your wording, the kernel handles writes to the vapic area. If the vapic area is only made special after initialization of the option rom, we wouldn't have this issue.


Ah, yes, and I'm even now writing a reset handler to make this happen.

--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux