linux-kernel - Re: [PATCH] powerpc/prom_init: remove linux,stdout-package property

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: <87a7qoq3z2.fsf@concordia.ellerman.id.au>
Date:   Wed, 18 Jul 2018 19:37:37 +1000
From:   Michael Ellerman <mpe@...erman.id.au>
To:     Murilo Opsfelder Araujo <muriloo@...ux.ibm.com>,
        linux-kernel@...r.kernel.org
Cc:     Murilo Opsfelder Araujo <muriloo@...ux.ibm.com>,
        Benjamin Herrenschmidt <benh@...nel.crashing.org>,
        Paul Mackerras <paulus@...ba.org>,
        Mathieu Malaterre <malat@...ian.org>,
        Michael Bringmann <mwb@...ux.vnet.ibm.com>,
        Nathan Fontenot <nfont@...ux.vnet.ibm.com>,
        Kees Cook <keescook@...omium.org>,
        Bharata B Rao <bharata@...ux.vnet.ibm.com>,
        Alexey Kardashevskiy <aik@...abs.ru>,
        Cédric Le Goater <clg@...d.org>,
        "Aneesh Kumar K . V" <aneesh.kumar@...ux.ibm.com>,
        Nicholas Piggin <npiggin@...il.com>,
        linuxppc-dev@...ts.ozlabs.org
Subject: Re: [PATCH] powerpc/prom_init: remove linux,stdout-package property

Hi Murilo,

Thanks for the patch.

Murilo Opsfelder Araujo <muriloo@...ux.ibm.com> writes:
> This property was added in 2004 by
>
>     https://github.com/mpe/linux-fullhistory/commit/689fe5072fe9a0dec914bfa4fa60aed1e54563e6
>
> and the only use of it, which was already inside `#if 0`, was removed a month
> later by
>
>     https://github.com/mpe/linux-fullhistory/commit/1fbe5a6d90f6cd4ea610737ef488719d1a875de7
>
> Fixes: https://github.com/linuxppc/linux/issues/125

That is going to confuse some scripts that are expecting that to be a
"Fixes: <some commit>" tag :)

The proper tag to use there would be "Link:".

But, I'd prefer we didn't add github issue links to the history, as I'm
not sure they won't bit-rot eventually. Not because I'm a anti-Microsoft
conspiracy person but just because it's a repo I set up and manage and
there's no long term plan for it necessarily.

> ---
>  arch/powerpc/kernel/prom_init.c | 2 --
>  1 file changed, 2 deletions(-)

Including the link here would be ideal, as it means it doesn't end up in
the commit history, but it does end up in the mail archive. So if we
ever really need to find it, it should be there.

cheers

> diff --git a/arch/powerpc/kernel/prom_init.c b/arch/powerpc/kernel/prom_init.c
> index 5425dd3d6a9f..c45fb463c9e5 100644
> --- a/arch/powerpc/kernel/prom_init.c
> +++ b/arch/powerpc/kernel/prom_init.c
> @@ -2102,8 +2102,6 @@ static void __init prom_init_stdout(void)
>  	stdout_node = call_prom("instance-to-package", 1, 1, prom.stdout);
>  	if (stdout_node != PROM_ERROR) {
>  		val = cpu_to_be32(stdout_node);
> -		prom_setprop(prom.chosen, "/chosen", "linux,stdout-package",
> -			     &val, sizeof(val));
>  
>  		/* If it's a display, note it */
>  		memset(type, 0, sizeof(type));
> -- 
> 2.17.1