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

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


  • To: "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx>
  • From: "Trolle Selander" <trolle.selander@xxxxxxxxx>
  • Date: Tue, 19 Aug 2008 17:41:52 +0100
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Tue, 19 Aug 2008 09:42:19 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:references; b=U2rY0KSCqHT+egm78Eo787hDzuvv2nQgaJmaAQXJ7CJx+8wStbPWAbFAceVtwGMr5u iVQAPSrKVtKK4j20QRxdCqoUCsBxBcGWoealXhOTDJmo5DfHrY/HixVxAIqpBnq4PavS G79gQ+OtdIMRkEk/rK2PCc8xuMFTLhdodpBnU=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Got a build error on CentOS 5.2/i386 (but not on Fedora 8 x86_64)

emulate.c: In function 'hvmemul_linear_to_phys':
emulate.c:233: warning: passing argument 2 of 'hvmemul_linear_to_phys'
from incompatible pointer type

Patch attached.

-- Trolle

On Tue, Aug 19, 2008 at 4:04 PM, Keir Fraser <keir.fraser@xxxxxxxxxxxxx> wrote:
> 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
>

Attachment: emulate_type_fix.patch
Description: Text Data

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

 


Rackspace

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