Re: [PATCH] rd: Mark ramdisk buffers heads dirty

[ebiederm@xmission.com (Eric W. Biederman)]

Christian Borntraeger <borntraeger@de.ibm.com> writes:

> Am Dienstag, 16. Oktober 2007 schrieb Eric W. Biederman:
>
>> fs/buffer.c |    3 +++
>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>  drivers/block/rd.c |   13 +------------
>>  1 files changed, 1 insertions(+), 12 deletions(-)
>
> Your patches look sane so far. I have applied both patches, and the problem 
> seems gone. I will try to get these patches to our testers.
>
> As long as they dont find new problems:

Sounds good.  Please make certain to test reiserfs as well as ext2+ as
it seems to strain the ramdisk code more aggressively.

> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
>
> Nick, Eric. What is the best patch for the stable series? Both patches from
> Eric or mine? I CCed stable, so they know that something is coming.

My gut feel says my patches assuming everything tests out ok, as
they actually fix the problem instead of papering over it, and there
isn't really a size difference.

In addition the change to init_page_buffers is interesting all by
itself.  With that patch we now have the option of running block
devices in mode where we don't bother to cache the buffer heads
which should reduce memory pressure a bit.  Not that an enhancement
like that will show up in stable, but...

Eric

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>