On Tue, Nov 18, 2025 at 10:46 PM Ira Weiny <[email protected]> wrote:
>
> Michal Clapinski wrote:
> > Signed-off-by: Michal Clapinski <[email protected]>
> > Reviewed-by: Dan Williams <[email protected]>
>
> Sorry for the delay.  I picked up this series but I find that this breaks
> the device-dax and daxctl-create.sh.
>
> I was able to fix device-dax with a sleep, see below.
>
> I'm not 100% sure what to do about this.
>
> I don't want to sprinkle sleeps around the tests.  daxctl-create.sh also
> randomly fail due to the races introduced.  So not sure exactly where to
> sprinkle them without more work.

I see 2 possible solutions here:
1. Modify the tests to just poll for the devices to appear.
2. Modify ndctl to poll for the devices to appear before returning.

What do you think about those?

> Could dropping just this patch and landing the others achieve most of what
> you need?

No, device-dax is the only one I actually care about.

Reply via email to