Re: (take 2)[PATCH] JBD: positively dispose the unmapped data buffers in journal_commit_transaction

[Toshiyuki Okajima <toshi.okajima@jp.fujitsu.com>]

Hi,

Jan Kara wrote:
>   Hi,
> 
>> I updated my patch and introduction article for it by reflecting
>> the comment of Andrew's.
<SNIP>
>>
>> Signed-off-by: Toshiyuki Okajima <toshi.okajima@jp.fujitsu.com>
>   I agree with the change. It's true that we can leave some anonymous
> pages behind and it's nicer to the MM to release them earlier when we
> know they will be never needed again. The patch looks fine to me, you
> can add
>   Acked-by: Jan Kara <jack@suse.cz>

Thank you for confirming.

>   How much have you stressed the patched kernel? I suggest you use
> fsxlinux and put some memory pressure to the system...

I have stressed it for 72 or more hours.
Stresser does:
  - allocates/frees big memory(1.7GB) which was almost system
    memory size(2GB) repeatedly.
Confirmation of integrity of patched Filesystem(jbd) does:
  - creates files, and copies 3 files from created each file
    (3 copies run concurrently), and confirms whether there is
    no difference between created files and copied files.
  (20 processes runs these works concurrently and repeatedly.)
Above 2 jobs run concurrently.

I haven't ever used 'fsxlinux'.
Next time, I will use it.

Thanks,
---
Toshiyuki Okajima