All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Thomas Hellström" <thomas@shipmail.org>
To: Pekka Paalanen <pq@iki.fi>
Cc: nouveau@lists.freedesktop.org, thellstrom@vmware.com,
	dri-devel@lists.sf.net
Subject: Re: Why was old TTM removed from drm.git?
Date: Wed, 24 Jun 2009 19:26:41 +0200	[thread overview]
Message-ID: <4A4261D1.8070204@shipmail.org> (raw)
In-Reply-To: <20090624194306.0730cb2f@daedalus.pq.iki.fi>

Hi, Pekka!

I'm sorry for this breakage. I thought drm master was currently used 
only for libdrm development, but
I see now that I didn't pay enough attention. Just prior to the commit I 
sent out a message explaining what I was going to do and why, but 
apparently it didn't make it to the list (which seems to be the case of 
quite a few mails these days).

Please feel free to revert that commit and pull it in again when you 
think it's OK, or perhaps anchor a nouveau drm branch just ahead of this 
commit. The old TTM is completely unmaintained and blocks any attempt to 
bring the drm.git master repo reasonably up to sync with what's in the 
kernel, so the sooner it goes away the better.

Again, Sorry for the breakage.

/Thomas


Pekka Paalanen wrote:
> Hi Thomas,
>
> I meant to ask you this 24h ago:
> why did you deliberately break Nouveau in drm.git?
>
> The commit 9a33f62be1c478334572ea9384af60 "drm: Strip old ttm." not only
> removes the old TTM, it explicitly removes Nouveau from Makefile. IMHO
> this means you knew it broke Nouveau and you did not care. And there is
> no explanation in the commit as to why. Did you ask any Nouveau developer
> if it was ok?
>
> It is true, that Nouveau is aiming for its own kernel tree for DRM
> development, but it is not for all end users yet. Majority of our users
> use the drm.git kernel modules for Nouveau, and now we get to explain
> how it is not there anymore, and we have no guide to point them to.
>
> 2.6.31-rc1 is coming, bearing newTTM. If you had waited a week, we would
> have had the kernel tree up and user documentation written on how to
> build that, and breaking Nouveau in drm.git would not have mattered.
>
> Well, rc1 is so near, that we probably won't bother fixing Nouveau in
> drm.git. I just wish the transition would have been Nouveau developers'
> decision.
>
>
> Regards, pq
>
>   




------------------------------------------------------------------------------
--

  reply	other threads:[~2009-06-24 17:26 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-24 16:43 Why was old TTM removed from drm.git? Pekka Paalanen
2009-06-24 17:26 ` Thomas Hellström [this message]
     [not found]   ` <4A4261D1.8070204-4+hqylr40dJg9hUCZPvPmw@public.gmane.org>
2009-06-24 18:04     ` Pekka Paalanen
2009-06-25  6:28     ` Michel Dänzer
     [not found]       ` <1245911291.20064.3828.camel-2h6evNeVVYGs1BDpvl8NfQ@public.gmane.org>
2009-06-25  8:55         ` Thomas Hellström

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=4A4261D1.8070204@shipmail.org \
    --to=thomas@shipmail.org \
    --cc=dri-devel@lists.sf.net \
    --cc=nouveau@lists.freedesktop.org \
    --cc=pq@iki.fi \
    --cc=thellstrom@vmware.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.