All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Jones <davej@suse.de>
To: Andries.Brouwer@cwi.nl
Cc: andersg@0x63.nu, linux-kernel@vger.kernel.org,
	whitney@adsl-209-76-109-63.dsl.snfc21.pacbell.net
Subject: Re: /proc/partitions broken in 2.5.23
Date: Fri, 21 Jun 2002 01:34:36 +0200	[thread overview]
Message-ID: <20020621013436.V29373@suse.de> (raw)
In-Reply-To: <UTC200206202321.g5KNLeT22807.aeb@smtp.cwi.nl>; from Andries.Brouwer@cwi.nl on Fri, Jun 21, 2002 at 01:21:40AM +0200

On Fri, Jun 21, 2002 at 01:21:40AM +0200, Andries.Brouwer@cwi.nl wrote:

 >     > I got a bug report about an issue with LVM in 2.5.22-dj1, which turns
 >     > out to be caused by broken /proc/partitions in mainline.
 >     > 
 >     > (davej@mesh:davej)$ cat /proc/partitions 
 >     > major minor  #blocks  name
 >     > 
 >     >    8     0          0 sda
 >     >   22     0 1515870810 hdc
 >     > 
 >     > Note the huge numbers in hex are 0x5a5a5a5a, so something
 >     > seems to be getting poisoned somewhere.
 > 
 > Is this LVM?

No.
 
 > I don't see how LVM could produce such values.
 > (And in fact LVM does not even compile, so only a patched LVM
 > could produce anything at all.)

The original person who reported a problem to me used LVM, and 
in the course of discussion, the proc/partitions bug came to light.
The values pasted above are from a box with no LVM compiled.

 > Normally the nr_real field indicates how many devices are
 > present. But LVM sets that to 256 even when nothing is present.
 > So, indeed, when all size fields are set to 0 this would probably
 > yield a list of 256 absent LVM devices.
 > Maybe LVM has to be fixed, or this patch fragment reverted, or both.

As I mentioned in another mail, it seems that removable devices with
no media have no valid #blocks, and is thus getting poisoned.

        Dave

-- 
| Dave Jones.        http://www.codemonkey.org.uk
| SuSE Labs

  reply	other threads:[~2002-06-20 23:34 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-20 23:21 /proc/partitions broken in 2.5.23 Andries.Brouwer
2002-06-20 23:34 ` Dave Jones [this message]
2002-06-20 23:42 ` Anders Gustafsson
     [not found] <641481775@toto.iv>
2002-06-26  3:22 ` Peter Chubb
  -- strict thread matches above, loose matches on Subject: below --
2002-06-19  9:02 Dave Jones
2002-06-19 11:32 ` Andries Brouwer
2002-06-19 11:44   ` Dave Jones
2002-06-19 12:51     ` Andries Brouwer
2002-06-19 13:16       ` Dave Jones
2002-06-19 17:18   ` Wayne Whitney

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=20020621013436.V29373@suse.de \
    --to=davej@suse.de \
    --cc=Andries.Brouwer@cwi.nl \
    --cc=andersg@0x63.nu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=whitney@adsl-209-76-109-63.dsl.snfc21.pacbell.net \
    /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.