From: Linus Walleij <linus.walleij@linaro.org>
To: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Greg KH <gregkh@linuxfoundation.org>,
ksummit-discuss@lists.linuxfoundation.org
Subject: Re: [Ksummit-discuss] [MAINTAINERS SUMMIT] Challenges in Upstream vs. Embargoed Development in Intel Graphics.
Date: Thu, 6 Sep 2018 11:54:17 +0200 [thread overview]
Message-ID: <CACRpkdYP1_kG+nzg0Op-3VoLqRm8nUpXf8Daf_1B17-wWJyLjA@mail.gmail.com> (raw)
In-Reply-To: <CAKMK7uHLACdr4vGexqO-oh+QrRqxq0_KyJdhnr_eAZy6ApwWUQ@mail.gmail.com>
On Wed, Sep 5, 2018 at 11:00 AM Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> I have no idea how this works outside of intel or graphics, I tend to
> not chat with those people as much as I do with graphics people. Would
> be interesting to know how it is outside of the graphics bubble
> indeed.
More or less any SoC company for routers, handsets, tablets etc
have the same problem.
At one point I was made responsible for such a scenario. The
approach I developed was a bit ad hoc but contained some of this:
- Classify the components into embargoed and non-embargoed
so anything not affected by embargo can be pushed upstream.
(OK it's maybe obvious).
- Get management to provide a cut-off date for embargo. Like
"after this point in time we certainly do not care what code you
publish pertaining to X" and make that formal so that when that
day comes developers can simply start sending the code without
having to ask permission again, because having to do that is
pointless and bureaucratic. This is a property of the company
"FOSS-OUT legal process" that is simple but still often
forgotten. It relieves the developers for the need to hammer
management for approvals.
- Use internal developers with high upstream experience and
NDA to make internal reviews and try to anticipate any problems
that will be seen when posting the code to the community and
fix it before it happens. Get the code in upstream shape.
(This is a good reason to hire kernel maintainers and have them
spend time upstream BTW.)
- Minimize hamming distance to mainline, which means rebase
internal development as often as possible, track mainline,
bring in entire branches of development if need be just to not
deviate, because deviating too much from mainline is inviting
disaster. This goes counter to the conventional wisdom that
says you should use stable releases and baselines and LTS
and what not. I am not a big fan of the latter. Rebase on
Torvald's -rcN or even linux-next if you are aiming for upstream,
else you are not really aiming for upstream, but secondary
goals such as feature completion, "not disturbing development",
or getting tests to pass cleanly all the time. Get your priorities
straight: is upstream first really what you want? Then that
should be priority number one, not feature completion, not
"not disturbing developers", not passing internal tests.
If you find upstream first isn't really your number one priority
then call your strategy upstream second or upstream
third so that it reflects reality instead of trying to sound good.
The last point was always the most controversial.
Just my €0.01
Linus Walleij
next prev parent reply other threads:[~2018-09-06 9:54 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-04 19:54 Rodrigo Vivi
2018-09-05 4:22 ` Leon Romanovsky
2018-09-05 4:49 ` Rodrigo Vivi
2018-09-05 7:38 ` Leon Romanovsky
2018-09-05 7:48 ` Greg KH
2018-09-05 8:17 ` Daniel Vetter
2018-09-05 8:31 ` Greg KH
2018-09-05 9:00 ` Daniel Vetter
2018-09-05 9:34 ` Leon Romanovsky
2018-09-05 22:45 ` Rodrigo Vivi
2018-09-06 13:56 ` Leon Romanovsky
2018-09-05 11:21 ` Mark Brown
2018-09-06 9:54 ` Linus Walleij [this message]
2018-09-06 10:15 ` Jani Nikula
2018-09-06 10:27 ` Mark Brown
2018-09-06 10:25 ` Arnd Bergmann
2018-09-06 10:43 ` Linus Walleij
2018-09-06 10:51 ` Mark Brown
2018-09-06 12:49 ` Sean Paul
2018-09-06 16:00 ` Jon Masters
2018-09-06 20:41 ` Rodrigo Vivi
2018-09-06 20:35 ` Rodrigo Vivi
2018-09-05 11:13 ` Mark Brown
2018-09-05 7:48 ` Greg KH
-- strict thread matches above, loose matches on Subject: below --
2018-09-04 17:42 Rodrigo Vivi
2018-09-06 20:09 ` Rodrigo Vivi
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=CACRpkdYP1_kG+nzg0Op-3VoLqRm8nUpXf8Daf_1B17-wWJyLjA@mail.gmail.com \
--to=linus.walleij@linaro.org \
--cc=daniel.vetter@ffwll.ch \
--cc=gregkh@linuxfoundation.org \
--cc=ksummit-discuss@lists.linuxfoundation.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