Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Michael Ellerman
Steven Rostedt writes: > On Thu, 02 Mar 2017 22:18:30 +1100 > Michael Ellerman wrote: >> Michael Ellerman writes: >> > David Daney writes: >> >> Strict alignment became necessary with commit 3821fd35b58d

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Jessica Yu
+++ David Daney [02/03/17 11:24 -0800]: On 03/02/2017 10:26 AM, Jessica Yu wrote: +++ Steven Rostedt [02/03/17 13:11 -0500]: Can I get an Ack from a module maintainer? Acked-by: Jessica Yu Thanks! Jessica Thanks Jessica, Can you also add scripts/module-common.lds to

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread David Daney
On 03/02/2017 10:26 AM, Jessica Yu wrote: +++ Steven Rostedt [02/03/17 13:11 -0500]: Can I get an Ack from a module maintainer? Acked-by: Jessica Yu Thanks! Jessica Thanks Jessica, Can you also add scripts/module-common.lds to MAINTAINERS so that get_maintainers.pl

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Jessica Yu
+++ Steven Rostedt [02/03/17 13:11 -0500]: Can I get an Ack from a module maintainer? Acked-by: Jessica Yu Thanks! Jessica On Wed, 1 Mar 2017 14:04:53 -0800 David Daney wrote: For powerpc the __jump_table section in modules is not aligned,

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Steven Rostedt
Can I get an Ack from a module maintainer? Thanks! -- Steve On Wed, 1 Mar 2017 14:04:53 -0800 David Daney wrote: > For powerpc the __jump_table section in modules is not aligned, this > causes a WARN_ON() splat when loading a module containing a __jump_table. > >

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Steven Rostedt
On Thu, 02 Mar 2017 22:18:30 +1100 Michael Ellerman wrote: > Michael Ellerman writes: > > David Daney writes: > >> Strict alignment became necessary with commit 3821fd35b58d > >> ("jump_label: Reduce the size of struct

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Jason Baron
On 03/01/2017 05:04 PM, David Daney wrote: For powerpc the __jump_table section in modules is not aligned, this causes a WARN_ON() splat when loading a module containing a __jump_table. Strict alignment became necessary with commit 3821fd35b58d ("jump_label: Reduce the size of struct

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Michael Ellerman
Michael Ellerman writes: > David Daney writes: >> Strict alignment became necessary with commit 3821fd35b58d >> ("jump_label: Reduce the size of struct static_key"), currently in >> linux-next, which uses the two least significant bits of pointers to

Re: [PATCH] module: set __jump_table alignment to 8

2017-03-02 Thread Michael Ellerman
David Daney writes: > For powerpc the __jump_table section in modules is not aligned, this > causes a WARN_ON() splat when loading a module containing a __jump_table. Thanks for doing the patch. If it helps: Acked-by: Michael Ellerman (powerpc) >

[PATCH] module: set __jump_table alignment to 8

2017-03-01 Thread David Daney
For powerpc the __jump_table section in modules is not aligned, this causes a WARN_ON() splat when loading a module containing a __jump_table. Strict alignment became necessary with commit 3821fd35b58d ("jump_label: Reduce the size of struct static_key"), currently in linux-next, which uses the