Hi Thomas, On 2018/12/9 6:07, Thomas Jespersen wrote: > This fixes warning reported by sparse (with -Wsparse-all). > > Signed-off-by: Thomas Jespersen <[email protected]> > --- > drivers/staging/erofs/internal.h | 4 ++-- > drivers/staging/erofs/utils.c | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/erofs/internal.h > b/drivers/staging/erofs/internal.h > index 57575c7f56355424a..bf180a803446655d4 100644 > --- a/drivers/staging/erofs/internal.h > +++ b/drivers/staging/erofs/internal.h > @@ -232,9 +232,9 @@ static inline bool erofs_workgroup_get(struct > erofs_workgroup *grp, int *ocnt) > /* spin if it is temporarily locked at the reclaim path */ > if (unlikely(o == locked)) { > #if defined(CONFIG_SMP) || defined(CONFIG_DEBUG_SPINLOCK) > - do > + do { > cpu_relax(); > - while (atomic_read(&grp->refcount) == locked); > + } while (atomic_read(&grp->refcount) == locked); > #endif > goto repeat; > }
Could you check the latest source code at https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/tree/drivers/staging/erofs?h=staging-next the above you sent doesn't seem the latest source code... and send your patch to Greg and the staging mailing list as well? if you want to upstream immediately. To: Greg Kroah-Hartman <[email protected]> linux-staging mailing list <[email protected]> linux-erofs mailing list <[email protected]> Gao Xiang <[email protected]> Chao Yu <[email protected]> Thanks, Gao Xiang
