Re: Suspend2 2.1.9.8 for 2.6.12: 622-swapwriter.patch

From: Pekka J Enberg
Date: Fri Jul 08 2005 - 00:57:44 EST


On Thu, 2005-07-07 at 23:32, Pekka Enberg wrote:
> > @@ -0,0 +1,817 @@
> > +struct io_info {
> > + struct bio * sys_struct;
> > + long block[PAGE_SIZE/512];

Nigel Cunningham writes:
No... it's the maximum number of blocks per page. Depending upon how
the user has set the blocksize when they created the filesystem (in the
case of filesystems), the number of blocks we use per page might be 1,
2, 4 or 8.

It's long because a sector number is stored in it.

The field block wants a better name. block_sectors, perhaps?

Pekka

-
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/