On 12/6/23 18:59, Dan Williams wrote:
> Greg lamented:
> "Ick, sorry about that, obviously this test isn't actually built by any
> bots :("
>
> A quick and dirty way to prevent this problem going forward is to always
> compile ndtest.ko whenever nfit_test is built. While this still does not
> expose the test code to any of the known build bots, it at least makes
> it the case that anyone that runs the x86 tests also compiles the
> powerpc test.
>
> I.e. the Intel NVDIMM maintainers are less likely to fall into this hole
> in the future.
>
> Link: http://lore.kernel.org/r/2023112729-aids-drainable-5744@gregkh
> Cc: Greg KH <gre...@linuxfoundation.org>
> Cc: Yi Zhang <yi.zh...@redhat.com>
> Cc: Dave Jiang <dave.ji...@intel.com>
> Cc: Ira Weiny <ira.we...@intel.com>
> Signed-off-by: Dan Williams <dan.j.willi...@intel.com>
Reviewed-by: Dave Jiang <dave.ji...@intel.com>
> ---
> tools/testing/nvdimm/test/Kbuild | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/testing/nvdimm/test/Kbuild
> b/tools/testing/nvdimm/test/Kbuild
> index 197bcb2b7f35..003d48f5f24f 100644
> --- a/tools/testing/nvdimm/test/Kbuild
> +++ b/tools/testing/nvdimm/test/Kbuild
> @@ -7,6 +7,7 @@ obj-m += nfit_test_iomap.o
>
> ifeq ($(CONFIG_ACPI_NFIT),m)
> nfit_test-y := nfit.o
> + obj-m += ndtest.o
> else
> nfit_test-y := ndtest.o
> endif
>