public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* 2.4.20-pre4 + LVM = hosed /proc/partitions
@ 2002-08-21  2:27 Erik Andersen
  2002-08-21  3:04 ` Erik Andersen
  0 siblings, 1 reply; 6+ messages in thread
From: Erik Andersen @ 2002-08-21  2:27 UTC (permalink / raw)
  To: linux-kernel

Try compiling CONFIG_BLK_DEV_LVM into 2.4.20-pre4 and then run
'cat /proc/partitions' for some amusement. I really like the way
it prints /dev/lvmX, where X is first a-z, and then proceeds to
apparently increment X another 229 times...

    $ cat /proc/partitions | wc -l
      259

And /proc/partitions now contains only LVM entries...

 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 2.4.20-pre4 + LVM = hosed /proc/partitions
  2002-08-21  2:27 2.4.20-pre4 + LVM = hosed /proc/partitions Erik Andersen
@ 2002-08-21  3:04 ` Erik Andersen
  2002-08-21  3:21   ` Jeff Chua
  2002-08-21 13:45   ` Alan Cox
  0 siblings, 2 replies; 6+ messages in thread
From: Erik Andersen @ 2002-08-21  3:04 UTC (permalink / raw)
  To: linux-kernel

On Tue Aug 20, 2002 at 08:27:32PM -0600, Erik wrote:
> Try compiling CONFIG_BLK_DEV_LVM into 2.4.20-pre4 and then run
> 'cat /proc/partitions' for some amusement. I really like the way

It also seems to occur for md and ataraid.

 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 2.4.20-pre4 + LVM = hosed /proc/partitions
  2002-08-21  3:04 ` Erik Andersen
@ 2002-08-21  3:21   ` Jeff Chua
  2002-08-21  3:26     ` Erik Andersen
  2002-08-21 13:45   ` Alan Cox
  1 sibling, 1 reply; 6+ messages in thread
From: Jeff Chua @ 2002-08-21  3:21 UTC (permalink / raw)
  To: Erik Andersen; +Cc: linux-kernel


On Tue, 20 Aug 2002, Erik Andersen wrote:

> On Tue Aug 20, 2002 at 08:27:32PM -0600, Erik wrote:
> > Try compiling CONFIG_BLK_DEV_LVM into 2.4.20-pre4 and then run
> > 'cat /proc/partitions' for some amusement. I really like the way
>
> It also seems to occur for md and ataraid.

You should consider switching to LVM2. LVM2 doesn't have such problem, and
it seems more stable too.

Jeff



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 2.4.20-pre4 + LVM = hosed /proc/partitions
  2002-08-21  3:21   ` Jeff Chua
@ 2002-08-21  3:26     ` Erik Andersen
  0 siblings, 0 replies; 6+ messages in thread
From: Erik Andersen @ 2002-08-21  3:26 UTC (permalink / raw)
  To: Jeff Chua; +Cc: linux-kernel

On Wed Aug 21, 2002 at 11:21:04AM +0800, Jeff Chua wrote:
> 
> On Tue, 20 Aug 2002, Erik Andersen wrote:
> 
> > On Tue Aug 20, 2002 at 08:27:32PM -0600, Erik wrote:
> > > Try compiling CONFIG_BLK_DEV_LVM into 2.4.20-pre4 and then run
> > > 'cat /proc/partitions' for some amusement. I really like the way
> >
> > It also seems to occur for md and ataraid.
> 
> You should consider switching to LVM2. LVM2 doesn't have such problem, and
> it seems more stable too.

I'm not actually using it.  Just compiled it in and saw wierd
stuff happening...

 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 2.4.20-pre4 + LVM = hosed /proc/partitions
  2002-08-21  3:04 ` Erik Andersen
  2002-08-21  3:21   ` Jeff Chua
@ 2002-08-21 13:45   ` Alan Cox
  2002-08-21 14:06     ` [PATCH] 2.4.20-pre[234] hosed /proc/partitions fix Erik Andersen
  1 sibling, 1 reply; 6+ messages in thread
From: Alan Cox @ 2002-08-21 13:45 UTC (permalink / raw)
  To: andersen; +Cc: linux-kernel

On Wed, 2002-08-21 at 04:04, Erik Andersen wrote:
> On Tue Aug 20, 2002 at 08:27:32PM -0600, Erik wrote:
> > Try compiling CONFIG_BLK_DEV_LVM into 2.4.20-pre4 and then run
> > 'cat /proc/partitions' for some amusement. I really like the way
> 
> It also seems to occur for md and ataraid.

Fixed in -ac for a while - not had time to do another sync with Marcelo
though


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] 2.4.20-pre[234] hosed /proc/partitions fix
  2002-08-21 13:45   ` Alan Cox
@ 2002-08-21 14:06     ` Erik Andersen
  0 siblings, 0 replies; 6+ messages in thread
From: Erik Andersen @ 2002-08-21 14:06 UTC (permalink / raw)
  To: Alan Cox, Marcelo Tosatti; +Cc: linux-kernel

On Wed Aug 21, 2002 at 02:45:06PM +0100, Alan Cox wrote:
> On Wed, 2002-08-21 at 04:04, Erik Andersen wrote:
> > On Tue Aug 20, 2002 at 08:27:32PM -0600, Erik wrote:
> > > Try compiling CONFIG_BLK_DEV_LVM into 2.4.20-pre4 and then run
> > > 'cat /proc/partitions' for some amusement. I really like the way
> > 
> > It also seems to occur for md and ataraid.
> 
> Fixed in -ac for a while - not had time to do another sync with Marcelo
> though

Sigh.  I wished I'd known an hour ago, since it would have
saved me the bother of figuring it out myself.

Marcello,
Please apply the following patch to 2.4.20-pre5.  The
patch that went into -pre2 had a nasty bug in it that
causes /proc/partitions to behave very badly.  This
patch restores it to sanity.

 
--- drivers/block/genhd.c.orig	Wed Aug 21 07:51:21 2002
+++ drivers/block/genhd.c	Wed Aug 21 08:03:48 2002
@@ -194,9 +194,7 @@
 
 	/* show the full disk and all non-0 size partitions of it */
 	for (n = 0; n < (gp->nr_real << gp->minor_shift); n++) {
-		int mask = (1<<gp->minor_shift) - 1;
-
-		if (!(n & mask) || gp->part[n].nr_sects) {
+		if (gp->part[n].nr_sects) {
 #ifdef CONFIG_BLK_STATS
 			struct hd_struct *hd = &gp->part[n];
 

 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2002-08-21 14:02 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-08-21  2:27 2.4.20-pre4 + LVM = hosed /proc/partitions Erik Andersen
2002-08-21  3:04 ` Erik Andersen
2002-08-21  3:21   ` Jeff Chua
2002-08-21  3:26     ` Erik Andersen
2002-08-21 13:45   ` Alan Cox
2002-08-21 14:06     ` [PATCH] 2.4.20-pre[234] hosed /proc/partitions fix Erik Andersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox