linux-kernel - Re: [git pull] drm request 3

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LFD.2.00.1003041015080.3588@localhost.localdomain>
Date:	Thu, 4 Mar 2010 10:18:03 -0800 (PST)
From:	Linus Torvalds <torvalds@...ux-foundation.org>
To:	Dave Airlie <airlied@...ux.ie>
cc:	linux-kernel@...r.kernel.org, dri-devel@...ts.sf.net
Subject: Re: [git pull] drm request 3

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 <bskeggs@...hat.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 <bskeggs@...hat.com>
	    Signed-off-by: Francisco Jerez <currojerez@...eup.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.

		Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/