lkml.org 
[lkml]   [2020]   [Jun]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH RFC v5 13/13] vhost: drop head based APIs
On Mon, Jun 08, 2020 at 11:57:48AM +0800, Jason Wang wrote:
>
> On 2020/6/7 下午10:11, Michael S. Tsirkin wrote:
> > Everyone's using buf APIs, no need for head based ones anymore.
> >
> > Signed-off-by: Michael S. Tsirkin<mst@redhat.com>
> > ---
> > drivers/vhost/vhost.c | 36 ++++++++----------------------------
> > drivers/vhost/vhost.h | 12 ------------
> > 2 files changed, 8 insertions(+), 40 deletions(-)
> >
> > diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
> > index 72ee55c810c4..e6931b760b61 100644
> > --- a/drivers/vhost/vhost.c
> > +++ b/drivers/vhost/vhost.c
> > @@ -2299,12 +2299,12 @@ static int fetch_buf(struct vhost_virtqueue *vq)
> > return 1;
> > }
> > -/* Reverse the effect of vhost_get_vq_desc. Useful for error handling. */
> > +/* Revert the effect of fetch_buf. Useful for error handling. */
> > +static
> > void vhost_discard_vq_desc(struct vhost_virtqueue *vq, int n)
> > {
> > vq->last_avail_idx -= n;
> > }
> > -EXPORT_SYMBOL_GPL(vhost_discard_vq_desc);
>
>
> The same question as previous version.
>
> Do we need to rewind cached descriptor here?
>
> Thanks

Good point. This needs more thought, we need to also
rewind the avail idx each time we flush the descriptor cache.

--
MST

\
 
 \ /
  Last update: 2020-06-08 08:47    [W:0.060 / U:0.632 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site