* [PATCH] Just inform and dont warn when DM_DEV_REMOVE is tried on a open device
@ 2010-04-20 7:16 Nikanth Karthikesan
2010-04-20 8:00 ` Milan Broz
0 siblings, 1 reply; 3+ messages in thread
From: Nikanth Karthikesan @ 2010-04-20 7:16 UTC (permalink / raw)
To: Alasdair G Kergon, device-mapper development, Stefan Assmann
Dont warn when DM_DEV_REMOVE ioctl is tried on a open device.
On openSUSE when cryptsetup and udev interact, cryptsetup sometimes tries to
remove a dm device currently opened by blkid, which is not a serious issue to
print a kernel warning. The application should retry if -EBUSY is returned. So
let us make it an informational message instead.
Signed-off-by: Nikanth Karthikesan <knikanth@suse.de>
---
diff --git a/drivers/md/dm-ioctl.c b/drivers/md/dm-ioctl.c
index d7500e1..dcfa48a 100644
--- a/drivers/md/dm-ioctl.c
+++ b/drivers/md/dm-ioctl.c
@@ -729,7 +729,7 @@ static int dev_remove(struct dm_ioctl *param, size_t param_size)
*/
r = dm_lock_for_deletion(md);
if (r) {
- DMWARN("unable to remove open device %s", hc->name);
+ DMINFO("unable to remove open device %s", hc->name);
up_write(&_hash_lock);
dm_put(md);
return r;
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] Just inform and dont warn when DM_DEV_REMOVE is tried on a open device
2010-04-20 7:16 [PATCH] Just inform and dont warn when DM_DEV_REMOVE is tried on a open device Nikanth Karthikesan
@ 2010-04-20 8:00 ` Milan Broz
2010-04-20 8:43 ` Nikanth Karthikesan
0 siblings, 1 reply; 3+ messages in thread
From: Milan Broz @ 2010-04-20 8:00 UTC (permalink / raw)
To: device-mapper development
Cc: Nikanth Karthikesan, Alasdair G Kergon, Stefan Assmann
On 04/20/2010 09:16 AM, Nikanth Karthikesan wrote:
> Dont warn when DM_DEV_REMOVE ioctl is tried on a open device.
>
> On openSUSE when cryptsetup and udev interact, cryptsetup sometimes tries to
> remove a dm device currently opened by blkid, which is not a serious issue to
> print a kernel warning. The application should retry if -EBUSY is returned. So
> let us make it an informational message instead.
> - DMWARN("unable to remove open device %s", hc->name);
> + DMINFO("unable to remove open device %s", hc->name);
NAK.
Firstly, it is security problem if someone from udev rules tries to open
temporary-cryptsetup device, it contains only private key material.
Secondly, it must be solved on application level and not paper kernel using
patches which hides errors.
FYI there is already patch in upstream cryptsetup which uses udev cookie mechanism.
It still cannot solve artificial change events (which causes scan from rules randomly),
but should help here. (will be in cryptsetup 1.1.1)
Milan
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] Just inform and dont warn when DM_DEV_REMOVE is tried on a open device
2010-04-20 8:00 ` Milan Broz
@ 2010-04-20 8:43 ` Nikanth Karthikesan
0 siblings, 0 replies; 3+ messages in thread
From: Nikanth Karthikesan @ 2010-04-20 8:43 UTC (permalink / raw)
To: Milan Broz; +Cc: device-mapper development, Alasdair G Kergon, Stefan Assmann
On Tuesday 20 April 2010 13:30:18 Milan Broz wrote:
> On 04/20/2010 09:16 AM, Nikanth Karthikesan wrote:
> > Dont warn when DM_DEV_REMOVE ioctl is tried on a open device.
> >
> > On openSUSE when cryptsetup and udev interact, cryptsetup sometimes tries
> > to remove a dm device currently opened by blkid, which is not a serious
> > issue to print a kernel warning. The application should retry if -EBUSY
> > is returned. So let us make it an informational message instead.
> >
> > - DMWARN("unable to remove open device %s", hc->name);
> > + DMINFO("unable to remove open device %s", hc->name);
>
> NAK.
>
> Firstly, it is security problem if someone from udev rules tries to open
> temporary-cryptsetup device, it contains only private key material.
>
Okay.
> Secondly, it must be solved on application level and not paper kernel using
> patches which hides errors.
>
Agreed, it must be solved in the application level.
/me wonders whether this is an error worth a warning from kernel? Returning
-EBUSY seems enough. May be a KERN_INFO message... But yes rest of DM prints
warnings for these kind of events.
> FYI there is already patch in upstream cryptsetup which uses udev cookie
> mechanism. It still cannot solve artificial change events (which causes
> scan from rules randomly), but should help here. (will be in cryptsetup
> 1.1.1)
>
Oh, ok. Thanks for the pointer.
Thanks
Nikanth
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-04-20 8:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-04-20 7:16 [PATCH] Just inform and dont warn when DM_DEV_REMOVE is tried on a open device Nikanth Karthikesan
2010-04-20 8:00 ` Milan Broz
2010-04-20 8:43 ` Nikanth Karthikesan
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.