From: Alexander Duyck <alexander.h.duyck@intel•com>
To: Stefan Assmann <sassmann@kpanic•de>
Cc: e1000-devel@lists•sourceforge.net, netdev@vger•kernel.org,
davem@davemloft•net
Subject: Re: [PATCH net-next] igbvf: do not force carrier off in igbvf_msix_other()
Date: Thu, 08 Aug 2013 09:09:39 -0700 [thread overview]
Message-ID: <5203C2C3.3030205@intel.com> (raw)
In-Reply-To: <1375963002-1624-1-git-send-email-sassmann@kpanic.de>
On 08/08/2013 04:56 AM, Stefan Assmann wrote:
> Currently carrier is forced off in igbvf_msix_other(). This seems
> unnecessary and causes multiple calls to igbvf_watchdog_task(), resulting
> in multiple link up messages when calling dhclient for example.
> [ 111.818106] igbvf 0000:00:04.0: Link is Up 1000 Mbps Full Duplex
> [ 111.819347] IPv6: ADDRCONF(NETDEV_UP): eth5: link is not ready
> [ 111.820509] IPv6: ADDRCONF(NETDEV_CHANGE): eth5: link becomes ready
> [ 111.822983] igbvf 0000:00:04.0: Link is Up 1000 Mbps Full Duplex
> [ 115.152421] igbvf 0000:00:04.0: Link is Up 1000 Mbps Full Duplex
> compared to
> [ 1040.422161] igbvf 0000:00:04.0: Link is Up 1000 Mbps Full Duplex
> [ 1040.423447] IPv6: ADDRCONF(NETDEV_UP): eth5: link is not ready
> [ 1040.424622] IPv6: ADDRCONF(NETDEV_CHANGE): eth5: link becomes ready
> when this patch is applied.
>
> Signed-off-by: Stefan Assmann <sassmann@kpanic•de>
> ---
> drivers/net/ethernet/intel/igbvf/netdev.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/igbvf/netdev.c b/drivers/net/ethernet/intel/igbvf/netdev.c
> index 93eb7ee..f041586 100644
> --- a/drivers/net/ethernet/intel/igbvf/netdev.c
> +++ b/drivers/net/ethernet/intel/igbvf/netdev.c
> @@ -876,8 +876,6 @@ static irqreturn_t igbvf_msix_other(int irq, void *data)
>
> adapter->int_counter1++;
>
> - netif_carrier_off(netdev);
> - hw->mac.get_link_status = 1;
> if (!test_bit(__IGBVF_DOWN, &adapter->state))
> mod_timer(&adapter->watchdog_timer, jiffies + 1);
>
While this patch helps to squelch the messages, did you test to see what
happens if for example you bring the PF interface down while the VFs are
trying to function? The reason for switching the carrier off is because
most interrupts on the mailbox indicate that something has been changed
in the underlying interface. If for example the PF is about to disable
the interfaces it should be triggering this interrupt. Otherwise you
are just setting up the VF to dump out a number of Tx hang and watchdog
messages.
Thanks,
Alex
------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
E1000-devel mailing list
E1000-devel@lists•sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
next prev parent reply other threads:[~2013-08-08 16:09 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-08 11:56 [PATCH net-next] igbvf: do not force carrier off in igbvf_msix_other() Stefan Assmann
2013-08-08 16:09 ` Alexander Duyck [this message]
2013-08-09 7:02 ` Stefan Assmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5203C2C3.3030205@intel.com \
--to=alexander.h.duyck@intel$(echo .)com \
--cc=davem@davemloft$(echo .)net \
--cc=e1000-devel@lists$(echo .)sourceforge.net \
--cc=netdev@vger$(echo .)kernel.org \
--cc=sassmann@kpanic$(echo .)de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox