linux-kernel - Re: [PATCH] init/Kconfig: Add ENDIAN attributes for all architectures using

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <201409151555.03997.arnd@arndb.de>
Date:	Mon, 15 Sep 2014 15:55:03 +0200
From:	Arnd Bergmann <arnd@...db.de>
To:	Chen Gang <gang.chen.5i5j@...il.com>
Cc:	Paul Gortmaker <paul.gortmaker@...driver.com>,
	"H. Peter Anvin" <hpa@...or.com>, eparis@...hat.com,
	paulmck@...ux.vnet.ibm.com,
	Geert Uytterhoeven <geert@...ux-m68k.org>,
	zhenglong.cai@...c.com.cn, khilman@...aro.org, ak@...ux.intel.com,
	mcgrof@...e.com, fabf@...net.be,
	"dhowells@...hat.com" <dhowells@...hat.com>, pefoley2@...oley.com,
	mgorman@...e.de, biederm@...ssion.com,
	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Jean Delvare <jdelvare@...e.de>
Subject: Re: [PATCH] init/Kconfig: Add ENDIAN attributes for all architectures using

On Sunday 14 September 2014, Chen Gang wrote:
> Hello Maintainers:
> 
> Is this patch worthy enough to be applied?
> 
> Welcome any ideas, suggestions or completions.

I find it hard to judge whether it's worth it or not. You gave one example
for a driver that needs this, but what would be your estimate on how
many other drivers have the same problem, either producing a #warning
or misbehaving if neither symbol is set?

Having all architectures consistently report the endianess would be nice,
but we have to weigh the advantages (currently broken code, potential
simplifications) against the regression risk.

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/