Re: [PATCH v3 01/13] iommu: Change type of pasid to unsigned int

From: Fenghua Yu
Date: Mon Jun 22 2020 - 14:09:28 EST


Hi, Christoph,

On Thu, Jun 18, 2020 at 12:12:06AM -0700, Christoph Hellwig wrote:
> On Wed, Jun 17, 2020 at 11:23:41AM -0700, Fenghua Yu wrote:
> > PASID is defined as a few different types in iommu including "int",
> > "u32", and "unsigned int". To be consistent and to match with ioasid's
> > type, define PASID and its variations (e.g. max PASID) as "unsigned int".
>
> Wouldn't u32 be a little more explicit (besides being shorter and
> matching the uapi definitions)?

u32 is better than unsigned int. I updated patch 1 to use u32.

Does the updated patch 1 look good for you?

Thank you for your review!