Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread Richard Weinberger
Am 27.02.2015 um 11:29 schrieb DaeSeok Youn: > Hi, > > 2015-02-27 18:41 GMT+09:00 Richard Weinberger : >> On Fri, Feb 27, 2015 at 10:27 AM, DaeSeok Youn >> wrote: >>> Hi, >>> >>> I think this patch was already queued. Can I send this patch again >>> after update changelog? >> >> One thing that

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread DaeSeok Youn
Hi, 2015-02-27 18:41 GMT+09:00 Richard Weinberger : > On Fri, Feb 27, 2015 at 10:27 AM, DaeSeok Youn wrote: >> Hi, >> >> I think this patch was already queued. Can I send this patch again >> after update changelog? > > One thing that is much more important, how did you test this patch? Did you?

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread Richard Weinberger
On Fri, Feb 27, 2015 at 10:27 AM, DaeSeok Youn wrote: > Hi, > > I think this patch was already queued. Can I send this patch again > after update changelog? One thing that is much more important, how did you test this patch? Did you? -- Thanks, //richard -- To unsubscribe from this list: send

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread DaeSeok Youn
Hi, I think this patch was already queued. Can I send this patch again after update changelog? Thanks. Regards, Daeseok Youn 2015-02-27 6:33 GMT+09:00 Andrew Morton : > On Mon, 23 Feb 2015 19:38:10 +0900 Daeseok Youn > wrote: > >> Use 'retval' instead of 'status'. >> > > The patch does a lot

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread Richard Weinberger
Am 27.02.2015 um 11:29 schrieb DaeSeok Youn: Hi, 2015-02-27 18:41 GMT+09:00 Richard Weinberger richard.weinber...@gmail.com: On Fri, Feb 27, 2015 at 10:27 AM, DaeSeok Youn daeseok.y...@gmail.com wrote: Hi, I think this patch was already queued. Can I send this patch again after update

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread Richard Weinberger
On Fri, Feb 27, 2015 at 10:27 AM, DaeSeok Youn daeseok.y...@gmail.com wrote: Hi, I think this patch was already queued. Can I send this patch again after update changelog? One thing that is much more important, how did you test this patch? Did you? -- Thanks, //richard -- To unsubscribe

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread DaeSeok Youn
Hi, I think this patch was already queued. Can I send this patch again after update changelog? Thanks. Regards, Daeseok Youn 2015-02-27 6:33 GMT+09:00 Andrew Morton a...@linux-foundation.org: On Mon, 23 Feb 2015 19:38:10 +0900 Daeseok Youn daeseok.y...@gmail.com wrote: Use 'retval'

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-27 Thread DaeSeok Youn
Hi, 2015-02-27 18:41 GMT+09:00 Richard Weinberger richard.weinber...@gmail.com: On Fri, Feb 27, 2015 at 10:27 AM, DaeSeok Youn daeseok.y...@gmail.com wrote: Hi, I think this patch was already queued. Can I send this patch again after update changelog? One thing that is much more important,

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-26 Thread Andrew Morton
On Mon, 23 Feb 2015 19:38:10 +0900 Daeseok Youn wrote: > Use 'retval' instead of 'status'. > The patch does a lot more than this. It causes __ocfs2_add_entry to propagate error codes which were previously dropped on the floor. Please update the changelog to fully explain the functional

Re: [PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-26 Thread Andrew Morton
On Mon, 23 Feb 2015 19:38:10 +0900 Daeseok Youn daeseok.y...@gmail.com wrote: Use 'retval' instead of 'status'. The patch does a lot more than this. It causes __ocfs2_add_entry to propagate error codes which were previously dropped on the floor. Please update the changelog to fully explain

[PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-23 Thread Daeseok Youn
Use 'retval' instead of 'status'. Signed-off-by: Daeseok Youn --- fs/ocfs2/dir.c | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/fs/ocfs2/dir.c b/fs/ocfs2/dir.c index b08050b..1478a50 100644 --- a/fs/ocfs2/dir.c +++ b/fs/ocfs2/dir.c @@ -1617,7 +1617,7 @@ int

[PATCH 1/3] ocfs2: remove unneeded variable 'status'

2015-02-23 Thread Daeseok Youn
Use 'retval' instead of 'status'. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- fs/ocfs2/dir.c | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/fs/ocfs2/dir.c b/fs/ocfs2/dir.c index b08050b..1478a50 100644 --- a/fs/ocfs2/dir.c +++ b/fs/ocfs2/dir.c @@