Re: [PATCH 2/2] iommupt: Describe @bitnr parameter

From: Jason Gunthorpe

Date: Thu Nov 06 2025 - 19:35:43 EST


On Thu, Nov 06, 2025 at 03:48:10PM -0800, Randy Dunlap wrote:
> > diff --git a/drivers/iommu/generic_pt/pt_common.h b/drivers/iommu/generic_pt/pt_common.h
> > index b5628f47e0db40..54c16355be2842 100644
> > --- a/drivers/iommu/generic_pt/pt_common.h
> > +++ b/drivers/iommu/generic_pt/pt_common.h
> > @@ -354,6 +354,7 @@ static inline unsigned int pt_max_sw_bit(struct pt_common *common);
> > /**
> > * pt_test_sw_bit_acquire() - Read a software bit in an item
> > * @pts: Entry to set
> > + * @bitnr: Bit to set
>
> Shouldn't both of these (above) to "to read" instead of "to set"?

Yes, that's right, Bagas could you fold that into a v2?

Jason