2017-04-24 17:50:14

by Karim Eshapa

[permalink] [raw]
Subject: PATCH drivers:net:cris/eth_v10: alternate string char arrary

static char pointer creates two variables in final assembly.
static string and pointer to it according to
Jeff Garzik janitors TODO.

Signed-off-by: Karim Eshapa <[email protected]>
---
drivers/net/cris/eth_v10.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/cris/eth_v10.c b/drivers/net/cris/eth_v10.c
index 91c876a..370765f 100644
--- a/drivers/net/cris/eth_v10.c
+++ b/drivers/net/cris/eth_v10.c
@@ -44,7 +44,7 @@
* io regions, irqs and dma channels
*/

-static const char* cardname = "ETRAX 100LX built-in ethernet controller";
+static const char cardname[] = "ETRAX 100LX built-in ethernet controller";

/* A default ethernet address. Highlevel SW will set the real one later */

--
2.7.4


2017-04-24 18:19:52

by David Miller

[permalink] [raw]
Subject: Re: PATCH drivers:net:cris/eth_v10: alternate string char arrary

From: Karim Eshapa <[email protected]>
Date: Mon, 24 Apr 2017 19:49:39 +0200

> static char pointer creates two variables in final assembly.
> static string and pointer to it according to
> Jeff Garzik janitors TODO.
>
> Signed-off-by: Karim Eshapa <[email protected]>

Instead of trusting some document written more than 10 years ago on
the internet, why don't you build the source file in question and take
a look at what actually happens?

I'm not applying this, sorry.

2017-04-24 19:55:57

by Karim Eshapa

[permalink] [raw]
Subject: RE:PATCH drivers:net:cris/eth_v10: alternate string char arrary

On Mon, 24 Apr 2017 14:18:58 -0400 (EDT), David Miller wrote:
> Mon, 24 Apr 2017 19:49:39 +0200, Karim Eshapa wrote:
>>
>> static char pointer creates two variables in final assembly.
>> static string and pointer to it according to
>> Jeff Garzik janitors TODO.
>
> Instead of trusting some document written more than 10 years ago on
> the internet, why don't you build the source file in question and take
> a look at what actually happens?
>

I've just dumped the assembly and symbols. I didn't find
extraordinary difference between the two strings.
sorry for that, but why is that still there in
TODO kerneljanitors at kernelnewbies.

Thanks,
Karim

2017-04-24 20:04:38

by David Miller

[permalink] [raw]
Subject: Re: PATCH drivers:net:cris/eth_v10: alternate string char arrary

From: Karim Eshapa <[email protected]>
Date: Mon, 24 Apr 2017 21:54:42 +0200

> On Mon, 24 Apr 2017 14:18:58 -0400 (EDT), David Miller wrote:
>> Mon, 24 Apr 2017 19:49:39 +0200, Karim Eshapa wrote:
>>>
>>> static char pointer creates two variables in final assembly.
>>> static string and pointer to it according to
>>> Jeff Garzik janitors TODO.
>>
>> Instead of trusting some document written more than 10 years ago on
>> the internet, why don't you build the source file in question and take
>> a look at what actually happens?
>>
>
> I've just dumped the assembly and symbols. I didn't find
> extraordinary difference between the two strings.
> sorry for that, but why is that still there in
> TODO kerneljanitors at kernelnewbies.

Because nobody is updating the documentation.