Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1275413 > unrolled thread

Re: severe regression in alx ethernet driver

Started byLdap Tester <ldap.tester@gmail.com>
First post2015-11-23 15:10 +0100
Last post2015-11-26 05:20 +0100
Articles 6 — 4 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: severe regression in alx ethernet driver Ldap Tester <ldap.tester@gmail.com> - 2015-11-23 15:10 +0100
    Re: severe regression in alx ethernet driver David Miller <davem@davemloft.net> - 2015-11-23 17:20 +0100
      Re: severe regression in alx ethernet driver David Miller <davem@davemloft.net> - 2015-11-23 17:20 +0100
        Re: severe regression in alx ethernet driver Jarod Wilson <jarod@redhat.com> - 2015-11-24 17:10 +0100
          Re: severe regression in alx ethernet driver Francois Romieu <romieu@fr.zoreil.com> - 2015-11-25 23:50 +0100
            Re: severe regression in alx ethernet driver David Miller <davem@davemloft.net> - 2015-11-26 05:20 +0100

#1275413 — Re: severe regression in alx ethernet driver

FromLdap Tester <ldap.tester@gmail.com>
Date2015-11-23 15:10 +0100
SubjectRe: severe regression in alx ethernet driver
Message-ID<qxZYB-2N4-1@gated-at.bofh.it>
The severe regression in alx ethernet driver is still present in the
latest released kernel.  Patches have been posted in
https://bugzilla.kernel.org/show_bug.cgi?id=70761 There are reports
that these patches have resolved the problem.  Could you please review
these patches and see if they can be included upstream?  I have not
been able to update my kernel in almost six months.  Many other users
are suffering from this same problem.

On Mon, Sep 14, 2015 at 6:47 PM, Ldap Tester <ldap.tester@gmail.com> wrote:
> There is a serious regression in the alx ethernet driver.  The driver
> stopped working after upgrading the kernel from 4.0.x to 4.1.x.
> Please see https://bugzilla.redhat.com/show_bug.cgi?id=1251434 and
> https://bugzilla.kernel.org/show_bug.cgi?id=70761 This regression is
> urgent, as I cannot update my kernel to include the latest security
> fixes.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1275539

FromDavid Miller <davem@davemloft.net>
Date2015-11-23 17:20 +0100
Message-ID<qy20q-4cV-23@gated-at.bofh.it>
In reply to#1275413
From: Ldap Tester <ldap.tester@gmail.com>
Date: Mon, 23 Nov 2015 11:00:08 -0300

> https://bugzilla.kernel.org/show_bug.cgi?id=70761

Patches sitting in a bugzilla entry will simply rot.

For upstream inclusion someone must formally submit them to the netdev
mailing list, with proper commit message, signoffs, etc. and then the
patch must be fully reviewed by the community.

Only then will it be applied and make it upstream.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1275543

FromDavid Miller <davem@davemloft.net>
Date2015-11-23 17:20 +0100
Message-ID<qy20q-4cV-33@gated-at.bofh.it>
In reply to#1275539
From: David Miller <davem@davemloft.net>
Date: Mon, 23 Nov 2015 11:15:30 -0500 (EST)

> From: Ldap Tester <ldap.tester@gmail.com>
> Date: Mon, 23 Nov 2015 11:00:08 -0300
> 
>> https://bugzilla.kernel.org/show_bug.cgi?id=70761
> 
> Patches sitting in a bugzilla entry will simply rot.
> 
> For upstream inclusion someone must formally submit them to the netdev
> mailing list, with proper commit message, signoffs, etc. and then the
> patch must be fully reviewed by the community.
> 
> Only then will it be applied and make it upstream.

And amusingly, the final comment in that bugzilla is someone asking
the patch author to do exactly this.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1276596

FromJarod Wilson <jarod@redhat.com>
Date2015-11-24 17:10 +0100
Message-ID<qyokh-1XH-17@gated-at.bofh.it>
In reply to#1275543
On Mon, Nov 23, 2015 at 11:16:10AM -0500, David Miller wrote:
> From: David Miller <davem@davemloft.net>
> Date: Mon, 23 Nov 2015 11:15:30 -0500 (EST)
> 
> > From: Ldap Tester <ldap.tester@gmail.com>
> > Date: Mon, 23 Nov 2015 11:00:08 -0300
> > 
> >> https://bugzilla.kernel.org/show_bug.cgi?id=70761
> > 
> > Patches sitting in a bugzilla entry will simply rot.
> > 
> > For upstream inclusion someone must formally submit them to the netdev
> > mailing list, with proper commit message, signoffs, etc. and then the
> > patch must be fully reviewed by the community.
> > 
> > Only then will it be applied and make it upstream.
> 
> And amusingly, the final comment in that bugzilla is someone asking
> the patch author to do exactly this.

They do at least have a signed-off-by in the patches attached to the bug,
so I'm working on touching up the descriptions and formatting, regression
testing them on my laptop that has an alx-driven E2200 in it (which isn't
affected by this bug), and then I can ship them up for proper review.

-- 
Jarod Wilson
jarod@redhat.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1277855

FromFrancois Romieu <romieu@fr.zoreil.com>
Date2015-11-25 23:50 +0100
Message-ID<qyR2V-3ZJ-5@gated-at.bofh.it>
In reply to#1276596
Jarod Wilson <jarod@redhat.com> :
[...]
> They do at least have a signed-off-by in the patches attached to the bug,
> so I'm working on touching up the descriptions and formatting, regression
> testing them on my laptop that has an alx-driven E2200 in it (which isn't
> affected by this bug), and then I can ship them up for proper review.

You can forget the last refill-me-harder patch and save a roundtrip.
The driver should 1) prohibit holes in its receive ring, 2) allocate before
pushing data up in the stack and 3) drop packets when it can't allocate a
fresh buffer.

-- 
Ueimor
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1277967

FromDavid Miller <davem@davemloft.net>
Date2015-11-26 05:20 +0100
Message-ID<qyWch-7RQ-1@gated-at.bofh.it>
In reply to#1277855
From: Francois Romieu <romieu@fr.zoreil.com>
Date: Wed, 25 Nov 2015 23:47:08 +0100

> Jarod Wilson <jarod@redhat.com> :
> [...]
>> They do at least have a signed-off-by in the patches attached to the bug,
>> so I'm working on touching up the descriptions and formatting, regression
>> testing them on my laptop that has an alx-driven E2200 in it (which isn't
>> affected by this bug), and then I can ship them up for proper review.
> 
> You can forget the last refill-me-harder patch and save a roundtrip.
> The driver should 1) prohibit holes in its receive ring, 2) allocate before
> pushing data up in the stack and 3) drop packets when it can't allocate a
> fresh buffer.

+1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web