From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gioh Kim Subject: [PATCH] [RFC] CMA: clear buffer-head lru before page migration Date: Fri, 04 Jul 2014 17:25:09 +0900 Message-ID: <53B664E5.5060102@lge.com> Mime-Version: 1.0 Content-Type: text/plain; charset=EUC-KR Content-Transfer-Encoding: 7bit Cc: =?EUC-KR?B?wMywx8ij?= , Gi-Oh Kim To: Andrew Morton , Laura Abbott , Michal Nazarewicz , Marek Szyprowski , Joonsoo Kim , Alexander Viro , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Return-path: Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org Hi, For page migration of CMA, buffer-heads of lru should be dropped. Please refer to https://lkml.org/lkml/2014/6/23/932 for the detail. I'm attaching a patch to drop all buffer-head on lru with invalidate_bh_lrus. Please look into this. Thanks ------------------------------ 8< --------------------------------- >>From d90cd3d13b73f7278c60d8fe625840664adcbb6a Mon Sep 17 00:00:00 2001 From: Gioh Kim Date: Fri, 4 Jul 2014 16:53:22 +0900 Subject: [PATCH] [RFC] CMA: clear buffer-head lru before page migration When CMA try to migrate page, some buffer-heads can exist on lru. The bh on lru has non-zero count value so that it cannot be dropped even-if it is not used. We can drop only buffers related to the migrated page, but it can take long time more than dropping all because of searching list. There all buffers in lru are dropped. Signed-off-by: Laura Abbott Signed-off-by: Gioh Kim --- fs/buffer.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/fs/buffer.c b/fs/buffer.c index eba6e4f..4f11b7a 100644 --- a/fs/buffer.c +++ b/fs/buffer.c @@ -3233,6 +3233,19 @@ int try_to_free_buffers(struct page *page) if (PageWriteback(page)) return 0; +#ifdef CONFIG_CMA + /* + * When CMA try to migrate page, some buffer-heads can exist on lru. + * The bh on lru has non-zero count value so that it cannot + * be dropped even-if it is not used. + * We can drop only buffers related to the migrated page, + * but it can take long time more than dropping all + * because of searching list. + * There all buffers in lru are dropped first. + */ + invalidate_bh_lrus(); +#endif + if (mapping == NULL) { /* can this still happen? */ ret = drop_buffers(page, &buffers_to_free); goto out; -- 1.7.9.5