Re: [PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-23 Thread Miroslav Benes
On Fri, 16 Dec 2016, Josh Poimboeuf wrote: > On Fri, Dec 16, 2016 at 05:55:55PM +0100, Petr Mladek wrote: > > On Thu 2016-12-08 12:08:36, Josh Poimboeuf wrote: > > > The sysfs enabled value is a boolean, so kstrtobool() is a better fit > > > for parsing the input string since it does the range

Re: [PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-23 Thread Miroslav Benes
On Fri, 16 Dec 2016, Josh Poimboeuf wrote: > On Fri, Dec 16, 2016 at 05:55:55PM +0100, Petr Mladek wrote: > > On Thu 2016-12-08 12:08:36, Josh Poimboeuf wrote: > > > The sysfs enabled value is a boolean, so kstrtobool() is a better fit > > > for parsing the input string since it does the range

Re: [PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-16 Thread Josh Poimboeuf
On Fri, Dec 16, 2016 at 05:55:55PM +0100, Petr Mladek wrote: > On Thu 2016-12-08 12:08:36, Josh Poimboeuf wrote: > > The sysfs enabled value is a boolean, so kstrtobool() is a better fit > > for parsing the input string since it does the range checking for us. > > > > Suggested-by: Petr Mladek

Re: [PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-16 Thread Josh Poimboeuf
On Fri, Dec 16, 2016 at 05:55:55PM +0100, Petr Mladek wrote: > On Thu 2016-12-08 12:08:36, Josh Poimboeuf wrote: > > The sysfs enabled value is a boolean, so kstrtobool() is a better fit > > for parsing the input string since it does the range checking for us. > > > > Suggested-by: Petr Mladek >

Re: [PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-16 Thread Petr Mladek
On Thu 2016-12-08 12:08:36, Josh Poimboeuf wrote: > The sysfs enabled value is a boolean, so kstrtobool() is a better fit > for parsing the input string since it does the range checking for us. > > Suggested-by: Petr Mladek > Signed-off-by: Josh Poimboeuf

Re: [PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-16 Thread Petr Mladek
On Thu 2016-12-08 12:08:36, Josh Poimboeuf wrote: > The sysfs enabled value is a boolean, so kstrtobool() is a better fit > for parsing the input string since it does the range checking for us. > > Suggested-by: Petr Mladek > Signed-off-by: Josh Poimboeuf > --- > kernel/livepatch/core.c | 11

[PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-08 Thread Josh Poimboeuf
The sysfs enabled value is a boolean, so kstrtobool() is a better fit for parsing the input string since it does the range checking for us. Suggested-by: Petr Mladek Signed-off-by: Josh Poimboeuf --- kernel/livepatch/core.c | 11 --- 1 file

[PATCH v3 11/15] livepatch: use kstrtobool() in enabled_store()

2016-12-08 Thread Josh Poimboeuf
The sysfs enabled value is a boolean, so kstrtobool() is a better fit for parsing the input string since it does the range checking for us. Suggested-by: Petr Mladek Signed-off-by: Josh Poimboeuf --- kernel/livepatch/core.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-)