All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Greg KH <greg@kroah.com>
Cc: kay.sievers@vrfy.org, bugme-daemon@bugzilla.kernel.org,
	linux-kernel@vger.kernel.org, genanr@emsphone.com
Subject: Re: [Bugme-new] [Bug 11323] New: /proc/diskstats does not contain all disk devices
Date: Wed, 13 Aug 2008 17:20:41 -0700	[thread overview]
Message-ID: <20080813172041.9c59167d.akpm@linux-foundation.org> (raw)
In-Reply-To: <20080813235112.GC32154@kroah.com>

On Wed, 13 Aug 2008 16:51:12 -0700
Greg KH <greg@kroah.com> wrote:

> On Wed, Aug 13, 2008 at 01:01:58PM -0700, Andrew Morton wrote:
> > > Problem Description: /proc/diskstats does not contain all the block devices it
> > > should. /sys/block has all the devices, but /proc/diskstats does not.
> > > 
> > > Steps to reproduce: boot a system with >9 (10?) disk devices (24 block
> > > devices?)
> > 
> > The below would be a prime suspect.
> > 
> > Unfortunately a simple revert results in an uncompilable kernel.
> > 
> > 
> > (It drives me up the wall and across the ceiling how the patch has a
> > commit "date" of three months prior to the 2.6.26 release, however it
> > wasn't present in 2.6.26.  What a dumb feature.  How do I make it stop
> > doing this?  gitk kind of gets it right, but isn't useful across DSL)
> 
> $ git show --pretty=fuller 27f302519148f311307637d4c9a6d0fd87d07e4c

<writes a script>

> commit 27f302519148f311307637d4c9a6d0fd87d07e4c
> Author:     Greg Kroah-Hartman <gregkh@suse.de>
> AuthorDate: Thu May 22 17:21:08 2008 -0400
> Commit:     Greg Kroah-Hartman <gregkh@suse.de>
> CommitDate: Mon Jul 21 21:54:49 2008 -0700
> 
> There is a commit date, and the date the patch was written.  Both are
> preserved in git.
> 
> And even if it was committed to a branch before 2.6.26 was released, and
> then pulled in, that's fine, it's distributed development :)

It's useless.  I have never ever ever ever wanted to know when random
person X committed a patch to some local tree.  The overwhelmingly most
common question is "when did that go into Linux".  Sigh.

> $ git describe --contains 27f302519148f311307637d4c9a6d0fd87d07e4c
> v2.6.27-rc1~866^2~40
> 
> showing it first showed up on 2.6.27-rc1.

Spose that works.  My usual recourse is searching the commits list,
which has useful-to-humans ordering information.

Is the date at which it went into mainline recorded?

> Anyway, I don't have any systems with such a large number of devices to
> test with.

I suppose that partitioning a junk disk with lots of little partitions
will show it.  parted wants to go stupid on me though.

>  Running git-bisect should narrow the problem down, you can't
> just revert this patch as later-on patches relied on it, as you found
> out...
> 
> Also, what is the output of these files, what exactly is missing?


      reply	other threads:[~2008-08-14  0:21 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <bug-11323-10286@http.bugzilla.kernel.org/>
2008-08-13 20:01 ` [Bugme-new] [Bug 11323] New: /proc/diskstats does not contain all disk devices Andrew Morton
2008-08-13 23:51   ` Greg KH
2008-08-14  0:20     ` Andrew Morton [this message]

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=20080813172041.9c59167d.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=bugme-daemon@bugzilla.kernel.org \
    --cc=genanr@emsphone.com \
    --cc=greg@kroah.com \
    --cc=kay.sievers@vrfy.org \
    --cc=linux-kernel@vger.kernel.org \
    /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.