linux-kernel - Re: [PATCH 0/1] net: ethtool: add SmartNIC reset support

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: <e8f5b581-2613-0dda-d9cc-e826a14ee394@broadcom.com>
Date:   Wed, 18 Oct 2017 12:31:28 -0700
From:   Scott Branden <scott.branden@...adcom.com>
To:     Andrew Lunn <andrew@...n.ch>
Cc:     "David S. Miller" <davem@...emloft.net>,
        "Allan W. Nielsen" <allan.nielsen@...rosemi.com>,
        Raju Lakkaraju <Raju.Lakkaraju@...rosemi.com>,
        Florian Fainelli <f.fainelli@...il.com>,
        BCM Kernel Feedback <bcm-kernel-feedback-list@...adcom.com>,
        linux-kernel@...r.kernel.org, Steve Lin <steven.lin1@...adcom.com>
Subject: Re: [PATCH 0/1] net: ethtool: add SmartNIC reset support

Hi Andrew,


On 17-10-18 12:16 PM, Andrew Lunn wrote:
>> Yes, there is also a management processor.
> O.K.
>
> Maybe it would be nice to add some more text to the commit message to
> make this clear. Define what an application processor is, and how it
> differs from a management processor. But othersize:
OK -will add more description to differentiate management processor vs. 
application processor(s).
>
> Reviewed-by: Andrew Lunn <andrew@...n.ch>
>
> I assume you have another kernel patch to actually make use of this?
> It is normal to post the user of a new API in the same series as the
> API.
I actually wanted to get agreement that the bit define could be added to 
ethtool before implementing it in driver.
If this direction approved we'll implement in driver and submit with 
this patch series.
>
>      Andrew
Thanks,
  Scott