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

From: Bagas Sanjaya

Date: Fri Nov 07 2025 - 00:14:39 EST


On Thu, Nov 06, 2025 at 08:35:41PM -0400, Jason Gunthorpe wrote:
> 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?

OK, thanks!

--
An old man doll... just what I always wanted! - Clara

Attachment: signature.asc
Description: PGP signature