linux-kernel - Re: [RESEND PATCH v3 3/3] sh: add the sh

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:   Tue, 23 Oct 2018 12:10:46 +0100
From:   Marc Zyngier <marc.zyngier@....com>
To:     Bartosz Golaszewski <brgl@...ev.pl>
Cc:     Sekhar Nori <nsekhar@...com>, Kevin Hilman <khilman@...nel.org>,
        David Lechner <david@...hnology.com>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>, Arnd Bergmann <arnd@...db.de>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Mark Rutland <mark.rutland@....com>,
        Yoshinori Sato <ysato@...rs.sourceforge.jp>,
        Rich Felker <dalias@...c.org>,
        Andy Shevchenko <andy.shevchenko@...il.com>,
        "Rafael J . Wysocki" <rafael.j.wysocki@...el.com>,
        Peter Rosin <peda@...ntia.se>, Jiri Slaby <jslaby@...e.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Geert Uytterhoeven <geert@...ux-m68k.org>,
        Magnus Damm <magnus.damm@...il.com>,
        Johan Hovold <johan@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Frank Rowand <frowand.list@...il.com>,
        Philippe Ombredanne <pombredanne@...b.com>,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        devicetree@...r.kernel.org, linux-arch@...r.kernel.org,
        Bartosz Golaszewski <bgolaszewski@...libre.com>
Subject: Re: [RESEND PATCH v3 3/3] sh: add the sh_ prefix to early platform symbols

Hi Bartosz,

On Mon, 27 Aug 2018 10:06:51 +0100,
Bartosz Golaszewski <brgl@...ev.pl> wrote:
> 
> From: Bartosz Golaszewski <bgolaszewski@...libre.com>
> 
> Old early platform device support is now sh-specific. Before moving on
> to implementing new early platform framework based on real platform
> devices, prefix all early platform symbols with 'sh_'.
> 
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@...libre.com>
> ---
>  arch/sh/drivers/platform_early.c       | 66 +++++++++++++-------------
>  arch/sh/include/asm/platform_early.h   | 30 ++++++------
>  arch/sh/kernel/cpu/sh2/setup-sh7619.c  |  2 +-
>  arch/sh/kernel/cpu/sh2a/setup-mxg.c    |  2 +-
>  arch/sh/kernel/cpu/sh2a/setup-sh7201.c |  2 +-
>  arch/sh/kernel/cpu/sh2a/setup-sh7203.c |  2 +-
>  arch/sh/kernel/cpu/sh2a/setup-sh7206.c |  2 +-
>  arch/sh/kernel/cpu/sh2a/setup-sh7264.c |  2 +-
>  arch/sh/kernel/cpu/sh2a/setup-sh7269.c |  2 +-
>  arch/sh/kernel/cpu/sh3/setup-sh7705.c  |  2 +-
>  arch/sh/kernel/cpu/sh3/setup-sh770x.c  |  2 +-
>  arch/sh/kernel/cpu/sh3/setup-sh7710.c  |  2 +-
>  arch/sh/kernel/cpu/sh3/setup-sh7720.c  |  2 +-
>  arch/sh/kernel/cpu/sh4/setup-sh4-202.c |  2 +-
>  arch/sh/kernel/cpu/sh4/setup-sh7750.c  |  8 ++--
>  arch/sh/kernel/cpu/sh4/setup-sh7760.c  |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7343.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7366.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7722.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7723.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7724.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7734.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7757.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7763.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7770.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7780.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7785.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-sh7786.c |  2 +-
>  arch/sh/kernel/cpu/sh4a/setup-shx3.c   |  2 +-
>  arch/sh/kernel/cpu/sh5/setup-sh5.c     |  2 +-
>  arch/sh/kernel/setup.c                 |  2 +-
>  arch/sh/kernel/time.c                  |  4 +-
>  drivers/clocksource/sh_cmt.c           |  6 +--
>  drivers/clocksource/sh_mtu2.c          |  6 +--
>  drivers/clocksource/sh_tmu.c           |  6 +--
>  drivers/tty/serial/sh-sci.c            |  4 +-
>  include/linux/platform_device.h        |  2 +-
>  37 files changed, 94 insertions(+), 94 deletions(-)
> 
> diff --git a/arch/sh/drivers/platform_early.c b/arch/sh/drivers/platform_early.c
> index 3dfe9f3541bb..f6d148451dfc 100644
> --- a/arch/sh/drivers/platform_early.c
> +++ b/arch/sh/drivers/platform_early.c
> @@ -4,8 +4,8 @@
>  #include <linux/mod_devicetable.h>
>  #include <linux/pm.h>
>  
> -static __initdata LIST_HEAD(early_platform_driver_list);
> -static __initdata LIST_HEAD(early_platform_device_list);
> +static __initdata LIST_HEAD(sh_early_platform_driver_list);
> +static __initdata LIST_HEAD(sh_early_platform_device_list);

As discussed offline at ELC, I don't think this patch is useful at
this stage, and in the context of this particular series. I believe it
should be dropped and only the first two patches should be merged. If
the SH maintainer is not responsive, then maybe Greg could take them
as they remove cruft from core code.

As for the whole "early device" debate, my take is that we'd be much
better off making the device model available early, rather than trying
to architect something that is simply papering over the issue.

Thanks,

	M.

-- 
Jazz is not dead, it just smell funny.