Re: [PATCH] compat: Fix endian issue in union sigval

2015-02-16 Thread Bamvor Jian Zhang
On 2015/2/14 19:22, Catalin Marinas wrote: On Fri, Feb 13, 2015 at 04:56:29PM -0500, Chris Metcalf wrote: On 2/13/2015 5:44 AM, Catalin Marinas wrote: On Fri, Feb 13, 2015 at 04:00:43PM +0800, Bamvor Jian Zhang wrote: diff --git a/arch/arm64/kernel/signal32.c b/arch/arm64/kernel/signal32.c

Re: [PATCH] compat: Fix endian issue in union sigval

2015-02-16 Thread Bamvor Jian Zhang
On 2015/2/13 18:44, Catalin Marinas wrote: On Fri, Feb 13, 2015 at 04:00:43PM +0800, Bamvor Jian Zhang wrote: On 2015/2/11 23:40, Catalin Marinas wrote: On Wed, Feb 11, 2015 at 07:22:08PM +0800, Bamvor Jian Zhang wrote: On 2015/2/10 20:27, Catalin Marinas wrote: On Tue, Feb 10, 2015 at 10:10

Re: [PATCH] compat: Fix endian issue in union sigval

2015-02-13 Thread Bamvor Jian Zhang
On 2015/2/11 23:40, Catalin Marinas wrote: > On Wed, Feb 11, 2015 at 07:22:08PM +0800, Bamvor Jian Zhang wrote: >> On 2015/2/10 20:27, Catalin Marinas wrote: >>> On Tue, Feb 10, 2015 at 10:10:11AM +, Zhang Jian(Bamvor) wrote: ... > The native sigval_t is also a union but

Re: [PATCH] compat: Fix endian issue in union sigval

2015-02-13 Thread Bamvor Jian Zhang
On 2015/2/11 23:40, Catalin Marinas wrote: On Wed, Feb 11, 2015 at 07:22:08PM +0800, Bamvor Jian Zhang wrote: On 2015/2/10 20:27, Catalin Marinas wrote: On Tue, Feb 10, 2015 at 10:10:11AM +, Zhang Jian(Bamvor) wrote: ... The native sigval_t is also a union but on 64-bit big endian

Re: [PATCH] compat: Fix endian issue in union sigval

2015-02-11 Thread Bamvor Jian Zhang
On 2015/2/10 20:27, Catalin Marinas wrote: > cc'ing linux-arch as well. > > On Tue, Feb 10, 2015 at 10:10:11AM +, Zhang Jian(Bamvor) wrote: >> In 64bit architecture, sigval_int is the high 32bit of sigval_ptr in >> big endian kernel compare with low 32bit of sigval_ptr in little >> endian

Re: [PATCH] compat: Fix endian issue in union sigval

2015-02-11 Thread Bamvor Jian Zhang
On 2015/2/10 20:27, Catalin Marinas wrote: cc'ing linux-arch as well. On Tue, Feb 10, 2015 at 10:10:11AM +, Zhang Jian(Bamvor) wrote: In 64bit architecture, sigval_int is the high 32bit of sigval_ptr in big endian kernel compare with low 32bit of sigval_ptr in little endian kernel.

<    1   2