Re: [PATCH v10 3/6] iio: core: Add new DMABUF interface infrastructure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Markus,

Le lundi 17 juin 2024 à 08:56 +0200, Markus Elfring a écrit :
>
> > +++ b/drivers/iio/industrialio-buffer.c
>
> >  static int iio_buffer_chrdev_release(struct inode *inode, struct
> > file *filep)
> >  {
>
> >  	wake_up(&buffer->pollq);
> > 
> > +	mutex_lock(&buffer->dmabufs_mutex);
> > +
> > +	/* Close all attached DMABUFs */
>
> > +	mutex_unlock(&buffer->dmabufs_mutex);
> > +
> >  	kfree(ib);
>
> 
> Would you become interested to apply a statement like
> “guard(mutex)(&buffer->dmabufs_mutex);”?
> https://elixir.bootlin.com/linux/v6.10-rc3/source/include/linux/mutex.h#L196

I'll update the patch to use it.

Cheers,
-Paul





[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux