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

Re: [Xen-devel] [PATCH v3 03/14] AMD/IOMMU: use bit field for control register


  • To: "Woods, Brian" <Brian.Woods@xxxxxxx>
  • From: Jan Beulich <JBeulich@xxxxxxxx>
  • Date: Mon, 22 Jul 2019 08:55:45 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1;spf=pass smtp.mailfrom=suse.com;dmarc=pass action=none header.from=suse.com;dkim=pass header.d=suse.com;arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=znqLsLSp99gT3tl0p4DcHDPfLV4Ez3HJWmtFFFj+2DU=; b=ENTNXcDKeQEqJBBGBTWzgFEp6BMTPGTasgEO0kotZEND54fEc4+4iADDuedptQyuFbcVjCJPlfUAgDSsTdOnWY2B8V36F7QZml+t1J6k0kDxLte1uZRYXuxiwJfv45OI0dbSW210fqViUvIeiayxXyqsm743jx2kb1pZgWL2TAL+jPm7wID6bEG1EOY16mAaYzMdo4S1agN/HCrfp7Wajb+NooI1UCIwnbo6GsISzXC71/9G+HB/CDnLZWzekhtTlsSkXd4Dpdi+aScZQa6ZHo0mAt9SaJziYr0EZbKGUEyw2sqr/UAG5+OCihaFcd1ehK932kYTnmHygearyXKtwA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=I8yDyilI1ae5EThEaI3VUq5FLWnHofedTsmnWLIQZhE7IGEGOrjCOoZMgOwyjWFAk5l1lh/dge2/zDa2M+SUYTrF4gT7O4Y0lsxSafZUgy2aWPF4Bc8OU/UfqWouEYJqBjHtSCet8rsYLf8/wI8qgAQAQvmvyu2bxINyDRyOSoiHdK6i53bYWWnxmvGYCTWuj8K/PncnMqgCMmYTLa3phOOaofuMgi0yVB/LDrS0fEMYEuQOsT0qpegv602Q93NYks1MN/04ifXfcb+ecmkcxEeytIdLMN0Ge+xGkyaBlGr3B4yBvChO5PC87tBB2S6GeMe89V10Z+gPRmTHL5684w==
  • Authentication-results: spf=none (sender IP is ) smtp.mailfrom=JBeulich@xxxxxxxx;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Suravee Suthikulpanit <Suravee.Suthikulpanit@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 22 Jul 2019 09:00:29 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHVO/SRnAccqyvNqEup1oh+z/xKQqbSRheAgAQYd4A=
  • Thread-topic: [PATCH v3 03/14] AMD/IOMMU: use bit field for control register

On 19.07.2019 20:23,  Woods, Brian  wrote:
> On Tue, Jul 16, 2019 at 04:36:06PM +0000, Jan Beulich wrote:
>> Also introduce a field in struct amd_iommu caching the most recently
>> written control register. All writes should now happen exclusively from
>> that cached value, such that it is guaranteed to be up to date.
>>
>> Take the opportunity and add further fields. Also convert a few boolean
>> function parameters to bool, such that use of !! can be avoided.
>>
>> Because of there now being definitions beyond bit 31, writel() also gets
>> replaced by writeq() when updating hardware.
>>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
>> Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> 
> Acked-by: Brian Woods <brian.woods@xxxxxxx>

Thanks for this and the other acks. I notice though that you skipped
patches 2 and 13: Are there concerns there? Patch 8 still has a
discussion to settle, so I realize you probably wouldn't want to ack
that one yet.

Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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