linux-kernel - Re: [PATCH v12 00/15] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X)

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]

Date:	Fri, 23 Jan 2015 13:08:37 +0000
From:	Russell King - ARM Linux <linux@....linux.org.uk>
To:	Geert Uytterhoeven <geert@...ux-m68k.org>
Cc:	Arnd Bergmann <arnd@...db.de>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	Daniel Thompson <daniel.thompson@...aro.org>,
	Paul Bolle <pebolle@...cali.nl>,
	linaro-kernel <linaro-kernel@...ts.linaro.org>,
	"patches@...aro.org" <patches@...aro.org>, spear-devel@...t.st.com,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
Subject: Re: [PATCH v12 00/15] arm: Fix DEBUG_LL for multi-platform kernels
 (without PL01X)

On Fri, Jan 23, 2015 at 02:05:22PM +0100, Geert Uytterhoeven wrote:
> Hi Arnd,
> 
> On Fri, Oct 24, 2014 at 5:46 PM, Arnd Bergmann <arnd@...db.de> wrote:
> > There is still a related bug that we should also fix, but I'd say let's
> > take your current patches first and then add whatever is missing
> > on top. Specifically, a snippet like this
> >
> >         default 0xd4017000 if DEBUG_MMP_UART2
> >         default 0xd4018000 if DEBUG_MMP_UART3
> >         default 0xe0000000 if ARCH_SPEAR13XX
> >         default 0xe4007000 if DEBUG_HIP04_UART
> >         default 0xf0000be0 if ARCH_EBSA110
> >
> > still means you get the wrong default when you build a multiplatform
> > kernel that you want to boot on HIP04 and you set DEBUG_HIP04_UART
> > but you happen to also have ARCH_SPEAR13XX enabled.
> >
> > I have a patch that I use locally for randconfig builds that tries
> > to fix this. It has some overlaps with your work but most parts are
> > distinct. See below.
> 
> This is still desperately needed for anyone who wants to enable DEBUG_LL
> on other platforms using multi_v7_defconfig.

I'd actually prefer to kill all these defaults and let people enter the
value they want - for exactly this reason.

However, I realise that would not be popular, because removing them
means that people have to refer to some kind of documentation to find
the correct value, and that's hassle.

So, the existing defaulting behaviour is my compromise, but as I say,
I'd much prefer the unpopular move of killing the entire set of defaults.

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
--
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/