Re: [PATCH] mm: Add WARN_ON for possibility of infinite loop if empty lists in free_pcppages_bulk'

From: Pankaj Gupta
Date: Tue Aug 23 2016 - 04:49:11 EST



> > > > While debugging issue in realtime kernel i found a scenario
> > > > which resulted in infinite loop resulting because of empty pcp->lists
> > > > and valid 'to_free' value. This patch is to add 'WARN_ON' in function
> > > > 'free_pcppages_bulk' if there is possibility of infinite loop because
> > > > of any bug in code.
> > > >
> > >
> > > What was the bug that allowed this situation to occur? It would imply
> > > the pcp count was somehow out of sync.
> >
> > Yes pcp count was out of sync. It was a bug in the downstream code.
>
> If the bug is not in the mainline code, I think it would be inappropriate
> to add unnecessary code to a relatively hot path. At most, it should be
> a VM_BUG_ON but the soft lockup should be clear enough.

yes 'VM_BUG_ON' is right thing here. This could help in realtime kernel where
'free_pcppages_bulk' is divided into two functions 'isolate_pcp_pages' and
'free_pcppages_bulk' where 'isolate_pcp_pages' isolate the 'batch/count' number
of pages and 'free_pcppages_bulk' just free these pages.

I was just thinking if there is any possibility of out of sync with count and temporary
list this might help. But looking more at the code does not seems like there is any
chance until any other potential bug somewhere else in code result this scenario.

I will drop this patch.

Thanks for the review.

>
> --
> Mel Gorman
> SUSE Labs
>