On Wed, Jan 03, 2024 at 04:12:51PM -0800, Darrick J. Wong wrote: > > + if (error) > > + return error; > > + > > + page = folio_file_page(folio, pos >> PAGE_SHIFT); > > + if (PageHWPoison(page)) { > > + folio_put(folio); > > We need to unlock the folio here, right? On the error return? Yes.