Re: [Intel-gfx] [PATCH v2] drm/i915: Enable GSE interrupt on BDW+

2016-05-23 Thread Mark Kettenis
errupt handler was always prepared for it. > > Let's enable it and see what happens. > > > > Credit to Mark Kettenis who fixed this in the OpenBSD fork of the > > driver. He reports that it fixed the "ACPI _BCM/_BCQ-based > > brightness mechanism on a MacBookPro12,1

Re: [Intel-gfx] drm modeset identifiers and xf86-video-intel

2016-04-08 Thread Mark Kettenis
> Date: Thu, 7 Apr 2016 21:49:22 +0100 > From: Chris Wilson > > On Thu, Apr 07, 2016 at 08:20:15PM +0200, Mark Kettenis wrote: > > On OpenBSD I implemented idr_alloc() to return random IDs. While the > > xf86-video-modesetting driver is perfectly happy with this, the >

[Intel-gfx] drm modeset identifiers and xf86-video-intel

2016-04-07 Thread Mark Kettenis
On OpenBSD I implemented idr_alloc() to return random IDs. While the xf86-video-modesetting driver is perfectly happy with this, the xf86-video-intel driver doesn't like it very much. I quickly figured out that that driver truncates the identifiers to 8-bits when it stores the values in its sna_c

[Intel-gfx] Enable GSE interrupt on Broadwell and later?

2015-12-01 Thread Mark Kettenis
I just committed the following cange to the OpenBSD inteldrm(4) driver, which currently is mostly a port of the Linux 3.14 codebase. This enables the GSE interrupt on Broadwell. Without this interrupt, the ASLE backlight brightness mechanism doesn't work. I've verified that this fixed the ACPI _B

Re: [Intel-gfx] [PATCH] drm/i915: Extend GET_APERTURE ioctl to report available map space

2014-09-22 Thread Mark Kettenis
Hi Chris, > This patch extends the current DRM_I915_GEM_GET_APERTURE ioctl to > include a couple of new fields in its reply to userspace - the total > amount of space available in the mappable region of the aperture and > also the single largest block available. Would it be possible to include th

Re: [Intel-gfx] [PATCH] xf86-video-intel: Unbreak build on OpenBSD

2014-02-18 Thread Mark Kettenis
> Date: Tue, 18 Feb 2014 08:06:36 + > From: Chris Wilson > > On Sat, Feb 15, 2014 at 09:29:42PM +0100, Mark Kettenis wrote: > > Fallout from the backlight helper changes. > > Apologies for missing this earlier. I have rearranged the code once > again to try to

[Intel-gfx] [PATCH] xf86-video-intel: Unbreak build on OpenBSD

2014-02-15 Thread Mark Kettenis
Fallout from the backlight helper changes. diff --git a/src/backlight.c b/src/backlight.c index b04b5f7..adc3e94 100644 --- a/src/backlight.c +++ b/src/backlight.c @@ -59,7 +59,9 @@ #ifdef __OpenBSD__ +#include #include +#include int backlight_set(struct backlight *b, int level) { dif

Re: [Intel-gfx] What to do with xf86-video-intel backlight control when running Xorg as non root

2014-02-13 Thread Mark Kettenis
> Date: Thu, 13 Feb 2014 20:37:47 +0100 > From: Hans de Goede > > Hi, Hi Hans, Apologies in advance for jumping into the discussion at a somewhat random point. > On 02/13/2014 05:40 PM, Chris Wilson wrote: > On Thu, Feb 13, 2014 > at 04:52:59PM +0100, Hans de Goede wrote: >> Hi All, > >> > >>

[Intel-gfx] [PATCH] xf86-video-intel build fix for OpenBSD

2014-01-20 Thread Mark Kettenis
Commit 50a45a1cdd4d8319ba9358974d241069689591c5 introduced the use of "struct stat" but doesn't include . Presumably that leaks in trough some other header on Linux, but on OpenBSD compilation fails because the struct isn't known. diff --git a/src/sna/kgem.c b/src/sna/kgem.c index 8e073e2..66f5e7

Re: [Intel-gfx] sna: buffer overrun

2013-11-03 Thread Mark Kettenis
> Date: Sun, 3 Nov 2013 19:43:48 + > From: Chris Wilson > > On Sun, Nov 03, 2013 at 01:22:52PM +0100, Mark Kettenis wrote: > > I ran into a "regression" in xf86-video-intel master. X would spin > > for several seconds and eventually I'd s

[Intel-gfx] sna: buffer overrun

2013-11-03 Thread Mark Kettenis
I ran into a "regression" in xf86-video-intel master. X would spin for several seconds and eventually I'd see a message like: [ 170.724] kgem_bo_write: failed to write 3600 bytes into BO handle=175: 14 in Xorg.0.log Bisected it down to the following commit: commit 4f41bf3de059c4e0a03fb161fb

Re: [Intel-gfx] [PATCH xf86-video-intel] sna: Fix compilation with older GCC

2013-08-04 Thread Mark Kettenis
> Date: Sun, 4 Aug 2013 12:52:18 +0100 > From: Chris Wilson > > On Sun, Aug 04, 2013 at 01:38:10PM +0200, Mark Kettenis wrote: > > From: Mark Kettenis > > I just wrote that patch as well. :( Must be right then! Cheers, Mark _

[Intel-gfx] [PATCH xf86-video-intel] sna: Fix compilation with older GCC

2013-08-04 Thread Mark Kettenis
From: Mark Kettenis Signed-off-by: Mark Kettenis --- src/sna/compiler.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/sna/compiler.h b/src/sna/compiler.h index 2f5dfc7..28d3351 100644 --- a/src/sna/compiler.h +++ b/src/sna/compiler.h @@ -67,6 +67,8 @@ #if HAS_GCC(4, 6

Re: [Intel-gfx] [PATCH xf86-video-intel] sna: Fix OpenBSD backlight control implementation

2013-03-26 Thread Mark Kettenis
> Date: Tue, 26 Mar 2013 09:06:54 + > From: Chris Wilson > > On Mon, Mar 25, 2013 at 10:40:35PM +0100, Mark Kettenis wrote: > > From: Mark Kettenis > > > > Fix cut'n'paste error such that this actually compiles. > > Mea culpa. Thanks a lot,

[Intel-gfx] [PATCH xf86-video-intel] sna: Use a more portable way to determine total RAM size

2013-03-26 Thread Mark Kettenis
From: Mark Kettenis The sysinfo function is Linux-specific. sysconf(_SC_PHYS_PAGES), while not truly portable, is available on many more systems, including Linux, Solaris, NetBSD, FreeBSD and OpenBSD. So use that instead. Verified that this results in the same value as the sysinfo call on a

[Intel-gfx] [RFC xf86-video-intel] sna: OpenBSD drm_i915_gem_execbuffer2 support

2013-03-25 Thread Mark Kettenis
The OpenBSD version of struct drm_i915_execbuffer2 doesn't include the member for cliprect support. Not entirely sure why. Perhaps when Owain Ainsworth added GEM support, it was already obvious that cliprects were not going to be supported on new hardware and therefore not worth supporting. Unfo

[Intel-gfx] [PATCH xf86-video-intel] sna: Fix OpenBSD backlight control implementation

2013-03-25 Thread Mark Kettenis
From: Mark Kettenis Fix cut'n'paste error such that this actually compiles. Signed-off-by: Mark Kettenis --- src/sna/sna_display.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/sna/sna_display.c b/src/sna/sna_display.c index 7891c8d..6d61650 100644

[Intel-gfx] [PATCH xf86-video-intel] Add OpenBSD backlight control implementation

2013-03-23 Thread Mark Kettenis
From: Mark Kettenis Signed-off-by: Mark Kettenis --- src/intel_display.c | 75 - 1 file changed, 69 insertions(+), 6 deletions(-) diff --git a/src/intel_display.c b/src/intel_display.c index 5ee955e..6e6a7a2 100644 --- a/src/intel_display.c

Re: [Intel-gfx] [PATCH xf86-video-intel] Add OpenBSD backlight control implementation

2013-03-22 Thread Mark Kettenis
> Date: Fri, 22 Mar 2013 11:30:45 + > From: Chris Wilson > > On Fri, Mar 22, 2013 at 11:23:11AM +0100, Mark Kettenis wrote: > > From: Mark Kettenis > > Thank you for the patch, I presume it works ;-) Pushed, It actually doe

[Intel-gfx] [PATCH xf86-video-intel] Add OpenBSD backlight control implementation

2013-03-22 Thread Mark Kettenis
From: Mark Kettenis Signed-off-by: Mark Kettenis --- src/intel_display.c | 75 - 1 file changed, 69 insertions(+), 6 deletions(-) diff --git a/src/intel_display.c b/src/intel_display.c index 5ee955e..6e6a7a2 100644 --- a/src/intel_display.c