* [PATCH] blockgroup_lock.h: simplify definition of NR_BG_LOCKS
@ 2016-09-14 19:28 Eric Biggers
2016-09-14 21:09 ` Andreas Dilger
0 siblings, 1 reply; 3+ messages in thread
From: Eric Biggers @ 2016-09-14 19:28 UTC (permalink / raw)
To: tytso, adilger.kernel; +Cc: linux-ext4, linux-kernel, Eric Biggers
We can use ilog2() to more easily produce the desired NR_BG_LOCKS. This
works because ilog2() is evaluated at compile-time when its argument is
a compile-time constant.
I did not change the chosen NR_BG_LOCKS values.
Signed-off-by: Eric Biggers <ebiggers@google.com>
---
include/linux/blockgroup_lock.h | 22 ++--------------------
1 file changed, 2 insertions(+), 20 deletions(-)
diff --git a/include/linux/blockgroup_lock.h b/include/linux/blockgroup_lock.h
index 61b583d..225bdb7 100644
--- a/include/linux/blockgroup_lock.h
+++ b/include/linux/blockgroup_lock.h
@@ -10,28 +10,10 @@
#include <linux/cache.h>
#ifdef CONFIG_SMP
-
-/*
- * We want a power-of-two. Is there a better way than this?
- */
-
-#if NR_CPUS >= 32
-#define NR_BG_LOCKS 128
-#elif NR_CPUS >= 16
-#define NR_BG_LOCKS 64
-#elif NR_CPUS >= 8
-#define NR_BG_LOCKS 32
-#elif NR_CPUS >= 4
-#define NR_BG_LOCKS 16
-#elif NR_CPUS >= 2
-#define NR_BG_LOCKS 8
+#define NR_BG_LOCKS (4 << ilog2(NR_CPUS < 32 ? NR_CPUS : 32))
#else
-#define NR_BG_LOCKS 4
-#endif
-
-#else /* CONFIG_SMP */
#define NR_BG_LOCKS 1
-#endif /* CONFIG_SMP */
+#endif
struct bgl_lock {
spinlock_t lock;
--
2.8.0.rc3.226.g39d4020
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] blockgroup_lock.h: simplify definition of NR_BG_LOCKS
2016-09-14 19:28 [PATCH] blockgroup_lock.h: simplify definition of NR_BG_LOCKS Eric Biggers
@ 2016-09-14 21:09 ` Andreas Dilger
2016-09-15 22:29 ` Theodore Ts'o
0 siblings, 1 reply; 3+ messages in thread
From: Andreas Dilger @ 2016-09-14 21:09 UTC (permalink / raw)
To: Eric Biggers; +Cc: Theodore Ts'o, linux-ext4, LKML
[-- Attachment #1: Type: text/plain, Size: 1480 bytes --]
On Sep 14, 2016, at 1:28 PM, Eric Biggers <ebiggers@google.com> wrote:
>
> We can use ilog2() to more easily produce the desired NR_BG_LOCKS. This
> works because ilog2() is evaluated at compile-time when its argument is
> a compile-time constant.
>
> I did not change the chosen NR_BG_LOCKS values.
>
> Signed-off-by: Eric Biggers <ebiggers@google.com>
Reviewed-by: Andreas Dilger <adilger@dilger.ca>
> ---
> include/linux/blockgroup_lock.h | 22 ++--------------------
> 1 file changed, 2 insertions(+), 20 deletions(-)
>
> diff --git a/include/linux/blockgroup_lock.h b/include/linux/blockgroup_lock.h
> index 61b583d..225bdb7 100644
> --- a/include/linux/blockgroup_lock.h
> +++ b/include/linux/blockgroup_lock.h
> @@ -10,28 +10,10 @@
> #include <linux/cache.h>
>
> #ifdef CONFIG_SMP
> -
> -/*
> - * We want a power-of-two. Is there a better way than this?
> - */
> -
> -#if NR_CPUS >= 32
> -#define NR_BG_LOCKS 128
> -#elif NR_CPUS >= 16
> -#define NR_BG_LOCKS 64
> -#elif NR_CPUS >= 8
> -#define NR_BG_LOCKS 32
> -#elif NR_CPUS >= 4
> -#define NR_BG_LOCKS 16
> -#elif NR_CPUS >= 2
> -#define NR_BG_LOCKS 8
> +#define NR_BG_LOCKS (4 << ilog2(NR_CPUS < 32 ? NR_CPUS : 32))
> #else
> -#define NR_BG_LOCKS 4
> -#endif
> -
> -#else /* CONFIG_SMP */
> #define NR_BG_LOCKS 1
> -#endif /* CONFIG_SMP */
> +#endif
>
> struct bgl_lock {
> spinlock_t lock;
> --
> 2.8.0.rc3.226.g39d4020
>
Cheers, Andreas
[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] blockgroup_lock.h: simplify definition of NR_BG_LOCKS
2016-09-14 21:09 ` Andreas Dilger
@ 2016-09-15 22:29 ` Theodore Ts'o
0 siblings, 0 replies; 3+ messages in thread
From: Theodore Ts'o @ 2016-09-15 22:29 UTC (permalink / raw)
To: Andreas Dilger; +Cc: Eric Biggers, linux-ext4, LKML
On Wed, Sep 14, 2016 at 03:09:30PM -0600, Andreas Dilger wrote:
> On Sep 14, 2016, at 1:28 PM, Eric Biggers <ebiggers@google.com> wrote:
> >
> > We can use ilog2() to more easily produce the desired NR_BG_LOCKS. This
> > works because ilog2() is evaluated at compile-time when its argument is
> > a compile-time constant.
> >
> > I did not change the chosen NR_BG_LOCKS values.
> >
> > Signed-off-by: Eric Biggers <ebiggers@google.com>
>
> Reviewed-by: Andreas Dilger <adilger@dilger.ca>
Applied, thanks.
- Ted
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-15 22:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-14 19:28 [PATCH] blockgroup_lock.h: simplify definition of NR_BG_LOCKS Eric Biggers
2016-09-14 21:09 ` Andreas Dilger
2016-09-15 22:29 ` Theodore Ts'o
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).