From: Bert Kenward <bkenw...@solarflare.com> Date: Thu, 10 Dec 2015 13:30:08 +0000
> Without this filter insertion on a VF would fail if only one channel > was in use. This would include the unicast station filter and therefore > no traffic would be received. > > Signed-off-by: Bert Kenward <bkenw...@solarflare.com> > --- > drivers/net/ethernet/sfc/ef10.c | 23 +++++++++++------------ > drivers/net/ethernet/sfc/efx.h | 5 +++++ > drivers/net/ethernet/sfc/farch.c | 2 +- > 3 files changed, 17 insertions(+), 13 deletions(-) > > diff --git a/drivers/net/ethernet/sfc/ef10.c b/drivers/net/ethernet/sfc/ef10.c > index c4a0e8a..368e98e 100644 > --- a/drivers/net/ethernet/sfc/ef10.c > +++ b/drivers/net/ethernet/sfc/ef10.c > @@ -3309,7 +3309,7 @@ static int efx_ef10_filter_remove_internal(struct > efx_nic *efx, > > new_spec.priority = EFX_FILTER_PRI_AUTO; > new_spec.flags = (EFX_FILTER_FLAG_RX | > - EFX_FILTER_FLAG_RX_RSS); > + (efx_rss_enabled(efx) ? EFX_FILTER_FLAG_RX_RSS : 0)); This was indented properly before your change, please don't damage the indentation like this. There must be 4 TAB characters and 2 SPACE characters on that second line of the assignment so that the first character is precisely at the first column after the openning parenthesis of the first line. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html