linux-kernel - Re: [PATCH 02/13] ARM: multi_v7_defconfig: Add missing systemd options

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:   Sun, 25 Nov 2018 14:25:04 -0800
From:   Olof Johansson <olof@...om.net>
To:     Otavio Salvador <otavio@...ystems.com.br>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux ARM Mailing List <linux-arm-kernel@...ts.infradead.org>,
        "open list:ARM/Rockchip SoC..." <linux-rockchip@...ts.infradead.org>,
        fabio.berton@...ystems.com.br, yannick.fertre@...com,
        Arnd Bergmann <arnd@...db.de>, Eric Anholt <eric@...olt.net>,
        Alexandre TORGUE <alexandre.torgue@...com>,
        Stefan Wahren <stefan.wahren@...e.com>,
        Stefan Agner <stefan@...er.ch>,
        Tony Lindgren <tony@...mide.com>,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        Russell King <linux@...linux.org.uk>,
        Simon Horman <horms+renesas@...ge.net.au>
Subject: Re: [PATCH 02/13] ARM: multi_v7_defconfig: Add missing systemd options

On Sun, Nov 25, 2018 at 1:19 PM Otavio Salvador <otavio@...ystems.com.br> wrote:
>
> From: Fabio Berton <fabio.berton@...ystems.com.br>
>
> Signed-off-by: Fabio Berton <fabio.berton@...ystems.com.br>
> Signed-off-by: Otavio Salvador <otavio@...ystems.com.br>

Hi Otavio,

Thanks for these patches. I don't have a reason to _not_ enable
systemd features, but would you mind providing a brief description of
what's not working without this change? Is it completely broken, or is
it just the random-UID featues that don't work, or something else?.
Useful for others to know what's expected to break if someone turns
them off, etc.

Also, I noticed you posted a series of 13 patches, which includes some
dts files for rv1108, some network driver changes and some of these
more generic defconfig changes. I would suggest that you split up your
patches into independent series where possible, especially when they
end up going to different subsystems. Providing suitable cover letters
for the series when you post them is also useful to help reviewers by
summarizing the series, including changes from last time it was
posted, etc. Please address it to the superset of patch receivers on
the series.

I'm asking for the cover letter because this time I was wondering if
there was a reason for the mix of patches so I went looking for it.
Maybe there is a need to group them together even though the
sub-series seem somewhat independent -- I can't tell from the way they
were posted.

Thanks!

-Olof