From: Alejandro Colomar <colomar.6.4.3@gmail.com>
To: Lucas Segarra Fernandez <lucas.segarra.fernandez@intel.com>,
herbert@gondor.apana.org.au, linux-kernel@vger.kernel.org
Cc: linux-crypto@vger.kernel.org, qat-linux@intel.com,
andriy.shevchenko@intel.com,
Giovanni Cabiddu <giovanni.cabiddu@intel.com>,
Andy Shevchenko <andy.shevchenko@gmail.com>
Subject: Re: [PATCH 1/4] linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header
Date: Thu, 17 Aug 2023 22:24:11 +0200 [thread overview]
Message-ID: <121ae7d6-db88-4ec7-a7d5-e7b1c14b49d4@gmail.com> (raw)
In-Reply-To: <20230817143352.132583-2-lucas.segarra.fernandez@intel.com>
[-- Attachment #1.1: Type: text/plain, Size: 10665 bytes --]
Hi Lucas,
On 2023-08-17 16:33, Lucas Segarra Fernandez wrote:
> From: Alejandro Colomar <alx.manpages@gmail.com>
>
> 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@gmail.com>
> Reviewed-by: Giovanni Cabiddu <giovanni.cabiddu@intel.com>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
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@kernel.org>
Signed-off-by: Alejandro Colomar <alx@kernel.org>
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@opensource.wolfsonmicro.com>
> */
>
> +#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 #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2023-08-17 20:25 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-17 14:33 [PATCH 0/4] Add debugfs pm_status for qat driver Lucas Segarra Fernandez
2023-08-17 14:33 ` [PATCH 1/4] linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header Lucas Segarra Fernandez
2023-08-17 20:24 ` Alejandro Colomar [this message]
2023-08-17 14:33 ` [PATCH 2/4] linux/array_size.h: Add ARRAY_SIZE_OF_FIELD() Lucas Segarra Fernandez
2023-08-17 20:34 ` Alejandro Colomar
2023-08-17 21:30 ` Andy Shevchenko
2023-08-17 22:03 ` Alejandro Colomar
2023-08-17 14:33 ` [PATCH 3/4] crypto: qat - refactor included headers Lucas Segarra Fernandez
2023-08-17 14:33 ` [PATCH 4/4] crypto: qat - add pm_status debugfs file Lucas Segarra Fernandez
2023-08-18 5:28 ` Herbert Xu
2023-08-18 8:46 ` Andy Shevchenko
2023-08-18 8:51 ` Herbert Xu
2023-08-20 19:52 ` [PATCH] linux/container_of.h: Add memberof() Alejandro Colomar
2023-08-21 11:18 ` Andy Shevchenko
2023-08-21 11:23 ` Alejandro Colomar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=121ae7d6-db88-4ec7-a7d5-e7b1c14b49d4@gmail.com \
--to=colomar.6.4.3@gmail.com \
--cc=alx@kernel.org \
--cc=andriy.shevchenko@intel.com \
--cc=andy.shevchenko@gmail.com \
--cc=giovanni.cabiddu@intel.com \
--cc=herbert@gondor.apana.org.au \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lucas.segarra.fernandez@intel.com \
--cc=qat-linux@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox