Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2015-05-22 Thread Vincent Cheng 
Hi Steven,

On Thu, May 21, 2015 at 5:21 PM, Steven Chamberlain ste...@pyro.eu.org wrote:
 fixed 740719 xserver-xorg-video-intel/2:2.21.15-2+kbsd8u1
 found 740719 xserver-xorg-video-intel/2:2.99.917-1
 thanks

 Hi,

 FYI I'm making an upload to our jessie-kfreebsd suite with this patch.
 I have suitable hardware now to test it.  We do need this for KMS to
 work out-of-the-box, this is the only change needed, and indeed it is
 working well.

 Please consider applying it in sid.  It is a kfreebsd-specific change
 guarded by ifdefs.  I've attached a new version rebased for 2:2.99.917.

Please forward your patch upstream [1].

Regards,
Vincent

[1] http://lists.freedesktop.org/mailman/listinfo/intel-gfx


-- 
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/caczd_td0bac4ayymozucb1syczdzfgpogi6kspkgssdfgsh...@mail.gmail.com

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2015-05-21 Thread Steven Chamberlain 
fixed 740719 xserver-xorg-video-intel/2:2.21.15-2+kbsd8u1
found 740719 xserver-xorg-video-intel/2:2.99.917-1
thanks

Hi,

FYI I'm making an upload to our jessie-kfreebsd suite with this patch.
I have suitable hardware now to test it.  We do need this for KMS to
work out-of-the-box, this is the only change needed, and indeed it is
working well.

Please consider applying it in sid.  It is a kfreebsd-specific change
guarded by ifdefs.  I've attached a new version rebased for 2:2.99.917.

Thanks,
Regards,
-- 
Steven Chamberlain
ste...@pyro.eu.org
diff -u xserver-xorg-video-intel-2.99.917/debian/changelog 
xserver-xorg-video-intel-2.99.917/debian/changelog
--- xserver-xorg-video-intel-2.99.917/debian/changelog
+++ xserver-xorg-video-intel-2.99.917/debian/changelog
@@ -1,3 +1,9 @@
+xserver-xorg-video-intel (2:2.99.917-2) unstable; urgency=medium
+
+  * Autoload i915kms module on kfreebsd (Closes: #740719).
+
+ -- Steven Chamberlain ste...@pyro.eu.org  Fri, 22 May 2015 01:07:26 +0100
+
 xserver-xorg-video-intel (2:2.99.917-1) unstable; urgency=medium
 
   * Upload to unstable. (Closes: #748753)
only in patch2:
unchanged:
--- xserver-xorg-video-intel-2.99.917.orig/src/intel_device.c
+++ xserver-xorg-video-intel-2.99.917/src/intel_device.c
@@ -399,7 +399,11 @@
 
ret = drmCheckModesettingSupported(id);
if (ret) {
+#ifdef __FreeBSD_kernel__
+   if (xf86LoadKernelModule(i915kms))
+#else
if (xf86LoadKernelModule(i915))
+#endif
ret = drmCheckModesettingSupported(id);
if (ret)
return -1;


signature.asc
Description: Digital signature

Processed: Re: Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2015-05-21 Thread Debian Bug Tracking System 
Processing commands for cont...@bugs.debian.org:

 fixed 740719 xserver-xorg-video-intel/2:2.21.15-2+kbsd8u1
Bug #740719 [xserver-xorg-video-intel] xserver-xorg-video-intel: requires KMS 
but loads non-KMS i915.ko on kfreebsd
The source xserver-xorg-video-intel and version 2:2.21.15-2+kbsd8u1 do not 
appear to match any binary packages
Marked as fixed in versions xserver-xorg-video-intel/2:2.21.15-2+kbsd8u1.
 found 740719 xserver-xorg-video-intel/2:2.99.917-1
Bug #740719 [xserver-xorg-video-intel] xserver-xorg-video-intel: requires KMS 
but loads non-KMS i915.ko on kfreebsd
Marked as found in versions xserver-xorg-video-intel/2:2.99.917-1.
 thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
740719: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=740719
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


--
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/handler.s.c.143225412022000.transcr...@bugs.debian.org

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-10-23 Thread Christoph Egger 
Control: -1 fixed 2:2.21.15-2+b2

Christoph Egger christ...@debian.org writes:
 Steven Chamberlain ste...@pyro.eu.org writes:
 I assume this patch to xserver-xorg-video-intel is still needed, in
 order to autoload i915kms+drm2 instead of i915?

 I'm pretty sure I'm running a unmodified xorg stack these days. let me
 check tomorrow.

I have xserver-xorg-video-intel=2:2.21.15-2+b2 which is fine.

  Christoph


-- 
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/87egtzi2rn@mitoraj.siccegge.de

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-10-23 Thread Steven Chamberlain 
found 740719 2:2.21.15-2
tags 740719 + jessie sid patch
thanks

Steven Chamberlain wrote:
 Christoph Egger wrote:
  I have xserver-xorg-video-intel=2:2.21.15-2+b2 which is fine.
 
 Hopefully you're not loading i915kms on boot;  if someone tries a
 clean jessie install with i915 graphics please let us know if KMS
 is working out-of-the-box.

I think you must be loading i915kms manually before Xorg, as Markus
observed i915.ko being loaded instead:
http://lists.debian.org/5448e48f.40...@gambaru.de

That makes sense looking at the code also;  which the patch is supposed
to fix:
https://bugs.debian.org/cgi-bin/bugreport.cgi?msg=5;filename=i915kms.diff;att=1;bug=740719

Regards,
-- 
Steven Chamberlain
ste...@pyro.eu.org


-- 
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20141023113611.ge22...@squeeze.pyro.eu.org

Processed: Re: Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-10-23 Thread Debian Bug Tracking System 
Processing commands for cont...@bugs.debian.org:

 found 740719 2:2.21.15-2
Bug #740719 {Done: Steven Chamberlain ste...@pyro.eu.org} 
[xserver-xorg-video-intel] xserver-xorg-video-intel: requires KMS but loads 
non-KMS i915.ko on kfreebsd
Marked as found in versions xserver-xorg-video-intel/2:2.21.15-2 and reopened.
 tags 740719 + jessie sid patch
Bug #740719 [xserver-xorg-video-intel] xserver-xorg-video-intel: requires KMS 
but loads non-KMS i915.ko on kfreebsd
Added tag(s) sid and jessie.
 thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
740719: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=740719
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


--
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/handler.s.c.141406418320177.transcr...@bugs.debian.org

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-10-23 Thread Christoph Egger 
Steven Chamberlain ste...@pyro.eu.org writes:
 I think you must be loading i915kms manually before Xorg, as Markus
 observed i915.ko being loaded instead:
 http://lists.debian.org/5448e48f.40...@gambaru.de

Indeed I doo

  Christoph


-- 
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/87y4s72d3p@mitoraj.siccegge.de

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-10-22 Thread Steven Chamberlain 
severity 740719 important
thanks

Hi Christoph,

I assume this patch to xserver-xorg-video-intel is still needed, in
order to autoload i915kms+drm2 instead of i915?

At least the source at src/intel_device.c:138 still says:
| if (xf86LoadKernelModule(i915))

Regards,
-- 
Steven Chamberlain
ste...@pyro.eu.org


-- 
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/544822ec.9070...@pyro.eu.org

Processed: Re: Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-10-22 Thread Debian Bug Tracking System 
Processing commands for cont...@bugs.debian.org:

 severity 740719 important
Bug #740719 [xserver-xorg-video-intel] xserver-xorg-video-intel: requires KMS 
but loads non-KMS i915.ko on kfreebsd
Severity set to 'important' from 'normal'
 thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
740719: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=740719
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


--
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/handler.s.c.14140136834090.transcr...@bugs.debian.org

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-10-22 Thread Christoph Egger 
Steven Chamberlain ste...@pyro.eu.org writes:
 I assume this patch to xserver-xorg-video-intel is still needed, in
 order to autoload i915kms+drm2 instead of i915?

I'm pretty sure I'm running a unmodified xorg stack these days. let me
check tomorrow.

  Christoph

-- 
9FED 5C6C E206 B70A 5857  70CA 9655 22B9 D49A E731
Debian Developer | Lisp Hacker | CaCert Assurer


-- 
To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/87r3xzu63x@anonymous.siccegge.de

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-03-04 Thread Robert Millan 
Package: xserver-xorg-video-intel
Version: 2.21.15
Tags: patch
User: debian-...@lists.debian.org
Usertags: kfreebsd

On 04/03/2014 10:43, Christoph Egger wrote:
 Robert Millan r...@debian.org writes:
 With latest kfreebsd-11 manual kldload drm2 shouldn't be required anymore, 
 just
 kldload i915kms. Does this work for you too?
 
 Is there a recomended way to do that automatically? I have had i915kms
 loaded from /etc/modules for a long time now and it works well on my
 kfreebsd-10. However when I boot into kfreebsd-11 it seems to load
 i915.ko. Unloading i915 and loading i915kms later on works fine and I
 get a nice newcons and proper X after that but I'd like to have it
 activated on boot!

The intel driver is supposed to load the KMS version of the module (i915kms) 
instead of
the non-KMS one (i915).

Attached patch should fix this.

(please CC the bug number if you followup)

-- 
Robert Millan
diff -ur xserver-xorg-video-intel-2.21.15.old/src/intel_device.c xserver-xorg-video-intel-2.21.15/src/intel_device.c
--- xserver-xorg-video-intel-2.21.15.old/src/intel_device.c	2013-08-21 13:20:29.0 +0200
+++ xserver-xorg-video-intel-2.21.15/src/intel_device.c	2014-03-04 12:41:21.998263738 +0100
@@ -135,7 +135,11 @@
 
 		ret = drmCheckModesettingSupported(id);
 		if (ret) {
+#ifdef __FreeBSD_kernel__
+			if (xf86LoadKernelModule(i915kms))
+#else
 			if (xf86LoadKernelModule(i915))
+#endif
 ret = drmCheckModesettingSupported(id);
 			if (ret)
 return -1;

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-03-04 Thread Julien Cristau 
On Tue, Mar  4, 2014 at 11:45:33 +, Robert Millan wrote:

 Package: xserver-xorg-video-intel
 Version: 2.21.15
 Tags: patch
 User: debian-...@lists.debian.org
 Usertags: kfreebsd
 
 On 04/03/2014 10:43, Christoph Egger wrote:
  Robert Millan r...@debian.org writes:
  With latest kfreebsd-11 manual kldload drm2 shouldn't be required 
  anymore, just
  kldload i915kms. Does this work for you too?
  
  Is there a recomended way to do that automatically? I have had i915kms
  loaded from /etc/modules for a long time now and it works well on my
  kfreebsd-10. However when I boot into kfreebsd-11 it seems to load
  i915.ko. Unloading i915 and loading i915kms later on works fine and I
  get a nice newcons and proper X after that but I'd like to have it
  activated on boot!
 
 The intel driver is supposed to load the KMS version of the module (i915kms) 
 instead of
 the non-KMS one (i915).
 
 Attached patch should fix this.
 
 (please CC the bug number if you followup)
 
 -- 
 Robert Millan

 diff -ur xserver-xorg-video-intel-2.21.15.old/src/intel_device.c 
 xserver-xorg-video-intel-2.21.15/src/intel_device.c
 --- xserver-xorg-video-intel-2.21.15.old/src/intel_device.c   2013-08-21 
 13:20:29.0 +0200
 +++ xserver-xorg-video-intel-2.21.15/src/intel_device.c   2014-03-04 
 12:41:21.998263738 +0100
 @@ -135,7 +135,11 @@
  
   ret = drmCheckModesettingSupported(id);
   if (ret) {
 +#ifdef __FreeBSD_kernel__
 + if (xf86LoadKernelModule(i915kms))
 +#else
   if (xf86LoadKernelModule(i915))
 +#endif
   ret = drmCheckModesettingSupported(id);
   if (ret)
   return -1;

shouldn't that be #if defined(__FreeBSD_kernel__) ||
defined(__FreeBSD__)?  Or does regular freebsd use i915 as module name?

Cheers,
Julien


signature.asc
Description: Digital signature

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-03-04 Thread Steven Chamberlain 
On 04/03/14 14:32, Julien Cristau wrote:
 shouldn't that be #if defined(__FreeBSD_kernel__) ||
 defined(__FreeBSD__)?

It would be redundant, because upstream FreeBSD also defines
__FreeBSD_kernel__ in 8.3-RELEASE and later[0], and that predates KMS
implementation in the Intel GPU driver, in 9.1-RELEASE[1].

[0]:
http://svnweb.freebsd.org/base/release/8.3.0/sys/sys/param.h?view=log#rev230432

[1]:
http://svnweb.freebsd.org/base/release/9.1.0/sys/dev/drm2/i915/i915_drm.h?view=log#rev235783

Regards,
-- 
Steven Chamberlain
ste...@pyro.eu.org



signature.asc
Description: OpenPGP digital signature

Bug#740719: xserver-xorg-video-intel: requires KMS but loads non-KMS i915.ko on kfreebsd

2014-03-04 Thread Julien Cristau 
On Tue, Mar  4, 2014 at 15:05:06 +, Steven Chamberlain wrote:

 On 04/03/14 14:32, Julien Cristau wrote:
  shouldn't that be #if defined(__FreeBSD_kernel__) ||
  defined(__FreeBSD__)?
 
 It would be redundant, because upstream FreeBSD also defines
 __FreeBSD_kernel__ in 8.3-RELEASE and later[0], and that predates KMS
 implementation in the Intel GPU driver, in 9.1-RELEASE[1].
 
Oh, I wasn't aware that this was also defined by upstream freebsd now.
Thanks.

Cheers,
Julien


signature.asc
Description: Digital signature