On 2018?01?09? 09:46, Tonghao Zhang wrote:> The patch (7235acdb1) changed the way of the work
> flushing in which the queued seq, done seq, and the
> flushing are not used anymore. Then remove them now.
>
> Fixes: 7235acdb1 ("vhost: simplify work flushing")
> Cc: Jason Wang <jasowang at redhat.com>
> Signed-off-by: Tonghao Zhang <xiangxia.m.yue at gmail.com>
> ---
> drivers/vhost/vhost.c | 1 -
> drivers/vhost/vhost.h | 4 ----
> 2 files changed, 5 deletions(-)
>
> diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
> index 33ac2b186b85..9b04cad91d65 100644
> --- a/drivers/vhost/vhost.c
> +++ b/drivers/vhost/vhost.c
> @@ -181,7 +181,6 @@ void vhost_work_init(struct vhost_work *work,
vhost_work_fn_t fn)
> {
> clear_bit(VHOST_WORK_QUEUED, &work->flags);
> work->fn = fn;
> - init_waitqueue_head(&work->done);
> }
> EXPORT_SYMBOL_GPL(vhost_work_init);
>
> diff --git a/drivers/vhost/vhost.h b/drivers/vhost/vhost.h
> index 79c6e7a60a5e..749fe13e061c 100644
> --- a/drivers/vhost/vhost.h
> +++ b/drivers/vhost/vhost.h
> @@ -20,10 +20,6 @@ typedef void (*vhost_work_fn_t)(struct vhost_work
*work);
> struct vhost_work {
> struct llist_node node;
> vhost_work_fn_t fn;
> - wait_queue_head_t done;
> - int flushing;
> - unsigned queue_seq;
> - unsigned done_seq;
> unsigned long flags;
> };
>
Thanks for the patch, but please use get_maintainer.pl to cc correct
list and maintainers.
You can add my Acked-by when reposting.