linux-kernel - Re: [PATCH] Export shmem_file_setup and shmem

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]

Date:	Sun, 10 Aug 2008 21:34:13 -0400
From:	Christoph Hellwig <hch@...radead.org>
To:	Arjan van de Ven <arjan@...radead.org>
Cc:	Stephane Marchesin <marchesin@...s.u-strasbg.fr>,
	Keith Packard <keithp@...thp.com>,
	John Stoffel <john@...ffel.org>,
	Hugh Dickins <hugh@...itas.com>,
	Nick Piggin <nickpiggin@...oo.com.au>,
	Christoph Hellwig <hch@...radead.org>,
	Eric Anholt <eric@...olt.net>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Export shmem_file_setup and shmem_getpage for DRM-GEM

On Wed, Aug 06, 2008 at 10:24:37AM -0700, Arjan van de Ven wrote:
> the ATI driver guys already have the interface working.
> 
> In addition.. since when is "oh you must also make it work for THAT" a
> requirement? Traditionally, it's up to the second person to make
> generic code work for them (within reason of course, but I'll argue
> that the bar of reasonableness has been met here)

Important difference between code and interfaces.  GEM is adding new
userspace interface, and it's current form rather ugly ones.  We'd
better think about these hard, and make sure they are useful for the
whole usecase and not a limited subset.

--
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/