Skip to content

Commit

Permalink
lan78xx: Debounce link events to minimize poll storm
Browse files Browse the repository at this point in the history
The bInterval is set to 4 (i.e. 8 microframes => 1ms) and the only bit
that the driver pays attention to is "link was reset". If there's a
flapping status bit in that endpoint data, (such as if PHY negotiation
needs a few tries to get a stable link) then polling at a slower rate
would act as a de-bounce.

See: #2447
  • Loading branch information
jemele authored and popcornmix committed Jul 8, 2024
1 parent 9c7bf9a commit e6fd543
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion drivers/net/usb/lan78xx.c
Original file line number Diff line number Diff line change
Expand Up @@ -618,6 +618,11 @@ static bool enable_tso;
module_param(enable_tso, bool, 0644);
MODULE_PARM_DESC(enable_tso, "Enables TCP segmentation offload");

#define INT_URB_MICROFRAMES_PER_MS 8
static int int_urb_interval_ms = 8;
module_param(int_urb_interval_ms, int, 0);
MODULE_PARM_DESC(int_urb_interval_ms, "Override usb interrupt urb interval");

static int lan78xx_read_reg(struct lan78xx_net *dev, u32 index, u32 *data)
{
u32 *buf;
Expand Down Expand Up @@ -4456,7 +4461,13 @@ static int lan78xx_probe(struct usb_interface *intf,
if (ret < 0)
goto out4;

period = ep_intr->desc.bInterval;
if (int_urb_interval_ms <= 0)
period = ep_intr->desc.bInterval;
else
period = int_urb_interval_ms * INT_URB_MICROFRAMES_PER_MS;

netif_notice(dev, probe, netdev, "int urb period %d\n", period);

maxp = usb_maxpacket(dev->udev, dev->pipe_intr);
buf = kmalloc(maxp, GFP_KERNEL);
if (!buf) {
Expand Down

0 comments on commit e6fd543

Please sign in to comment.