Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-18 Thread Alexander Duyck
On Mon, Aug 17, 2020 at 8:22 PM Wei Yang wrote: > > On Mon, Aug 17, 2020 at 09:05:32AM -0700, Alexander Duyck wrote: > > > > > >On 8/17/2020 2:35 AM, David Hildenbrand wrote: > >> On 17.08.20 10:48, Wei Yang wrote: > >> > If "page" is the list head, list_for_each_entry_safe() would stop > >> >

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-18 Thread Wei Yang
On Tue, Aug 18, 2020 at 09:23:12AM +0200, David Hildenbrand wrote: >On 18.08.20 05:05, Wei Yang wrote: >> On Mon, Aug 17, 2020 at 07:07:04PM +0200, David Hildenbrand wrote: >>> On 17.08.20 18:05, Alexander Duyck wrote: On 8/17/2020 2:35 AM, David Hildenbrand wrote: > On 17.08.20

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-18 Thread David Hildenbrand
On 18.08.20 05:05, Wei Yang wrote: > On Mon, Aug 17, 2020 at 07:07:04PM +0200, David Hildenbrand wrote: >> On 17.08.20 18:05, Alexander Duyck wrote: >>> >>> >>> On 8/17/2020 2:35 AM, David Hildenbrand wrote: On 17.08.20 10:48, Wei Yang wrote: > If "page" is the list head,

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-17 Thread Wei Yang
On Mon, Aug 17, 2020 at 09:05:32AM -0700, Alexander Duyck wrote: > > >On 8/17/2020 2:35 AM, David Hildenbrand wrote: >> On 17.08.20 10:48, Wei Yang wrote: >> > If "page" is the list head, list_for_each_entry_safe() would stop >> > iteration. >> > >> > Signed-off-by: Wei Yang >> > --- >> >

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-17 Thread Wei Yang
On Mon, Aug 17, 2020 at 07:07:04PM +0200, David Hildenbrand wrote: >On 17.08.20 18:05, Alexander Duyck wrote: >> >> >> On 8/17/2020 2:35 AM, David Hildenbrand wrote: >>> On 17.08.20 10:48, Wei Yang wrote: If "page" is the list head, list_for_each_entry_safe() would stop iteration.

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-17 Thread Wei Yang
On Mon, Aug 17, 2020 at 11:35:29AM +0200, David Hildenbrand wrote: >On 17.08.20 10:48, Wei Yang wrote: >> If "page" is the list head, list_for_each_entry_safe() would stop >> iteration. >> >> Signed-off-by: Wei Yang >> --- >> mm/page_reporting.c | 2 +- >> 1 file changed, 1 insertion(+), 1

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-17 Thread Alexander Duyck
On 8/17/2020 2:35 AM, David Hildenbrand wrote: On 17.08.20 10:48, Wei Yang wrote: If "page" is the list head, list_for_each_entry_safe() would stop iteration. Signed-off-by: Wei Yang --- mm/page_reporting.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-17 Thread David Hildenbrand
On 17.08.20 18:05, Alexander Duyck wrote: > > > On 8/17/2020 2:35 AM, David Hildenbrand wrote: >> On 17.08.20 10:48, Wei Yang wrote: >>> If "page" is the list head, list_for_each_entry_safe() would stop >>> iteration. >>> >>> Signed-off-by: Wei Yang >>> --- >>> mm/page_reporting.c | 2 +- >>>

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-17 Thread David Hildenbrand
On 17.08.20 10:48, Wei Yang wrote: > If "page" is the list head, list_for_each_entry_safe() would stop > iteration. > > Signed-off-by: Wei Yang > --- > mm/page_reporting.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/page_reporting.c b/mm/page_reporting.c > index

[PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-17 Thread Wei Yang
If "page" is the list head, list_for_each_entry_safe() would stop iteration. Signed-off-by: Wei Yang --- mm/page_reporting.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/page_reporting.c b/mm/page_reporting.c index 3bbd471cfc81..3605123d 100644 ---