Re: [PATCH] proc: fix duplicate line in /proc/devices

From: Neil Horman
Date: Wed Mar 22 2006 - 14:19:34 EST


On Wed, Mar 22, 2006 at 11:12:31AM -0800, Andrew Morton wrote:
> Neil Horman <nhorman@xxxxxxxxxxxxx> wrote:
> >
> > Patch to fix a potential duplicate block device line printed after the "Block
> > device" header in /proc/devices. Tested and confirmed working by the reporter.
> >
>
> <looks>
>
> Block devices:
> 1 ramdisk
> 1 ramdisk
> 2 fd
> 8 sd
>
> >
> > diff --git a/fs/proc/proc_misc.c b/fs/proc/proc_misc.c
> > index 1d24fea..b097958 100644
> > --- a/fs/proc/proc_misc.c
> > +++ b/fs/proc/proc_misc.c
> > @@ -312,7 +312,6 @@ static void *devinfo_next(struct seq_fil
> > case BLK_HDR:
> > info->state = BLK_LIST;
> > (*pos)++;
> > - break;
> > case BLK_LIST:
> > if (get_blkdev_info(info->blkdev,&idummy,&ndummy)) {
> > /*
>
> OK, so that's the same as CHR_HDR and lines up with the fallthroughs in
> devinfo_show(). A somewhat strange way of implementing a state machine,
> but I guess it saves a few cycles...
>
> I'll copy the stable-kernel guys - this is 2.6.16.1 material, thanks.
Fantastic. Thanks Andrew.
Neil

> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

--
/***************************************************
*Neil Horman
*Software Engineer
*gpg keyid: 1024D / 0x92A74FA1 - http://pgp.mit.edu
***************************************************/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/