netdev - Re: [PATCH net-next v2 2/2] usb: class: cdc-wdm: WWAN framework integration

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: <878s4kxldn.fsf@miraculix.mork.no>
Date:   Wed, 12 May 2021 13:05:40 +0200
From:   Bjørn Mork <bjorn@...k.no>
To:     Loic Poulain <loic.poulain@...aro.org>
Cc:     Aleksander Morgado <aleksander@...ksander.es>,
        Jakub Kicinski <kuba@...nel.org>,
        Oliver Neukum <oliver@...kum.org>,
        "David S. Miller" <davem@...emloft.net>,
        Network Development <netdev@...r.kernel.org>,
        dcbw@...ps.redhat.com
Subject: Re: [PATCH net-next v2 2/2] usb: class: cdc-wdm: WWAN framework
 integration

Loic Poulain <loic.poulain@...aro.org> writes:

> I'm however open discussing other strategies:
> - Do not change anything, keep USB WWAN devices out of the WWAN subsystem.
> - Migrate cdc-wdm completely to WWAN and get rid of the cdc-wdm chardev
> - Build time choice, if CONFIG_WWAN, registered to WWAN, otherwise
> exposed through cdc-wdm chardev.
> - Run time choice, use either the new WWAN chardev or the legacy
> cdc-wdm chardev (this patch)
> - ...
>
> I would be interested in getting input from others/maintainers on this.

I appreciate the work you do.  When I am silent on the issue, it's
because I don't know the best solution (this is a well proven fact :-)
and don't want to affect the results by arbitrary comments.

There is one thing I can say though:  No build time options, please.


Bjørn