netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Donald Becker <becker@scyld.com>
Cc: Roger Luethi <rl@hellgate.ch>,
	netdev@oss.sgi.com,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: pci-skeleton duplex check
Date: Thu, 12 Dec 2002 16:39:20 -0500	[thread overview]
Message-ID: <3DF90208.3010208@pobox.com> (raw)
In-Reply-To: <Pine.LNX.4.44.0212121539110.10674-100000@beohost.scyld.com>

Donald Becker wrote:
> On Thu, 12 Dec 2002, Jeff Garzik wrote:
> 
>>Donald Becker wrote:
>>
>>>[[ I don't know why I bother. The people that now control what goes into
>>>the kernel would rather put in random patches from other people than
>>>accept a correct fix from me. ]]
>>
>>I'm very interested in applying fixes from you!  I am publicly begging 
>>you to do so, and even CC'ing lkml on my request.
> 
> 
> This is very disingenuous statement.


Oh come on, it's far less disingenuous than what you said:

	[[ I don't know why I bother. The people that now control what
	   goes into the kernel would rather put in random patches from
	   other people than accept a correct fix from me. ]]

I'm sure you'll continue making snide comments on every mailing list you 
maintain, but the fact remains:

I would much rather accept a fix from you.

That hasn't changed in the past year.  or two.  or any amount of time. 
Your input is very valuable, and I typically save quite a few of your 
emails.


> The drivers in the kernel are now heavily modified and have significantly
> diverged from my version.  Sure, you are fine with having someone else
> do the difficult and unrewarding debugging and maintainence work, while
> you work on just the latest cool hardware, change the interfaces and are
> concerned only with the current kernel version.

While I disagree with this assessment, I think we can safely draw the 
conclusion that the problem is _not_ people ignoring your patches, or 
preferring other patches over yours.


> I've been actively developing Linux drivers for over a decade, and run
> about two dozen mailing lists for specific drivers.  I write diagnostic
> routines for every released driver.  I thoroughly test and frequently
> update the driver set I maintain.  And since about 2000, my patches were
> ignored while the first notice I've have gotten to changes in my drivers
> is the bug reports.  And the response: "submit a patch to fix those
> newly introduced bugs".  I've even had patches ignore in favor of people
> that wrote "I don't have the NIC, but here is a change".

I don't recall _ever_ getting a patch from you or seeing one posted on 
lkml or netdev.  How can you be ignored if you're not sending patches?


> A good example is the tulip driver.  You repeatedly restructured my
> driver in the kernel, splitting into different files.  It was still 90+%
> my code, but the changes made it impossible to track the modification
> history.  The kernel driver was long-broken with 21143-SYM cards, but no
> one took the responsibility for fixing it.

s/was/is/
I take responsibility for fixing it, I just haven't fixed it yet :)


> It's easy to make the first few patches, when you don't have to deal
> with reversion testing, many different models, and have an unlimited
> sandbox where it doesn't matter if a specific release works or not.  But
> it takes a huge of work to keep a stable, tracable driver development
> process that works with many different kernel versions and hardware
> environments.


We're slowly getting there, in terms of regression and stress testing.

Since you don't send patches anymore for a long time, I was really the 
only one [stupid enough?] to stand up and even bother to help collecting 
and reviewing net driver changes.

I would love to integrate your drivers directly, but they don't come 
anywhere close to using current kernel APIs.  The biggie of biggies is 
not using the pci_driver API.  So, given that we cannot directly merge 
your drivers, and you don't send patches to kernel developers, what is 
the next best alternative?  (a) let kernel net drivers bitrot, or (b) 
maintain them as best we can without Don Becker patches?  I say that "b" 
is far better than "a" for Linux users.

	Jeff

  reply	other threads:[~2002-12-12 21:39 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-12-11 13:24 pci-skeleton duplex check Roger Luethi
2002-12-12  2:42 ` Donald Becker
2002-12-12 13:20   ` Roger Luethi
2002-12-12 14:11     ` Donald Becker
2002-12-12 20:32       ` Jeff Garzik
2002-12-12 21:11         ` Donald Becker
2002-12-12 21:39           ` Jeff Garzik [this message]
2002-12-13  1:18             ` Donald Becker
2002-12-13  9:17               ` David S. Miller
2002-12-13 16:56                 ` Donald Becker
2002-12-13 18:29                   ` David S. Miller
2002-12-13 20:57               ` Jeff Garzik
2002-12-12 23:08           ` Ben Greear
2002-12-14  0:19           ` Michael Richardson
2002-12-14  1:43             ` Oliver Xymoron

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=3DF90208.3010208@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=becker@scyld.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@oss.sgi.com \
    --cc=rl@hellgate.ch \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).