Jan, This patch is the latest. Should have addressed all of your concerns. Please take a look. Thanks, Wei Signed-off-by: Wei Wang <wei.wang2@xxxxxxx> On Monday 11 April 2011 09:23:38 Jan Beulich wrote: > >>> On 08.04.11 at 18:52, Wei Wang2 <wei.wang2@xxxxxxx> wrote: > > > > Jan, How dose this one look like to you? > > Much better, but still not quite there: The unmasking must happen > *after* the writing of the upper half (if that's what is being modified). > > You could also skip the unmasking altogether if saved_mask == 1. > > And if you start using __io_apic_write() (which I find very desirable) > is there a reason not to use it (and __io_apic_read()) in all of the > other places you touch anyway, too? > > Jan
Attachment:
fix_intremap_v3.patch
Description: fix_intremap_v3.patch
_______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel