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:37:28 +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 02:38:02 -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; bh=KFSfshi45o/pI6vtjh9q/0hb9zrB7ETlCs7+GoW62so=; b=qDOssPoO6gS6zJVfqLI0kmI0FxulQPI7J+2DB04XWn6xDhn11tOlKuXbFNUWDXxZF9 pxeA8LJhnaNwmscI4H9NAWBusSP56j4DW9XdUgOzip8AoP1nhb6V9vpWtQ8tllwNZNyK aS2AO5zlYhK8Dkgfp8EPGkI6a9JyoRIIdLPm8=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4DA30384020000780003ADEA@xxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Is this a candidate for backporting to 4.1?


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>