From: Jesse Brandeburg <jesse.brandeburg@intel.com>
To: kuba@kernel.org, davem@davemloft.net
Cc: netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org,
alice.michael@intel.com, alan.brady@intel.com
Subject: [RFC net-next] iavf: refactor plan proposal
Date: Mon, 8 Mar 2021 16:28:58 -0800 [thread overview]
Message-ID: <20210308162858.00004535@intel.com> (raw)
Hello,
We plan to refactor the iavf module and would appreciate community and
maintainer feedback on our plans. We want to do this to realize the
usefulness of the common code module for multiple drivers. This
proposal aims to avoid disrupting current users.
The steps we plan are something like:
1) Continue upstreaming of the iecm module (common module) and
the initial feature set for the idpf driver[1] utilizing iecm.
2) Introduce the refactored iavf code as a "new" iavf driver with the
same device ID, but Kconfig default to =n to enable testing.
a. Make this exclusive so if someone opts in to "new" iavf,
then it disables the original iavf (?)
b. If we do make it exclusive in Kconfig can we use the same
name?
3) Plan is to make the "new" iavf driver the default iavf once
extensive regression testing can be completed.
a. Current proposal is to make CONFIG_IAVF have a sub-option
CONFIG_IAVF_V2 that lets the user adopt the new code,
without changing the config for existing users or breaking
them.
We are looking to make sure that the mode of our refactoring will meet
the community's expectations. Any advice or feedback is appreciated.
Thanks,
Jesse, Alice, Alan
[1]
https://lore.kernel.org/netdev/20200824173306.3178343-1-anthony.l.nguyen@intel.com/
next reply other threads:[~2021-03-09 0:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-09 0:28 Jesse Brandeburg [this message]
2021-03-09 6:09 ` [RFC net-next] iavf: refactor plan proposal Leon Romanovsky
2021-03-10 5:11 ` Jesse Brandeburg
2021-03-10 5:50 ` Leon Romanovsky
2021-03-09 22:17 ` Jakub Kicinski
2021-03-10 5:12 ` Jesse Brandeburg
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=20210308162858.00004535@intel.com \
--to=jesse.brandeburg@intel.com \
--cc=alan.brady@intel.com \
--cc=alice.michael@intel.com \
--cc=davem@davemloft.net \
--cc=intel-wired-lan@lists.osuosl.org \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
/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).