At 09/28/2012 04:17 AM, KOSAKI Motohiro Wrote: > On Thu, Sep 27, 2012 at 1:45 AM, <we...@cn.fujitsu.com> wrote: >> From: Wen Congyang <we...@cn.fujitsu.com> >> >> hwpoisoned may set when we offline a page by the sysfs interface >> /sys/devices/system/memory/soft_offline_page or >> /sys/devices/system/memory/hard_offline_page. If we don't clear >> this flag when onlining pages, this page can't be freed, and will >> not in free list. So we can't offline these pages again. So we >> should clear this flag when onlining pages. > > This seems wrong fix to me. After offline, memory may or may not > change with new one. Thus we can't assume any memory status. Thus, > we should just forget hwpoison status at _offline_ event. >
Yes, agree with you. I will update this patch. Thanks for reviewing. Wen Congyang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
