From: "Steve Wise" <swise@opengridcomputing.com>
To: "David Miller" <davem@davemloft.net>
Cc: "'Raju Rangoju'" <rajur@chelsio.com>, <dledford@redhat.com>,
<linux-rdma@vger.kernel.org>, "'Jason Gunthorpe'" <jgg@ziepe.ca>,
"netdev" <netdev@vger.kernel.org>,
"Potnuri Bharat Teja" <bharat@chelsio.com>,
"Ganesh Goudar" <ganeshgr@chelsio.com>,
"Rahul Lakkireddy" <rahul.lakkireddy@chelsio.com>
Subject: interdependencies with cxgb4 and iw_cxgb4
Date: Wed, 14 Mar 2018 10:31:24 -0500 [thread overview]
Message-ID: <022f01d3bba9$82e94870$88bbd950$@opengridcomputing.com> (raw)
Hey Dave,
We've been discussing dependency issues between cxgb4 and the upper layer
drivers including iw_cxgb4 on linux-rdma [1], and I want to broaden this
discussion to include netdev and you, since this involves both linux-rdma
and netdev. (note the SRQ patch submission that triggered this discussion
was sent to linux-rdma, but it should have also included netdev. This was
an oversight and will be corrected on the next submission depending on how
chelsio decides to proceed)
In a nutshell, Chelsio has a few patch series to submit that require
enabling some feature in cxgb4 that is then used by iw_cxgb4. SRQ support is
one such series. The series as a whole spans both netdev and linux-rdma
maintainers. Because of the compile dependency between iw_cxgb4 and cxgb4
introduced by this series, the cxgb4 parts and iw_cxgb4 parts cannot be
split across maintainers. Further, if one maintainer merges the entire
series, then there are issues if commits are submitted to the other
maintainer that conflict.
This issue has also been dealt-with for Mellanox drivers, I believe. I take
it the solution for them was a k.o. signed repo, that they maintain, where
both linux-rdma and netdev take PRs from for commits that are needed in both
repos. Then these are reconciled when both repos are merged into Linus'
repo. (I hope my understanding of this is correct)
For Chelsio, this is perhaps a possibility, but I'm wondering if there is a
simpler solution? A few other option we've been discussing include:
1) submit the cxgb4-only changes to netdev in release cycle X, and then only
submit the iw_cxgb4 (or other upper drivers) changes that use them in
release cycle X+1. The pro of this is simplicity. The con is timeliness -
it takes 2 release cycles to get the feature upstream.
2) run the entire series through one maintainer's repo (with all
maintainers' ACK on the content and plan, of course), and ensuring no
conflicting commits are submitted for the rest of that release cycle. I'm
not really sure that this is feasible given anyone could create commits for
upstream drivers. So how could Chelsio really control this?
Do you have any suggestions on how we should proceed?
[1] https://www.spinics.net/lists/linux-rdma/msg61856.html
Thanks,
Steve.
next reply other threads:[~2018-03-14 15:31 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-14 15:31 Steve Wise [this message]
2018-03-16 16:21 ` interdependencies with cxgb4 and iw_cxgb4 David Miller
2018-03-19 19:50 ` Steve Wise
2018-03-19 23:34 ` David Miller
2018-03-20 13:47 ` Steve Wise
2018-03-20 14:03 ` Leon Romanovsky
2018-03-20 14:40 ` David Miller
2018-03-20 15:08 ` Steve Wise
2018-03-20 15:18 ` David Miller
2018-03-20 15:20 ` Steve Wise
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='022f01d3bba9$82e94870$88bbd950$@opengridcomputing.com' \
--to=swise@opengridcomputing.com \
--cc=bharat@chelsio.com \
--cc=davem@davemloft.net \
--cc=dledford@redhat.com \
--cc=ganeshgr@chelsio.com \
--cc=jgg@ziepe.ca \
--cc=linux-rdma@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=rahul.lakkireddy@chelsio.com \
--cc=rajur@chelsio.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 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).