Re: [PATCH 1/4] linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header

From: Alejandro Colomar
Date: Thu Aug 17 2023 - 16:25:02 EST

Next message: Rob Herring: "[PATCH v2] media: dt-bindings: Merge OV5695 into OV5693 binding"
Previous message: Meyer, Kyle: "Re: [PATCH] KVM: x86: Increase KVM_MAX_VCPUS to 4096"
In reply to: Lucas Segarra Fernandez: "[PATCH 1/4] linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header"
Next in thread: Lucas Segarra Fernandez: "[PATCH 3/4] crypto: qat - refactor included headers"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Lucas,

On 2023-08-17 16:33, Lucas Segarra Fernandez wrote:
> From: Alejandro Colomar <alx.manpages@xxxxxxxxx>
>
> Touching files so used for the kernel,
> forces 'make' to recompile most of the kernel.
>
> Having those definitions in more granular files
> helps avoid recompiling so much of the kernel.
>
> Signed-off-by: Alejandro Colomar <alx.manpages@xxxxxxxxx>
> Reviewed-by: Giovanni Cabiddu <giovanni.cabiddu@xxxxxxxxx>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>

Heh! This is really really old, isn't it? :p

Would you mind updating my email to use the kernel.org address?

From: Alejandro Colomar <alx@xxxxxxxxxx>
Signed-off-by: Alejandro Colomar <alx@xxxxxxxxxx>

Thanks for taking care of this patch!

Cheers,
Alex

> ---
> include/linux/array_size.h | 13 +++++++++++++
> include/linux/clk-provider.h | 1 +
> include/linux/counter.h | 1 +
> include/linux/genl_magic_func.h | 1 +
> include/linux/hashtable.h | 1 +
> include/linux/kernel.h | 7 +------
> include/linux/kfifo.h | 1 +
> include/linux/kvm_host.h | 1 +
> include/linux/moduleparam.h | 2 ++
> include/linux/mtd/rawnand.h | 1 +
> include/linux/netfilter.h | 1 +
> include/linux/pagemap.h | 1 +
> include/linux/phy.h | 1 +
> include/linux/pinctrl/machine.h | 2 +-
> include/linux/property.h | 1 +
> include/linux/rcupdate_wait.h | 1 +
> include/linux/regmap.h | 1 +
> include/linux/skmsg.h | 1 +
> include/linux/string.h | 1 +
> include/linux/surface_aggregator/controller.h | 1 +
> 20 files changed, 33 insertions(+), 7 deletions(-)
> create mode 100644 include/linux/array_size.h
>
> diff --git a/include/linux/array_size.h b/include/linux/array_size.h
> new file mode 100644
> index 000000000000..06d7d83196ca
> --- /dev/null
> +++ b/include/linux/array_size.h
> @@ -0,0 +1,13 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +#ifndef _LINUX_ARRAY_SIZE_H
> +#define _LINUX_ARRAY_SIZE_H
> +
> +#include <linux/compiler.h>
> +
> +/**
> + * ARRAY_SIZE - get the number of elements in array @arr
> + * @arr: array to be sized
> + */
> +#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
> +
> +#endif /* _LINUX_ARRAY_SIZE_H */
> diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h
> index 0f0cd01906b4..4f4d4f4af0a6 100644
> --- a/include/linux/clk-provider.h
> +++ b/include/linux/clk-provider.h
> @@ -6,6 +6,7 @@
> #ifndef __LINUX_CLK_PROVIDER_H
> #define __LINUX_CLK_PROVIDER_H
>
> +#include <linux/array_size.h>
> #include <linux/of.h>
> #include <linux/of_clk.h>
>
> diff --git a/include/linux/counter.h b/include/linux/counter.h
> index b63746637de2..baf4ffcd8d18 100644
> --- a/include/linux/counter.h
> +++ b/include/linux/counter.h
> @@ -6,6 +6,7 @@
> #ifndef _COUNTER_H_
> #define _COUNTER_H_
>
> +#include <linux/array_size.h>
> #include <linux/cdev.h>
> #include <linux/device.h>
> #include <linux/kernel.h>
> diff --git a/include/linux/genl_magic_func.h b/include/linux/genl_magic_func.h
> index 2984b0cb24b1..cec9cae51f0b 100644
> --- a/include/linux/genl_magic_func.h
> +++ b/include/linux/genl_magic_func.h
> @@ -2,6 +2,7 @@
> #ifndef GENL_MAGIC_FUNC_H
> #define GENL_MAGIC_FUNC_H
>
> +#include <linux/array_size.h>
> #include <linux/build_bug.h>
> #include <linux/genl_magic_struct.h>
>
> diff --git a/include/linux/hashtable.h b/include/linux/hashtable.h
> index f6c666730b8c..09c5f1522b06 100644
> --- a/include/linux/hashtable.h
> +++ b/include/linux/hashtable.h
> @@ -7,6 +7,7 @@
> #ifndef _LINUX_HASHTABLE_H
> #define _LINUX_HASHTABLE_H
>
> +#include <linux/array_size.h>
> #include <linux/list.h>
> #include <linux/types.h>
> #include <linux/kernel.h>
> diff --git a/include/linux/kernel.h b/include/linux/kernel.h
> index 0d91e0af0125..7195c6f27a22 100644
> --- a/include/linux/kernel.h
> +++ b/include/linux/kernel.h
> @@ -11,6 +11,7 @@
> #ifndef _LINUX_KERNEL_H
> #define _LINUX_KERNEL_H
>
> +#include <linux/array_size.h>
> #include <linux/stdarg.h>
> #include <linux/align.h>
> #include <linux/limits.h>
> @@ -49,12 +50,6 @@
> #define READ 0
> #define WRITE 1
>
> -/**
> - * ARRAY_SIZE - get the number of elements in array @arr
> - * @arr: array to be sized
> - */
> -#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
> -
> #define PTR_IF(cond, ptr) ((cond) ? (ptr) : NULL)
>
> #define u64_to_user_ptr(x) ( \
> diff --git a/include/linux/kfifo.h b/include/linux/kfifo.h
> index 0b35a41440ff..b6fdd92ccb56 100644
> --- a/include/linux/kfifo.h
> +++ b/include/linux/kfifo.h
> @@ -36,6 +36,7 @@
> * to lock the reader.
> */
>
> +#include <linux/array_size.h>
> #include <linux/kernel.h>
> #include <linux/spinlock.h>
> #include <linux/stddef.h>
> diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h
> index 9d3ac7720da9..a629b398a592 100644
> --- a/include/linux/kvm_host.h
> +++ b/include/linux/kvm_host.h
> @@ -3,6 +3,7 @@
> #define __KVM_HOST_H
>
>
> +#include <linux/array_size.h>
> #include <linux/types.h>
> #include <linux/hardirq.h>
> #include <linux/list.h>
> diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h
> index 962cd41a2cb5..3cecef5fa1cf 100644
> --- a/include/linux/moduleparam.h
> +++ b/include/linux/moduleparam.h
> @@ -2,6 +2,8 @@
> #ifndef _LINUX_MODULE_PARAMS_H
> #define _LINUX_MODULE_PARAMS_H
> /* (C) Copyright 2001, 2002 Rusty Russell IBM Corporation */
> +
> +#include <linux/array_size.h>
> #include <linux/init.h>
> #include <linux/stringify.h>
> #include <linux/kernel.h>
> diff --git a/include/linux/mtd/rawnand.h b/include/linux/mtd/rawnand.h
> index 5159d692f9ce..cd27ef633a4f 100644
> --- a/include/linux/mtd/rawnand.h
> +++ b/include/linux/mtd/rawnand.h
> @@ -13,6 +13,7 @@
> #ifndef __LINUX_MTD_RAWNAND_H
> #define __LINUX_MTD_RAWNAND_H
>
> +#include <linux/array_size.h>
> #include <linux/mtd/mtd.h>
> #include <linux/mtd/nand.h>
> #include <linux/mtd/flashchip.h>
> diff --git a/include/linux/netfilter.h b/include/linux/netfilter.h
> index d4fed4c508ca..f9ca506c4261 100644
> --- a/include/linux/netfilter.h
> +++ b/include/linux/netfilter.h
> @@ -2,6 +2,7 @@
> #ifndef __LINUX_NETFILTER_H
> #define __LINUX_NETFILTER_H
>
> +#include <linux/array_size.h>
> #include <linux/init.h>
> #include <linux/skbuff.h>
> #include <linux/net.h>
> diff --git a/include/linux/pagemap.h b/include/linux/pagemap.h
> index 716953ee1ebd..7a3de980ed9d 100644
> --- a/include/linux/pagemap.h
> +++ b/include/linux/pagemap.h
> @@ -5,6 +5,7 @@
> /*
> * Copyright 1995 Linus Torvalds
> */
> +#include <linux/array_size.h>
> #include <linux/mm.h>
> #include <linux/fs.h>
> #include <linux/list.h>
> diff --git a/include/linux/phy.h b/include/linux/phy.h
> index 11c1e91563d4..39e88b570ead 100644
> --- a/include/linux/phy.h
> +++ b/include/linux/phy.h
> @@ -11,6 +11,7 @@
> #ifndef __PHY_H
> #define __PHY_H
>
> +#include <linux/array_size.h>
> #include <linux/compiler.h>
> #include <linux/spinlock.h>
> #include <linux/ethtool.h>
> diff --git a/include/linux/pinctrl/machine.h b/include/linux/pinctrl/machine.h
> index 0639b36f43c5..ee8803f6ad07 100644
> --- a/include/linux/pinctrl/machine.h
> +++ b/include/linux/pinctrl/machine.h
> @@ -11,7 +11,7 @@
> #ifndef __LINUX_PINCTRL_MACHINE_H
> #define __LINUX_PINCTRL_MACHINE_H
>
> -#include <linux/kernel.h> /* ARRAY_SIZE() */
> +#include <linux/array_size.h>
>
> #include <linux/pinctrl/pinctrl-state.h>
>
> diff --git a/include/linux/property.h b/include/linux/property.h
> index 8c3c6685a2ae..f7889c7c3a66 100644
> --- a/include/linux/property.h
> +++ b/include/linux/property.h
> @@ -10,6 +10,7 @@
> #ifndef _LINUX_PROPERTY_H_
> #define _LINUX_PROPERTY_H_
>
> +#include <linux/array_size.h>
> #include <linux/bits.h>
> #include <linux/fwnode.h>
> #include <linux/stddef.h>
> diff --git a/include/linux/rcupdate_wait.h b/include/linux/rcupdate_wait.h
> index 699b938358bf..a321404eeec0 100644
> --- a/include/linux/rcupdate_wait.h
> +++ b/include/linux/rcupdate_wait.h
> @@ -6,6 +6,7 @@
> * RCU synchronization types and methods:
> */
>
> +#include <linux/array_size.h>
> #include <linux/rcupdate.h>
> #include <linux/completion.h>
>
> diff --git a/include/linux/regmap.h b/include/linux/regmap.h
> index 8fc0b3ebce44..af0430dc0945 100644
> --- a/include/linux/regmap.h
> +++ b/include/linux/regmap.h
> @@ -10,6 +10,7 @@
> * Author: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
> */
>
> +#include <linux/array_size.h>
> #include <linux/list.h>
> #include <linux/rbtree.h>
> #include <linux/ktime.h>
> diff --git a/include/linux/skmsg.h b/include/linux/skmsg.h
> index 054d7911bfc9..d7e3b9f46d58 100644
> --- a/include/linux/skmsg.h
> +++ b/include/linux/skmsg.h
> @@ -4,6 +4,7 @@
> #ifndef _LINUX_SKMSG_H
> #define _LINUX_SKMSG_H
>
> +#include <linux/array_size.h>
> #include <linux/bpf.h>
> #include <linux/filter.h>
> #include <linux/scatterlist.h>
> diff --git a/include/linux/string.h b/include/linux/string.h
> index dbfc66400050..3c920b6d609b 100644
> --- a/include/linux/string.h
> +++ b/include/linux/string.h
> @@ -2,6 +2,7 @@
> #ifndef _LINUX_STRING_H_
> #define _LINUX_STRING_H_
>
> +#include <linux/array_size.h>
> #include <linux/compiler.h> /* for inline */
> #include <linux/types.h> /* for size_t */
> #include <linux/stddef.h> /* for NULL */
> diff --git a/include/linux/surface_aggregator/controller.h b/include/linux/surface_aggregator/controller.h
> index cb7980805920..dcce0b663a3a 100644
> --- a/include/linux/surface_aggregator/controller.h
> +++ b/include/linux/surface_aggregator/controller.h
> @@ -12,6 +12,7 @@
> #ifndef _LINUX_SURFACE_AGGREGATOR_CONTROLLER_H
> #define _LINUX_SURFACE_AGGREGATOR_CONTROLLER_H
>
> +#include <linux/array_size.h>
> #include <linux/completion.h>
> #include <linux/device.h>
> #include <linux/types.h>

--
<http://www.alejandro-colomar.es/>
GPG key fingerprint: A9348594CE31283A826FBDD8D57633D441E25BB5

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

Next message: Rob Herring: "[PATCH v2] media: dt-bindings: Merge OV5695 into OV5693 binding"
Previous message: Meyer, Kyle: "Re: [PATCH] KVM: x86: Increase KVM_MAX_VCPUS to 4096"
In reply to: Lucas Segarra Fernandez: "[PATCH 1/4] linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header"
Next in thread: Lucas Segarra Fernandez: "[PATCH 3/4] crypto: qat - refactor included headers"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]