Re: [PATCH xserver v2] glamor: add support for NV12 in Xv

2018-09-11 Thread Alex Deucher
.0 videotestsrc ! video/x-raw, format=NV12 ! xvimagesink > > v2: Combine the two texture2Ds on u_sampler. > > Signed-off-by: Julien Isorce Reviewed-by: Alex Deucher > --- > glamor/glamor_xv.c | 180 > + > 1 file changed

Re: [PATCH xserver 0/3] Xv: add NV12 support in glamor

2018-09-11 Thread Alex Deucher
hat can use glamor. > > Test: gst-launch-1.0 videotestsrc ! video/x-raw, format=NV12 ! xvimagesink > > Julien Isorce (3): > xfree86: define FOURCC_NV12 and XVIMAGE_NV12 > glamor: add support for GL_RG > glamor: add support for NV12 in Xv Series is: Reviewed-

Re: [V2] modesetting: Fix X crash in ms_dirty_update()

2018-08-08 Thread Alex Deucher
pScreen-> > SharedPixmapNotifyDamage. > > On the platform, since the dGPU is GPU device, so that the iGPU is > output master device. SharedPixmapNotifyDamage() should be called when > current device is output master. > > Change-Id: I8fa6922a4f75b5e068970fc4d362f778052379f2 &g

Re: [PATCH] DRI2: Sync radeonsi_pci_ids.h from Mesa

2018-05-14 Thread Alex Deucher
this one, time to look into something > more scalable? > > Signed-off-by: Bas Nieuwenhuizen Reviewed-by: Alex Deucher > --- > hw/xfree86/dri2/pci_ids/radeonsi_pci_ids.h | 30 ++ > 1 file changed, 30 insertions(+) > > diff --git a/hw/xfree86/dri2/pci_i

Re: [PATCH xrandr v5 0/6] xrandr: improve option parsing and documentation

2018-02-27 Thread Alex Deucher
add grammar fix patch > v4: single value accepted for --gamma too, monitor options documented > v5: additional patch to check for positive values > > The patchset is also available from: > > git://git.oblomov.eu/xorg/xrandr parse-doc-fixes > > up to c9755465412cbcf533d

Re: [PATCH xserver] glamor: unifdef XORG_VERSION_CURRENT

2018-02-26 Thread Alex Deucher
On Mon, Feb 26, 2018 at 3:26 PM, Adam Jackson wrote: > This is always true now that glamor is in-tree. > > Signed-off-by: Adam Jackson Reviewed-by: Alex Deucher > --- > glamor/glamor_utils.h | 4 > 1 file changed, 4 deletions(-) > > diff --git a/glamor

Re: [PATCH xserver] glamor: Add 30bit RGB color format support

2018-01-25 Thread Alex Deucher
On Thu, Jan 25, 2018 at 12:03 PM, Michel Dänzer wrote: > From: Hawking Zhang > > Signed-off-by: Hawking Zhang > > [ Michel Dänzer: Adapt to glamor changes since 1.19 ] > > Signed-off-by: Michel Dänzer Acked-by: Alex Deucher > --- > glamor/glamor_egl.c

Re: [PATCH 00/27] edid-decode: fixes and new features and checks

2017-08-31 Thread Alex Deucher
specs, but I did a quick review of the patches and they look good to me. Series is: Acked-by: Alex Deucher > > Regards, > > Hans Verkuil > > Hans Verkuil (27): > edid-decode: add HDMI Forum VSDB support > edid-decode: print the chromaticities > edid-d

Re: [PATCH 0/5] Improve Present support in Xwayland with per window flips

2017-08-30 Thread Alex Deucher
On Wed, Aug 30, 2017 at 5:15 AM, Roman Gilg wrote: > On Wed, Aug 30, 2017 at 4:42 AM, Michel Dänzer wrote: >> >> >> Hi Roman, >> >> >> On 30/08/17 12:24 AM, Roman Gilg wrote: >> > >> > Originating from the bug report >> > >> > https://bugs.freedesktop.org/show_bug.cgi?id=99702 >> > >> > and my ow

Re: [PATCH xserver] glx: Fix error generation for non-reply vendor private requests

2017-08-22 Thread Alex Deucher
On Tue, Aug 22, 2017 at 12:19 PM, Adam Jackson wrote: > Discarding the return value here is just wrong. > > Signed-off-by: Adam Jackson Reviewed-by: Alex Deucher > --- > glx/glxcmds.c | 3 +-- > glx/glxcmdsswap.c | 3 +-- > 2 files changed, 2 insertions(+), 4 deleti

Re: [PATCH xserver 1/3] glx: Remove True/False defines

2017-08-16 Thread Alex Deucher
On Wed, Aug 16, 2017 at 2:49 PM, Adam Jackson wrote: > Those are xlib spellings, we say TRUE/FALSE pretty consistently > elsewhere in the server. > > Signed-off-by: Adam Jackson Series is: Reviewed-by: Alex Deucher > --- > glx/createcontext.c| 16 ---

Re: [PATCH xserver] dri2: sort DRI2InfoPtr::version checking in ascending order

2017-08-03 Thread Alex Deucher
On Thu, Aug 3, 2017 at 2:43 PM, Emil Velikov wrote: > From: Emil Velikov > > Makes it easer to follow if 8 is between 7 and 9 ;-) Agreed :) Reviewed-by: Alex Deucher > > Signed-off-by: Emil Velikov > --- > hw/xfree86/dri2/dri2.c | 11 +-- > 1 file cha

Re: [PATCH xserver 1/3] modesetting: Hide cursor during DPMS

2017-08-03 Thread Alex Deucher
setting is > finished, xf86DisableUnusedFunctions will get called and that will call > xf86CursorResetCursor to make sure the cursor is enabled everywhere it > should be. > > Here's a replacement for the modesetting patch which does this up in > xfree86/modes: Reviewed-by: A

Re: [PATCH xserver 0/1] Good bye GlxSetVisualConfigs

2017-07-31 Thread Alex Deucher
So here it is - update all the drivers (about 200+ lines > dropped in each) and finally remove the symbol from Xserver. > > All the drivers are built tested, unless stated otherwise. Series is: Reviewed-by: Alex Deucher > > Thanks > Emil >

Re: [PATCH xserver 2/2] glx: don't export __glXDRISWRastProvider

2017-07-31 Thread Alex Deucher
On Mon, Jul 31, 2017 at 9:06 AM, Emil Velikov wrote: > From: Emil Velikov > > The symbol is used only internally and is not part of the API/ABI. > > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > glx/glxdriswrast.c | 2 +- > 1 file changed, 1

Re: [PATCH xserver 1/2] glx: fix typo becuase -> because

2017-07-31 Thread Alex Deucher
On Mon, Jul 31, 2017 at 9:06 AM, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > glx/glxcmds.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/glx/glxcmds.c b/glx/glxcmds.c >

Re: [Mesa-dev] [PATCH dri3proto v2] Add modifier/multi-plane requests, bump to v1.1

2017-07-26 Thread Alex Deucher
On Wed, Jul 26, 2017 at 8:15 AM, Nicolai Hähnle wrote: > On 26.07.2017 08:29, Michel Dänzer wrote: >> >> On 25/07/17 05:28 PM, Nicolai Hähnle wrote: >>> >>> On 22.07.2017 14:00, Daniel Stone wrote: I don't have any great solution off the top of my head, but I'd be inclined to b

Re: Multi gpu display

2017-07-14 Thread Alex Deucher
On Fri, Jul 14, 2017 at 4:27 PM, Alex Deucher wrote: > On Fri, Jul 14, 2017 at 4:36 AM, zhoucm1 wrote: >> >> >> On 2017年07月06日 10:30, zhoucm1 wrote: >>> >>> >>> >>> On 2017年07月06日 00:15, Alex Deucher wrote: >>>> >>>>

Re: Multi gpu display

2017-07-14 Thread Alex Deucher
On Fri, Jul 14, 2017 at 4:36 AM, zhoucm1 wrote: > > > On 2017年07月06日 10:30, zhoucm1 wrote: >> >> >> >> On 2017年07月06日 00:15, Alex Deucher wrote: >>> >>> On Tue, Jul 4, 2017 at 6:26 AM, zhoucm1 wrote: >>>> >>>> >>>

Re: Multi gpu display

2017-07-05 Thread Alex Deucher
On Tue, Jul 4, 2017 at 6:26 AM, zhoucm1 wrote: > > > On 2017年07月04日 13:43, zhoucm1 wrote: >> >> >> >> On 2017年07月04日 13:34, Alex Deucher wrote: >>> >>> On Tue, Jul 4, 2017 at 1:22 AM, zhoucm1 wrote: >>>> >>>> >>>

Re: Multi gpu display

2017-07-03 Thread Alex Deucher
On Tue, Jul 4, 2017 at 1:22 AM, zhoucm1 wrote: > > > On 2017年07月04日 11:41, Alex Deucher wrote: >> >> On Mon, Jul 3, 2017 at 4:32 AM, zhoucm1 wrote: >>> >>> Hi xorg-devel guys, >>> >>> Anyone can help me? >>> >>> I t

Re: Multi gpu display

2017-07-03 Thread Alex Deucher
On Mon, Jul 3, 2017 at 4:32 AM, zhoucm1 wrote: > Hi xorg-devel guys, > > Anyone can help me? > > I tried two gpu card with four monitors on Unbuntu 16.04, each gpu has two > monitors. > By default config, I can see four monitors all are lighting with its > separate desktop instance, and the runnin

Re: [PATCH xserver 0/4] xfree86: Miscellaneous fixes for PCI domain in BusID

2017-06-19 Thread Alex Deucher
> xfree86: Document BusID PCI domain format in xorg.conf manpage > xfree86: Document BusID PCI domain format in ddxDesign.xml Series is: Reviewed-by: Alex Deucher > > hw/xfree86/common/xf86pciBus.c | 6 +++--- > hw/xfree86/doc/ddxDesign.xml | 3 ++- > hw/xfree86/man/xorg

Re: [PATCH] xfree86: Add Option "PreferCloneMode"

2017-05-25 Thread Alex Deucher
ferCloneMode" allows > setting this preferClone member. > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > hw/xfree86/man/xorg.conf.man | 5 + > hw/xfree86/modes/xf86Crtc.c | 4 > 2 files changed, 9 insertions(+) > > diff --git a/hw/xfre

Re: [PATCH xserver] modesetting: Add the "DPI" connector type.

2017-04-25 Thread Alex Deucher
On Tue, Apr 25, 2017 at 2:36 PM, Eric Anholt wrote: > The number for it was merged to drm_mode.h in kernel 4.7, and the > output_names[] array just requires that we slot in new strings in > order. > > Signed-off-by: Eric Anholt Reviewed-by: Alex Deucher > --- > hw/xfree8

Re: [PATCH xserver] xfree86/modes: Make colormap/gamma glue code work with RandR disabled

2017-04-12 Thread Alex Deucher
94 > Fixes: 62f44052573b ("xfree86/modes: Move gamma initialization to > xf86RandR12Init12 v2") > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > hw/xfree86/modes/xf86RandR12.c | 136 > +++-

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-04-10 Thread Alex Deucher
On Tue, Apr 4, 2017 at 3:02 AM, Daniel Vetter wrote: > On Mon, Apr 03, 2017 at 03:50:33PM -0700, Keith Packard wrote: >> Daniel Vetter writes: >> >> > Also if this confuses VR, then another reason why we want to make leases >> > invariant and only allow pure revoke, not changing the list. >> >> I

Re: voting confirmation

2017-04-06 Thread Alex Deucher
On Thu, Apr 6, 2017 at 10:39 AM, Antoine Martin wrote: > Hi, > > Apologies for sending an image link to this list, but I can't think of a > better way to explain the somewhat confusing page that showed up after I > cast my ballot: > http://imgur.com/r8nSblg > Did it take my vote into account? Did

Re: [PATCH xserver] xf86_crtc_supports_gamma: Return FALSE if RandR is disabled

2017-04-05 Thread Alex Deucher
s.freedesktop.org/100293 > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > hw/xfree86/modes/xf86Crtc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/hw/xfree86/modes/xf86Crtc.c b/hw/xfree86/modes/xf86Crtc.c > index 3f9857b4a..9ce303d

Re: [PATCH xserver] Xephyr: Handle source-only pictures in ephyrPrepare/DoneComposite

2017-03-22 Thread Alex Deucher
On Wed, Mar 22, 2017 at 11:42 PM, Michel Dänzer wrote: > From: Michel Dänzer > > There is no pixmap associated with source-only pictures. > > Fixes Xephyr -fakexa crashing on startup. > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > hw/kd

Re: [PATCH xserver 2/2] glamor: Always purge the FBO cache in BlockHandler

2017-03-03 Thread Alex Deucher
reserving the main > benefit of the cache for bursts of drawing operations. > > Signed-off-by: Michel Dänzer Series is: Reviewed-by: Alex Deucher > --- > glamor/glamor.c | 6 ++ > glamor/glamor_fbo.c | 24 > glamor/glamor_priv.h | 6 -- &

2017 X.Org Foundation Membership deadline for voting in the election

2017-02-16 Thread Alex Deucher
upcoming election is 05 March 2017 at 23:59 UTC. If you are interested in joining the X.Org Foundation or in renewing your membership, please visit the membership system site at: https://members.x.org/ Alex Deucher, on behalf of the X.Org elections committee

Re: [PATCH xserver 5/5] mi: Add include guards to migc.h

2017-02-16 Thread Alex Deucher
On Thu, Feb 16, 2017 at 1:03 PM, Adam Jackson wrote: > Signed-off-by: Adam Jackson Series is: Reviewed-by: Alex Deucher > --- > mi/migc.h | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/mi/migc.h b/mi/migc.h > index 8d06a6f..add601a 100644 > --- a/mi/

Re: [PATCH xserver 1/3] xfake: Remove

2017-02-16 Thread Alex Deucher
On Thu, Feb 16, 2017 at 12:46 PM, Adam Jackson wrote: > We already have Xvfb for a dummy DDX. > > Signed-off-by: Adam Jackson Seems reasonable to me. Series is: Acked-by: Alex Deucher > --- > configure.ac | 3 - > hw/kdrive/Makefile.am | 9 +-

Re: [PATCH 0/7] ScrnInfoRec cleanup

2017-02-14 Thread Alex Deucher
to follow this with is removing fixed dotclock > support, on the way to unifying the pre- and post-RANDR-1.2 setup paths. > As far as I can tell this would only affect a few ancient mach64, trident, > or chips (ahem) chips. Quite possibly all of them are already unsupported > because they'

Re: [PATCH xserver v2] damage: Validate source pictures bound to windows before unwrapping

2017-02-08 Thread Alex Deucher
On Wed, Feb 8, 2017 at 1:06 PM, Adam Jackson wrote: > On Tue, 2017-02-07 at 11:57 -0500, Alex Deucher wrote: >> On Tue, Feb 7, 2017 at 3:38 AM, Michel Dänzer >> wrote: >> > From: Michel Dänzer >> > >> > The lower layers also do this, but no damage m

Re: [PATCH xserver v2] damage: Validate source pictures bound to windows before unwrapping

2017-02-07 Thread Alex Deucher
Dänzer Makes sense to me. Reviewed-by: Alex Deucher > --- > > v2: Update code comment > > This is an alternative to https://patchwork.freedesktop.org/patch/136721/ . > > configure.ac | 2 +- > miext/damage/damage.c | 10 ++ > 2 files changed,

X.Org Foundation Membership Renewal

2017-02-02 Thread Alex Deucher
and click the renewal link. For new members, click the Join Now link. The X.org Foundation is a non-profit organization under the SPI umbrella which acts as a steward for the X Window System and related projects. Board elections are coming up so renew or join today! Thanks! Alex Deucher

Re: [PATCH xserver 0/2] present: Allow flipping with PRIME slave outputs

2017-02-02 Thread Alex Deucher
nt: Allow flipping with PRIME slave outputs For the series: Reviewed-by: Alex Deucher > > dix/pixmap.c | 16 > present/present.c | 16 > 2 files changed, 12 insertions(+), 20 deletions(-) > > -- > 2.11.0 > > ___

Re: [PATCH v2 xserver 00/11] modesetting: MS_ALL_IN_ONE

2017-01-09 Thread Alex Deucher
On Mon, Jan 9, 2017 at 3:27 AM, Hans de Goede wrote: > Hi, > > On 09-01-17 02:52, Yu, Qiang wrote: >> >> >> Hi Hans, >> >> I forgot there is another difference, PRIME solution has to copy screen >> once before display >> to iGPU while MS_ALL_IN_ONE don't have to and can display what full screen >>

Re: [PATCH xserver v2] present: Only call present_flip_notify if vblank->queued == FALSE

2017-01-07 Thread Alex Deucher
On Sat, Jan 7, 2017 at 3:36 AM, Michel Dänzer wrote: > > Any takers? > > In the absence of a negative review, I'll push this next week, since > more people are running into this. Reviewed-by: Alex Deucher > > > On 28/11/16 10:54 AM, Michel Dänzer wrote: >> F

Re: [PATCH fbdev] Fix shadow fb allocation size (v2)

2017-01-04 Thread Alex Deucher
On Wed, Jan 4, 2017 at 10:30 AM, Adam Jackson wrote: > ->bitsPerPixel is rather obviously eight times too large. > > v2: Use ->displayWidth - the pitch - not ->virtualX (Keith Packard) > > Signed-off-by: Adam Jackson Reviewed-by: Alex Deucher > --- > src/fbdev.c

Re: [PATCH fbdev] Fix shadow fb allocation size

2017-01-03 Thread Alex Deucher
On Tue, Jan 3, 2017 at 4:25 PM, Adam Jackson wrote: > ->bitsPerPixel is rather obviously eight times too large. > > Signed-off-by: Adam Jackson Reviewed-by: Alex Deucher > --- > src/fbdev.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH] edid: Add quirk for ADA 1024x600 7" display.

2016-12-16 Thread Alex Deucher
On Thu, Dec 15, 2016 at 12:56 AM, Kai-Heng Feng wrote: > Detailed mode reports 108 mm x 68 mm which is for smaller display. > Maximum image size reports 15 cm x 10 cm which aligns with its physical > size, use this size instead. > > Signed-off-by: Kai-Heng Feng Acked-b

Re: [PATCH cirrus] alpine: Default to 16bpp

2016-12-08 Thread Alex Deucher
On Thu, Dec 8, 2016 at 9:39 AM, Adam Jackson wrote: > 24bpp support is going away, so since we can't do 32bpp and these cards > have basically no VRAM to begin with, drop to 16bpp. > > Signed-off-by: Adam Jackson Reviewed-by: Alex Deucher > --- > src/alp_driver.c |

Re: [PATCH xserver] DRI2: Sync radeonsi_pci_ids.h from Mesa

2016-10-17 Thread Alex Deucher
On Mon, Oct 17, 2016 at 5:48 AM, Michel Dänzer wrote: > From: Michel Dänzer > > Fixes DRI2 client driver name mapping for newer AMD GPUs with the > modesetting driver, allowing the DRI2 extension to initialize. > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher &

Re: [PATCH xserver 2/2] xf86RandR12: Fix XF86VidModeSetGamma triggering a BadImplementation error

2016-09-19 Thread Alex Deucher
On Mon, Sep 19, 2016 at 11:31 AM, Alex Deucher wrote: > On Sat, Sep 17, 2016 at 6:00 AM, Hans de Goede wrote: >> Commit b4e46c0444bb ("xfree86: Hook up colormaps and RandR 1.2 gamma code") >> dropped the providing of a pScrn->ChangeGamma callback from the xf86Ra

Re: [PATCH xserver 2/2] xf86RandR12: Fix XF86VidModeSetGamma triggering a BadImplementation error

2016-09-19 Thread Alex Deucher
-off-by: Hans de Goede Not too familiar with the code in question, but it looks sane to me. Reviewed-by: Alex Deucher > --- > hw/xfree86/modes/xf86RandR12.c | 29 + > 1 file changed, 29 insertions(+) > > diff --git a/hw/xfree86/modes/xf86RandR12.c b/hw

Re: [PATCH xserver] Add SyncSharedPixmap ScreenRec hook

2016-08-31 Thread Alex Deucher
On Wed, Aug 31, 2016 at 4:56 AM, Michel Dänzer wrote: > From: Michel Dänzer > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > include/scrnintstr.h | 19 +++ > 1 file changed, 19 insertions(+) > > diff --git a/include/scrnintstr.

Re: [PATCH xserver v3] glamor: Handle bitplane in glamor_copy_fbo_cpu

2016-08-18 Thread Alex Deucher
fully more readable by > re-using the existing src_* locals in the bitplane case as well. > > Reported-by: Keith Raghubar > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > glamor/glamor_copy.c | 40 +--- > 1 file chang

Re: [PATCH xserver] xfree86/modes: Handle no palette case better in xf86RandR12CrtcSetGamma

2016-08-01 Thread Alex Deucher
rmaps, e.g. modesetting. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97154 > Reviewed-by: Alex Deucher > >> Signed-off-by: Michel Dänzer >> --- >> hw/xfree86/modes/xf86RandR12.c | 10 +- >> 1 file changed, 9 insertions(+), 1 deletion(-)

Re: [PATCH RFC xserver] dix: Work around non-premultiplied ARGB cursor data

2016-07-15 Thread Alex Deucher
play broken games > * Less overhead than a corresponding workaround in the driver > > Con: > * Makes the problem completely invisible to game developers, so once > it's in we can probably never remove it again > > Opinions? Seems like a good idea to me. Reviewe

Re: [PATCH v3 xserver 3/3] xfree86: Hook up colormaps and RandR 1.2 gamma code

2016-06-22 Thread Alex Deucher
gt; --- > > v3: Free randrp->palette in xf86RandR12CloseScreen, fixes memory leak. Reviewed-by: Alex Deucher > > hw/xfree86/common/xf86Helper.c | 7 +- > hw/xfree86/common/xf86cmap.c | 55 ++--- > hw/xfree86/modes/xf86RandR12.c | 175 > +---

Re: [PATCH v2 xserver 3/3] xfree86: Hook up colormaps and RandR 1.2 gamma code

2016-06-21 Thread Alex Deucher
; v2: Fixes for issues introduced in the last minute before sending out v1: > * No CamelCase for palette struct members > * Initialize palette_size and palette struct members, fixes crash on > server startup I'm not really an expert with respect to the colormap code, but the changes

Re: [PATCH xserver] glamor: Adjust for drawable x/y in composite's copy optimization

2016-05-27 Thread Alex Deucher
ues > passed in need to be converted from drawable-relative to absolute by > adding the drawable x/y values. > > Signed-off-by: Keith Packard Reviewed-by: Alex Deucher > --- > glamor/glamor_render.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/glamo

Re: [PATCH xserver] glamor: Check for composite operations which are equivalent to copies

2016-05-25 Thread Alex Deucher
On Wed, May 25, 2016 at 6:09 AM, Michel Dänzer wrote: > From: Michel Dänzer > > Increases x11perf -compwinwin500 numbers by a factor of 10 for me with > radeonsi. > > Conditions copied from exaComposite(). > > Signed-off-by: Michel Dänzer Reviewed-by: Alex D

Re: [PATCH xserver] glamor: Fix sampling outside of RGBx source/mask pictures

2016-05-25 Thread Alex Deucher
channel, so we have to use the RepeatFix handling > in that case. > > Also, only force alpha = 1.0 when sampling inside of RGBx source/mask > pictures. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94514 > Signed-off-by: Michel Dänzer Reviewed-by: Alex

Re: [PATCH xserver] glamor: Disable logic ops when doing compositing

2016-05-13 Thread Alex Deucher
On Fri, May 13, 2016 at 7:29 AM, Keith Packard wrote: > If the logic op gets left enabled, it overrides the blending > operation, causing incorrect contents on the display. > > Signed-off-by: Keith Packard Makes sense. Reviewed-by: Alex Deucher > --- > glamor/glamor_program

Re: [PATCH 2/2] modesetting: port clean start code from amdgpu. (v2)

2016-05-02 Thread Alex Deucher
il call of EnterVT from > ScreenInit, it really isn't necessary and causes us to set a dirty > mode with -modesetting always anyways. > > v2: reorder set desired modes vs block handler as done for amdgpu. > > Reviewed-by: Eric Anholt > Signed-off-by: Dave Airlie For th

Re: misc prime/modesetting fixes

2016-04-29 Thread Alex Deucher
ently leaking > the slave BO, his fix wasn't very symmetrical, however to make a > nice fix I had to break the ABI by passing a NULL where we didn't > before. patches 1-5 are: Reviewed-by: Alex Deucher > > Dave. > > ___ >

Re: [PATCH xserver 4/4] xfree86/parser: simplify #ifdef ladder

2016-04-18 Thread Alex Deucher
On Sun, Apr 17, 2016 at 3:30 PM, Emil Velikov wrote: > Rather than 'hacking' around symbol names and providing macros such as > 'Local' just fold things and make the code more readable. > > Signed-off-by: Emil Velikov Series is: Reviewed-by: Alex Deucher &

Re: [PATCH xserver] dri3: remove unused file dri3int.h

2016-04-18 Thread Alex Deucher
On Sun, Apr 17, 2016 at 3:34 PM, Emil Velikov wrote: > Copied during the prototyping stage and never used. > > Cc: Keith Packard > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > dri3/dri3int.h | 26 -- > 1 file changed, 26 deleti

Re: [xrandr v2] Select NearestNeighbour filtering for pixel exact scaling

2016-04-04 Thread Alex Deucher
ilter they want using --filter, but > default to automatic selection. > > References: https://bugs.freedesktop.org/show_bug.cgi?id=94816 Also: https://bugs.freedesktop.org/show_bug.cgi?id=94820 Reviewed-by: Alex Deucher > Signed-off-by: Chris Wilson > --- > configure.

Re: [PATCH xserver] DRI2: add Polaris PCI IDs

2016-03-28 Thread Alex Deucher
On Mon, Mar 28, 2016 at 3:36 AM, Michel Dänzer wrote: > From: Sonny Jiang > > Signed-off-by: Sonny Jiang > Reviewed-by: Alex Deucher (Polaris10) > Reviewed-by: Michel Dänzer (Polaris11) > > (Ported from Mesa commit f00c840578a70e479ffb99f6b64c73dc420179fa) > > S

Re: [PATCHv2] modesetting: Add common widescreen resolutions

2016-02-22 Thread Alex Deucher
On Sun, Feb 21, 2016 at 6:12 AM, Adel Gadllah wrote: > Stolen from the intel ddx driver, see: > https://cgit.freedesktop.org/xorg/driver/xf86-video-intel/commit/?id=26fd6bec > https://bugs.freedesktop.org/show_bug.cgi?id=37858 > > it makes those common modes selectable when using the modesetting d

Google Summer of Code 2016

2016-02-18 Thread Alex Deucher
Hi, It's time to start thinking about the Google Summer of Code for 2016! The Xorg GSoC project is open to all projects related to graphics, windowing systems, and related technologies (Mesa, Wayland, X, input, kernel, OpenGL, OpenCL, etc.). For mentors and developers, please start thinking of id

Re: Hybrid graphics - "radeon" driver unloaded, "modesetting" loaded?

2016-02-10 Thread Alex Deucher
On Wed, Feb 10, 2016 at 7:58 AM, Lennert Van Alboom wrote: > On Fri, Feb 05, 2016 at 11:24:12AM +0900, Michel Dänzer wrote: >> On 04.02.2016 19:05, Lennert Van Alboom wrote: >> > Hello. I've got a HP Elitebook 850g1 with hybrid graphics: >> > >> > $ lspci | grep VGA >> > 00:02.0 VGA compat

Re: [PATCH rendercheck 1/5] Start using stdbool.h instead of Xlib or custom bools.

2016-02-01 Thread Alex Deucher
On Mon, Feb 1, 2016 at 4:48 PM, Eric Anholt wrote: > I have a hard time typing anything else at this point. > > Signed-off-by: Eric Anholt > --- > main.c | 21 ++ > ops.c| 2 +- > rendercheck.h| 58 +++

Re: [PATCH xserver] modesetting: Require sufficiently new libdrm

2016-01-27 Thread Alex Deucher
On Wed, Jan 27, 2016 at 11:50 AM, Adam Jackson wrote: > Bugzilla: https://bugs.freedesktop.org/93883 > Signed-off-by: Adam Jackson Reviewed-by: Alex Deucher > --- > configure.ac | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/configure.ac b

Re: [PATCH xserver] xfree86/modes: Set RandR primary output from CreateScreenResources

2016-01-14 Thread Alex Deucher
quot; in Section "Monitor". > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > hw/xfree86/modes/xf86RandR12.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/hw/xfree86/modes/xf86RandR12.c b/hw/xfree86/modes/xf86RandR12.c > index eae

Re: [PATCH xserver] prime: Damage full destination rectangle when we start dirty tracking

2015-12-03 Thread Alex Deucher
On Thu, Dec 3, 2015 at 3:04 AM, Michel Dänzer wrote: > From: Michel Dänzer > > This makes sure that the destination pixmap contents will be fully > initialized. Without this, a PRIME output starts out with garbage. > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deuc

Re: [PATCH xserver 3/4] modesetting: Remove XF86_CRTC_VERSION checks

2015-10-30 Thread Alex Deucher
Daniel Martin For 1-3: Reviewed-by: Alex Deucher > --- > hw/xfree86/drivers/modesetting/drmmode_display.c | 7 +-- > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/hw/xfree86/drivers/modesetting/drmmode_display.c > b/hw/xfree86/drivers/modesetting/drmmode

Re: [PATCH xserver] DRI2: Sync radeonsi_pci_ids.h from Mesa

2015-10-27 Thread Alex Deucher
On Mon, Oct 26, 2015 at 10:51 PM, Michel Dänzer wrote: > From: Michel Dänzer > > Fixes DRI2 client driver name mapping for newer AMD GPUs with the > modesetting driver, allowing the DRI2 extension to initialize. > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher &

Re: [PATCH] Only align screen / scanout pixmap height where necessary

2015-10-02 Thread Alex Deucher
n > > This doesn't seem to make any difference when using EXA for > acceleration. > > Compared to Stephen Chandler's patch, this drops the height alignment > in most places and only keeps it where it's really necessary. > > Reported-by: Stephen Chandler Paul &g

Re: via_ati_questions

2015-08-06 Thread Alex Deucher
On Thu, Aug 6, 2015 at 9:55 AM, Doug Lasse wrote: > Hi, > > I'm writing to you cause I have several questions about my 2 graphics card > which are VIA K8N800/K8M800/K8N800A Unichrome Pro IGP and ATI Radeon 8500. > > For my VIA : I've tried openchrome and proprietary via driver but without > any su

Re: [PATCH] [RFC] modesetting: add zaphod support (v2)

2015-08-06 Thread Alex Deucher
Maybe we could squash all the Zaphod patches into one patch, throwing away > my commit messages, so we have one nice patch to show the state of the art > in ZaphodHeads enablement? Both Dave and Mario's patches are: Reviewed-by: Alex Deucher > > thanks, > -mario > > >

Re: [PATCH xf86-video-sis] Remove upload / download EXA hooks

2015-08-03 Thread Alex Deucher
On Mon, Aug 3, 2015 at 1:24 PM, Connor Behan wrote: > Support based on something other than libc memcpy was never added, so > these functions did not improve upon software fallback at all. > > Signed-off-by: Connor Behan Reviewed-by: Alex Deucher > --- > src/s

Re: [PATCH 1/2] Add Bonaire 0x665F

2015-07-28 Thread Alex Deucher
On Sun, Jul 26, 2015 at 5:35 PM, Josef Larsson wrote: > The PCI ID was missing so the xorg radeon driver could not find my > graphics card (XFX Radeon R7 360), so I added it and now it works fine: > > # lspci | grep VGA > 01:00.0 VGA compatible controller: Advanced Micro Devices, Inc. > [AMD/ATI]

Re: [PATCH 1/4] modesetting: Implement 32->24 bpp conversion in shadow update

2015-07-22 Thread Alex Deucher
viewed-by: Adam Jackson > Signed-off-by: Dave Airlied For the series: Reviewed-by: Alex Deucher > --- > hw/xfree86/drivers/modesetting/Makefile.am | 2 + > hw/xfree86/drivers/modesetting/driver.c | 41 --- > hw/xfree86/drivers/modesetting/drmmode_display.c | 24 +

Re: [PATCH] prime: add rotation support for offloaded outputs (v2)

2015-07-01 Thread Alex Deucher
ap, and > avoid any strange hobbits and rotations. > > This breaks ABI in two places. > > Signed-off-by: Dave Airlie Reviewed-by: Alex Deucher > --- > dix/pixmap.c | 180 > +-- > hw/xfree86/driv

Re: [PATCH 6/7] glamor: Mark a bunch of single-file data static.

2015-07-01 Thread Alex Deucher
by: Eric Anholt Reviewed-by: Alex Deucher > --- > glamor/glamor_core.c| 2 +- > glamor/glamor_program.c | 8 > glamor/glamor_render.c | 2 +- > 3 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/glamor/glamor_core.c b/glamor/glamor_core.c >

Re: [PATCH 5/7] glamor: Make a bunch of single-file glamor functions static.

2015-07-01 Thread Alex Deucher
On Tue, Jun 30, 2015 at 6:58 PM, Eric Anholt wrote: > Signed-off-by: Eric Anholt Reviewed-by: Alex Deucher > --- > glamor/glamor_largepixmap.c | 10 +++--- > glamor/glamor_priv.h| 22 -- > glamor/glamor_render.c | 4 ++-- > 3 files cha

Re: [PATCH 4/7] glamor: Drop dead glamor_es2_pixmap_read_prepare().

2015-07-01 Thread Alex Deucher
On Tue, Jun 30, 2015 at 6:58 PM, Eric Anholt wrote: > It's been unused since I killed glamor_download_pixmap_to_cpu(). > > Signed-off-by: Eric Anholt Reviewed-by: Alex Deucher > --- > glamor/glamor_pixmap.c | 70 > --

Re: [PATCH 3/7] glamor: Restore the hook to glamor_composite_rectangles().

2015-07-01 Thread Alex Deucher
On Tue, Jun 30, 2015 at 6:58 PM, Eric Anholt wrote: > It was apparently accidentally dropped in keithp's removal of _nf > functions in 90d326fcc687e6d6d4b308f6272ededcf8145a17. > > Signed-off-by: Eric Anholt Reviewed-by: Alex Deucher > --- > glamor/glamor.c | 2

Re: [PATCH 2/7] glamor: Fix up some weird formatting in _glamor_create_fbo_array().

2015-07-01 Thread Alex Deucher
On Tue, Jun 30, 2015 at 6:58 PM, Eric Anholt wrote: > Signed-off-by: Eric Anholt Reviewed-by: Alex Deucher > --- > glamor/glamor_fbo.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/glamor/glamor_fbo.c b/glamor/glamor_fbo.c > index ca

Re: [PATCH 1/7] glamor: Drop a redundant check.

2015-07-01 Thread Alex Deucher
On Tue, Jun 30, 2015 at 6:58 PM, Eric Anholt wrote: > Above, we've already checked for ->fbo && ->fbo->fb and returned. > > Signed-off-by: Eric Anholt Reviewed-by: Alex Deucher > --- > glamor/glamor_pixmap.c | 3 +-- > 1 file changed, 1 insertion(+)

Re: [PATCH 7/7] glamor: Use ARRAY_SIZE in a couple more places for consistency.

2015-07-01 Thread Alex Deucher
On Tue, Jun 30, 2015 at 6:58 PM, Eric Anholt wrote: > Signed-off-by: Eric Anholt Reviewed-by: Alex Deucher > --- > glamor/glamor_program.c | 6 ++ > glamor/glamor_render.c | 5 + > 2 files changed, 3 insertions(+), 8 deletions(-) > > diff --git a/glamor/glam

Re: [PATCH] [RFC] cursor: drop ARGB_CURSOR

2015-06-09 Thread Alex Deucher
t. > > Signed-off-by: Dave Airlie Reviewed-by: Alex Deucher > --- > dix/cursor.c | 8 > hw/kdrive/ephyr/ephyrcursor.c | 4 > hw/xfree86/modes/xf86Cursors.c | 8 > hw/xfree86/ramdac/xf86Cursor.c | 2 -- > hw/xfree86/ramdac/xf86Cu

Re: [PATCH 1/4] present: static cleanup

2015-06-02 Thread Alex Deucher
On Tue, Jun 2, 2015 at 11:37 AM, Adam Jackson wrote: > Signed-off-by: Adam Jackson For the series: Reviewed-by: Alex Deucher > --- > present/present_event.c | 2 +- > present/present_priv.h| 2 -- > present/present_request.c | 4 ++-- > 3 files changed, 3 insertio

Re: [PATCH] xfree86: attempt to autoconfig gpu slave devices (v3)

2015-04-22 Thread Alex Deucher
ill get assigned to the GPUDevice >> sections for the screen. >> >> v2: fix the skipping unsuitable screen logic (Aaron) >> v3: fix segfault if not conf file (me, 5s after sending v2) > > Anyone care to r-b this? I'm not too familiar with the code in question

Re: [PATCH] Add amdgpu as a default driver for AMD GPUs

2015-04-20 Thread Alex Deucher
On Mon, Apr 20, 2015 at 11:08 PM, Michel Dänzer wrote: > From: Michel Dänzer > > Its Probe hook bails cleanly when it can't initialize, in which case the > ati driver will be tried next. > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > hw/xf

Re: [PATCH 0/5] Trivial randr cleanups

2015-03-20 Thread Alex Deucher
decided to > remove them. Alternatively we can change them to LogMessageVerb(X_INFO, > 3...) or similar. > > While there, my editor pointed out a few long lines so I've decided to > address those as well :-) > > Any and all input will be appreciated. For the series:

Re: [PATCH 2/2] modesetting: Include dix-config.h from dumb_bo.c

2015-03-17 Thread Alex Deucher
On Mon, Mar 16, 2015 at 9:21 PM, Michel Dänzer wrote: > From: Michel Dänzer > > Fixes mmap failures with 32-bit builds. > > Signed-off-by: Michel Dänzer For the series: Reviewed-by: Alex Deucher > --- > hw/xfree86/drivers/modesetting/dumb_bo.c | 4 > 1 fi

Re: [PATCH] int10: Fix mapping the interrupt vector

2015-03-04 Thread Alex Deucher
On Wed, Mar 4, 2015 at 12:51 PM, Adam Jackson wrote: > pci_device_map_legacy returns 0 on success, and an errno value on > failure. It works a lot better if we don't treat 0 as failure. > > Signed-off-by: Adam Jackson Good catch! Reviewed-by: Alex Deucher > --- > hw/

Re: [PATCH 2/2] exa: Verify non-null pSrc in mgaDownloadFromScreen()

2015-03-03 Thread Alex Deucher
On Tue, Mar 3, 2015 at 3:50 PM, Tormod Volden wrote: > On Tue, Mar 3, 2015 at 9:21 PM, Alex Deucher wrote: >> On Tue, Mar 3, 2015 at 2:21 PM, Tormod Volden wrote: >>> On Tue, Mar 3, 2015 at 3:18 AM, Michel Dänzer wrote: >>>> About patch 1, since mgaDownloadFromScre

Re: [PATCH 2/2] exa: Verify non-null pSrc in mgaDownloadFromScreen()

2015-03-03 Thread Alex Deucher
On Tue, Mar 3, 2015 at 2:21 PM, Tormod Volden wrote: > On Tue, Mar 3, 2015 at 3:18 AM, Michel Dänzer wrote: >> On 03.03.2015 05:01, Tormod Volden wrote: >>> mgaDownloadFromScreen(PixmapPtr pSrc, int x, int y, int w, int h, >>>char *dst, int dst_pitch) >>> { >>> +if (!

GSoC Ideas!

2015-02-11 Thread Alex Deucher
Hi everyone! It's that time of year again. Time to start coming up with GSoC ideas. Martin and I are organizing this year's Xorg entry. We need to fill in our ideas page with some good possible projects for students. The project ideas should be something that a student could accomplish over th

Re: [PATCH] glamor: check max native ALU instructions

2015-01-28 Thread Alex Deucher
On Wed, Jan 28, 2015 at 4:55 PM, Ian Romanick wrote: > On 01/28/2015 01:00 PM, Olivier Fourdan wrote: >> On 28/01/15 21:17, Ian Romanick wrote: >>> >>> Do we know which GPUs cannot meet this limit and also do not have "true" >>> integer support? I wonder if we should go ahead and do the >>> GL_ME

Re: [xf86-video-ati] dri2: Enable BufferAge support

2015-01-20 Thread Alex Deucher
radeon does not support TripleBuffering so we do not have to > worry about perserving the flags when injecting the third buffer. > > Signed-off-by: Chris Wilson > Cc: Dave Airlie > Cc: Jerome Glisse > Cc: Alex Deucher > Cc: Michel Dänzer Reviewed-by: Alex Deuch

Re: [PATCH] randr/prime: Don't stop on the first pipe when disabling ReplaceScanoutPixmap

2015-01-16 Thread Alex Deucher
Unwind upon failure >> >> Reported-by: Christoph Haag >> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84653 >> > Signed-off-by: Chris Wilson >> > Cc: Dave Airlie >> Tested-by: Christoph Haag > > Ping? Reviewed-by: Alex Deucher > -Chris >

  1   2   3   4   >