diff for duplicates of <20200205124329.GE26758@MiWiFi-R3L-srv> diff --git a/a/1.txt b/N1/1.txt index 16fd9f8..f91f277 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -29,7 +29,7 @@ On 02/04/20 at 03:42pm, 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); @@ -46,7 +46,7 @@ On 02/04/20 at 03:42pm, 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 ab5a934..4d458cd 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -4,22 +4,22 @@ "ref\0e0006cc4-d448-89c6-38c0-51da7fc08715@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 12:43:29 +0000\0" + "Date\0Wed, 5 Feb 2020 20:43:29 +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/04/20 at 03:42pm, David Hildenbrand wrote:\n" @@ -53,7 +53,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" @@ -70,9 +70,9 @@ "\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. -8769ba6be2d99fe1b1bc9a368c2b4b56d6202415dbc4b88890a2dc91532fdddd +8b744442ff0ae698511a108f61108381b3bd5eb1638de62af75519938958a2a3
diff --git a/a/1.txt b/N2/1.txt index 16fd9f8..f91f277 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -29,7 +29,7 @@ On 02/04/20 at 03:42pm, 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); @@ -46,7 +46,7 @@ On 02/04/20 at 03:42pm, 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 ab5a934..45b88d4 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -4,7 +4,7 @@ "ref\0e0006cc4-d448-89c6-38c0-51da7fc08715@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 12:43:29 +0000\0" + "Date\0Wed, 5 Feb 2020 20:43:29 +0800\0" "To\0David Hildenbrand <david@redhat.com>\0" "Cc\0linux-s390@vger.kernel.org" Michal Hocko <mhocko@suse.com> @@ -53,7 +53,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" @@ -70,9 +70,9 @@ "\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. -8769ba6be2d99fe1b1bc9a368c2b4b56d6202415dbc4b88890a2dc91532fdddd +e05d6d4c41b967af15758367bfd160cde64f1875ff79c01d5b1f158539535b55
diff --git a/a/1.txt b/N3/1.txt index 16fd9f8..32ccf68 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -29,7 +29,7 @@ On 02/04/20 at 03:42pm, 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); @@ -46,7 +46,13 @@ On 02/04/20 at 03:42pm, 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. + + +_______________________________________________ +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 ab5a934..0e4406f 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -4,7 +4,7 @@ "ref\0e0006cc4-d448-89c6-38c0-51da7fc08715@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 12:43:29 +0000\0" + "Date\0Wed, 5 Feb 2020 20:43:29 +0800\0" "To\0David Hildenbrand <david@redhat.com>\0" "Cc\0linux-s390@vger.kernel.org" Michal Hocko <mhocko@suse.com> @@ -53,7 +53,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" @@ -70,9 +70,15 @@ "\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. + "confusion.\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 -8769ba6be2d99fe1b1bc9a368c2b4b56d6202415dbc4b88890a2dc91532fdddd +39101da23c0e61ba4dc2a574e0023e232545493fbc9dc7dea586403590e4cae1
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.