Re: [PATCH xserver] modesetting: Fix page flipping under DRI 3.2.

2018-03-27 Thread Louis-Francis Ratté-Boulianne
Hi Mario, Could you try with the following patch and let me know how things are working out: https://gitlab.collabora.com/lfrb/xserver/commit/d26b45afc19547ab3ff689 581db2b01abb4e1ed2 It shouldn't differ much from your patch though and I still don't understand why pageflip would fail for you.

Re: [PATCH xorgproto] Fix MAN_SUBSTS libdir replacement

2018-03-27 Thread Adam Jackson
On Tue, 2018-03-27 at 23:10 +0200, Julien Cristau wrote: > __libdir__ doesn't seem to actually be used anywhere in the man page, > but... > > Signed-off-by: Julien Cristau Reviewed-by: Adam Jackson - ajax ___

[PATCH xorgproto] Fix MAN_SUBSTS libdir replacement

2018-03-27 Thread Julien Cristau
__libdir__ doesn't seem to actually be used anywhere in the man page, but... Signed-off-by: Julien Cristau --- Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile.am b/Makefile.am index 8c1faf8..a58faca 100644 --- a/Makefile.am +++

Re: [PATCH xserver] meson: Add option to set default font path (v2)

2018-03-27 Thread Adam Jackson
On Tue, 2018-03-27 at 12:57 -0400, Adam Jackson wrote: > Reversed and merged, thanks: > > remote: E: failed to find patch for rev > 73a0562615aa1adfb934b953e23b1e69b126db4f. > remote: I: 0 patch(es) updated to state Accepted. > To ssh://git.freedesktop.org/git/xorg/xserver >

Re: [PATCH xserver] meson: Add option to set default font path (v2)

2018-03-27 Thread Adam Jackson
On Tue, 2018-03-27 at 09:47 -0700, Keith Packard wrote: > Adam Jackson writes: > > > +default_font_path = ','.join([ > > +join_paths(fontrootdir, 'misc'), > > +join_paths(fontrootdir, 'TTF'), > > +join_paths(fontrootdir, 'OTF'), > > +

Re: [PATCH xserver] meson: Add option to set default font path (v2)

2018-03-27 Thread Keith Packard
Adam Jackson writes: > +default_font_path = ','.join([ > +join_paths(fontrootdir, 'misc'), > +join_paths(fontrootdir, 'TTF'), > +join_paths(fontrootdir, 'OTF'), > +join_paths(fontrootdir, 'Type1'), > +join_paths(fontrootdir, '75dpi'),

[PATCH xserver] meson: Add option to set default font path (v2)

2018-03-27 Thread Adam Jackson
The autotools build gets this from some macros in fontutil, but they're just wrappers around pkgconfig. v2: Use same default as autotools (Keith Packard) Signed-off-by: Adam Jackson --- include/meson.build | 2 +- meson.build | 18 +- meson_options.txt

Re: [PATCH xserver 08/12] meson: Build cvt and gtf

2018-03-27 Thread Keith Packard
Adam Jackson writes: > What'd be Real Nice is if xrandr knew how to generate modes for given > sizes. Yeah, I've started to add cvt to xrandr a couple of times but never finished. Would also be nice to let you add a mode, associate with an output and set that mode all in one

Re: [PATCH xserver 05/12] meson: Build Xorg suid wrapper

2018-03-27 Thread Keith Packard
Adam Jackson writes: > Probably this path should add a warning() to alert the builder? Yeah, probably. -- -keith signature.asc Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives:

Re: [PATCH xserver 02/12] man: s/__/@/g

2018-03-27 Thread Adam Jackson
On Tue, 2018-03-27 at 16:50 +0200, Olivier Fourdan wrote: > Hi, > > On Tue, Mar 27, 2018 at 12:42 AM, Adam Jackson wrote: > > A cosmetic change for automake (though we have to replicate some of > > xorg-macros.m4 in manpages.am now), but meson's configure_file() wants > >

Re: [PATCH xserver 02/12] man: s/__/@/g

2018-03-27 Thread Olivier Fourdan
Hi, On Tue, Mar 27, 2018 at 12:42 AM, Adam Jackson wrote: > A cosmetic change for automake (though we have to replicate some of > xorg-macros.m4 in manpages.am now), but meson's configure_file() wants > @-delimited strings. > [...] Note sure if it's related but the builds

Re: [PATCH xserver 06/12] meson: Add option to set default font path

2018-03-27 Thread Adam Jackson
On Mon, 2018-03-26 at 18:47 -0600, Keith Packard wrote: > Adam Jackson writes: > > > +option('default_font_path', type: 'string', > > + value: 'catalogue:/etc/X11/fontpath.d,built-ins') > > This isn't the autotools default; seems like you should probably just > use that?

Re: [PATCH xserver 08/12] meson: Build cvt and gtf

2018-03-27 Thread Adam Jackson
On Mon, 2018-03-26 at 18:48 -0600, Keith Packard wrote: > Adam Jackson writes: > > > Signed-off-by: Adam Jackson > > These shouldn't be in the X server sources, but... > > Reviewed-by: Keith Packard What'd be Real Nice is if xrandr knew

Re: [PATCH xserver 05/12] meson: Build Xorg suid wrapper

2018-03-27 Thread Adam Jackson
On Mon, 2018-03-26 at 18:45 -0600, Keith Packard wrote: > Adam Jackson writes: > > > Signed-off-by: Adam Jackson > > +if get_option('suid_wrapper') > > +executable('Xorg.wrap', > > +'xorg-wrapper.c', > > +include_directories: [inc,

Re: [PATCH xserver 3/3] modesetting/drmmode: Use drmModeGetFB2

2018-03-27 Thread Daniel Stone
Hi Emil, On 26 March 2018 at 16:22, Emil Velikov wrote: > On 23 March 2018 at 13:50, Daniel Stone wrote: >> Much like AddFB -> AddFB2, GetFB2 lets us get multiple buffers back as >> well as modifier information. This lets us use -background none

Re: [PATCH xserver 2/3] modesetting/drmmode: Use glamor_pixmap_from_fds

2018-03-27 Thread Daniel Stone
Hi, On 26 March 2018 at 16:11, Emil Velikov wrote: > On 23 March 2018 at 13:50, Daniel Stone wrote: >> +for (i = 0; i < 4 && handles[i]; i++) { >> +err = drmPrimeHandleToFD(drmmode->fd, handles[i], O_CLOEXEC, >> [i]); >> +if

Re: [PATCH xserver] modesetting: Fix page flipping under DRI 3.2.

2018-03-27 Thread Olivier Fourdan
Hi, On Mon, Mar 26, 2018 at 7:54 PM, Mario Kleiner wrote: > [...] > I haven't had time to retest with the latest patch on master of Adam > "modesetting: Fix up some XXX from removing GLAMOR_HAS_DRM_*" > yet, so maybe that makes a positive difference? My testing was

Re: [PATCH v3 app/xrandr] Document that --dpi and --fbmm options set DPI of whole X screen

2018-03-27 Thread Giuseppe Bilotta
Hello, On Fri, Mar 23, 2018 at 4:53 PM, Pali Rohár wrote: > On Monday 12 March 2018 09:19:57 Giuseppe Bilotta wrote: >> On Sat, Mar 10, 2018 at 4:23 PM, Pali Rohár wrote: >> > Explicitly document and make it clear that those options do not change >> >