[SRU][Zesty][PATCH 1/1] Revert "vhost: cache used event for better performance"

Joseph Salisbury joseph.salisbury at canonical.com
Mon Aug 28 14:11:30 UTC 2017

On 08/28/2017 10:05 AM, Seth Forshee wrote:
> On Mon, Aug 28, 2017 at 09:14:28AM -0400, Joseph Salisbury wrote:
>> On 08/28/2017 08:54 AM, Seth Forshee wrote:
>>> On Fri, Aug 25, 2017 at 12:07:55PM -0400, Joseph Salisbury wrote:
>>>> BugLink: http://bugs.launchpad.net/bugs/1711251
>>>> This reverts commit 809ecb9bca6a9424ccd392d67e368160f8b76c92.
>>> I'm curious why you reverted directly instead of cherry picking the
>>> upstream revert. In either case the patch needs your sign-off, which is
>>> missing here.
>> Cherry picking the revert from upstream was not clean and required a
>> backport.  However, If I did the revert directly to Zesty it was clean. 
>> Do you think it would be better to cherry pick from upstream and back port?
> No, that's a good enough reason. However I think it probably does need
> to be marked as sauce in that case, and I think it would be useful to
> mention the upstream commit and the reason for not cherry picking it in
> the patch description.

Thanks for the feedback, Seth.  I'll NAK this one and send a v2.

More information about the kernel-team mailing list