public inbox for linux-kernel@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox