On 19/09/17 17:38, Bjørn Mork wrote: > Andreas Böhler <news@xxxxxxxxxxx> writes: > >> On 19/09/17 14:25, Bjørn Mork wrote: >>> Bjørn Mork <bjorn@xxxxxxx> writes: >>> >>> Maybe we should simply ignore stalls completely? They should not happen >>> with any real WDM device anyway. >>> >>> What happens if you e.g. change the if-test here: >>> >>> /* >>> * only set a new error if there is no previous error. >>> * Errors are only cleared during read/open >>> */ >>> if (desc->rerr == 0) >>> desc->rerr = status; >>> >>> >>> to >>> >>> if (desc->rerr == 0 && status != -EPIPE) >> >> Well, this looks very promising: Although the error messages persist >> every now and then, the device now works! >> >> I'll make the new module the default and I'll test it more thouroughly >> during the next few days. As promised I made the new module the default and tested it for a few days. Although the error messages persist, I did not notice any negative side effects. In fact, it also fixes a problem where mbim-proxy was at 100% load. Two other users also reported success [1], so I'm looking forward to having it fixed in the default kernel tree. In the meantime, I'll keep the module patched using DKMS. Thanks again for your help, Andreas [1] https://bugs.freedesktop.org/show_bug.cgi?id=100938 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
