Re: [ndctl PATCH v3 00/16] ndctl: Meson support

From: Jane Chu <jane.chu@oracle.com>
To: "Verma, Vishal L" <vishal.l.verma@intel.com>,
	"Williams, Dan J" <dan.j.williams@intel.com>
Cc: "jmoyer@redhat.com" <jmoyer@redhat.com>,
	"msuchanek@suse.de" <msuchanek@suse.de>,
	"linux-cxl@vger.kernel.org" <linux-cxl@vger.kernel.org>,
	"nvdimm@lists.linux.dev" <nvdimm@lists.linux.dev>,
	"breno.leitao@gmail.com" <breno.leitao@gmail.com>,
	"vaibhav@linux.ibm.com" <vaibhav@linux.ibm.com>,
	"kilobyte@angband.pl" <kilobyte@angband.pl>
Subject: Re: [ndctl PATCH v3 00/16] ndctl: Meson support
Date: Fri, 14 Jan 2022 22:30:38 +0000	[thread overview]
Message-ID: <c6be7804-419e-d547-062d-6b852494bceb@oracle.com> (raw)
In-Reply-To: <d4a57facb2b778867e3bbe8564f03868b58e2f72.camel@intel.com>

Hi,

Should the README.md file get updated accordingly?

thanks!
-jane

On 1/14/2022 8:04 AM, Verma, Vishal L wrote:
> On Wed, 2022-01-05 at 13:31 -0800, Dan Williams wrote:
>> Changes since v2 [1]:
>>
>> - Rebase on v72
>>    - Add Meson support for the new config file directory definitions.
>>    - Add Meson support for landing the daxctl udev rule
>>      daxdev-reconfigure service in the right directories
>> - Include the deprecation of BLK Aperture test infrastructure
>> - Include a miscellaneous doc clarification for 'ndctl update-firmware'
>> - Fix the tests support for moving the build directory out-of-line
>> - Include a fix for the deprectation of the dax_pmem_compat module
>>    pending in the libnvdimm-for-next tree.
>>
>> [1]: https://lore.kernel.org/r/163061537869.1943957.8491829881215255815.stgit@dwillia2-desk3.amr.corp.intel.com
>>
>> ---
>>
>> As mentioned in patch 14 the motiviation for converting to Meson is
>> primarily driven by speed (an order of magnitude in some scenarios), but
>> Meson also includes better test and debug-build support. The build
>> language is easier to read, write, and debug. Meson is all around better
>> suited to the next phase of the ndctl project that will include all
>> things "device memory" related (ndctl, daxctl, and cxl).
>>
>> In order to simplify the conversion the old BLK-aperture test
>> infrastructure is jettisoned and it will also be removed upstream. Some
>> other refactorings and fixups are included as well to better organize
>> the utilty infrastructure between truly common and sub-tool specific.
>>
>> Vishal,
>>
>> In preparation for ndctl-v73 please consider pulling in this series
>> early mainly for my own sanity of not needing to forward port more
>> updates to the autotools infrastructure.
>>
> Hi Dan,
> 
> These look great, thanks a lot for this work, it is an awesome workflow
> improvement!  I've merged it into pending, and will also merge to
> master shortly to encourage all new submissions to be based on this.
> 
> Also CC'ing a few distro maintainers - this will be a change to
> packaging specs etc. that are maintained outside of the ndctl repo.
> This change can be expected to land in the v73 release, which should be
> in the next 2-4 weeks.
> 
> Thanks,
> Vishal