Re: [PATCH 01/15] mm: poison struct page for ptlock

From: Hugh Dickins
Date: Wed Nov 09 2005 - 21:23:52 EST


On Wed, 9 Nov 2005, Andrew Morton wrote:
>
> It does everything we want.

I don't think so: the union leaves us just as vulnerable to some
subsystem using fields of the other half of the union, doesn't it?

Which is not really a problem, but is a part of what's worrying you.

> Of course, it would be nice to retain 2.95.x support. The reviled
> page_private(() would help us do that. But the now-to-be-reviled
> page_mapping() does extraneous stuff, and we'd need a ton of page_lru()'s.
>
> So it'd be a big patch, converting page->lru to page->u.s.lru in lots of
> places.
>
> But I think either a big patch or 2.95.x abandonment is preferable to this
> approach.

Hmm, that's a pity.

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