* [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) @ 2019-08-21 0:18 Caitlyn 2019-08-21 0:18 ` [PATCH 1/2] staging/erofs/xattr.h: Fixed misaligned function arguments Caitlyn ` (2 more replies) 0 siblings, 3 replies; 12+ messages in thread From: Caitlyn @ 2019-08-21 0:18 UTC (permalink / raw) To: Gao Xiang, Chao Yu, Greg Kroah-Hartman Cc: devel, Tobin C . Harding, linux-erofs, linux-kernel, Caitlyn Hello! This patch series cleans up some checkpatch fixes in erofs. The patches include balancing conditional braces and fixing some indentation. No testing done, all patches build and checkpath cleanly. Caitlyn (2): staging/erofs/xattr.h: Fixed misaligned function arguments. staging/erofs: Balanced braces around a few conditional statements. drivers/staging/erofs/inode.c | 4 ++-- drivers/staging/erofs/unzip_vle.c | 12 ++++++------ drivers/staging/erofs/xattr.h | 6 +++--- 3 files changed, 11 insertions(+), 11 deletions(-) -- 2.7.4 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/2] staging/erofs/xattr.h: Fixed misaligned function arguments. 2019-08-21 0:18 [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) Caitlyn @ 2019-08-21 0:18 ` Caitlyn 2019-08-21 2:20 ` Greg Kroah-Hartman 2019-08-21 0:18 ` [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements Caitlyn 2019-08-21 0:39 ` [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) Gao Xiang via Linux-erofs 2 siblings, 1 reply; 12+ messages in thread From: Caitlyn @ 2019-08-21 0:18 UTC (permalink / raw) To: Gao Xiang, Chao Yu, Greg Kroah-Hartman Cc: devel, Tobin C . Harding, linux-erofs, linux-kernel, Caitlyn Indented some function arguments to fix checkpath warnings. Signed-off-by: Caitlyn <caitlynannefinn@gmail.com> --- drivers/staging/erofs/xattr.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/erofs/xattr.h b/drivers/staging/erofs/xattr.h index 35ba5ac..d86f5cd 100644 --- a/drivers/staging/erofs/xattr.h +++ b/drivers/staging/erofs/xattr.h @@ -74,14 +74,14 @@ int erofs_getxattr(struct inode *, int, const char *, void *, size_t); ssize_t erofs_listxattr(struct dentry *, char *, size_t); #else static int __maybe_unused erofs_getxattr(struct inode *inode, int index, - const char *name, - void *buffer, size_t buffer_size) + const char *name, void *buffer, + size_t buffer_size) { return -ENOTSUPP; } static ssize_t __maybe_unused erofs_listxattr(struct dentry *dentry, - char *buffer, size_t buffer_size) + char *buffer, size_t buffer_size) { return -ENOTSUPP; } -- 2.7.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] staging/erofs/xattr.h: Fixed misaligned function arguments. 2019-08-21 0:18 ` [PATCH 1/2] staging/erofs/xattr.h: Fixed misaligned function arguments Caitlyn @ 2019-08-21 2:20 ` Greg Kroah-Hartman 0 siblings, 0 replies; 12+ messages in thread From: Greg Kroah-Hartman @ 2019-08-21 2:20 UTC (permalink / raw) To: Caitlyn; +Cc: devel, linux-erofs, linux-kernel On Tue, Aug 20, 2019 at 08:18:19PM -0400, Caitlyn wrote: > Indented some function arguments to fix checkpath warnings. > > Signed-off-by: Caitlyn <caitlynannefinn@gmail.com> > --- > drivers/staging/erofs/xattr.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > Hi, This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him a patch that has triggered this response. He used to manually respond to these common problems, but in order to save his sanity (he kept writing the same thing over and over, yet to different people), I was created. Hopefully you will not take offence and will fix the problem in your patch and resubmit it so that it can be accepted into the Linux kernel tree. You are receiving this message because of the following common error(s) as indicated below: - It looks like you did not use your "real" name for the patch on either the Signed-off-by: line, or the From: line (both of which have to match). Please read the kernel file, Documentation/SubmittingPatches for how to do this correctly. If you wish to discuss this problem further, or you have questions about how to resolve this issue, please feel free to respond to this email and Greg will reply once he has dug out from the pending patches received from other developers. thanks, greg k-h's patch email bot ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 0:18 [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) Caitlyn 2019-08-21 0:18 ` [PATCH 1/2] staging/erofs/xattr.h: Fixed misaligned function arguments Caitlyn @ 2019-08-21 0:18 ` Caitlyn 2019-08-21 0:40 ` Gao Xiang via Linux-erofs 2019-08-21 2:26 ` Joe Perches 2019-08-21 0:39 ` [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) Gao Xiang via Linux-erofs 2 siblings, 2 replies; 12+ messages in thread From: Caitlyn @ 2019-08-21 0:18 UTC (permalink / raw) To: Gao Xiang, Chao Yu, Greg Kroah-Hartman Cc: devel, Tobin C . Harding, linux-erofs, linux-kernel, Caitlyn Balanced braces to fix some checkpath warnings in inode.c and unzip_vle.c Signed-off-by: Caitlyn <caitlynannefinn@gmail.com> --- drivers/staging/erofs/inode.c | 4 ++-- drivers/staging/erofs/unzip_vle.c | 12 ++++++------ 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/erofs/inode.c b/drivers/staging/erofs/inode.c index 4c3d8bf..8de6fcd 100644 --- a/drivers/staging/erofs/inode.c +++ b/drivers/staging/erofs/inode.c @@ -278,9 +278,9 @@ struct inode *erofs_iget(struct super_block *sb, vi->nid = nid; err = fill_inode(inode, isdir); - if (likely(!err)) + if (likely(!err)) { unlock_new_inode(inode); - else { + } else { iget_failed(inode); inode = ERR_PTR(err); } diff --git a/drivers/staging/erofs/unzip_vle.c b/drivers/staging/erofs/unzip_vle.c index f0dab81..f431614 100644 --- a/drivers/staging/erofs/unzip_vle.c +++ b/drivers/staging/erofs/unzip_vle.c @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block *sb, mutex_lock(&work->lock); nr_pages = work->nr_pages; - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { pages = pages_onstack; - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && - mutex_trylock(&z_pagemap_global_lock)) + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && + mutex_trylock(&z_pagemap_global_lock)) { pages = z_pagemap_global; - else { + } else { repeat: pages = kvmalloc_array(nr_pages, sizeof(struct page *), GFP_KERNEL); /* fallback to global pagemap for the lowmem scenario */ if (unlikely(!pages)) { - if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) + if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) { goto repeat; - else { + } else { mutex_lock(&z_pagemap_global_lock); pages = z_pagemap_global; } -- 2.7.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 0:18 ` [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements Caitlyn @ 2019-08-21 0:40 ` Gao Xiang via Linux-erofs 2019-08-21 0:55 ` Caitlyn Finn 2019-08-21 2:26 ` Joe Perches 1 sibling, 1 reply; 12+ messages in thread From: Gao Xiang via Linux-erofs @ 2019-08-21 0:40 UTC (permalink / raw) To: Caitlyn; +Cc: devel, Greg Kroah-Hartman, linux-kernel, linux-erofs On Tue, Aug 20, 2019 at 08:18:20PM -0400, Caitlyn wrote: > Balanced braces to fix some checkpath warnings in inode.c and > unzip_vle.c > > Signed-off-by: Caitlyn <caitlynannefinn@gmail.com> > --- > drivers/staging/erofs/inode.c | 4 ++-- > drivers/staging/erofs/unzip_vle.c | 12 ++++++------ > 2 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/erofs/inode.c b/drivers/staging/erofs/inode.c > index 4c3d8bf..8de6fcd 100644 > --- a/drivers/staging/erofs/inode.c > +++ b/drivers/staging/erofs/inode.c > @@ -278,9 +278,9 @@ struct inode *erofs_iget(struct super_block *sb, > vi->nid = nid; > > err = fill_inode(inode, isdir); > - if (likely(!err)) > + if (likely(!err)) { > unlock_new_inode(inode); The only valid place is here. Thanks, Gao Xiang > - else { > + } else { > iget_failed(inode); > inode = ERR_PTR(err); > } > diff --git a/drivers/staging/erofs/unzip_vle.c b/drivers/staging/erofs/unzip_vle.c > index f0dab81..f431614 100644 > --- a/drivers/staging/erofs/unzip_vle.c > +++ b/drivers/staging/erofs/unzip_vle.c > @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block *sb, > mutex_lock(&work->lock); > nr_pages = work->nr_pages; > > - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) > + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { > pages = pages_onstack; > - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > - mutex_trylock(&z_pagemap_global_lock)) > + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > + mutex_trylock(&z_pagemap_global_lock)) { > pages = z_pagemap_global; > - else { > + } else { > repeat: > pages = kvmalloc_array(nr_pages, sizeof(struct page *), > GFP_KERNEL); > > /* fallback to global pagemap for the lowmem scenario */ > if (unlikely(!pages)) { > - if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) > + if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) { > goto repeat; > - else { > + } else { > mutex_lock(&z_pagemap_global_lock); > pages = z_pagemap_global; > } > -- > 2.7.4 > > _______________________________________________ > devel mailing list > devel@linuxdriverproject.org > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 0:40 ` Gao Xiang via Linux-erofs @ 2019-08-21 0:55 ` Caitlyn Finn 0 siblings, 0 replies; 12+ messages in thread From: Caitlyn Finn @ 2019-08-21 0:55 UTC (permalink / raw) To: Gao Xiang Cc: devel, Greg Kroah-Hartman, linux-kernel, linux-erofs, Tobin C . Harding [-- Attachment #1: Type: text/plain, Size: 3074 bytes --] Gao, Thanks very much for your help and the prompt reply! Sorry for my mistake, I will resolve and re-submit when I have an appropriate patch series. Caitlyn Finn On Tue, Aug 20, 2019 at 8:40 PM Gao Xiang <hsiangkao@aol.com> wrote: > On Tue, Aug 20, 2019 at 08:18:20PM -0400, Caitlyn wrote: > > Balanced braces to fix some checkpath warnings in inode.c and > > unzip_vle.c > > > > Signed-off-by: Caitlyn <caitlynannefinn@gmail.com> > > --- > > drivers/staging/erofs/inode.c | 4 ++-- > > drivers/staging/erofs/unzip_vle.c | 12 ++++++------ > > 2 files changed, 8 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/staging/erofs/inode.c > b/drivers/staging/erofs/inode.c > > index 4c3d8bf..8de6fcd 100644 > > --- a/drivers/staging/erofs/inode.c > > +++ b/drivers/staging/erofs/inode.c > > @@ -278,9 +278,9 @@ struct inode *erofs_iget(struct super_block *sb, > > vi->nid = nid; > > > > err = fill_inode(inode, isdir); > > - if (likely(!err)) > > + if (likely(!err)) { > > unlock_new_inode(inode); > > The only valid place is here. > > Thanks, > Gao Xiang > > > - else { > > + } else { > > iget_failed(inode); > > inode = ERR_PTR(err); > > } > > diff --git a/drivers/staging/erofs/unzip_vle.c > b/drivers/staging/erofs/unzip_vle.c > > index f0dab81..f431614 100644 > > --- a/drivers/staging/erofs/unzip_vle.c > > +++ b/drivers/staging/erofs/unzip_vle.c > > @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block > *sb, > > mutex_lock(&work->lock); > > nr_pages = work->nr_pages; > > > > - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) > > + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { > > pages = pages_onstack; > > - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > - mutex_trylock(&z_pagemap_global_lock)) > > + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > + mutex_trylock(&z_pagemap_global_lock)) { > > pages = z_pagemap_global; > > - else { > > + } else { > > repeat: > > pages = kvmalloc_array(nr_pages, sizeof(struct page *), > > GFP_KERNEL); > > > > /* fallback to global pagemap for the lowmem scenario */ > > if (unlikely(!pages)) { > > - if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) > > + if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) { > > goto repeat; > > - else { > > + } else { > > mutex_lock(&z_pagemap_global_lock); > > pages = z_pagemap_global; > > } > > -- > > 2.7.4 > > > > _______________________________________________ > > devel mailing list > > devel@linuxdriverproject.org > > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel > [-- Attachment #2: Type: text/html, Size: 4461 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 0:18 ` [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements Caitlyn 2019-08-21 0:40 ` Gao Xiang via Linux-erofs @ 2019-08-21 2:26 ` Joe Perches 2019-08-21 2:31 ` Gao Xiang 1 sibling, 1 reply; 12+ messages in thread From: Joe Perches @ 2019-08-21 2:26 UTC (permalink / raw) To: Caitlyn, Gao Xiang, Chao Yu, Greg Kroah-Hartman Cc: devel, linux-erofs, linux-kernel, Tobin C . Harding On Tue, 2019-08-20 at 20:18 -0400, Caitlyn wrote: > Balanced braces to fix some checkpath warnings in inode.c and > unzip_vle.c [] > diff --git a/drivers/staging/erofs/unzip_vle.c b/drivers/staging/erofs/unzip_vle.c [] > @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block *sb, > mutex_lock(&work->lock); > nr_pages = work->nr_pages; > > - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) > + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { > pages = pages_onstack; > - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > - mutex_trylock(&z_pagemap_global_lock)) > + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > + mutex_trylock(&z_pagemap_global_lock)) { Extra space after tab > pages = z_pagemap_global; > - else { > + } else { > repeat: > pages = kvmalloc_array(nr_pages, sizeof(struct page *), > GFP_KERNEL); > > /* fallback to global pagemap for the lowmem scenario */ > if (unlikely(!pages)) { > - if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) > + if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) { > goto repeat; > - else { > + } else { Unnecessary else > mutex_lock(&z_pagemap_global_lock); > pages = z_pagemap_global; > } ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 2:26 ` Joe Perches @ 2019-08-21 2:31 ` Gao Xiang 2019-08-21 15:13 ` Tobin C. Harding 0 siblings, 1 reply; 12+ messages in thread From: Gao Xiang @ 2019-08-21 2:31 UTC (permalink / raw) To: Joe Perches Cc: devel, Greg Kroah-Hartman, linux-kernel, Caitlyn, linux-erofs, Tobin C . Harding On Tue, Aug 20, 2019 at 07:26:46PM -0700, Joe Perches wrote: > On Tue, 2019-08-20 at 20:18 -0400, Caitlyn wrote: > > Balanced braces to fix some checkpath warnings in inode.c and > > unzip_vle.c > [] > > diff --git a/drivers/staging/erofs/unzip_vle.c b/drivers/staging/erofs/unzip_vle.c > [] > > @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block *sb, > > mutex_lock(&work->lock); > > nr_pages = work->nr_pages; > > > > - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) > > + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { > > pages = pages_onstack; > > - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > - mutex_trylock(&z_pagemap_global_lock)) > > + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > + mutex_trylock(&z_pagemap_global_lock)) { > > Extra space after tab There is actually balanced braces in linux-next. https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/staging/erofs/zdata.c#n762 > > > pages = z_pagemap_global; > > - else { > > + } else { > > repeat: > > pages = kvmalloc_array(nr_pages, sizeof(struct page *), > > GFP_KERNEL); > > > > /* fallback to global pagemap for the lowmem scenario */ > > if (unlikely(!pages)) { > > - if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) > > + if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES) { > > goto repeat; > > - else { > > + } else { > > Unnecessary else There is not the "goto repeat" in linux-next anymore. https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/staging/erofs/zdata.c#n765 Thanks, Gao Xiang > > > mutex_lock(&z_pagemap_global_lock); > > pages = z_pagemap_global; > > } > > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 2:31 ` Gao Xiang @ 2019-08-21 15:13 ` Tobin C. Harding 2019-08-21 15:52 ` Gao Xiang via Linux-erofs 0 siblings, 1 reply; 12+ messages in thread From: Tobin C. Harding @ 2019-08-21 15:13 UTC (permalink / raw) To: Gao Xiang Cc: devel, Greg Kroah-Hartman, linux-kernel, Joe Perches, linux-erofs, Caitlyn On Wed, Aug 21, 2019 at 10:31:22AM +0800, Gao Xiang wrote: > On Tue, Aug 20, 2019 at 07:26:46PM -0700, Joe Perches wrote: > > On Tue, 2019-08-20 at 20:18 -0400, Caitlyn wrote: > > > Balanced braces to fix some checkpath warnings in inode.c and > > > unzip_vle.c > > [] > > > diff --git a/drivers/staging/erofs/unzip_vle.c b/drivers/staging/erofs/unzip_vle.c > > [] > > > @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block *sb, > > > mutex_lock(&work->lock); > > > nr_pages = work->nr_pages; > > > > > > - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) > > > + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { > > > pages = pages_onstack; > > > - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > > - mutex_trylock(&z_pagemap_global_lock)) > > > + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > > + mutex_trylock(&z_pagemap_global_lock)) { > > > > Extra space after tab > > There is actually balanced braces in linux-next. > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/staging/erofs/zdata.c#n762 Which tree did these changes go in through please Gao? I believe Caitlyn was working off of the staging-next branch of Greg's staging tree. thanks, Tobin. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 15:13 ` Tobin C. Harding @ 2019-08-21 15:52 ` Gao Xiang via Linux-erofs 2019-08-21 20:11 ` Caitlyn Finn 0 siblings, 1 reply; 12+ messages in thread From: Gao Xiang via Linux-erofs @ 2019-08-21 15:52 UTC (permalink / raw) To: Tobin C. Harding Cc: devel, Greg Kroah-Hartman, linux-kernel, Joe Perches, linux-erofs, Caitlyn Hi Tobin, On Wed, Aug 21, 2019 at 08:13:35AM -0700, Tobin C. Harding wrote: > On Wed, Aug 21, 2019 at 10:31:22AM +0800, Gao Xiang wrote: > > On Tue, Aug 20, 2019 at 07:26:46PM -0700, Joe Perches wrote: > > > On Tue, 2019-08-20 at 20:18 -0400, Caitlyn wrote: > > > > Balanced braces to fix some checkpath warnings in inode.c and > > > > unzip_vle.c > > > [] > > > > diff --git a/drivers/staging/erofs/unzip_vle.c b/drivers/staging/erofs/unzip_vle.c > > > [] > > > > @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block *sb, > > > > mutex_lock(&work->lock); > > > > nr_pages = work->nr_pages; > > > > > > > > - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) > > > > + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { > > > > pages = pages_onstack; > > > > - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > > > - mutex_trylock(&z_pagemap_global_lock)) > > > > + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > > > + mutex_trylock(&z_pagemap_global_lock)) { > > > > > > Extra space after tab > > > > There is actually balanced braces in linux-next. > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/staging/erofs/zdata.c#n762 > > Which tree did these changes go in through please Gao? I believe > Caitlyn was working off of the staging-next branch of Greg's staging > tree. I don't think so, the reason is that unzip_vle.c was renamed to zdata.c months ago, see: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/tree/drivers/staging/erofs?h=staging-next so I think the patch is outdated when I first look at it. Thanks, Gao Xiang > > thanks, > Tobin. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements. 2019-08-21 15:52 ` Gao Xiang via Linux-erofs @ 2019-08-21 20:11 ` Caitlyn Finn 0 siblings, 0 replies; 12+ messages in thread From: Caitlyn Finn @ 2019-08-21 20:11 UTC (permalink / raw) To: Gao Xiang Cc: devel, Greg Kroah-Hartman, linux-kernel, Joe Perches, linux-erofs, Tobin C . Harding On Wed, Aug 21, 2019 at 11:52 AM Gao Xiang <hsiangkao@aol.com> wrote: > > Hi Tobin, > > On Wed, Aug 21, 2019 at 08:13:35AM -0700, Tobin C. Harding wrote: > > On Wed, Aug 21, 2019 at 10:31:22AM +0800, Gao Xiang wrote: > > > On Tue, Aug 20, 2019 at 07:26:46PM -0700, Joe Perches wrote: > > > > On Tue, 2019-08-20 at 20:18 -0400, Caitlyn wrote: > > > > > Balanced braces to fix some checkpath warnings in inode.c and > > > > > unzip_vle.c > > > > [] > > > > > diff --git a/drivers/staging/erofs/unzip_vle.c b/drivers/staging/erofs/unzip_vle.c > > > > [] > > > > > @@ -915,21 +915,21 @@ static int z_erofs_vle_unzip(struct super_block *sb, > > > > > mutex_lock(&work->lock); > > > > > nr_pages = work->nr_pages; > > > > > > > > > > - if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) > > > > > + if (likely(nr_pages <= Z_EROFS_VLE_VMAP_ONSTACK_PAGES)) { > > > > > pages = pages_onstack; > > > > > - else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > > > > - mutex_trylock(&z_pagemap_global_lock)) > > > > > + } else if (nr_pages <= Z_EROFS_VLE_VMAP_GLOBAL_PAGES && > > > > > + mutex_trylock(&z_pagemap_global_lock)) { > > > > > > > > Extra space after tab > > > > > > There is actually balanced braces in linux-next. > > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/staging/erofs/zdata.c#n762 > > > > Which tree did these changes go in through please Gao? I believe > > Caitlyn was working off of the staging-next branch of Greg's staging > > tree. > > I don't think so, the reason is that unzip_vle.c was renamed to zdata.c > months ago, see: > https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/tree/drivers/staging/erofs?h=staging-next > > so I think the patch is outdated when I first look at it. > > Thanks, > Gao Xiang Gao, I see now that I was on an outdated revision (Linux 5.3-rc4) of the staging-next branch of Greg's staging tree, from before that change was merged. I'll be certain that I'm fully up-to-date before submitting future patches. Thanks all for your time and assistance, and Gao and Joe for the review comments as well, I'll review and submit an appropriate patch series at a later time. Thanks, Caitlyn Finn ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) 2019-08-21 0:18 [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) Caitlyn 2019-08-21 0:18 ` [PATCH 1/2] staging/erofs/xattr.h: Fixed misaligned function arguments Caitlyn 2019-08-21 0:18 ` [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements Caitlyn @ 2019-08-21 0:39 ` Gao Xiang via Linux-erofs 2 siblings, 0 replies; 12+ messages in thread From: Gao Xiang via Linux-erofs @ 2019-08-21 0:39 UTC (permalink / raw) To: Caitlyn; +Cc: devel, Greg Kroah-Hartman, linux-kernel, linux-erofs Hi Caitlyn, On Tue, Aug 20, 2019 at 08:18:18PM -0400, Caitlyn wrote: > Hello! > > This patch series cleans up some checkpatch fixes in erofs. The patches > include balancing conditional braces and fixing some indentation. No testing > done, all patches build and checkpath cleanly. I think you need to work on the latest staging tree or linux-next. This patchset cannot be applied (there is the only valid place in inode.c, I will reply in the following patch.) Thanks, Gao Xiang > > Caitlyn (2): > staging/erofs/xattr.h: Fixed misaligned function arguments. > staging/erofs: Balanced braces around a few conditional statements. > > drivers/staging/erofs/inode.c | 4 ++-- > drivers/staging/erofs/unzip_vle.c | 12 ++++++------ > drivers/staging/erofs/xattr.h | 6 +++--- > 3 files changed, 11 insertions(+), 11 deletions(-) > > -- > 2.7.4 > > _______________________________________________ > devel mailing list > devel@linuxdriverproject.org > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2019-08-21 20:11 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-08-21 0:18 [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) Caitlyn 2019-08-21 0:18 ` [PATCH 1/2] staging/erofs/xattr.h: Fixed misaligned function arguments Caitlyn 2019-08-21 2:20 ` Greg Kroah-Hartman 2019-08-21 0:18 ` [PATCH 2/2] staging/erofs: Balanced braces around a few conditional statements Caitlyn 2019-08-21 0:40 ` Gao Xiang via Linux-erofs 2019-08-21 0:55 ` Caitlyn Finn 2019-08-21 2:26 ` Joe Perches 2019-08-21 2:31 ` Gao Xiang 2019-08-21 15:13 ` Tobin C. Harding 2019-08-21 15:52 ` Gao Xiang via Linux-erofs 2019-08-21 20:11 ` Caitlyn Finn 2019-08-21 0:39 ` [PATCH 0/2] Submitting my first patch series (Checkpatch fixes) Gao Xiang via Linux-erofs
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).