Re: [PATCH v3 0/2] memory-hotplug.rst: complete admin-guide overhaul

From: David Hildenbrand <david@redhat.com>
To: Jonathan Corbet <corbet@lwn.net>, linux-kernel@vger.kernel.org
Cc: linux-mm@kvack.org, Andrew Morton <akpm@linux-foundation.org>,
	Anshuman Khandual <anshuman.khandual@arm.com>,
	Dave Hansen <dave.hansen@linux.intel.com>,
	Matthew Wilcox <willy@infradead.org>,
	Michal Hocko <mhocko@suse.com>,
	Mike Kravetz <mike.kravetz@oracle.com>,
	Mike Rapoport <rppt@kernel.org>,
	Mike Rapoport <rppt@linux.ibm.com>,
	Muchun Song <songmuchun@bytedance.com>,
	Oscar Salvador <osalvador@suse.de>,
	Pavel Tatashin <pasha.tatashin@soleen.com>,
	Stephen Rothwell <sfr@canb.auug.org.au>
Subject: Re: [PATCH v3 0/2] memory-hotplug.rst: complete admin-guide overhaul
Date: Mon, 14 Jun 2021 18:16:01 +0200	[thread overview]
Message-ID: <300305e3-b225-623b-f02f-0973f4909705@redhat.com> (raw)
In-Reply-To: <87fsxka5zh.fsf@meer.lwn.net>

On 14.06.21 16:14, Jonathan Corbet wrote:
> David Hildenbrand <david@redhat.com> writes:
> 
>> Essentially against Andrew's mmotm, which already contains v1. Note
>> mmotm was chosen due to a conflict in:
>>
>> https://lkml.kernel.org/r/20210510030027.56044-1-songmuchun@bytedance.com
>>
>> I'd suggest we take this via Andrew's tree, unless you have other
>> preference on how to handle the conflict. Thanks!
> 
> That's fine with me - one less thing for me to deal with :)
> 

Thanks!

FWIW, the patches are fine. My setup simply seems to compress the file
paths in the diff stats more aggressively:

$ git format-patch -2 -v3  > /dev/null; cat v3-0000-cover-letter.patch | tail -6
  .../admin-guide/mm/memory-hotplug.rst         | 800 ++++++++++--------
  1 file changed, 455 insertions(+), 345 deletions(-)

When I force to a bigger terminal size, I get longer paths:

$ git format-patch -2 -v3 --stat=80 > /dev/null; cat v3-0000-cover-letter.patch | tail -6
  Documentation/admin-guide/mm/memory-hotplug.rst | 800 ++++++++++++++----------
  1 file changed, 455 insertions(+), 345 deletions(-)

(maybe my awesome-wm + xterm setup confuses git, or there is a
BUG somewhere in my software stack)

-- 
Thanks,

David / dhildenb