On Thu, May 22, 2025 at 07:47:56AM -0700, Marc Herbert wrote: > On 2025-05-21 21:47, Alison Schofield wrote: > > > Thanks for doing this Marc! > > > > I'm wondering about the need to delineate between what is needed to load > > and use the cxl-test or nfit-test modules as opposed to what is required to > > run all the unit tests. > > > > I believe my environment, and yours, and most other folks using these > > environments are doing so in a VM so it's no big deal to load up all the > > things. > > > > Maybe just a gentle separator in the list showing required and optional. > > I unfortunately don't know and understand these enough to remember that > and it would be very time-consuming to re-test them one by one. > > More generally speaking, this sort of list looks deceptively simple but > it almost never is. That's basically why I initially asked in > https://lore.kernel.org/nvdimm/aed71134-1029-4b88-ab20-8dfa527a7...@linux.intel.com/ > if someone more knowledgeable could do this (based on the > run_qemu.git/.github/workflows/*.cfg files) Also, this stuff tends to > evolve. > > Now that tested versions can be found in run_qemu.bit, it's less > critical to update this README.md file. The current README.md version is > inconveniently non working but at least some functional versions can be > found somewhere else.
Hi Marc, As you probably just got notified, I went ahead and massaged this one and applied to pending: https://github.com/pmem/ndctl/commit/fb2f28cd280c69a978753e54a0f67267e54ffbda There is clear agreement on the great need for this as we encourage more folks to join the cxl-test club. My comment about module vs unit test was a no-op because the README, it turns out, made no mention of that. I did pause a few times on the README and say to myself, 'oh that needs updating' but decided to get this out and come back around for better later. I'm going to ping Dan directly for an ACK, since he promised :) If there is any issue on pending I'm happy to fix it up. I'm planning to make the v82 release June 11-ish. -- Alison > > Cheers, > > Marc
