On Tue, Jun 15, 2021 at 07:41:34PM +0800, Ding Hui wrote:
> On 2021/6/14 10:12, Naoya Horiguchi wrote:
>
> > @@ -2171,6 +2177,8 @@ int soft_offline_page(unsigned long pfn, int flags)
> > return -EIO;
> > }
> > + mutex_lock(&mf_mutex);
> > +
> > if (PageHWPoison(page)) {
> > pr_info("%s: %#lx page already poisoned\n", __func__, pfn);
> > put_ref_page(ref_page);
>
> Did you miss mutex_unlock() here when page already poisoned ?
Thanks, you're right. I'll fix it.
- Naoya
>
> > @@ -2194,5 +2202,7 @@ int soft_offline_page(unsigned long pfn, int flags)
> > __func__, pfn, page->flags, &page->flags);
> > }
> > + mutex_unlock(&mf_mutex);
> > +
> > return ret;
> > }
> >
>
> --
> Thanks,
> - Ding Hui
>
