[PATCH xserver] ad hoc fix for mmap's truncated offset parameter on 32bit

Mark Kettenis mark.kettenis at xs4all.nl
Fri Apr 29 08:24:17 UTC 2016


> Date: Fri, 29 Apr 2016 09:34:08 +0200
> From: Julien Cristau <jcristau at debian.org>
> 
> On Thu, Apr 28, 2016 at 16:10:01 +0200, Mark Kettenis wrote:
> 
> > > From: Stefan Dirsch <sndirsch at suse.de>
> > > Date: Thu, 28 Apr 2016 14:35:00 +0200
> > > 
> > > Builtin modesetting driver didn't work on 32bit on cirrus KMS. See
> > > https://bugzilla.suse.com/show_bug.cgi?id=917385 for more details.
> > 
> > I think selectively compiling with -D_FILE_OFFSET_BITS=64 is a
> > seriously bad idea.  Either evrything should be compiled with that
> > switch, or nothing.  And adding a #define like that after including
> > some other header files is even worse.
> > 
> I think the right fix for the referenced bug was
> https://cgit.freedesktop.org/xorg/xserver/commit/?id=4962c8c08842d9d3ca66d254b1ce4cacc4fb3756
> so this patch should not be necessary anyway.

Yup.  That should do it.  I agree that this patch shouldn't be
necessary anymore.


More information about the xorg-devel mailing list