[PATCH xserver] modesetting: Fix and improve ms_kernel_msc_to_crtc_msc()

2018-05-05 Thread Mario Kleiner
The old 32-Bit wraparound handling didn't actually work, due to some integer casting bug, and the mapping was ill equipped to deal with input from the new true 64-bit GetCrtcSequence/QueueCrtcSequence api's introduced in Linux 4.15. For 32-Bit truncated input from pageflip events and old vblank

Re: [PATCH xscope] Improve help and manpage.

2018-05-05 Thread Alan Coopersmith
On 07/ 8/13 05:40 PM, Stéphane Aulery wrote: > Add some details on the verbosity and options -T and -A. > These two options are not very clear to me. > > Signed-off-by: Stéphane Aulery > --- > man/xscope.man | 89 > +--- >

Re: [PATCH xditview] make sure filename is a regular file.

2018-05-05 Thread Alan Coopersmith
Sorry for letting this get lost in our massive backlog, but it seemed to work well a quick test with /dev/null so I've pushed it to git master now: To ssh://git.freedesktop.org/git/xorg/app/xditview 5383f40..3cd0b7e master -> master Thanks for the patch! -alan- On 12/ 7/10 08:32

Re: [PATCH libXaw] Scrollbar.c: Add Btn4 & Btn5 to default translations

2018-05-05 Thread Alan Coopersmith
Sorry for letting this get lost in our massive backlog, but it seemed to work well a quick test with xmore so I've pushed it to git master now: To ssh://git.freedesktop.org/git/xorg/lib/libXaw 5a39226..2f298ec master -> master Thanks for the patch! On 01/19/11 12:20 PM, Roberto Branciforti

Re: [PATCH gccmakedep] Add parsing of GCC option '-std='.

2018-05-05 Thread Alan Coopersmith
On 03/23/12 10:59 AM, Anderson Luiz da Silva wrote: > Especially useful when forcing language standards. Correct error when > using experimental code in GCC (-std=c++0x or -std=gnu++0x.) > > Signed-off-by: Anderson Luiz da Silva > --- > gccmakedep.in |2 +- >

Re: [Mesa-dev] [PATCH 1/2] loader_dri3: Wait for pending swaps to complete before drawable_fini.

2018-05-05 Thread Roman Gilg
On Sat, May 5, 2018 at 8:50 PM, Mario Kleiner wrote: > Thanks. Can you see if you get any freezes in kwin_x11 by "violent > alt-tabbing" with patch 1? I've seen two such freezes within 8 hours > of normal use yesterday, each occuring when i alt-tabbed (normally) > and

[PATCH app/xprop] Add missing braces around else clause in Read_Quoted()

2018-05-05 Thread Alan Coopersmith
Found by gcc 7.3: xprop.c: In function ‘Read_Quoted’: xprop.c:163:8: warning: this ‘else’ clause does not guard... [-Wmisleading-indentation] } else ^~~~ xprop.c:164:17: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘else’

[PATCH app/xdm] Add missing brackets to multi-statement if in SaveServerAuthorizations

2018-05-05 Thread Alan Coopersmith
Introduced by commit 5222d28e8d8e5b4cc Reported by gcc 7.3: auth.c: In function ‘SaveServerAuthorizations’: auth.c:447:6: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] if (auths[i]->data_length > 0) ^~ auth.c:452:3: note: ...this statement, but the latter is

Re: [Mesa-dev] [PATCH 1/2] loader_dri3: Wait for pending swaps to complete before drawable_fini.

2018-05-05 Thread Mario Kleiner
On Sat, May 5, 2018 at 4:44 PM, Roman Gilg wrote: > Without this patch plasmashell on Xserver/Mesa master freezes on me > when opening the launcher menu (Kickoff). With the patch haven't > experienced freezes yet. > > Haven't tested the Steam client yet. Might be a different

[PATCH util/makedepend 2/3] Remove unused variable whitespace

2018-05-05 Thread Alan Coopersmith
Use of it was removed in commit 275c17136006f9d, but we kept setting it anyway. Resolves gcc 7.3 warning: main.c: In function ‘getnextline’: main.c:587:10: warning: variable ‘whitespace’ set but not used [-Wunused-but-set-variable] boolean whitespace = FALSE; ^~

[PATCH util/makedepend 1/3] Fix bad indentation in ifparser.c

2018-05-05 Thread Alan Coopersmith
ifparser.c: In function ‘parse_product’: ifparser.c:300:5: warning: this ‘else’ clause does not guard... [-Wmisleading-indentation] else ^~~~ ifparser.c:302:2: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘else’ break; ^

[PATCH util/makedepend 3/3] Simplify writing of output lines to Makefile

2018-05-05 Thread Alan Coopersmith
Instead of writing everything to a temporary buffer, and then using fwrite() to have it fputc() one character at a time into the stdio buffer, just use fprintf() directly to save a copy and write in larger blocks. Testing on Solaris on makedepend's own source files showed a reduction in memcpy's

[PATCH util/makedepend V2] Quote colons in filenames/paths

2018-05-05 Thread Alan Coopersmith
From: Antonio Larrosa Makefile doesn't like colons in filenames/paths so they must be quoted in the output. Otherwise makedepend doesn't work with full paths that contain a colon. V2: Use quoted filename when measuring name length Signed-off-by: Alan Coopersmith

Re: [PATCH] Quote colons in filenames/paths

2018-05-05 Thread Alan Coopersmith
Sorry for the slow response, but as you may have guessed by now, makedepend is one of the components no one is actively maintaining. But since we've got a number of fixes in git, we should probably do a release soon, and I saw this still sitting in the queue. This seems like a good idea, but

Re: [PATCH 2/2] loader_dri3: Variant 2: Wait for pending swaps to complete before drawable_fini.

2018-05-05 Thread Mike Lothian
Thank you for the clarification. On Sat, 5 May 2018 at 14:31 Daniel Stone wrote: > On 5 May 2018 at 10:15, Mike Lothian wrote: > > Out of interest can you try running the vulkan smoketest, I'm seeing > this: > > > > smoketest > > terminate called

Re: [Mesa-dev] [PATCH 1/2] loader_dri3: Wait for pending swaps to complete before drawable_fini.

2018-05-05 Thread Roman Gilg
Without this patch plasmashell on Xserver/Mesa master freezes on me when opening the launcher menu (Kickoff). With the patch haven't experienced freezes yet. Haven't tested the Steam client yet. Might be a different problem though. Tested-by: Roman Gilg On Fri, May 4, 2018

Re: [PATCH 2/2] loader_dri3: Variant 2: Wait for pending swaps to complete before drawable_fini.

2018-05-05 Thread Daniel Stone
On 5 May 2018 at 10:15, Mike Lothian wrote: > Out of interest can you try running the vulkan smoketest, I'm seeing this: > > smoketest > terminate called after throwing an instance of 'std::runtime_error' > what(): VkResult -101004 returned > Aborted (core dumped)

Re: [PATCH 2/2] loader_dri3: Variant 2: Wait for pending swaps to complete before drawable_fini.

2018-05-05 Thread Mike Lothian
Which is: VK_ERROR_OUT_OF_DATE_KHR = -101004, On Sat, 5 May 2018 at 10:15 Mike Lothian wrote: > Out of interest can you try running the vulkan smoketest, I'm seeing this: > > smoketest > terminate called after throwing an instance of 'std::runtime_error' > what():

Re: [PATCH 2/2] loader_dri3: Variant 2: Wait for pending swaps to complete before drawable_fini.

2018-05-05 Thread Mike Lothian
Out of interest can you try running the vulkan smoketest, I'm seeing this: smoketest terminate called after throwing an instance of 'std::runtime_error' what(): VkResult -101004 returned Aborted (core dumped) On Sat, 5 May 2018 at 05:25 Mario Kleiner wrote: >