linux-kernel - Re: [PATCH] ARM: multi_v7_defconfig: Enable Intel e1000e driver

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] [day] [month] [year] [list]

Message-ID: <CAOesGMhr_-H7Z86hepnTQbsMtJFv+tDRk=YcRbj2w7OU20yG=Q@mail.gmail.com>
Date:   Mon, 17 Oct 2016 15:03:47 -0700
From:   Olof Johansson <olof@...om.net>
To:     Scott Branden <scott.branden@...adcom.com>
Cc:     Arnd Bergmann <arnd@...db.de>,
        Russell King <linux@...linux.org.uk>,
        Florian Fainelli <f.fainelli@...il.com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        BCM Kernel Feedback <bcm-kernel-feedback-list@...adcom.com>,
        Ray Jui <rjui@...adcom.com>
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Enable Intel e1000e driver

On Tue, Oct 11, 2016 at 3:38 PM, Scott Branden
<scott.branden@...adcom.com> wrote:
> Hi Arnd,
>
> On 16-10-10 12:47 AM, Arnd Bergmann wrote:
>>
>> On Saturday, October 8, 2016 1:41:04 PM CEST Scott Branden wrote:
>>>
>>> Enable support for the Intel e1000e driver
>>>
>>> Signed-off-by: Ray Jui <rjui@...adcom.com>
>>> Signed-off-by: Scott Branden <scott.branden@...adcom.com>
>>>
>>
>> Can we make this a loadable module and group it with the other
>> ethernet drivers?
>>
> We use a fixed rootfs image to test the kernel.  For simplicity we don't
> construct a rootfs or load any modules.  We just test the kernel image. So
> for us every defconfig we use needs to be set to y to use the upstreamed
> kernel.
>
> Plus, how do you NFS mount a rootfs if the ethernet driver is a loadable
> module?

Yeah, agreed that it makes sense to enable it here. Applied to fixes.


-Olof