Re: [PATCH v2 09/19] iommu/vt-d: Enlightened PASID allocation

From: Jacob Pan
Date: Fri Apr 26 2019 - 11:03:09 EST


On Fri, 26 Apr 2019 09:24:29 +0200
Auger Eric <eric.auger@xxxxxxxxxx> wrote:

> > Agreed
> >>> +#define VCMD_VRSP_RESULE(e) (((e) >> 8) &
> >>> 0xfffff)
> >> nit: s/RESULE/RSLT?
> > yes. Also the mask bits should be 8 to 63
> > s/0xfffff/GENMASK_ULL(63, 8))/
> Well the macro definition looks correct as 63:28 is RsvdZ

you are right, I misread the spec.

Thanks,

Jacob