[PATCH 11/13] Always define MAX_RAW_MINORS as 65535 in userspace
DORMANTno replies
REVIEWED: 2 (0M)
From: Palmer Dabbelt <hidden>
Date: 2015-09-14 22:50:45
Also in:
linux-arch, linux-fsdevel
Subsystem:
char and misc drivers, the rest · Maintainers:
Arnd Bergmann, Greg Kroah-Hartman, Linus Torvalds
2 review trailers.
Possibly related (same subject, not in this thread)
- 2015-09-17 · Re: [PATCH 11/13] Always define MAX_RAW_MINORS as 65535 in userspace · Palmer Dabbelt <palmer@dabbelt.com>
- 2015-09-15 · Re: [PATCH 11/13] Always define MAX_RAW_MINORS as 65535 in userspace · H. Peter Anvin <hidden>
- 2015-09-14 · [PATCH 11/13] Always define MAX_RAW_MINORS as 65535 in userspace · Palmer Dabbelt <palmer@dabbelt.com>
- 2015-09-14 · [PATCH 11/13] Always define MAX_RAW_MINORS as 65535 in userspace · Palmer Dabbelt <palmer@dabbelt.com>
- 2015-09-14 · [PATCH 11/13] Always define MAX_RAW_MINORS as 65535 in userspace · Palmer Dabbelt <palmer@dabbelt.com>
While I don't think this was ever meant to be exposed to userspace, if anyone is using it then this will at least provide a correct (if unlikely) definition. MAX_RAW_MINORS used to be used in the kernel, where it's been replaced with CONFIG_MAX_RAW_MINORS. Note that there's a checkpatch.pl warning about a split config string here, but I've left that alone. Signed-off-by: Palmer Dabbelt <redacted> Reviewed-by: Andrew Waterman <waterman-aFE07iDfcCIb0cFwG/AQJIdd74u8MsAO@public.gmane.org> Reviewed-by: Albert Ou <aou-aFE07iDfcCIb0cFwG/AQJIdd74u8MsAO@public.gmane.org> --- drivers/char/raw.c | 7 ++++--- include/uapi/linux/raw.h | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/char/raw.c b/drivers/char/raw.c
index 60316fbaf295..362d7a6511b9 100644
--- a/drivers/char/raw.c
+++ b/drivers/char/raw.c@@ -36,7 +36,7 @@ static struct raw_device_data *raw_devices; static DEFINE_MUTEX(raw_mutex); static const struct file_operations raw_ctl_fops; /* forward declaration */ -static int max_raw_minors = MAX_RAW_MINORS; +static int max_raw_minors = CONFIG_MAX_RAW_MINORS; module_param(max_raw_minors, int, 0); MODULE_PARM_DESC(max_raw_minors, "Maximum number of raw devices (1-65536)");
@@ -317,8 +317,9 @@ static int __init raw_init(void) if (max_raw_minors < 1 || max_raw_minors > 65536) { printk(KERN_WARNING "raw: invalid max_raw_minors (must be" - " between 1 and 65536), using %d\n", MAX_RAW_MINORS); - max_raw_minors = MAX_RAW_MINORS; + " between 1 and 65536), using %d\n", + CONFIG_MAX_RAW_MINORS); + max_raw_minors = CONFIG_MAX_RAW_MINORS; } raw_devices = vzalloc(sizeof(struct raw_device_data) * max_raw_minors);
diff --git a/include/uapi/linux/raw.h b/include/uapi/linux/raw.h
index 62d543e70603..f0390b3e8530 100644
--- a/include/uapi/linux/raw.h
+++ b/include/uapi/linux/raw.h@@ -13,6 +13,6 @@ struct raw_config_request __u64 block_minor; }; -#define MAX_RAW_MINORS CONFIG_MAX_RAW_DEVS +#define MAX_RAW_MINORS 65535 #endif /* __LINUX_RAW_H */
--
2.4.6