On Thu, 4 Mar 2010 10:18:03 -0800 (PST) Linus Torvalds <torva...@linux-foundation.org> wrote:
> > > Hmm. What the hell am I supposed to do about > > (II) NOUVEAU(0): [drm] nouveau interface version: 0.0.16 > (EE) NOUVEAU(0): [drm] wrong version, expecting 0.0.15 > (EE) NOUVEAU(0): 879: > > now? > > What happened to the whole backwards compatibility thing? I wasn't even > warned that this breaks existing user space. That makes it impossible to > _test_ new kernels. Upgrading X and the kernel in lock-step is not a valid > model, lots of people are just using some random distribution (F12 in my > case), and you just broke it. > > I see the commit that does this was very aware of it: > > commit a1606a9596e54da90ad6209071b357a4c1b0fa82 > Author: Ben Skeggs <bske...@redhat.com> > Date: Fri Feb 12 10:27:35 2010 +1000 > > drm/nouveau: new gem pushbuf interface, bump to 0.0.16 > > This commit breaks the userspace interface, and requires a new > libdrm for > nouveau to operate again. > > The multiple GEM_PUSHBUF ioctls that were present in 0.0.15 for > compatibility purposes are now gone, and replaced with the new > ioctl which > allows for multiple push buffers to be submitted (necessary for hw > index > buffers in the nv50 3d driver) and relocations to be applied on any > buffer. > > A number of other ioctls (CARD_INIT, GEM_PIN, GEM_UNPIN) that were > needed > for userspace modesetting have also been removed. > > Signed-off-by: Ben Skeggs <bske...@redhat.com> > Signed-off-by: Francisco Jerez <curroje...@riseup.net> > > but why the hell wasn't I made aware of it before-hand? Quite frankly, I > probably wouldn't have pulled it. > > We can't just go around breaking peoples setups. This driver is, like it > or not, used by Fedora-12 (and probably other distros). It may say > "staging", but that doesn't change the fact that it's in production use by > huge distributions. Flag days aren't acceptable. Whoa, so breaking ABI in staging drivers isn't ok? Lots of other staging drivers are shipped by distros with compatible userspaces, but I thought the whole point of staging was to fix up ABIs before they became mainstream and had backwards compat guarantees, meaning that breakage was to be expected? Yes, it sucks, but what else should the nouveau developers have done? They didn't want to push nouveau into mainline because they weren't happy with the ABI yet, but it ended up getting pushed anyway as a staging driver at your request, and now they're stuck? Sorry this whole thing is a bit of a wtf... Yes Dave probably should have mentioned it in his pull request, but that doesn't seem to be a good reason not to pull imo... -- Jesse Barnes, Intel Open Source Technology Center ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
