diff mbox series

ubd: Remove unused mutex 'ubd_mutex'

Message ID 20240505001508.255096-1-linux@treblig.org
State New
Headers show
Series ubd: Remove unused mutex 'ubd_mutex' | expand

Commit Message

Dr. David Alan Gilbert May 5, 2024, 12:15 a.m. UTC 
From: "Dr. David Alan Gilbert" <linux@treblig.org>

Commit fb5d1d389c9e ("ubd: open the backing files in ubd_add")

removed the last use of ubd_mutex.
Remove it.

Build and kernel startup test only.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 arch/um/drivers/ubd_kern.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Christoph Hellwig May 6, 2024, 7:06 a.m. UTC | #1 
Looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>
Dr. David Alan Gilbert May 20, 2024, 3:12 p.m. UTC | #2 
* Christoph Hellwig (hch@lst.de) wrote:
> Looks good:
> 
> Reviewed-by: Christoph Hellwig <hch@lst.de>

Thanks Christoph; any ideas who might take this?

Dave
Christoph Hellwig May 20, 2024, 3:20 p.m. UTC | #3 
On Mon, May 20, 2024 at 03:12:20PM +0000, Dr. David Alan Gilbert wrote:
> * Christoph Hellwig (hch@lst.de) wrote:
> > Looks good:
> > 
> > Reviewed-by: Christoph Hellwig <hch@lst.de>
> 
> Thanks Christoph; any ideas who might take this?

Probably Richard who is alredy on Cc.
Richard Weinberger May 20, 2024, 7:24 p.m. UTC | #4 
----- Ursprüngliche Mail -----
> Von: "hch" <hch@lst.de>
> An: "Dr. David Alan Gilbert" <linux@treblig.org>
> CC: "hch" <hch@lst.de>, "richard" <richard@nod.at>, "linux-um" <linux-um@lists.infradead.org>, "linux-kernel"
> <linux-kernel@vger.kernel.org>
> Gesendet: Montag, 20. Mai 2024 17:20:49
> Betreff: Re: [PATCH] ubd: Remove unused mutex 'ubd_mutex'

> On Mon, May 20, 2024 at 03:12:20PM +0000, Dr. David Alan Gilbert wrote:
>> * Christoph Hellwig (hch@lst.de) wrote:
>> > Looks good:
>> > 
>> > Reviewed-by: Christoph Hellwig <hch@lst.de>
>> 
>> Thanks Christoph; any ideas who might take this?
> 
> Probably Richard who is alredy on Cc.

Yes, after the merge window.

Thanks,
//richard
Dr. David Alan Gilbert May 23, 2024, 3:10 p.m. UTC | #5 
* Richard Weinberger (richard@nod.at) wrote:
> ----- Ursprüngliche Mail -----
> > Von: "hch" <hch@lst.de>
> > An: "Dr. David Alan Gilbert" <linux@treblig.org>
> > CC: "hch" <hch@lst.de>, "richard" <richard@nod.at>, "linux-um" <linux-um@lists.infradead.org>, "linux-kernel"
> > <linux-kernel@vger.kernel.org>
> > Gesendet: Montag, 20. Mai 2024 17:20:49
> > Betreff: Re: [PATCH] ubd: Remove unused mutex 'ubd_mutex'
> 
> > On Mon, May 20, 2024 at 03:12:20PM +0000, Dr. David Alan Gilbert wrote:
> >> * Christoph Hellwig (hch@lst.de) wrote:
> >> > Looks good:
> >> > 
> >> > Reviewed-by: Christoph Hellwig <hch@lst.de>
> >> 
> >> Thanks Christoph; any ideas who might take this?
> > 
> > Probably Richard who is alredy on Cc.
> 
> Yes, after the merge window.

Thanks; no rush, just trying to keep an eye on where each patch is
going.

Dave

> Thanks,
> //richard
>
diff mbox series

Patch

diff --git a/arch/um/drivers/ubd_kern.c b/arch/um/drivers/ubd_kern.c
index 63fc062add708..2b855240fbabf 100644
--- a/arch/um/drivers/ubd_kern.c
+++ b/arch/um/drivers/ubd_kern.c
@@ -106,7 +106,6 @@  static inline void ubd_set_bit(__u64 bit, unsigned char *data)
 #define DRIVER_NAME "uml-blkdev"
 
 static DEFINE_MUTEX(ubd_lock);
-static DEFINE_MUTEX(ubd_mutex); /* replaces BKL, might not be needed */
 
 static int ubd_ioctl(struct block_device *bdev, blk_mode_t mode,
 		     unsigned int cmd, unsigned long arg);