mvp / uhubctl

uhubctl - USB hub per-port power control
Other
2.23k stars 232 forks source link

Can't toggle power to disk using Linux 6rc3 #457

Closed emsaks closed 2 years ago

emsaks commented 2 years ago

I've been using uhubctl to toggle power to frozen/hung disks. But now, under Linux 6 (rc3) if the disk is opened (e.g. a process is reading it, or in my case, dmsetup has a dependency on it) the command will hang until the disk is closed. (I've also tried using the sysfs-disable branch).

mvp commented 2 years ago

@mgrzeschik, @hnez would know more why it broke in Linux 6 rc3.

emsaks commented 2 years ago

How can I contact them (or will they be notified by the @ mention)?

mvp commented 2 years ago

I guess. I recommend to use older Linux kernel for now. Using Linux 6 is living on the edge at the moment.

mgrzeschik commented 2 years ago

I for now can not see what change could have introduced this behavior. This could also be an issue with some specific peripheral/controller driver. At least I can say that the patch introducing the /sys/bus/usb/devices/...//port/disable sysfs property is not involved in this. :)

mvp commented 2 years ago

@emsaks, now that #450 is merged, can you try this again - build uhubctl from master branch?

mvp commented 2 years ago

Closing for inactivity. Please feel free to reopen if you have anything to add @emsaks

mgrzeschik commented 2 years ago

Did you use the latest version of uhubctl to trigger this. Including the recent changes of #450 that was merged lately?

emsaks commented 2 years ago

No; I built your tree myself before the merge.

mvp commented 2 years ago

Can you try building it from master branch and check if it works for you @emsaks?

emsaks commented 2 years ago

I'll try when I can. I've downgraded to an earlier kernel in the meantime...

On Thu, Sep 22, 2022 at 9:30 PM mvp @.***> wrote:

@emsaks https://github.com/emsaks, now that #450 https://github.com/mvp/uhubctl/pull/450 is merged, can you try this again - build uhubctl from master branch?

— Reply to this email directly, view it on GitHub https://github.com/mvp/uhubctl/issues/457#issuecomment-1255399117, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC5AMJDKYR3TXCBIAKYZ5DDV7SQTTANCNFSM6AAAAAAQLKVEEA . You are receiving this because you were mentioned.Message ID: @.***>

mvp commented 2 years ago

Thanks anyway! BTW, official kernel 6.0 was released few days ago.