Re: [PATCH] mod_devicetable.h: reduce sizeof(struct of_device_id) by 80 bytes

2019-05-02 Thread Rasmus Villemoes
On 02/05/2019 14.29, Jeff Mahoney wrote: > On 5/2/19 5:41 AM, Rasmus Villemoes wrote: >> But we cannot really know whether there is some userspace tool that >> parses the .ko ELF objects the same way that file2alias does, doing >> pattern matching on the symbol names etc. I cannot see why anybody

Re: [PATCH] mod_devicetable.h: reduce sizeof(struct of_device_id) by 80 bytes

2019-05-02 Thread Jeff Mahoney
On 5/2/19 5:41 AM, Rasmus Villemoes wrote: On 26/04/2019 11.27, Arnd Bergmann wrote: On Thu, Apr 25, 2019 at 10:31 PM Rasmus Villemoes wrote: For an arm imx_v6_v7_defconfig kernel, .rodata becomes 70K smaller; .init.data shrinks by another ~13K, making the whole kernel image about 83K, or

Re: [PATCH] mod_devicetable.h: reduce sizeof(struct of_device_id) by 80 bytes

2019-05-02 Thread Rasmus Villemoes
On 26/04/2019 11.27, Arnd Bergmann wrote: > On Thu, Apr 25, 2019 at 10:31 PM Rasmus Villemoes > wrote: >> >> For an arm imx_v6_v7_defconfig kernel, .rodata becomes 70K smaller; >> .init.data shrinks by another ~13K, making the whole kernel image >> about 83K, or 0.3%, smaller. >> >>

Re: [PATCH] mod_devicetable.h: reduce sizeof(struct of_device_id) by 80 bytes

2019-04-26 Thread Rasmus Villemoes
On 26/04/2019 13.05, Enrico Weigelt, metux IT consult wrote: > On 25.04.19 22:31, Rasmus Villemoes wrote: > > Hi, > >> A typical kernel image has hundreds of static struct of_device_id >> instances (a lot of which are sentinel all-zeroes), each occupying >> ~200 bytes. Nobody initializes the

Re: [PATCH] mod_devicetable.h: reduce sizeof(struct of_device_id) by 80 bytes

2019-04-26 Thread Enrico Weigelt, metux IT consult
On 25.04.19 22:31, Rasmus Villemoes wrote: Hi, > A typical kernel image has hundreds of static struct of_device_id > instances (a lot of which are sentinel all-zeroes), each occupying > ~200 bytes. Nobody initializes the .compatible member with strings > anywhere near 128 bytes, so a lot of that

Re: [PATCH] mod_devicetable.h: reduce sizeof(struct of_device_id) by 80 bytes

2019-04-26 Thread Arnd Bergmann
On Thu, Apr 25, 2019 at 10:31 PM Rasmus Villemoes wrote: > > A typical kernel image has hundreds of static struct of_device_id > instances (a lot of which are sentinel all-zeroes), each occupying > ~200 bytes. Nobody initializes the .compatible member with strings > anywhere near 128 bytes, so a