LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH v2] MTD: cfi: reduce stack size
@ 2015-03-09 19:34 Arnd Bergmann
  2015-03-09 20:29 ` Sergei Shtylyov
  0 siblings, 1 reply; 2+ messages in thread
From: Arnd Bergmann @ 2015-03-09 19:34 UTC (permalink / raw)
  To: David Woodhouse, Brian Norris
  Cc: Ezequiel Garcia, Jingoo Han, linux-mtd, linux-kernel, linux-arm-kernel

The cfi_staa_write_buffers function uses a large amount of kernel stack
whenever CONFIG_MTD_MAP_BANK_WIDTH_32 is set, and that results in a
warning on ARM allmodconfig builds:

drivers/mtd/chips/cfi_cmdset_0020.c: In function 'cfi_staa_write_buffers':
drivers/mtd/chips/cfi_cmdset_0020.c:651:1: warning: the frame size of 1208 bytes is larger than 1024 bytes [-Wframe-larger-than=]

It turns out that this is largely a result of a suboptimal implementation
of map_word_andequal(). Replacing this function with a straightforward
one reduces the stack size in this function by exactly 200 bytes,
shrinks the .text segment for this file from 27648 bytes to 26608 bytes,
and makes the warning go away.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>

diff --git a/include/linux/mtd/map.h b/include/linux/mtd/map.h
index 5f487d776411..7b5841663d4f 100644
--- a/include/linux/mtd/map.h
+++ b/include/linux/mtd/map.h
@@ -314,7 +314,15 @@ static inline map_word map_word_or(struct map_info *map, map_word val1, map_word
 	return r;
 }
 
-#define map_word_andequal(m, a, b, z) map_word_equal(m, z, map_word_and(m, a, b))
+static inline int map_word_andequal(struct map_info *map, map_word val1, map_word val2, map_word val3)
+{
+	int i;
+	for (i=0; i < map_words(map); i++) {
+		if ((val1.x[i] & val2.x[i]) != val3.x[i])
+			return 0;
+	}
+	return 1;
+}
 
 static inline int map_word_bitsset(struct map_info *map, map_word val1, map_word val2)
 {


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] MTD: cfi: reduce stack size
  2015-03-09 19:34 [PATCH v2] MTD: cfi: reduce stack size Arnd Bergmann
@ 2015-03-09 20:29 ` Sergei Shtylyov
  0 siblings, 0 replies; 2+ messages in thread
From: Sergei Shtylyov @ 2015-03-09 20:29 UTC (permalink / raw)
  To: Arnd Bergmann, David Woodhouse, Brian Norris
  Cc: Jingoo Han, linux-mtd, Ezequiel Garcia, linux-arm-kernel, linux-kernel

Hello.

On 3/9/2015 10:34 PM, Arnd Bergmann wrote:

> The cfi_staa_write_buffers function uses a large amount of kernel stack
> whenever CONFIG_MTD_MAP_BANK_WIDTH_32 is set, and that results in a
> warning on ARM allmodconfig builds:

> drivers/mtd/chips/cfi_cmdset_0020.c: In function 'cfi_staa_write_buffers':
> drivers/mtd/chips/cfi_cmdset_0020.c:651:1: warning: the frame size of 1208 bytes is larger than 1024 bytes [-Wframe-larger-than=]

> It turns out that this is largely a result of a suboptimal implementation
> of map_word_andequal(). Replacing this function with a straightforward
> one reduces the stack size in this function by exactly 200 bytes,
> shrinks the .text segment for this file from 27648 bytes to 26608 bytes,
> and makes the warning go away.

> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

> diff --git a/include/linux/mtd/map.h b/include/linux/mtd/map.h
> index 5f487d776411..7b5841663d4f 100644
> --- a/include/linux/mtd/map.h
> +++ b/include/linux/mtd/map.h
> @@ -314,7 +314,15 @@ static inline map_word map_word_or(struct map_info *map, map_word val1, map_word
>   	return r;
>   }
>
> -#define map_word_andequal(m, a, b, z) map_word_equal(m, z, map_word_and(m, a, b))
> +static inline int map_word_andequal(struct map_info *map, map_word val1, map_word val2, map_word val3)
> +{
> +	int i;

    I think empty line is needed here.

> +	for (i=0; i < map_words(map); i++) {

    Could you please be consistent and surround all binary operators with 
spaces, like on the line below?

> +		if ((val1.x[i] & val2.x[i]) != val3.x[i])
> +			return 0;
> +	}
> +	return 1;
> +}
[...]

WBR, Sergei


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-03-09 20:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-09 19:34 [PATCH v2] MTD: cfi: reduce stack size Arnd Bergmann
2015-03-09 20:29 ` Sergei Shtylyov

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).