* [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of t @ 2012-09-19 5:08 Fengguang Wu 2012-09-19 13:25 ` [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 Jeff Layton 0 siblings, 1 reply; 7+ messages in thread From: Fengguang Wu @ 2012-09-19 5:08 UTC (permalink / raw) To: kernel-janitors Hi Jeff, FYI, there are new coccinelle warnings show up in tree: git://git.samba.org/sfrench/cifs-2.6.git for-next head: 324b53bbcd1d9795062f00e92064296169c9b081 commit: a4a23914d0d1736eb6d98049d303d551131df83c [50/71] cifs: turn the pages list in cifs_readdata into an array All coccinelle warnings: + fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 vim +3157 fs/cifs/file.c 3147 rdata->marshal_iov = cifs_readpages_marshal_iov; 3148 > 3149 list_for_each_entry_safe(page, tpage, &tmplist, lru) { 3150 list_del(&page->lru); 3151 rdata->pages[rdata->nr_pages++] = page; 3152 } 3153 3154 rc = cifs_retry_async_readv(rdata); 3155 if (rc != 0) { 3156 for (i = 0; i < rdata->nr_pages; i++) { > 3157 lru_cache_add_file(page); 3158 unlock_page(page); 3159 page_cache_release(page); 3160 } 3161 kref_put(&rdata->refcount, cifs_readdata_release); 3162 break; 3163 } 3164 3165 kref_put(&rdata->refcount, cifs_readdata_release); 3166 } 3167 --- 0-DAY kernel build testing backend Open Source Technology Centre Fengguang Wu, Yuanhan Liu Intel Corporation ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable 2012-09-19 5:08 [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of t Fengguang Wu @ 2012-09-19 13:25 ` Jeff Layton 0 siblings, 0 replies; 7+ messages in thread From: Jeff Layton @ 2012-09-19 13:25 UTC (permalink / raw) To: Fengguang Wu Cc: kernel-janitors-u79uwXL29TY76Z2rM5mHXA, linux-cifs-u79uwXL29TY76Z2rM5mHXA, smfrench-Re5JQEeQqe8AvxtiuMwx3w On Wed, 19 Sep 2012 13:08:02 +0800 Fengguang Wu <fengguang.wu@intel.com> wrote: > Hi Jeff, > > FYI, there are new coccinelle warnings show up in > > tree: git://git.samba.org/sfrench/cifs-2.6.git for-next > head: 324b53bbcd1d9795062f00e92064296169c9b081 > commit: a4a23914d0d1736eb6d98049d303d551131df83c [50/71] cifs: turn the pages list in cifs_readdata into an array > > All coccinelle warnings: > > + fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 > > vim +3157 fs/cifs/file.c > 3147 rdata->marshal_iov = cifs_readpages_marshal_iov; > 3148 > > 3149 list_for_each_entry_safe(page, tpage, &tmplist, lru) { > 3150 list_del(&page->lru); > 3151 rdata->pages[rdata->nr_pages++] = page; > 3152 } > 3153 > 3154 rc = cifs_retry_async_readv(rdata); > 3155 if (rc != 0) { > 3156 for (i = 0; i < rdata->nr_pages; i++) { > > 3157 lru_cache_add_file(page); > 3158 unlock_page(page); > 3159 page_cache_release(page); > 3160 } > 3161 kref_put(&rdata->refcount, cifs_readdata_release); > 3162 break; > 3163 } > 3164 > 3165 kref_put(&rdata->refcount, cifs_readdata_release); > 3166 } > 3167 > > --- > 0-DAY kernel build testing backend Open Source Technology Centre > Fengguang Wu, Yuanhan Liu Intel Corporation Thanks Wu, good catch. I've got a fix that I'll push to Steve today. -- Jeff Layton <jlayton@redhat.com> ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 @ 2012-09-19 13:25 ` Jeff Layton 0 siblings, 0 replies; 7+ messages in thread From: Jeff Layton @ 2012-09-19 13:25 UTC (permalink / raw) To: Fengguang Wu Cc: kernel-janitors-u79uwXL29TY76Z2rM5mHXA, linux-cifs-u79uwXL29TY76Z2rM5mHXA, smfrench-Re5JQEeQqe8AvxtiuMwx3w On Wed, 19 Sep 2012 13:08:02 +0800 Fengguang Wu <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote: > Hi Jeff, > > FYI, there are new coccinelle warnings show up in > > tree: git://git.samba.org/sfrench/cifs-2.6.git for-next > head: 324b53bbcd1d9795062f00e92064296169c9b081 > commit: a4a23914d0d1736eb6d98049d303d551131df83c [50/71] cifs: turn the pages list in cifs_readdata into an array > > All coccinelle warnings: > > + fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 > > vim +3157 fs/cifs/file.c > 3147 rdata->marshal_iov = cifs_readpages_marshal_iov; > 3148 > > 3149 list_for_each_entry_safe(page, tpage, &tmplist, lru) { > 3150 list_del(&page->lru); > 3151 rdata->pages[rdata->nr_pages++] = page; > 3152 } > 3153 > 3154 rc = cifs_retry_async_readv(rdata); > 3155 if (rc != 0) { > 3156 for (i = 0; i < rdata->nr_pages; i++) { > > 3157 lru_cache_add_file(page); > 3158 unlock_page(page); > 3159 page_cache_release(page); > 3160 } > 3161 kref_put(&rdata->refcount, cifs_readdata_release); > 3162 break; > 3163 } > 3164 > 3165 kref_put(&rdata->refcount, cifs_readdata_release); > 3166 } > 3167 > > --- > 0-DAY kernel build testing backend Open Source Technology Centre > Fengguang Wu, Yuanhan Liu Intel Corporation Thanks Wu, good catch. I've got a fix that I'll push to Steve today. -- Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <20120919062508.70ba65eb-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>]
* Re: [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable [not found] ` <20120919062508.70ba65eb-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org> @ 2012-09-19 18:05 ` Jeff Layton 0 siblings, 0 replies; 7+ messages in thread From: Jeff Layton @ 2012-09-19 18:05 UTC (permalink / raw) To: smfrench-Re5JQEeQqe8AvxtiuMwx3w Cc: Fengguang Wu, kernel-janitors-u79uwXL29TY76Z2rM5mHXA, linux-cifs-u79uwXL29TY76Z2rM5mHXA On Wed, 19 Sep 2012 06:25:08 -0700 Jeff Layton <jlayton@redhat.com> wrote: > On Wed, 19 Sep 2012 13:08:02 +0800 > Fengguang Wu <fengguang.wu@intel.com> wrote: > > > Hi Jeff, > > > > FYI, there are new coccinelle warnings show up in > > > > tree: git://git.samba.org/sfrench/cifs-2.6.git for-next > > head: 324b53bbcd1d9795062f00e92064296169c9b081 > > commit: a4a23914d0d1736eb6d98049d303d551131df83c [50/71] cifs: turn the pages list in cifs_readdata into an array > > > > All coccinelle warnings: > > > > + fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 > > > > vim +3157 fs/cifs/file.c > > 3147 rdata->marshal_iov = cifs_readpages_marshal_iov; > > 3148 > > > 3149 list_for_each_entry_safe(page, tpage, &tmplist, lru) { > > 3150 list_del(&page->lru); > > 3151 rdata->pages[rdata->nr_pages++] = page; > > 3152 } > > 3153 > > 3154 rc = cifs_retry_async_readv(rdata); > > 3155 if (rc != 0) { > > 3156 for (i = 0; i < rdata->nr_pages; i++) { > > > 3157 lru_cache_add_file(page); > > 3158 unlock_page(page); > > 3159 page_cache_release(page); > > 3160 } > > 3161 kref_put(&rdata->refcount, cifs_readdata_release); > > 3162 break; > > 3163 } > > 3164 > > 3165 kref_put(&rdata->refcount, cifs_readdata_release); > > 3166 } > > 3167 > > > > --- > > 0-DAY kernel build testing backend Open Source Technology Centre > > Fengguang Wu, Yuanhan Liu Intel Corporation > > > Thanks Wu, good catch. I've got a fix that I'll push to Steve today. > FWIW, here's the (trivial) patch to fix this. Steve, I've respun the patch that introduced this bug and pushed it out to my cifs-next branch. This should be the only difference. You If you'd rather do a patch on top of your tree instead, let me know... ---------------------------[snip]----------------------------- diff --git a/fs/cifs/file.c b/fs/cifs/file.c index 1d43303..075f7cf 100644 --- a/fs/cifs/file.c +++ b/fs/cifs/file.c @@ -3280,6 +3280,7 @@ static int cifs_readpages(struct file *file, struct address_space *mapping, rc = cifs_retry_async_readv(rdata); if (rc != 0) { for (i = 0; i < rdata->nr_pages; i++) { + page = rdata->pages[i]; lru_cache_add_file(page); unlock_page(page); page_cache_release(page); ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 @ 2012-09-19 18:05 ` Jeff Layton 0 siblings, 0 replies; 7+ messages in thread From: Jeff Layton @ 2012-09-19 18:05 UTC (permalink / raw) To: smfrench-Re5JQEeQqe8AvxtiuMwx3w Cc: Fengguang Wu, kernel-janitors-u79uwXL29TY76Z2rM5mHXA, linux-cifs-u79uwXL29TY76Z2rM5mHXA On Wed, 19 Sep 2012 06:25:08 -0700 Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote: > On Wed, 19 Sep 2012 13:08:02 +0800 > Fengguang Wu <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote: > > > Hi Jeff, > > > > FYI, there are new coccinelle warnings show up in > > > > tree: git://git.samba.org/sfrench/cifs-2.6.git for-next > > head: 324b53bbcd1d9795062f00e92064296169c9b081 > > commit: a4a23914d0d1736eb6d98049d303d551131df83c [50/71] cifs: turn the pages list in cifs_readdata into an array > > > > All coccinelle warnings: > > > > + fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 > > > > vim +3157 fs/cifs/file.c > > 3147 rdata->marshal_iov = cifs_readpages_marshal_iov; > > 3148 > > > 3149 list_for_each_entry_safe(page, tpage, &tmplist, lru) { > > 3150 list_del(&page->lru); > > 3151 rdata->pages[rdata->nr_pages++] = page; > > 3152 } > > 3153 > > 3154 rc = cifs_retry_async_readv(rdata); > > 3155 if (rc != 0) { > > 3156 for (i = 0; i < rdata->nr_pages; i++) { > > > 3157 lru_cache_add_file(page); > > 3158 unlock_page(page); > > 3159 page_cache_release(page); > > 3160 } > > 3161 kref_put(&rdata->refcount, cifs_readdata_release); > > 3162 break; > > 3163 } > > 3164 > > 3165 kref_put(&rdata->refcount, cifs_readdata_release); > > 3166 } > > 3167 > > > > --- > > 0-DAY kernel build testing backend Open Source Technology Centre > > Fengguang Wu, Yuanhan Liu Intel Corporation > > > Thanks Wu, good catch. I've got a fix that I'll push to Steve today. > FWIW, here's the (trivial) patch to fix this. Steve, I've respun the patch that introduced this bug and pushed it out to my cifs-next branch. This should be the only difference. You If you'd rather do a patch on top of your tree instead, let me know... ---------------------------[snip]----------------------------- diff --git a/fs/cifs/file.c b/fs/cifs/file.c index 1d43303..075f7cf 100644 --- a/fs/cifs/file.c +++ b/fs/cifs/file.c @@ -3280,6 +3280,7 @@ static int cifs_readpages(struct file *file, struct address_space *mapping, rc = cifs_retry_async_readv(rdata); if (rc != 0) { for (i = 0; i < rdata->nr_pages; i++) { + page = rdata->pages[i]; lru_cache_add_file(page); unlock_page(page); page_cache_release(page); ^ permalink raw reply related [flat|nested] 7+ messages in thread
[parent not found: <20120919110518.05b707e5-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>]
* Re: [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable [not found] ` <20120919110518.05b707e5-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org> @ 2012-09-19 19:42 ` Steve French 0 siblings, 0 replies; 7+ messages in thread From: Steve French @ 2012-09-19 19:42 UTC (permalink / raw) To: Jeff Layton Cc: Fengguang Wu, kernel-janitors-u79uwXL29TY76Z2rM5mHXA, linux-cifs-u79uwXL29TY76Z2rM5mHXA I repulled jeff's cifs-next into my fcifs-2.6. for-next. Added the fixes from Pavel (SMB2 endian fix, and memory leak) and the fix to allow us to negotiate "vers=3.0" - am rebuilding now but expect it do continue to do ok for simple smb2.1 and smb3.0 functional testing (sans signing) - with the known problem of lack of symlinks and rename of open files and cifsacl (chmod) On Wed, Sep 19, 2012 at 1:05 PM, Jeff Layton <jlayton@redhat.com> wrote: > On Wed, 19 Sep 2012 06:25:08 -0700 > Jeff Layton <jlayton@redhat.com> wrote: > >> On Wed, 19 Sep 2012 13:08:02 +0800 >> Fengguang Wu <fengguang.wu@intel.com> wrote: >> >> > Hi Jeff, >> > >> > FYI, there are new coccinelle warnings show up in >> > >> > tree: git://git.samba.org/sfrench/cifs-2.6.git for-next >> > head: 324b53bbcd1d9795062f00e92064296169c9b081 >> > commit: a4a23914d0d1736eb6d98049d303d551131df83c [50/71] cifs: turn the pages list in cifs_readdata into an array >> > >> > All coccinelle warnings: >> > >> > + fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 >> > >> > vim +3157 fs/cifs/file.c >> > 3147 rdata->marshal_iov = cifs_readpages_marshal_iov; >> > 3148 >> > > 3149 list_for_each_entry_safe(page, tpage, &tmplist, lru) { >> > 3150 list_del(&page->lru); >> > 3151 rdata->pages[rdata->nr_pages++] = page; >> > 3152 } >> > 3153 >> > 3154 rc = cifs_retry_async_readv(rdata); >> > 3155 if (rc != 0) { >> > 3156 for (i = 0; i < rdata->nr_pages; i++) { >> > > 3157 lru_cache_add_file(page); >> > 3158 unlock_page(page); >> > 3159 page_cache_release(page); >> > 3160 } >> > 3161 kref_put(&rdata->refcount, cifs_readdata_release); >> > 3162 break; >> > 3163 } >> > 3164 >> > 3165 kref_put(&rdata->refcount, cifs_readdata_release); >> > 3166 } >> > 3167 >> > >> > --- >> > 0-DAY kernel build testing backend Open Source Technology Centre >> > Fengguang Wu, Yuanhan Liu Intel Corporation >> >> >> Thanks Wu, good catch. I've got a fix that I'll push to Steve today. >> > > FWIW, here's the (trivial) patch to fix this. Steve, I've respun the > patch that introduced this bug and pushed it out to my cifs-next > branch. This should be the only difference. > > You If you'd rather do a patch on top of your tree instead, let me > know... > > ---------------------------[snip]----------------------------- > diff --git a/fs/cifs/file.c b/fs/cifs/file.c > index 1d43303..075f7cf 100644 > --- a/fs/cifs/file.c > +++ b/fs/cifs/file.c > @@ -3280,6 +3280,7 @@ static int cifs_readpages(struct file *file, struct address_space *mapping, > rc = cifs_retry_async_readv(rdata); > if (rc != 0) { > for (i = 0; i < rdata->nr_pages; i++) { > + page = rdata->pages[i]; > lru_cache_add_file(page); > unlock_page(page); > page_cache_release(page); -- Thanks, Steve ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 @ 2012-09-19 19:42 ` Steve French 0 siblings, 0 replies; 7+ messages in thread From: Steve French @ 2012-09-19 19:42 UTC (permalink / raw) To: Jeff Layton Cc: Fengguang Wu, kernel-janitors-u79uwXL29TY76Z2rM5mHXA, linux-cifs-u79uwXL29TY76Z2rM5mHXA I repulled jeff's cifs-next into my fcifs-2.6. for-next. Added the fixes from Pavel (SMB2 endian fix, and memory leak) and the fix to allow us to negotiate "vers=3.0" - am rebuilding now but expect it do continue to do ok for simple smb2.1 and smb3.0 functional testing (sans signing) - with the known problem of lack of symlinks and rename of open files and cifsacl (chmod) On Wed, Sep 19, 2012 at 1:05 PM, Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote: > On Wed, 19 Sep 2012 06:25:08 -0700 > Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote: > >> On Wed, 19 Sep 2012 13:08:02 +0800 >> Fengguang Wu <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote: >> >> > Hi Jeff, >> > >> > FYI, there are new coccinelle warnings show up in >> > >> > tree: git://git.samba.org/sfrench/cifs-2.6.git for-next >> > head: 324b53bbcd1d9795062f00e92064296169c9b081 >> > commit: a4a23914d0d1736eb6d98049d303d551131df83c [50/71] cifs: turn the pages list in cifs_readdata into an array >> > >> > All coccinelle warnings: >> > >> > + fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 >> > >> > vim +3157 fs/cifs/file.c >> > 3147 rdata->marshal_iov = cifs_readpages_marshal_iov; >> > 3148 >> > > 3149 list_for_each_entry_safe(page, tpage, &tmplist, lru) { >> > 3150 list_del(&page->lru); >> > 3151 rdata->pages[rdata->nr_pages++] = page; >> > 3152 } >> > 3153 >> > 3154 rc = cifs_retry_async_readv(rdata); >> > 3155 if (rc != 0) { >> > 3156 for (i = 0; i < rdata->nr_pages; i++) { >> > > 3157 lru_cache_add_file(page); >> > 3158 unlock_page(page); >> > 3159 page_cache_release(page); >> > 3160 } >> > 3161 kref_put(&rdata->refcount, cifs_readdata_release); >> > 3162 break; >> > 3163 } >> > 3164 >> > 3165 kref_put(&rdata->refcount, cifs_readdata_release); >> > 3166 } >> > 3167 >> > >> > --- >> > 0-DAY kernel build testing backend Open Source Technology Centre >> > Fengguang Wu, Yuanhan Liu Intel Corporation >> >> >> Thanks Wu, good catch. I've got a fix that I'll push to Steve today. >> > > FWIW, here's the (trivial) patch to fix this. Steve, I've respun the > patch that introduced this bug and pushed it out to my cifs-next > branch. This should be the only difference. > > You If you'd rather do a patch on top of your tree instead, let me > know... > > ---------------------------[snip]----------------------------- > diff --git a/fs/cifs/file.c b/fs/cifs/file.c > index 1d43303..075f7cf 100644 > --- a/fs/cifs/file.c > +++ b/fs/cifs/file.c > @@ -3280,6 +3280,7 @@ static int cifs_readpages(struct file *file, struct address_space *mapping, > rc = cifs_retry_async_readv(rdata); > if (rc != 0) { > for (i = 0; i < rdata->nr_pages; i++) { > + page = rdata->pages[i]; > lru_cache_add_file(page); > unlock_page(page); > page_cache_release(page); -- Thanks, Steve ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-09-19 19:42 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-19 5:08 [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of t Fengguang Wu
2012-09-19 13:25 ` [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable Jeff Layton
2012-09-19 13:25 ` [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 Jeff Layton
[not found] ` <20120919062508.70ba65eb-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>
2012-09-19 18:05 ` [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable Jeff Layton
2012-09-19 18:05 ` [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 Jeff Layton
[not found] ` <20120919110518.05b707e5-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>
2012-09-19 19:42 ` [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable Steve French
2012-09-19 19:42 ` [cifs:for-next 50/71] fs/cifs/file.c:3157:23-27: ERROR: invalid reference to the index variable of the iterator on line 3149 Steve French
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.