* [PATCH] ubd: Remove unused mutex 'ubd_mutex'
@ 2024-05-05 0:15 linux
2024-05-06 7:06 ` Christoph Hellwig
0 siblings, 1 reply; 6+ messages in thread
From: linux @ 2024-05-05 0:15 UTC (permalink / raw)
To: richard, hch; +Cc: linux-um, linux-kernel, Dr. David Alan Gilbert
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(-)
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);
--
2.45.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ubd: Remove unused mutex 'ubd_mutex'
2024-05-05 0:15 [PATCH] ubd: Remove unused mutex 'ubd_mutex' linux
@ 2024-05-06 7:06 ` Christoph Hellwig
2024-05-20 15:12 ` Dr. David Alan Gilbert
0 siblings, 1 reply; 6+ messages in thread
From: Christoph Hellwig @ 2024-05-06 7:06 UTC (permalink / raw)
To: linux; +Cc: richard, hch, linux-um, linux-kernel
Looks good:
Reviewed-by: Christoph Hellwig <hch@lst.de>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ubd: Remove unused mutex 'ubd_mutex'
2024-05-06 7:06 ` Christoph Hellwig
@ 2024-05-20 15:12 ` Dr. David Alan Gilbert
2024-05-20 15:20 ` Christoph Hellwig
0 siblings, 1 reply; 6+ messages in thread
From: Dr. David Alan Gilbert @ 2024-05-20 15:12 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: richard, linux-um, linux-kernel
* Christoph Hellwig (hch@lst.de) wrote:
> Looks good:
>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
Thanks Christoph; any ideas who might take this?
Dave
--
-----Open up your eyes, open up your mind, open up your code -------
/ Dr. David Alan Gilbert | Running GNU/Linux | Happy \
\ dave @ treblig.org | | In Hex /
\ _________________________|_____ http://www.treblig.org |_______/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ubd: Remove unused mutex 'ubd_mutex'
2024-05-20 15:12 ` Dr. David Alan Gilbert
@ 2024-05-20 15:20 ` Christoph Hellwig
2024-05-20 19:24 ` Richard Weinberger
0 siblings, 1 reply; 6+ messages in thread
From: Christoph Hellwig @ 2024-05-20 15:20 UTC (permalink / raw)
To: Dr. David Alan Gilbert; +Cc: Christoph Hellwig, richard, linux-um, linux-kernel
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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ubd: Remove unused mutex 'ubd_mutex'
2024-05-20 15:20 ` Christoph Hellwig
@ 2024-05-20 19:24 ` Richard Weinberger
2024-05-23 15:10 ` Dr. David Alan Gilbert
0 siblings, 1 reply; 6+ messages in thread
From: Richard Weinberger @ 2024-05-20 19:24 UTC (permalink / raw)
To: hch; +Cc: Dr. David Alan Gilbert, linux-um, linux-kernel
----- 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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ubd: Remove unused mutex 'ubd_mutex'
2024-05-20 19:24 ` Richard Weinberger
@ 2024-05-23 15:10 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 6+ messages in thread
From: Dr. David Alan Gilbert @ 2024-05-23 15:10 UTC (permalink / raw)
To: Richard Weinberger; +Cc: hch, linux-um, linux-kernel
* 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
>
--
-----Open up your eyes, open up your mind, open up your code -------
/ Dr. David Alan Gilbert | Running GNU/Linux | Happy \
\ dave @ treblig.org | | In Hex /
\ _________________________|_____ http://www.treblig.org |_______/
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-05-23 15:10 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-05 0:15 [PATCH] ubd: Remove unused mutex 'ubd_mutex' linux
2024-05-06 7:06 ` Christoph Hellwig
2024-05-20 15:12 ` Dr. David Alan Gilbert
2024-05-20 15:20 ` Christoph Hellwig
2024-05-20 19:24 ` Richard Weinberger
2024-05-23 15:10 ` Dr. David Alan Gilbert
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox