[dpdk-dev] [PATCH v2 5/6] net/sfc: add support for MCDI proxy

Ferruh Yigit ferruh.yigit at intel.com
Wed Mar 15 12:13:23 CET 2017


On 3/9/2017 5:23 PM, Andrew Rybchenko wrote:
> From: Ivan Malov <ivan.malov at oktetlabs.ru>
> 
> The patch is to add support for MCDI proxy which comes in
> useful, particularly, while running over VF: few commands
> will normally fail with EPERM, but in some cases the host
> driver (i.e. running over the corresponding PF, typically,
> within a hypervisor) may set itself as a proxy to conduct
> authorization for the commands coming from VFs; these are
> forwarded to the corresponding access control application
> which may decline or approve authorization by replying to
> the requests; all in all, the guest driver has to process
> the replies forwarded back by the firmware MC in order to
> give up gracefully (by setting return code which could be
> understood by 'libefx') or re-issue the original commands

Hi Andrew, Ivan,

I noticed that all lines in the commit log end in same column,
interestingly this is true for other patches in this patchset [1],
is this mere coincidence or is there a poetic effort here :)

Thanks for the patches,
ferruh

[1]
    If Rx mode is unacceptable, in particular, when promiscuous
    or all-multicast filters are not allowed while running over
    PCI function which is not a member of appropriate privilege
    groups, the driver has to cope with the failures gracefully

...

    If periodic DMA statistics feature is absent (particularly,
    while running over VF), the PMD must provide an ability to
    cope with it using explicit update requests which are kept
    restrained according to 'stats_update_period_ms' parameter

...

    The patch is to make MAC statistics update interval tunable
    by means of 'stats_update_period_ms' kvarg parameter making
    it possible to use values different from 1000 ms in case of
    SFN8xxx boards provided that firmware version is 6.2.1.1033



> 
> Signed-off-by: Ivan Malov <ivan.malov at oktetlabs.ru>
> Signed-off-by: Andrew Rybchenko <arybchenko at solarflare.com>
> Reviewed-by: Andy Moreton <amoreton at solarflare.com>
<...>


More information about the dev mailing list