diff for duplicates of <20200205141254.GD8965@MiWiFi-R3L-srv> diff --git a/a/1.txt b/N1/1.txt index 3b576f9..056c7df 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -33,7 +33,7 @@ On 02/05/20 at 02:38pm, 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); @@ -50,8 +50,8 @@ On 02/05/20 at 02:38pm, 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 562676f..42584cc 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -8,22 +8,22 @@ "ref\02868343a-745b-e2b6-7e78-d5649c00ee31@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 14:12:54 +0000\0" + "Date\0Wed, 5 Feb 2020 22:12:54 +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:38pm, David Hildenbrand wrote:\n" @@ -61,7 +61,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" @@ -78,8 +78,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" @@ -106,4 +106,4 @@ "\n" Understood, sorry about the noise, David. I will think about this. -b24d9a7a6ac672680d86585608a44019fcbc2045fa3aec6338787e5a9285e225 +134d052459878abde0d1d0606d15d1968f54a4d08ec3b01f78582085eeabc8c4
diff --git a/a/1.txt b/N2/1.txt index 3b576f9..056c7df 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -33,7 +33,7 @@ On 02/05/20 at 02:38pm, 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); @@ -50,8 +50,8 @@ On 02/05/20 at 02:38pm, 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 562676f..05f41a0 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -8,7 +8,7 @@ "ref\02868343a-745b-e2b6-7e78-d5649c00ee31@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 14:12:54 +0000\0" + "Date\0Wed, 5 Feb 2020 22:12:54 +0800\0" "To\0David Hildenbrand <david@redhat.com>\0" "Cc\0linux-s390@vger.kernel.org" Michal Hocko <mhocko@suse.com> @@ -61,7 +61,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" @@ -78,8 +78,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" @@ -106,4 +106,4 @@ "\n" Understood, sorry about the noise, David. I will think about this. -b24d9a7a6ac672680d86585608a44019fcbc2045fa3aec6338787e5a9285e225 +c68eca1446da505e53da00c3d1ad5ed6a762133e3c0f1edf2fed83ac88ea3dfb
diff --git a/a/1.txt b/N3/1.txt index 3b576f9..fa5be2e 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -33,7 +33,7 @@ On 02/05/20 at 02:38pm, 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); @@ -50,8 +50,8 @@ On 02/05/20 at 02:38pm, 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. > >> @@ -77,3 +77,9 @@ On 02/05/20 at 02:38pm, David Hildenbrand wrote: > making this eventually unconditional?). Thanks! Understood, sorry about the noise, David. I will think about this. + + +_______________________________________________ +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 562676f..5bb27c0 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -8,7 +8,7 @@ "ref\02868343a-745b-e2b6-7e78-d5649c00ee31@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 14:12:54 +0000\0" + "Date\0Wed, 5 Feb 2020 22:12:54 +0800\0" "To\0David Hildenbrand <david@redhat.com>\0" "Cc\0linux-s390@vger.kernel.org" Michal Hocko <mhocko@suse.com> @@ -61,7 +61,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" @@ -78,8 +78,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" @@ -104,6 +104,12 @@ "> I am fairly busy right now. Can you send a patch (double-checking and\n" "> making this eventually unconditional?). Thanks!\n" "\n" - Understood, sorry about the noise, David. I will think about this. + "Understood, sorry about the noise, David. I will think about this.\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 -b24d9a7a6ac672680d86585608a44019fcbc2045fa3aec6338787e5a9285e225 +c54a4ba2c7e04e9a8b47254baf1be316eb39af4223d7a854511c284eef859402
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.