From: Lars Ellenberg <lars.ellenberg@linbit.com>
To: dm-devel@redhat.com
Subject: Re: [PATCH 1/3] Send KOBJ_ADD event after dm resume ioctl.
Date: Fri, 19 Mar 2010 10:06:56 +0100 [thread overview]
Message-ID: <20100319090656.GR6379@soda.linbit> (raw)
In-Reply-To: <ac3eb2511003190127n1aa2dda5pbd4e0841a46830bc@mail.gmail.com>
On Fri, Mar 19, 2010 at 09:27:35AM +0100, Kay Sievers wrote:
> On Thu, Mar 18, 2010 at 22:35, Milan Broz <mbroz@redhat.com> wrote:
>
> > But if your statement is "it is your broken activation model" as you said
> > in some discussion, I can do nothing, just disagree - it is different model,
> > not broken.
>
> Sure, if you are fine with that model, I'm fine with it too. You are
> the one sending patches to mangle basic driver-core definitions to
> paper-over some issues dm seem to have with it. I just object to such
> core changes, not to the way dm is doing things.
>
> I have no problem with dm creating "dead" devices, it's like this
> since a long time, but please don't try to fake things in the driver
> core to make it look different from what it is. We don't want /sys and
> /dev and events to be out of sync, like non-"add"-ed devices which are
> fully created in /sys, or "remove"-d devices which are still fully
> populated in /sys.
>
> /sys is the direct export of kernel objects, if you create objects,
> they appear, and they get announced. If you don't want them to be
> announced at that time, just don't register them at that time.
>
> Don't get me wrong, I'm not asking you to change the current state how
> dm is doing things, I just object to the patch which inconsistently
> tries to fake events, which do not match the state in /sys.
Would introducing a KOBJ_READY_TO_BE_CONSUMED_BY_UDEV help?
Or re-using KOBJ_ONLINE for that purpose,
even though it has different meaning elsewhere?
Lars
next prev parent reply other threads:[~2010-03-19 9:06 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-18 13:58 [PATCH 1/3] Send KOBJ_ADD event after dm resume ioctl Milan Broz
2010-03-18 13:58 ` [PATCH 2/3] Add genhd flag requesting notification of partition changes only Milan Broz
2010-03-18 13:58 ` [PATCH 3/3] Do not send multiple REMOVE events for kobjects Milan Broz
2010-03-18 16:13 ` [PATCH 1/3] Send KOBJ_ADD event after dm resume ioctl Kay Sievers
2010-03-18 17:24 ` Alasdair G Kergon
2010-03-18 21:35 ` Milan Broz
2010-03-19 8:27 ` Kay Sievers
2010-03-19 9:06 ` Lars Ellenberg [this message]
2010-03-19 9:24 ` Kay Sievers
2010-03-19 9:49 ` Milan Broz
2010-03-19 10:16 ` Kay Sievers
2010-03-19 11:14 ` Milan Broz
2010-03-19 11:44 ` Kay Sievers
2010-03-19 12:08 ` Milan Broz
2010-03-19 12:14 ` Kay Sievers
2010-03-19 11:50 ` Hannes Reinecke
2010-03-19 12:00 ` Alasdair G Kergon
2010-03-19 12:12 ` Alasdair G Kergon
2010-03-19 12:16 ` Kay Sievers
2010-03-19 13:17 ` Peter Rajnoha
2010-03-19 13:24 ` Peter Rajnoha
2010-03-19 13:43 ` Kay Sievers
2010-03-19 13:47 ` Alasdair G Kergon
2010-03-19 13:58 ` David Zeuthen
2010-03-19 14:34 ` Alasdair G Kergon
2010-03-19 14:59 ` David Zeuthen
2010-03-19 15:24 ` Alasdair G Kergon
2010-03-19 16:01 ` David Zeuthen
2010-03-19 16:36 ` Alasdair G Kergon
2010-03-22 10:11 ` Milan Broz
2010-03-19 15:55 ` Mike Snitzer
2010-03-19 16:08 ` David Zeuthen
2010-03-19 14:08 ` Hannes Reinecke
2010-03-19 15:05 ` Peter Rajnoha
2010-03-19 15:14 ` David Zeuthen
2010-03-19 15:51 ` Peter Rajnoha
2010-03-19 9:10 ` Milan Broz
2010-03-19 9:22 ` Kay Sievers
2010-03-19 9:42 ` Hannes Reinecke
2010-03-19 12:27 ` Alasdair G Kergon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100319090656.GR6379@soda.linbit \
--to=lars.ellenberg@linbit.com \
--cc=dm-devel@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.