On Thu, Aug 20, 2020 at 03:40:06AM +0000, Peter Chen wrote: > > > > One way or another, the existing code is wrong. I guess the best we can > > do for now is to let it remain wrong during the patch series, rather > > than changing it to be wrong in a different way. > > > > To put it another way, we already run the risk of clearing memory that > > has been freed. The series does not make that risk any worse, and it > > eventually fixes the problem. > > > > This means your patch should remain in its position at the end of the > > series. > > > > Thank. > > If you think my sequence during the patch series is OK, would you > please add your reviewed-by below? > > https://www.spinics.net/lists/linux-usb/msg199291.html That wouldn't make sense; I can't add Reviewed-by: to patches that I wrote myself. They already have my Signed-off-by:. However, it is okay to add Reviewed-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> to the 6/6 patch. Alan Stern