On 2022/01/28 22:00, Christoph Hellwig wrote:
> + if (atomic_inc_return(&lo->lo_refcnt) > 1)
> + return 0;
> +
> err = mutex_lock_killable(&lo->lo_mutex);
> if (err)
You did not notice my diff here...
> return err;
> - if (lo->lo_state == Lo_deleting)
> + if (lo->lo_state == Lo_deleting) {
> + atomic_dec(&lo->lo_refcnt);
> err = -ENXIO;
> - else
> - atomic_inc(&lo->lo_refcnt);
> + }
Why do we need [PATCH 1/8] [PATCH 2/8] [PATCH 3/8] in this series?
Shouldn't we first make a clean revert, and keep the changes for
this release cycle as small as possible?
