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/
Home Products Support Community News


Re: [Xen-devel] Re: [PATCH V3] AMD IOMMU: Fix an interrupt remapping iss

To: Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] Re: [PATCH V3] AMD IOMMU: Fix an interrupt remapping issue
From: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
Date: Tue, 19 Jul 2011 10:59:55 +0100
Cc: Wei Wang2 <wei.wang2@xxxxxxx>, Boris Ostrovsky <Boris.Ostrovsky@xxxxxxx>, Wei Huang2 <Wei.Huang2@xxxxxxx>, Keir Fraser <keir@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 19 Jul 2011 03:03:49 -0700
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=avtjSzUAk58+yW2dpPda6MkMVu1OtnySdNk8V6UdBHY=; b=o1WbfkxwS6j/UCKXZRMf4bH9YlHmQz9zSBhcH1msxwytNnxv92rs+oP4Tl4LXXIwiH X7+CO+0FRedYCYABJXXGLquhlBvuY1hzGQg0fyKNm6iJ8de5X0DlteKgNObmO6WyNaa5 YJdhdt5Acnq4/32f/yrLs9GM92JDE8sQe+QIA=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <CAFLBxZZ7u=o8J8KaPObmyYHSQn2oQwKRZ_pTDkpD2WiQsbbk6Q@xxxxxxxxxxxxxx>
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>
References: <201104081335.36718.wei.wang2@xxxxxxx> <201104081852.20738.wei.wang2@xxxxxxx> <4DA2C89A020000780003AD92@xxxxxxxxxxxxxxxxxx> <201104111231.24422.wei.wang2@xxxxxxx> <4DA30384020000780003ADEA@xxxxxxxxxxxxxxxxxx> <CAFLBxZZ7u=o8J8KaPObmyYHSQn2oQwKRZ_pTDkpD2WiQsbbk6Q@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Oops, n/m -- already in. :-)

On Tue, Jul 19, 2011 at 10:37 AM, George Dunlap
<George.Dunlap@xxxxxxxxxxxxx> wrote:
> Is this a candidate for backporting to 4.1?
>  -George
> On Mon, Apr 11, 2011 at 12:35 PM, Jan Beulich <JBeulich@xxxxxxxxxx> wrote:
>>>>> On 11.04.11 at 12:31, Wei Wang2 <wei.wang2@xxxxxxx> wrote:
>>> This patch is the latest. Should have addressed all of your concerns. Please
>>> take a look.
>> Looks good to me now. Assuming it also still addresses the problem
>> at hand:
>> Acked-by: Jan Beulich <jbeulich@xxxxxxxxxx>
>> Thanks, Jan
>>> 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
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>> http://lists.xensource.com/xen-devel

Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>