WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] hvmemul_virtual_to_linear() doesn't care about direction

To: "Byrne, John (HP Labs)" <john.l.byrne@xxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] hvmemul_virtual_to_linear() doesn't care about direction-flag?
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Tue, 19 Aug 2008 11:48:15 +0100
Cc:
Delivery-date: Tue, 19 Aug 2008 03:48:42 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C4D040C1.1C3E9%keir.fraser@xxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AckBYgqYPA8ffyuBQxqCY+AgRFffRQAclVJ+AAUtT5k=
Thread-topic: [Xen-devel] hvmemul_virtual_to_linear() doesn't care about direction-flag?
User-agent: Microsoft-Entourage/11.4.0.080122
This affects hvmemul_linear_to_phys() too, and would for example mean that a
backwards I/O string instruction in userspace that crosses a page boundary
would very likely cause I/O to/from the wrong physical pages. I've confirmed
this with a small testing patch to hvmloader. I think we need to work out
how to maintain a test suite of this kind of thing to check for regressions
in these kinds of rarer corner cases.

Obviously I'll fix this for 3.3.0 and probably roll out another release
candidate.

 -- Keir

On 19/8/08 09:20, "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx> wrote:

> Quite a nasty omission though, and quite easily fixed. Thanks for pointing
> it out.
> 
>  -- Keir
> 
> On 18/8/08 19:41, "Byrne, John (HP Labs)" <john.l.byrne@xxxxxx> wrote:
> 
>> I was following the emulation code around in xen-unstable cs 18335 and I
>> noticed that the direction flag doesn't get taken into account for the
>> segment
>> bounds checking in the 32-bit case anywhere I could see. Does anyone know
>> better?
>> 
>> Maybe no one will care, but I thought I'd mention it.
>> 
>> John Byrne
>> 
>> 
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>> http://lists.xensource.com/xen-devel
> 



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