On Sat, May 16, 2026 at 12:06:14PM +0800, Lance Yang wrote:
> 
> 
> On 2026/5/15 21:13, Breno Leitao wrote:
> [...]
> > > 
> > > Wonder if it would be simpler to just do a positive check near the top
> > > of get_any_page() instead. Something like:
> > > 
> > > static bool hwpoison_unrecoverable_kernel_page(struct page *page,
> > >                                           unsigned long flags)
> > 
> > Ack. We probably want to call it something like HWPoisonKernelOwned() to
> > follow the same naming sematics of these helpers, such as 
> > HWPoisonHandlable()
> > 
> > By the way, I will re-include the self test back to this patch series,
> > In case they are not useful, we do not merge it.
> > 
> 
> Sounds good :)
> 
> Can you also test the relevant page types if possible, especially
> the ones the new helper is supposed to classify?

Ack. I will expand the test to cover different page types as well!

Thanks,
--breno

Reply via email to