Messages in this thread |  | | | Date | Wed, 11 Jun 2008 05:19:26 +0200 | | From | Nick Piggin <> | | Subject | Re: [patch 3/7] mm: speculative page references |
On Tue, Jun 10, 2008 at 12:08:27PM -0700, Christoph Lameter wrote:
> On Thu, 5 Jun 2008, npiggin@suse.de wrote:
>
> > + * do the right thing (see comments above).
> > + */
> > + return 0;
> > + }
> > +#endif
> > + VM_BUG_ON(PageCompound(page) && (struct page *)page_private(page) != page);
>
> This is easier written as:
>
> == VM_BUG_ON(PageTail(page)
Yeah that would be nicer.
> And its also slightly incorrect since page_private(page) is not pointing
> to the head page for PageHead(page).
I see. Thanks.
|  |