Re: [PATCH V2 0/8] vhost flush cleanups
On Sun, May 15, 2022 at 03:29:14PM -0500, Mike Christie wrote: The following patches are Andrey Ryabinin's flush cleanups and some from me. They reduce the number of flush calls and remove some bogus ones where we don't even have a worker running anymore or they were based on outdated or incorrect assumptions. Jason, for the patches you gave me an explicit acked/reviewed tag I added it. For the replies where I answered your review questions and you only replied with an affirimative reply I did not add a tag, because I was not not 100% sure what you wanted to do. These patches will be used in the vhost threading patches which I think will make progress again now that I have Christian's email figured out :) (he had moved to microsoft but I've been using the ubuntu address). I think the patches are ok cleanups in general so I thought they could get merged separately if people agree. V2: - Added patch to rename vhost_work_dev_flush to just vhost_dev_flush to handle review comment from Jason about the naming not being so great. Thank you for this cleanup! I think there is only a small issue to fix in vhost/test.c, otherwise it looks good to me :-) Thanks, Stefano ___ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization
Re: [PATCH V2 0/8] vhost flush cleanups
在 2022/5/16 04:29, Mike Christie 写道: The following patches are Andrey Ryabinin's flush cleanups and some from me. They reduce the number of flush calls and remove some bogus ones where we don't even have a worker running anymore or they were based on outdated or incorrect assumptions. Jason, for the patches you gave me an explicit acked/reviewed tag I added it. For the replies where I answered your review questions and you only replied with an affirimative reply I did not add a tag, because I was not not 100% sure what you wanted to do. Fine, I will go through the codes. Thanks These patches will be used in the vhost threading patches which I think will make progress again now that I have Christian's email figured out :) (he had moved to microsoft but I've been using the ubuntu address). I think the patches are ok cleanups in general so I thought they could get merged separately if people agree. V2: - Added patch to rename vhost_work_dev_flush to just vhost_dev_flush to handle review comment from Jason about the naming not being so great. ___ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization
[PATCH V2 0/8] vhost flush cleanups
The following patches are Andrey Ryabinin's flush cleanups and some from me. They reduce the number of flush calls and remove some bogus ones where we don't even have a worker running anymore or they were based on outdated or incorrect assumptions. Jason, for the patches you gave me an explicit acked/reviewed tag I added it. For the replies where I answered your review questions and you only replied with an affirimative reply I did not add a tag, because I was not not 100% sure what you wanted to do. These patches will be used in the vhost threading patches which I think will make progress again now that I have Christian's email figured out :) (he had moved to microsoft but I've been using the ubuntu address). I think the patches are ok cleanups in general so I thought they could get merged separately if people agree. V2: - Added patch to rename vhost_work_dev_flush to just vhost_dev_flush to handle review comment from Jason about the naming not being so great. ___ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization