Re: [dpdk-dev] [PATCH] vhost: fix notification for packed ring

2018-10-11 Thread Tiwei Bie
On Thu, Oct 11, 2018 at 09:36:48AM -0400, Michael S. Tsirkin wrote: > On Thu, Oct 11, 2018 at 09:34:06PM +0800, Jason Wang wrote: > > > > > > On 2018年10月11日 21:06, Tiwei Bie wrote: > > > The notification can't be disabled in packed ring when > > > application tries to disable notification, becaus

Re: [dpdk-dev] [PATCH] vhost: fix notification for packed ring

2018-10-11 Thread Michael S. Tsirkin
On Thu, Oct 11, 2018 at 09:34:06PM +0800, Jason Wang wrote: > > > On 2018年10月11日 21:06, Tiwei Bie wrote: > > The notification can't be disabled in packed ring when > > application tries to disable notification, because the > > device event flags field is overwritten by an unexpected > > value. Th

Re: [dpdk-dev] [PATCH] vhost: fix notification for packed ring

2018-10-11 Thread Jason Wang
On 2018年10月11日 21:06, Tiwei Bie wrote: The notification can't be disabled in packed ring when application tries to disable notification, because the device event flags field is overwritten by an unexpected value. This patch fixes this issue. Fixes: b1cce26af1dc ("vhost: add notification for p

Re: [dpdk-dev] [PATCH] vhost: fix notification for packed ring

2018-10-11 Thread Maxime Coquelin
On 10/11/2018 03:06 PM, Tiwei Bie wrote: The notification can't be disabled in packed ring when application tries to disable notification, because the device event flags field is overwritten by an unexpected value. This patch fixes this issue. Fixes: b1cce26af1dc ("vhost: add notification for

[dpdk-dev] [PATCH] vhost: fix notification for packed ring

2018-10-11 Thread Tiwei Bie
The notification can't be disabled in packed ring when application tries to disable notification, because the device event flags field is overwritten by an unexpected value. This patch fixes this issue. Fixes: b1cce26af1dc ("vhost: add notification for packed ring") Cc: sta...@dpdk.org Signed-off