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

RE: [Xen-ia64-devel] [PATCH] RESEND: access_reflection fix


  • To: "Alex Williamson" <alex.williamson@xxxxxx>
  • From: "Xu, Anthony" <anthony.xu@xxxxxxxxx>
  • Date: Thu, 30 Mar 2006 12:09:28 +0800
  • Cc: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Thu, 30 Mar 2006 04:11:26 +0000
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
  • Thread-index: AcZTpcKWd9MJCeu/TNGLxzSrqJGfTQACbw+Q
  • Thread-topic: [Xen-ia64-devel] [PATCH] RESEND: access_reflection fix

I prefer the easier way. :-)

Thanks,
-Anthony 

>-----Original Message-----
>From: Alex Williamson [mailto:alex.williamson@xxxxxx]
>Sent: 2006?3?30? 10:57
>To: Xu, Anthony
>Cc: xen-ia64-devel
>Subject: RE: [Xen-ia64-devel] [PATCH] access_reflection fix
>
>On Thu, 2006-03-30 at 10:29 +0800, Xu, Anthony wrote:
>>
>> R22 comes from region register; bit 1 is reserved for region register.
>> So bit 1 should 0, VMM only need to reset bit0.
>> Then this patch should be OK,
>> -2 are same with ~1 :-).
>
>   Ok, that makes sense then.  It might still be good to clear both bits
>just so the operation makes sense wrt to fields of the register.  Please
>resubmit and either change the submit comment to reflect the code or the
>code the reflect the comment.  fwiw, I tend to prefer the one's
>compliment since it makes these mismatches a little more obvious.
>Thanks,
>
>       Alex
>
>--
>Alex Williamson                             HP Linux & Open Source Lab

Attachment: access_reflection1.diff
Description: access_reflection1.diff

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.