All of lore.kernel.org
 help / color / mirror / Atom feed
From: Phillip Susi <psusi@cfl.rr.com>
To: dm-devel@redhat.com, ataraid-list@redhat.com, linux-lvm@redhat.com
Subject: Re: Re: LVM on dmraid breakage
Date: Fri, 03 Aug 2007 17:30:00 -0400	[thread overview]
Message-ID: <46B39E58.2040305@cfl.rr.com> (raw)
In-Reply-To: <20070803195749.GA27271@percy.comedia.it>

Luca Berra wrote:
> On Fri, Aug 03, 2007 at 02:10:02PM -0400, Phillip Susi wrote:
>> I agree with moving the partition detection code to user space, but 
>> trying to undo it after the fact doesn't help because udev is already 
>> processing the add events.  Also you do not need to remove the 
>> partitions so long as pvscan understands that it shouldn't be using them.
> which is the modification i proposed to lvm tools, isn't it?

You suggested deleting the partition table after it has already been 
detected.  I am saying that while I agree in principal that partition 
detection should be moved out of the kernel, for now, it is in there and 
deleting them after they have already been detected doesn't help matters 
because pvscan may already be running on them.

>> Udev is supposed to be the new model for enumerating devices and 
> i know that, and i will withdraw from this discussion, since it might
> get to an useless flame war.
> 
> Is there any technical reason for not having lvm tools filter out 
> devices that
> are used by device mapper?
> 
> besides dmraid, think of multipath.

None that I can see at the moment, but that doesn't mean there isn't 
one, or won't be one in the future.  The other problem is that there are 
likely other factors besides being used already as a dm target that 
might give reason for lvm to not scan the volume.  These kind of policy 
decisions seem like they should be made by udev rather than hard coded 
into lvm.  If the admin wants a policy where lvm should look at volumes, 
or indeed, maybe only certain volumes, that already happen to be dm 
targets, he should be able to do that.  Likewise, there may be some 
other reason to not look at a disk for lvm pvs.  Editing a conf file to 
specify a filter list of devices by name is all well and good for a 
static system, but it does not play well in the modern udev managed plug 
and play world.

  reply	other threads:[~2007-08-03 21:30 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-01 20:19 LVM on dmraid breakage Phillip Susi
2007-08-01 20:29 ` Jonathan Brassow
2007-08-01 21:19   ` Phillip Susi
2007-08-01 21:44     ` Jonathan Brassow
2007-08-01 22:12       ` Chandra Seetharaman
2007-08-02  6:50 ` Luca Berra
2007-08-02  6:50   ` [linux-lvm] " Luca Berra
2007-08-02 10:45   ` Bryn M. Reeves
2007-08-02 10:45     ` [linux-lvm] " Bryn M. Reeves
2007-08-02 21:50     ` Phillip Susi
2007-08-03  0:40       ` David Robinson
2007-08-03  0:40         ` [linux-lvm] Re: [dm-devel] " David Robinson
2007-08-07 21:37         ` Alasdair G Kergon
2007-08-07 21:37           ` [linux-lvm] " Alasdair G Kergon
2007-08-08 21:00           ` Phillip Susi
2007-08-07 21:03       ` Alasdair G Kergon
2007-08-07 21:03         ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-08 21:04         ` Phillip Susi
2007-08-08 21:45           ` Alasdair G Kergon
2007-08-08 21:45             ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-09 17:27             ` Phillip Susi
2007-08-10  5:09               ` Luca Berra
2007-08-10  5:09                 ` [linux-lvm] Re: [dm-devel] " Luca Berra
2007-08-10 13:49               ` Alasdair G Kergon
2007-08-10 13:49                 ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-10 20:15                 ` Phillip Susi
2007-08-03  7:55     ` [linux-lvm] " Luca Berra
2007-08-02 22:04   ` Phillip Susi
2007-08-03  8:11     ` Luca Berra
2007-08-03  8:11       ` [linux-lvm] Re: [dm-devel] " Luca Berra
2007-08-03 18:10       ` Phillip Susi
2007-08-03 19:57         ` Luca Berra
2007-08-03 19:57           ` [linux-lvm] Re: [dm-devel] " Luca Berra
2007-08-03 21:30           ` Phillip Susi [this message]
2007-08-04  6:50             ` Luca Berra
2007-08-04  6:50               ` [linux-lvm] " Luca Berra
2007-08-06 14:45               ` Phillip Susi
2007-08-07  9:17                 ` Luca Berra
2007-08-07  9:17                   ` [linux-lvm] Re: [dm-devel] " Luca Berra
2007-08-07 15:58                   ` Phillip Susi
2007-08-07 22:01                   ` Alasdair G Kergon
2007-08-07 22:01                     ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-08 21:06                     ` Phillip Susi
2007-08-07 21:53             ` Alasdair G Kergon
2007-08-07 21:53               ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-07 21:50         ` Alasdair G Kergon
2007-08-07 21:50           ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-07 21:40       ` Alasdair G Kergon
2007-08-07 21:40         ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-07 21:28     ` Alasdair G Kergon
2007-08-07 21:28       ` [linux-lvm] Re: [dm-devel] " Alasdair G Kergon
2007-08-08 21:13       ` Phillip Susi
2007-08-07 21:02   ` Alasdair G Kergon
2007-08-07 21:02     ` [linux-lvm] " Alasdair G Kergon
2007-08-07 20:52 ` 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=46B39E58.2040305@cfl.rr.com \
    --to=psusi@cfl.rr.com \
    --cc=ataraid-list@redhat.com \
    --cc=dm-devel@redhat.com \
    --cc=linux-lvm@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.