Re: [PATCH] mm: clean up soft_offline_page()

From: Naoya Horiguchi
Date: Fri Dec 14 2012 - 10:38:02 EST


Hi Xishi,

On Fri, Dec 14, 2012 at 11:06:56AM +0800, Xishi Qiu wrote:
> On 2012/12/14 7:01, Naoya Horiguchi wrote:
...
> > + ret = get_any_page(page, pfn, flags);
> > + if (ret < 0)
> > + return ret;
> > + if (ret) { /* for in-use pages */
> > + if (PageHuge(page))
> > + soft_offline_huge_page(page, flags);
>
> ret = soft_offline_huge_page(page, flags);
>
> > + else
> > + __soft_offline_page(page, flags);
>
> ret = __soft_offline_page(page, flags); right?

Ah, you're right, Thanks!

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