mbox series

[0/3] net/virtio: missing/wrong read barriers.

Message ID 20190124165902.24178-1-i.maximets@samsung.com (mailing list archive)
Headers
Series net/virtio: missing/wrong read barriers. |

Message

Ilya Maximets Jan. 24, 2019, 4:58 p.m. UTC 
  Ilya Maximets (3):
  net/virtio: fix improper read barriers on packed Tx cleanup
  net/virtio: add barriers for extra descriptors on Rx split
  net/virtio: add missing read barrier for packed dequeue

 drivers/net/virtio/virtio_rxtx.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Tiwei Bie Jan. 25, 2019, 4:43 a.m. UTC | #1 
On Thu, Jan 24, 2019 at 07:58:59PM +0300, Ilya Maximets wrote:
> Ilya Maximets (3):
>   net/virtio: fix improper read barriers on packed Tx cleanup
>   net/virtio: add barriers for extra descriptors on Rx split
>   net/virtio: add missing read barrier for packed dequeue
> 
>  drivers/net/virtio/virtio_rxtx.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Reviewed-by: Tiwei Bie <tiwei.bie@intel.com>
Maxime Coquelin Jan. 25, 2019, 3:50 p.m. UTC | #2 
Hi Ilya,

On 1/24/19 5:58 PM, Ilya Maximets wrote:
> Ilya Maximets (3):
>    net/virtio: fix improper read barriers on packed Tx cleanup
>    net/virtio: add barriers for extra descriptors on Rx split
>    net/virtio: add missing read barrier for packed dequeue
> 
>   drivers/net/virtio/virtio_rxtx.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 


Thanks for the series, it is sadly too late for v19.02.
I'll pick it as soon as v19.05 merge windo opens.

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Maxime
Maxime Coquelin Feb. 8, 2019, 6:04 p.m. UTC | #3 
On 1/24/19 5:58 PM, Ilya Maximets wrote:
> Ilya Maximets (3):
>    net/virtio: fix improper read barriers on packed Tx cleanup
>    net/virtio: add barriers for extra descriptors on Rx split
>    net/virtio: add missing read barrier for packed dequeue
> 
>   drivers/net/virtio/virtio_rxtx.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 


Applied to dpdk-next-virtio/master.

Thanks,
Maxime