]> git.baikalelectronics.ru Git - kernel.git/commit
mic: vop: Fix broken virtqueues
authorVincent Whitchurch <vincent.whitchurch@axis.com>
Tue, 29 Jan 2019 10:22:07 +0000 (11:22 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 30 Jan 2019 14:42:26 +0000 (15:42 +0100)
commitab66b3948f922481e8870ae07132169fdca916dc
tree6692c50426c9e4151d5dca9ab46e5bbc2bbb3673
parentabfbd962b9061abde58e6b793e8606cb61b71fc6
mic: vop: Fix broken virtqueues

VOP is broken in mainline since commit 4a6fbf54cf6cdd088 ("virtio_ring:
introduce packed ring support"); attempting to use the virtqueues leads
to various kernel crashes.  I'm testing it with my not-yet-merged
loopback patches, but even the in-tree MIC hardware cannot work.

The problem is not in the referenced commit per se, but is due to the
following hack in vop_find_vq() which depends on the layout of private
structures in other source files, which that commit happened to change:

  /*
   * To reassign the used ring here we are directly accessing
   * struct vring_virtqueue which is a private data structure
   * in virtio_ring.c. At the minimum, a BUILD_BUG_ON() in
   * vring_new_virtqueue() would ensure that
   *  (&vq->vring == (struct vring *) (&vq->vq + 1));
   */
  vr = (struct vring *)(vq + 1);
  vr->used = used;

Fix vop by using __vring_new_virtqueue() to create the needed vring
layout from the start, instead of attempting to patch in the used ring
later.  __vring_new_virtqueue() was added way back in commit
2882a9ac20d3b7 ("virtio: Add improved queue allocation API") in order to
address mic's usecase, according to the commit message.

Fixes: 4a6fbf54cf6c ("virtio_ring: introduce packed ring support")
Signed-off-by: Vincent Whitchurch <vincent.whitchurch@axis.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/mic/vop/vop_main.c