| Message ID | 20241015-ep-msi-v3-1-cedc89a16c1a@nxp.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | PCI: EP: Add RC-to-EP doorbell with platform MSI controller | expand |
On 10/16/24 7:07 AM, Frank Li wrote: > Add a cleanup DEFINE_GUARD macro for msi_lock_descs() and > msi_unlock_descs() to simplify lock and unlock operations in error path. > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > --- > include/linux/msi.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/linux/msi.h b/include/linux/msi.h > index b10093c4d00ea..0b6cb7f303887 100644 > --- a/include/linux/msi.h > +++ b/include/linux/msi.h > @@ -228,6 +228,8 @@ int msi_setup_device_data(struct device *dev); > void msi_lock_descs(struct device *dev); > void msi_unlock_descs(struct device *dev); > > +DEFINE_GUARD(msi_descs, struct device *, msi_lock_descs(_T), msi_unlock_descs(_T)) > + This belongs with patch 3 since it is first used there. > struct msi_desc *msi_domain_first_desc(struct device *dev, unsigned int domid, > enum msi_desc_filter filter);
On Tue, Oct 15 2024 at 18:07, Frank Li wrote: > Add a cleanup DEFINE_GUARD macro for msi_lock_descs() and > msi_unlock_descs() to simplify lock and unlock operations in error > path. What for? Thanks, tglx
On Wed, Oct 16, 2024 at 06:21:50PM +0200, Thomas Gleixner wrote: > On Tue, Oct 15 2024 at 18:07, Frank Li wrote: > > > Add a cleanup DEFINE_GUARD macro for msi_lock_descs() and > > msi_unlock_descs() to simplify lock and unlock operations in error > > path. > > What for? See [PATCH v3 3/6] PCI: endpoint: Add RC-to-EP doorbell support using platform MSI controller scoped_guard(msi_descs, dev) msi_for_each_desc(desc, dev, MSI_DESC_ALL) { ... if (...) goto ... } So cleanup can simplify unlock at error branch. Frank > > Thanks, > > tglx
diff --git a/include/linux/msi.h b/include/linux/msi.h index b10093c4d00ea..0b6cb7f303887 100644 --- a/include/linux/msi.h +++ b/include/linux/msi.h @@ -228,6 +228,8 @@ int msi_setup_device_data(struct device *dev); void msi_lock_descs(struct device *dev); void msi_unlock_descs(struct device *dev); +DEFINE_GUARD(msi_descs, struct device *, msi_lock_descs(_T), msi_unlock_descs(_T)) + struct msi_desc *msi_domain_first_desc(struct device *dev, unsigned int domid, enum msi_desc_filter filter);
Add a cleanup DEFINE_GUARD macro for msi_lock_descs() and msi_unlock_descs() to simplify lock and unlock operations in error path. Signed-off-by: Frank Li <Frank.Li@nxp.com> --- include/linux/msi.h | 2 ++ 1 file changed, 2 insertions(+)