Re: [PATCH] fs: fix local var type

2018-08-23 Thread kbuild test robot
Hi Weikang, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v4.18 next-20180822] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] fs: fix local var type

2018-08-23 Thread kbuild test robot
Hi Weikang, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v4.18 next-20180822] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] fs: fix local var type

2018-08-23 Thread kbuild test robot
Hi Weikang, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v4.18 next-20180822] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] fs: fix local var type

2018-08-23 Thread kbuild test robot
Hi Weikang, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v4.18 next-20180822] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

RE: [PATCH] fs: fix local var type

2018-08-23 Thread David Laight
From: Michal Hocko > Sent: 23 August 2018 12:14 > > On Thu 23-08-18 01:59:14, Weikang Shi wrote: > > In the seq_hex_dump function,the remaining variable is int, but it receive > > a type of size_t > argument. > > So I change the type of remaining > > The changelog should explain _why_ we need

RE: [PATCH] fs: fix local var type

2018-08-23 Thread David Laight
From: Michal Hocko > Sent: 23 August 2018 12:14 > > On Thu 23-08-18 01:59:14, Weikang Shi wrote: > > In the seq_hex_dump function,the remaining variable is int, but it receive > > a type of size_t > argument. > > So I change the type of remaining > > The changelog should explain _why_ we need

Re: [PATCH] fs: fix local var type

2018-08-23 Thread Michal Hocko
On Thu 23-08-18 01:59:14, Weikang Shi wrote: > In the seq_hex_dump function,the remaining variable is int, but it receive a > type of size_t argument. > So I change the type of remaining The changelog should explain _why_ we need this fix. Is any of the code path overflowing? Besides that I do

Re: [PATCH] fs: fix local var type

2018-08-23 Thread Michal Hocko
On Thu 23-08-18 01:59:14, Weikang Shi wrote: > In the seq_hex_dump function,the remaining variable is int, but it receive a > type of size_t argument. > So I change the type of remaining The changelog should explain _why_ we need this fix. Is any of the code path overflowing? Besides that I do

[PATCH] fs: fix local var type

2018-08-23 Thread Weikang Shi
In the seq_hex_dump function,the remaining variable is int, but it receive a type of size_t argument. So I change the type of remaining Signed-off-by: Weikang Shi --- fs/seq_file.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fs/seq_file.c b/fs/seq_file.c index

[PATCH] fs: fix local var type

2018-08-23 Thread Weikang Shi
In the seq_hex_dump function,the remaining variable is int, but it receive a type of size_t argument. So I change the type of remaining Signed-off-by: Weikang Shi --- fs/seq_file.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fs/seq_file.c b/fs/seq_file.c index