All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Amar Mudrankit <amar.mudrankit@qlogic.com>
Cc: netdev@vger.kernel.org, rdreier@cisco.com,
	Ramachandra K <ramachandra.kuchimanchi@qlogic.com>,
	poornima.kamath@qlogic.com
Subject: Re: Fwd: [ofa-general] FW: QLogic vNIC Kernel Submission
Date: Tue, 17 Jun 2008 21:15:08 +0200	[thread overview]
Message-ID: <48580D3C.2070308@trash.net> (raw)
In-Reply-To: <485803D3.5020505@trash.net>

Patrick McHardy wrote:
> Amar Mudrankit wrote:
>> It looks as if my original email was "scrubbed" before it made the
>> mailing list so I am resending it...
>>
>> QLogic has been attempting to submit our virtual NIC (vNIC) driver to
>> the Linux kernel for several months.  We have made changes to the code
>> based on the feedback we have received over four rounds of
>> submissions. Among the feedback we received during this process was a
>> request to alter our code to use a single value per file for
>> configuration of our driver through sysfs interface.  After spending
>> much time and effort to complete this change to our design we
>> re-submitted the driver only to receive a response suggesting that we
>> change once again from this interface to a different API interface
>> called rtnl_link.  Needless to say I am very frustrated with this
>> process. This new API interface would require substantial changes to
>> our code.
> 
> Thats one of the reasons why it should be done before merging it.
> The other one being that an API can't be removed easily once its
> in the kernel.


Besides that, the state machine triggered device registration
and the sysfs based checksumming settings of also need to go (both
should be about 5 minutes of work by the way).


  parent reply	other threads:[~2008-06-17 19:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <99863D2ED484D449811D97A4C44C9CBD7C50F7@EPEXCH2.qlogic.org>
2008-06-17 17:18 ` Fwd: [ofa-general] FW: QLogic vNIC Kernel Submission Amar Mudrankit
2008-06-17 18:34   ` Patrick McHardy
2008-06-17 19:14     ` Stephen Hemminger
     [not found]       ` <C07C40DB2364324799506DE8FF12F8D859E90D@EPEXCH1.qlogic.org>
2008-06-17 20:12         ` John Russo
2008-06-17 20:49           ` Patrick McHardy
2008-06-17 23:59             ` Jeff Garzik
2008-06-17 19:15     ` Patrick McHardy [this message]
2008-06-18 10:43   ` Jeff Garzik
2008-06-18 11:19     ` Patrick McHardy

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=48580D3C.2070308@trash.net \
    --to=kaber@trash.net \
    --cc=amar.mudrankit@qlogic.com \
    --cc=netdev@vger.kernel.org \
    --cc=poornima.kamath@qlogic.com \
    --cc=ramachandra.kuchimanchi@qlogic.com \
    --cc=rdreier@cisco.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.