lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 13 Aug 2020 11:29:04 +0200 From: Auger Eric <eric.auger@...hat.com> To: Jacob Pan <jacob.jun.pan@...ux.intel.com>, iommu@...ts.linux-foundation.org, LKML <linux-kernel@...r.kernel.org>, Joerg Roedel <joro@...tes.org>, Alex Williamson <alex.williamson@...hat.com> Cc: Lu Baolu <baolu.lu@...ux.intel.com>, David Woodhouse <dwmw2@...radead.org>, Yi Liu <yi.l.liu@...el.com>, "Tian, Kevin" <kevin.tian@...el.com>, Raj Ashok <ashok.raj@...el.com>, Christoph Hellwig <hch@...radead.org>, Jean-Philippe Brucker <jean-philippe@...aro.com>, Jonathan Corbet <corbet@....net> Subject: Re: [PATCH v7 3/7] iommu/uapi: Introduce enum type for PASID data format Hi Jacob, On 7/30/20 2:21 AM, Jacob Pan wrote: > There can be multiple vendor-specific PASID data formats used in UAPI > structures. This patch adds enum type with a last entry which makes > range checking much easier. > > Suggested-by: Alex Williamson <alex.williamson@...hat.com> > Signed-off-by: Jacob Pan <jacob.jun.pan@...ux.intel.com> > --- > include/uapi/linux/iommu.h | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/include/uapi/linux/iommu.h b/include/uapi/linux/iommu.h > index d5e9014f690e..abf4455a3495 100644 > --- a/include/uapi/linux/iommu.h > +++ b/include/uapi/linux/iommu.h > @@ -294,11 +294,16 @@ struct iommu_gpasid_bind_data_vtd { > IOMMU_SVA_VTD_GPASID_PCD | \ > IOMMU_SVA_VTD_GPASID_PWT) > > +enum iommu_pasid_data_format { > + IOMMU_PASID_FORMAT_INTEL_VTD = 1, > + IOMMU_PASID_FORMAT_LAST, > +}; > + > /** > * struct iommu_gpasid_bind_data - Information about device and guest PASID binding > * @argsz: User filled size of this data > * @version: Version of this data structure > - * @format: PASID table entry format > + * @format: PASID table entry format of enum iommu_pasid_data_format type > * @flags: Additional information on guest bind request > * @gpgd: Guest page directory base of the guest mm to bind > * @hpasid: Process address space ID used for the guest mm in host IOMMU > @@ -317,7 +322,6 @@ struct iommu_gpasid_bind_data { > __u32 argsz; > #define IOMMU_GPASID_BIND_VERSION_1 1 > __u32 version; > -#define IOMMU_PASID_FORMAT_INTEL_VTD 1 > __u32 format; > __u32 addr_width; > #define IOMMU_SVA_GPASID_VAL (1 << 0) /* guest PASID valid */ > Reviewed-by: Eric Auger <eric.auger@...hat.com> Eric
Powered by blists - more mailing lists