All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Christian König" <deathsimple@vodafone.de>
To: "Marek Olšák" <maraeo@gmail.com>, "Matt Turner" <mattst88@gmail.com>
Cc: "mesa-dev@lists.freedesktop.org" <mesa-dev@lists.freedesktop.org>,
	dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [Mesa-dev] rules for merging patches to libdrm
Date: Tue, 19 Nov 2013 16:04:33 +0100	[thread overview]
Message-ID: <528B7E01.4080101@vodafone.de> (raw)
In-Reply-To: <CAAxE2A6U-+GdmBaxxG_tfrJQf=93TNNCoOOkt-A=WjEiBsL3CQ@mail.gmail.com>

For merging the patches for new hardware and/or features I agree on the 
process of merging things bottom up. (e.g kernel first, then libdrm, 
mesa last). But to give reasons for the merge into Linus tree it's 
usually better to have a "big picture" you can validate the code against.

So I think the very first step should be to publish everything on the 
appropriate lists, and not try an approach like releasing the kernel 
code first and waiting for it to show up upstream and then try to 
release the userspace code build on top of it.

I had this situation a couple of times where "management" people wanted 
to release things bit by bit to "speed things up" and it always took me 
a bit of time to explain that this wouldn't work in the open source 
world (at least not with the Linux kernel).

Christian.

Am 19.11.2013 15:26, schrieb Marek Olšák:
> Having patches on a mailing list is good enough, but generally if
> people *trust* you that you will have an open userspace, that's good
> enough too if you have people's trust.
>
> In my opinion, the required kernel code must land in Linus's tree
> first. If it's not there, it's like it didn't exist at all. Then you
> can merge and release dependent libdrm code. And after that, you can
> merge dependent Mesa code.
>
> This is really common sense and I don't think we need a strict process
> to enforce the rules.
>
> Marek
>
> On Fri, Nov 8, 2013 at 11:32 PM, Matt Turner <mattst88@gmail.com> wrote:
>> On Fri, Nov 8, 2013 at 11:29 AM, Dave Airlie <airlied@gmail.com> wrote:
>>> Since we seemed to have some confusion over this I'll state it clearly here.
>>>
>>> You should not merge kernel interface and ioctls to libdrm until they
>>> have appeared in a git commit upstream with a stable id, this
>>> generally means drm-next, but can also mean drm-intel-next.
>> How does this interact with the rule that kernel interfaces require an
>> open source userspace? Is "here are the mesa/libdrm patches that use
>> it" sufficient to get the kernel interface merged?
>>
>> libdrm is easy to change and its releases are cheap. What problem does
>> committing code that uses an in-progress kernel interface to libdrm
>> cause? I guess I'm not understanding something.
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

  reply	other threads:[~2013-11-19 15:04 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-08 19:29 rules for merging patches to libdrm Dave Airlie
2013-11-08 22:32 ` Matt Turner
2013-11-08 22:59   ` [Mesa-dev] " Kenneth Graunke
2013-11-09  8:11     ` Dave Airlie
2013-11-09 21:26       ` Ian Romanick
2013-11-18 13:29         ` Thierry Reding
2013-11-18 15:17           ` Rob Clark
2013-11-18 15:23             ` [Mesa-dev] " Thierry Reding
2013-11-18 16:21               ` Rob Clark
2013-11-18 16:41                 ` Thierry Reding
2013-11-18 18:38                   ` [Mesa-dev] " Jerome Glisse
2013-11-19  8:18                     ` Thierry Reding
2013-11-18 16:30         ` [Mesa-dev] " Maarten Lankhorst
2013-11-18 16:38           ` Thierry Reding
2013-11-19 14:26   ` Marek Olšák
2013-11-19 15:04     ` Christian König [this message]
2013-11-19 15:14       ` [Mesa-dev] " Martin Peres

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=528B7E01.4080101@vodafone.de \
    --to=deathsimple@vodafone.de \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=maraeo@gmail.com \
    --cc=mattst88@gmail.com \
    --cc=mesa-dev@lists.freedesktop.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 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.