Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 07b00272 authored by Sagi Grimberg's avatar Sagi Grimberg Committed by Greg Kroah-Hartman
Browse files

params: lift param_set_uint_minmax to common code 



[ Upstream commit 2a14c9ae15a38148484a128b84bff7e9ffd90d68 ]

It is a useful helper hence move it to common code so others can enjoy
it.

Suggested-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarChaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Reviewed-by: default avatarHannes Reinecke <hare@suse.com>
Signed-off-by: default avatarSagi Grimberg <sagi@grimberg.me>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Stable-dep-of: 3ebc46ca8675 ("tcp: Fix shift-out-of-bounds in dctcp_update_alpha().")
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 4a3fcf53

include/linux/moduleparam.h

+2 −0
Original line number Diff line number Diff line
@@ -359,6 +359,8 @@ extern int param_get_int(char *buffer, const struct kernel_param *kp); 
extern const struct kernel_param_ops param_ops_uint;

extern int param_set_uint(const char *val, const struct kernel_param *kp);

extern int param_get_uint(char *buffer, const struct kernel_param *kp);

int param_set_uint_minmax(const char *val, const struct kernel_param *kp,

		unsigned int min, unsigned int max);

#define param_check_uint(name, p) __param_check(name, p, unsigned int)



extern const struct kernel_param_ops param_ops_long;

kernel/params.c

+18 −0
Original line number Diff line number Diff line
@@ -242,6 +242,24 @@ STANDARD_PARAM_DEF(long, long, "%li", kstrtol); 
STANDARD_PARAM_DEF(ulong,	unsigned long,		"%lu",  kstrtoul);

STANDARD_PARAM_DEF(ullong,	unsigned long long,	"%llu", kstrtoull);



int param_set_uint_minmax(const char *val, const struct kernel_param *kp,

		unsigned int min, unsigned int max)

{

	unsigned int num;

	int ret;



	if (!val)

		return -EINVAL;

	ret = kstrtouint(val, 0, &num);

	if (ret)

		return ret;

	if (num < min || num > max)

		return -EINVAL;

	*((unsigned int *)kp->arg) = num;

	return 0;

}

EXPORT_SYMBOL_GPL(param_set_uint_minmax);



int param_set_charp(const char *val, const struct kernel_param *kp)

{

	if (strlen(val) > 1024) {

net/sunrpc/xprtsock.c

+0 −18
Original line number Diff line number Diff line
@@ -3302,24 +3302,6 @@ void cleanup_socket_xprt(void) 
	xprt_unregister_transport(&xs_bc_tcp_transport);

}



static int param_set_uint_minmax(const char *val,

		const struct kernel_param *kp,

		unsigned int min, unsigned int max)

{

	unsigned int num;

	int ret;



	if (!val)

		return -EINVAL;

	ret = kstrtouint(val, 0, &num);

	if (ret)

		return ret;

	if (num < min || num > max)

		return -EINVAL;

	*((unsigned int *)kp->arg) = num;

	return 0;

}



static int param_set_portnr(const char *val, const struct kernel_param *kp)

{

	return param_set_uint_minmax(val, kp,