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 16:04:45 +0100
Cc:
Delivery-date: Tue, 19 Aug 2008 08:05:14 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C4D0637F.26256%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+AAUtT5kACPVKTQ==
Thread-topic: [Xen-devel] hvmemul_virtual_to_linear() doesn't care about direction-flag?
User-agent: Microsoft-Entourage/11.4.0.080122
This should be fixed as of c/s 18340. Feel free to see if you can spot any
problems with it! I'll roll another release candidate tomorrow morning,
after the tree has been through automated testing.

Curently it's in staging only:
http://xenbits.xensource.com/staging/xen-unstable.hg

 -- Keir

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

> 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