[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1496960243-196898-1-git-send-email-babu.moger@oracle.com>
Date: Thu, 8 Jun 2017 15:17:21 -0700
From: Babu Moger <babu.moger@...cle.com>
To: ysato@...rs.sourceforge.jp, geert@...ux-m68k.org,
jonas@...thpole.se, stefan.kristiansson@...nalahti.fi,
shorne@...il.com, jejb@...isc-linux.org, deller@....de,
davem@...emloft.net, viro@...iv.linux.org.uk
Cc: mpe@...erman.id.au, peterz@...radead.org, mingo@...hat.com,
jcmvbkbc@...il.com, linux-kernel@...r.kernel.org,
uclinux-h8-devel@...ts.sourceforge.jp, linux-m68k@...r.kernel.org,
openrisc@...ts.librecores.org, linux-parisc@...r.kernel.org,
sparclinux@...r.kernel.org
Subject: [PATCH 0/2] Define CPU_BIG_ENDIAN or warn for inconsistencies
Found this problem while enabling queued rwlock on SPARC.
The parameter CONFIG_CPU_BIG_ENDIAN is used to clear the
specific byte in qrwlock structure. Without this parameter,
we clear the wrong byte. Here is the code.
static inline u8 *__qrwlock_write_byte(struct qrwlock *lock)
{
return (u8 *)lock + 3 * IS_BUILTIN(CONFIG_CPU_BIG_ENDIAN);
}
Here is the previous discussion.
http://www.spinics.net/lists/devicetree/msg178101.html
Based on the discussion, it was decided to add CONFIG_CPU_BIG_ENDIAN
for all the fixed big endian architecture(frv, h8300, m68k, openrisc,
parisc and sparc). And warn if there are inconsistencies in this definition.
Babu Moger (2):
arch: Define CPU_BIG_ENDIAN for all fixed big endian archs
include: warn for inconsistent endian config definition
arch/frv/Kconfig | 3 +++
arch/h8300/Kconfig | 3 +++
arch/m68k/Kconfig | 3 +++
arch/openrisc/Kconfig | 3 +++
arch/parisc/Kconfig | 3 +++
arch/sparc/Kconfig | 3 +++
include/linux/byteorder/big_endian.h | 4 ++++
include/linux/byteorder/little_endian.h | 4 ++++
8 files changed, 26 insertions(+), 0 deletions(-)
Powered by blists - more mailing lists