diff for duplicates of <20200205133442.GC8965@MiWiFi-R3L-srv> diff --git a/a/1.txt b/N1/1.txt index 9585aad..57f796c 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -31,7 +31,7 @@ On 02/05/20 at 02:20pm, David Hildenbrand wrote: > >>>> + zone->zone_start_pfn = 0; > >>>> + zone->spanned_pages = 0; > >>>> } -> >>>> } else if (zone_end_pfn = end_pfn) { +> >>>> } else if (zone_end_pfn == end_pfn) { > >>>> /* > >>>> @@ -405,34 +408,11 @@ static void shrink_zone_span(struct zone *zone, unsigned long start_pfn, > >>>> start_pfn); @@ -48,8 +48,8 @@ On 02/05/20 at 02:20pm, David Hildenbrand wrote: > > > > Yeah, I also think it's unlikely to come here. > > -> > The 'if (zone_start_pfn = start_pfn)' checking also covers the case -> > (zone_start_pfn = start_pfn && zone_end_pfn = end_pfn). So this +> > The 'if (zone_start_pfn == start_pfn)' checking also covers the case +> > (zone_start_pfn == start_pfn && zone_end_pfn == end_pfn). So this > > zone_start_pfn/spanned_pages resetting can be removed to avoid > > confusion. > diff --git a/a/content_digest b/N1/content_digest index e8129d4..594df94 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -6,22 +6,22 @@ "ref\0cd353848-301a-025d-dd66-44d76e1bbc44@redhat.com\0" "From\0Baoquan He <bhe@redhat.com>\0" "Subject\0Re: [PATCH v6 08/10] mm/memory_hotplug: Don't check for \"all holes\" in shrink_zone_span()\0" - "Date\0Wed, 05 Feb 2020 13:34:42 +0000\0" + "Date\0Wed, 5 Feb 2020 21:34:42 +0800\0" "To\0David Hildenbrand <david@redhat.com>\0" - "Cc\0linux-s390@vger.kernel.org" - Michal Hocko <mhocko@suse.com> + "Cc\0linux-kernel@vger.kernel.org" + linux-mm@kvack.org + linux-arm-kernel@lists.infradead.org linux-ia64@vger.kernel.org - Pavel Tatashin <pasha.tatashin@soleen.com> + linuxppc-dev@lists.ozlabs.org + linux-s390@vger.kernel.org linux-sh@vger.kernel.org x86@kernel.org - linux-kernel@vger.kernel.org - linux-mm@kvack.org - Wei Yang <richardw.yang@linux.intel.com> Andrew Morton <akpm@linux-foundation.org> - linuxppc-dev@lists.ozlabs.org + Oscar Salvador <osalvador@suse.de> + Michal Hocko <mhocko@suse.com> + Pavel Tatashin <pasha.tatashin@soleen.com> Dan Williams <dan.j.williams@intel.com> - linux-arm-kernel@lists.infradead.org - " Oscar Salvador <osalvador@suse.de>\0" + " Wei Yang <richardw.yang@linux.intel.com>\0" "\00:1\0" "b\0" "On 02/05/20 at 02:20pm, David Hildenbrand wrote:\n" @@ -57,7 +57,7 @@ "> >>>> +\t\t\tzone->zone_start_pfn = 0;\n" "> >>>> +\t\t\tzone->spanned_pages = 0;\n" "> >>>> \t\t}\n" - "> >>>> \t} else if (zone_end_pfn = end_pfn) {\n" + "> >>>> \t} else if (zone_end_pfn == end_pfn) {\n" "> >>>> \t\t/*\n" "> >>>> @@ -405,34 +408,11 @@ static void shrink_zone_span(struct zone *zone, unsigned long start_pfn,\n" "> >>>> \t\t\t\t\t start_pfn);\n" @@ -74,8 +74,8 @@ "> > \n" "> > Yeah, I also think it's unlikely to come here.\n" "> > \n" - "> > The 'if (zone_start_pfn = start_pfn)' checking also covers the case\n" - "> > (zone_start_pfn = start_pfn && zone_end_pfn = end_pfn). So this\n" + "> > The 'if (zone_start_pfn == start_pfn)' checking also covers the case\n" + "> > (zone_start_pfn == start_pfn && zone_end_pfn == end_pfn). So this\n" "> > zone_start_pfn/spanned_pages resetting can be removed to avoid\n" "> > confusion.\n" "> \n" @@ -93,4 +93,4 @@ "Yeah, noticed you told Wei the status in another patch thread, I am fine\n" with it, just leave it to you to decide. Thanks. -0ba01882a29df879bcf3b9e972af65f30307697104486b95e79828cab43f099e +097ba9ac7cbf6185f609d1d59df660156ec1a8b8e859b8db57d9d35617a159f0
diff --git a/a/1.txt b/N2/1.txt index 9585aad..57f796c 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -31,7 +31,7 @@ On 02/05/20 at 02:20pm, David Hildenbrand wrote: > >>>> + zone->zone_start_pfn = 0; > >>>> + zone->spanned_pages = 0; > >>>> } -> >>>> } else if (zone_end_pfn = end_pfn) { +> >>>> } else if (zone_end_pfn == end_pfn) { > >>>> /* > >>>> @@ -405,34 +408,11 @@ static void shrink_zone_span(struct zone *zone, unsigned long start_pfn, > >>>> start_pfn); @@ -48,8 +48,8 @@ On 02/05/20 at 02:20pm, David Hildenbrand wrote: > > > > Yeah, I also think it's unlikely to come here. > > -> > The 'if (zone_start_pfn = start_pfn)' checking also covers the case -> > (zone_start_pfn = start_pfn && zone_end_pfn = end_pfn). So this +> > The 'if (zone_start_pfn == start_pfn)' checking also covers the case +> > (zone_start_pfn == start_pfn && zone_end_pfn == end_pfn). So this > > zone_start_pfn/spanned_pages resetting can be removed to avoid > > confusion. > diff --git a/a/content_digest b/N2/content_digest index e8129d4..58cb921 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -6,7 +6,7 @@ "ref\0cd353848-301a-025d-dd66-44d76e1bbc44@redhat.com\0" "From\0Baoquan He <bhe@redhat.com>\0" "Subject\0Re: [PATCH v6 08/10] mm/memory_hotplug: Don't check for \"all holes\" in shrink_zone_span()\0" - "Date\0Wed, 05 Feb 2020 13:34:42 +0000\0" + "Date\0Wed, 5 Feb 2020 21:34:42 +0800\0" "To\0David Hildenbrand <david@redhat.com>\0" "Cc\0linux-s390@vger.kernel.org" Michal Hocko <mhocko@suse.com> @@ -57,7 +57,7 @@ "> >>>> +\t\t\tzone->zone_start_pfn = 0;\n" "> >>>> +\t\t\tzone->spanned_pages = 0;\n" "> >>>> \t\t}\n" - "> >>>> \t} else if (zone_end_pfn = end_pfn) {\n" + "> >>>> \t} else if (zone_end_pfn == end_pfn) {\n" "> >>>> \t\t/*\n" "> >>>> @@ -405,34 +408,11 @@ static void shrink_zone_span(struct zone *zone, unsigned long start_pfn,\n" "> >>>> \t\t\t\t\t start_pfn);\n" @@ -74,8 +74,8 @@ "> > \n" "> > Yeah, I also think it's unlikely to come here.\n" "> > \n" - "> > The 'if (zone_start_pfn = start_pfn)' checking also covers the case\n" - "> > (zone_start_pfn = start_pfn && zone_end_pfn = end_pfn). So this\n" + "> > The 'if (zone_start_pfn == start_pfn)' checking also covers the case\n" + "> > (zone_start_pfn == start_pfn && zone_end_pfn == end_pfn). So this\n" "> > zone_start_pfn/spanned_pages resetting can be removed to avoid\n" "> > confusion.\n" "> \n" @@ -93,4 +93,4 @@ "Yeah, noticed you told Wei the status in another patch thread, I am fine\n" with it, just leave it to you to decide. Thanks. -0ba01882a29df879bcf3b9e972af65f30307697104486b95e79828cab43f099e +854612413fba9bdc4de3a1ec443d7326c1ae0e8e0ad8c2acc32cb7875b800e7f
diff --git a/a/1.txt b/N3/1.txt index 9585aad..5f759bc 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -31,7 +31,7 @@ On 02/05/20 at 02:20pm, David Hildenbrand wrote: > >>>> + zone->zone_start_pfn = 0; > >>>> + zone->spanned_pages = 0; > >>>> } -> >>>> } else if (zone_end_pfn = end_pfn) { +> >>>> } else if (zone_end_pfn == end_pfn) { > >>>> /* > >>>> @@ -405,34 +408,11 @@ static void shrink_zone_span(struct zone *zone, unsigned long start_pfn, > >>>> start_pfn); @@ -48,8 +48,8 @@ On 02/05/20 at 02:20pm, David Hildenbrand wrote: > > > > Yeah, I also think it's unlikely to come here. > > -> > The 'if (zone_start_pfn = start_pfn)' checking also covers the case -> > (zone_start_pfn = start_pfn && zone_end_pfn = end_pfn). So this +> > The 'if (zone_start_pfn == start_pfn)' checking also covers the case +> > (zone_start_pfn == start_pfn && zone_end_pfn == end_pfn). So this > > zone_start_pfn/spanned_pages resetting can be removed to avoid > > confusion. > @@ -66,3 +66,9 @@ it doesn't impact anything. Yeah, noticed you told Wei the status in another patch thread, I am fine with it, just leave it to you to decide. Thanks. + + +_______________________________________________ +linux-arm-kernel mailing list +linux-arm-kernel@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/a/content_digest b/N3/content_digest index e8129d4..e57b481 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -6,7 +6,7 @@ "ref\0cd353848-301a-025d-dd66-44d76e1bbc44@redhat.com\0" "From\0Baoquan He <bhe@redhat.com>\0" "Subject\0Re: [PATCH v6 08/10] mm/memory_hotplug: Don't check for \"all holes\" in shrink_zone_span()\0" - "Date\0Wed, 05 Feb 2020 13:34:42 +0000\0" + "Date\0Wed, 5 Feb 2020 21:34:42 +0800\0" "To\0David Hildenbrand <david@redhat.com>\0" "Cc\0linux-s390@vger.kernel.org" Michal Hocko <mhocko@suse.com> @@ -57,7 +57,7 @@ "> >>>> +\t\t\tzone->zone_start_pfn = 0;\n" "> >>>> +\t\t\tzone->spanned_pages = 0;\n" "> >>>> \t\t}\n" - "> >>>> \t} else if (zone_end_pfn = end_pfn) {\n" + "> >>>> \t} else if (zone_end_pfn == end_pfn) {\n" "> >>>> \t\t/*\n" "> >>>> @@ -405,34 +408,11 @@ static void shrink_zone_span(struct zone *zone, unsigned long start_pfn,\n" "> >>>> \t\t\t\t\t start_pfn);\n" @@ -74,8 +74,8 @@ "> > \n" "> > Yeah, I also think it's unlikely to come here.\n" "> > \n" - "> > The 'if (zone_start_pfn = start_pfn)' checking also covers the case\n" - "> > (zone_start_pfn = start_pfn && zone_end_pfn = end_pfn). So this\n" + "> > The 'if (zone_start_pfn == start_pfn)' checking also covers the case\n" + "> > (zone_start_pfn == start_pfn && zone_end_pfn == end_pfn). So this\n" "> > zone_start_pfn/spanned_pages resetting can be removed to avoid\n" "> > confusion.\n" "> \n" @@ -91,6 +91,12 @@ "> priority. Thanks :)\n" "\n" "Yeah, noticed you told Wei the status in another patch thread, I am fine\n" - with it, just leave it to you to decide. Thanks. + "with it, just leave it to you to decide. Thanks.\n" + "\n" + "\n" + "_______________________________________________\n" + "linux-arm-kernel mailing list\n" + "linux-arm-kernel@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -0ba01882a29df879bcf3b9e972af65f30307697104486b95e79828cab43f099e +8ca4098fe4778d690131450655ef3324ca55394eec4099e646c79b966fcddf57
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.