On Sat, Sep 26, 2020 at 12:20:57AM +0900, Tetsuo Handa wrote: > On 2020/09/26 0:11, Greg KH wrote: > > On Tue, Sep 22, 2020 at 01:21:19PM +0200, Oliver Neukum wrote: > >> Stress testing has shown that CDC-WDM has some issues with hangs > >> and error reporting > >> > >> 1. wakeups are not correctly handled in multhreaded environments > >> 2. unresponsive hardware is not handled > >> 3. errors are not correctly reported. This needs flush() to be > >> implemented. > >> > >> This version makes wdm_flush() use interruptible sleep. > >> > >> For easier review all squashed together: > >> > > > > I have like 3 or 4 different "RFC" series here from you for this driver, > > which one is the "newest"? > > https://lkml.kernel.org/r/20200923092136.14824-1-oneukum@xxxxxxxx > > is the newest series from Oliver. But > > https://lkml.kernel.org/r/b27841ab-a88c-13e2-a66f-6df7af1f46b4@xxxxxxxxxxxxxxxxxxx > > is the squashed version with updated comments and deduplicated code. > > > > > And can you send a series that isn't RFC so that I can know you feel it > > is good enough to be merged? > > Do you want this fix as a series of patches (the former link)? > Since I think that the changeset should be atomically applied, I posted the latter link. A single patch would be good to send to me again, burried at the end of a long thread is hard to dig out. Also with proper authorship is needed, did Oliver write this, or did you? There is the co-developed-by: tag, which looks like it might be relevant here, can you do that? thanks, greg k-h