Questions - Re: [PATCH] documentation for mm.h

From: Anton Altaparmakov (aia21@cam.ac.uk)
Date: Thu Mar 08 2001 - 05:11:50 EST


At 22:33 07/03/2001, Rik van Riel wrote:
[snip]
> typedef struct page {
>+ struct list_head list; /* ->mapping has some page lists. */
>+ struct address_space *mapping; /* The inode (or ...) we belong to. */
>+ unsigned long index; /* Our offset within mapping. */

Assuming index is in bytes (it looks like it is): Shouldn't index of type
unsigned long long or __u64? Otherwise, AFAICS using the page cache
automatically results in an artificial 4Gib limit on file size, which is
not very good, even by todays standards.

[snip]
>+ * During disk I/O, PG_locked is used. This bit is set before I/O
>+ * and reset when I/O completes. page->wait is a wait queue of all
>+ * tasks waiting for the I/O on this page to complete.

Is this physical I/O only or does it include a driver writing/reading the page?

Thanks,

         Anton

-- 
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Linux NTFS Maintainer / WWW: http://sourceforge.net/projects/linux-ntfs/
ICQ: 8561279 / WWW: http://www-stu.christs.cam.ac.uk/~aia21/

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



This archive was generated by hypermail 2b29 : Thu Mar 15 2001 - 21:00:07 EST