Messages in this thread |  | | Date | Thu, 14 Sep 2000 22:00:39 +0200 | From | Christian Ehrhardt <> | Subject | Re: [PATCH 2.4.0-test8] mm/filemap.c |
| |
On Thu, Sep 14, 2000 at 07:05:20PM +0200, Juan J. Quintela wrote: [ Removed linus from cc, he probably knows this ]
> >>>>> "bill" == Bill Wendling <wendling@ganymede.isdn.uiuc.edu> writes: > Linus, please don't apply. > bill> - The `head = &mapping->pages;' statement is useless inside the ^^^ > bill> repeat, since head isn't modified inside the loop. > No, but we sleep inside the loop, and while we sleep, we don't have > locked the page cache :((((
There is no way that this can affect the statement in question. Note that head is the _address_ of mapping->pages.
regards Christian
-- THAT'S ALL FOLKS! - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
|  |