[lkml]   [2001]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: Re: BUG: cat /proc/partitions endless loop
Christoph Hellwig <> schrieb am 28.09.01:
> In article <> you wrote:
> > I traced the problem down to drivers/block/genhd.c,
> > where the function get_partition_list() outer loop does not
> > terminate due to the last element in the structured list starting
> > with gendisk_head is not initialized to NULL, by whatever reason.
> > My fix does not cure the pointered endless loop, but prevents
> > from looping when stepping thru the pointered list.
> I think the fix could be simpler. What about:

> + for (gp = gendisk_head; gp != gendisk_head; gp = gp->next) {

This will break your for loop immedeately, because the loop criteria
is already violated by the initialization !

But I tried another solution:
for (gp = gendisk_head; gp && (gp->next != gendisk_head); gp = gp->next) {

with no success - the cat /proc/partition only printed the heading line.
This proofed to me, that the pointered list is created the wrong way,
in other words, gendisk_head->next is a pointer to itself.
It looks to me, that the "next" field in
/*static*/ struct gendisk *gendisk_head;
is not initialized (by the compiler ?) correctly to NULL.

> if (gp->part[n].nr_sects == 0)
> continue;

1.000.000 DM gewinnen - kostenlos tippen -, 8MB Speicher, Verschluesselung -

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:03    [W:0.024 / U:0.876 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site