From patchwork Wed May 20 21:27:49 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Randy Dunlap X-Patchwork-Id: 25097 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n4KLR6YR008361 for ; Wed, 20 May 2009 21:27:06 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755700AbZETV1D (ORCPT ); Wed, 20 May 2009 17:27:03 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755611AbZETV1D (ORCPT ); Wed, 20 May 2009 17:27:03 -0400 Received: from rcsinet12.oracle.com ([148.87.113.124]:21052 "EHLO rgminet12.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755541AbZETV1B (ORCPT ); Wed, 20 May 2009 17:27:01 -0400 Received: from acsinet15.oracle.com (acsinet15.oracle.com [141.146.126.227]) by rgminet12.oracle.com (Switch-3.3.1/Switch-3.3.1) with ESMTP id n4KLPG63000810 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Wed, 20 May 2009 21:25:19 GMT Received: from abhmt009.oracle.com (abhmt009.oracle.com [141.146.116.18]) by acsinet15.oracle.com (Switch-3.3.1/Switch-3.3.1) with ESMTP id n4KLQ2Ub009216; Wed, 20 May 2009 21:26:02 GMT Received: from [192.168.1.4] (/71.182.83.218) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Wed, 20 May 2009 14:25:21 -0700 Message-ID: <4A1475D5.6080602@oracle.com> Date: Wed, 20 May 2009 14:27:49 -0700 From: Randy Dunlap Organization: Oracle Linux Engineering User-Agent: Thunderbird 2.0.0.6 (X11/20070801) MIME-Version: 1.0 To: Woody Suwalski CC: Randy Dunlap , "Rafael J. Wysocki" , Zorael , ACPI Devel Maling List , Len Brown , LKML Subject: Re: [i915] Kernel does not compile when set to use i915 kernel mode-setting per default (CONFIG_DRM_I915_KMS=y) References: <4A1464C9.9030603@xandros.com> In-Reply-To: <4A1464C9.9030603@xandros.com> X-Source-IP: abhmt009.oracle.com [141.146.116.18] X-Auth-Type: Internal IP X-CT-RefId: str=0001.0A010205.4A147543.013E:SCFSTAT5015188,ss=1,fgs=0 Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org Woody Suwalski wrote: > Randy Dunlap wrote: >> Woody Suwalski wrote: >> >>> Rafael J. Wysocki wrote: >>> >>>> On Monday 18 May 2009, Randy Dunlap wrote: >>>> >>>> >>>>> On Mon, 18 May 2009 21:01:06 +0200 Rafael J. Wysocki wrote: >>>>> >>>>> >>>>>> Adding CCs. >>>>>> >>>>>> On Monday 18 May 2009, you wrote: >>>>>> >>>>>>> (First post, so please CC to zorael@gmail.com in case it doesn't >>>>>>> automatically.) >>>>>>> >>>>>>> >>>>>>> I'm trying to compile 2.6.30-rc6 to enable kernel mode-setting *per >>>>>>> default*, for my Intel 945GME video chipset. Alas, when >>>>>>> compiling, it >>>>>>> halts with the following tidbit: >>>>>>> >>>>>>> ... >>>>>>> LD .tmp_vmlinux1 >>>>>>> drivers/built-in.o: In function `intel_opregion_free': >>>>>>> /usr/src/linux/drivers/gpu/drm/i915/i915_opregion.c:422: undefined >>>>>>> reference to `acpi_video_exit' >>>>>>> drivers/built-in.o: In function `intel_opregion_init': >>>>>>> /usr/src/linux/drivers/gpu/drm/i915/i915_opregion.c:393: undefined >>>>>>> reference to `acpi_video_register' >>>>>>> make[1]: *** [.tmp_vmlinux1] Error 1 >>>>>>> make[1]: Leaving directory `/usr/src/linux-2.6.29' >>>>>>> make: *** [debian/stamp/build/kernel] Error 2 >>>>>>> >>>>>>> >>>>>>> grepping acpi_video_register recursively I see it's defined as an >>>>>>> empty function in ./include/acpi/video.h, and as a more proper >>>>>>> function in ./drivers/acpi/video.c. Does somehow setting it to >>>>>>> use KMS >>>>>>> per default via CONFIG_DRM_I915_KMS=y stop it from reading those >>>>>>> files? >>>>>>> >>>>>>> I used to run with KMS by default in the early -30rc*s, but at some >>>>>>> point (rc2?), it stopped compiling succesfully. If I don't enable it >>>>>>> per default, it *does* compile, though I don't know how to >>>>>>> explicitly >>>>>>> enable it at boot-time. So I'm living without KMS for the time >>>>>>> being. >>>>>>> Woe. >>>>>>> >>>>>>> Anything obvious I'm doing wrong? >>>>>>> >>>>> Len posted a patch for this. It's below. >>>>> Len, do you have this patch queued for Linus? >>>>> >>>> Ah, it's commit ecb4aed78dcf09e48c8c34c8c2fa7f5c69344be6, which went in >>>> slightly after -rc6. So it should be fixed. >>>> >>>> Zorael, please test the current Linus' tree. >>>> >>>> >>> Len's patch idea does not work for me - if ACPI is build as a module, I >>> have no way to force acpi-video to be built in. >>> And kernel with i915 built-in breaks... >>> So to summarize: acpi=m with i915=y is broken... >>> >>> For now I am simply commenting-out the i915 acpi-video calls, but that >>> is not a good way... >>> We need to move the core acpi-video to the main kernel... >>> >> >> Please post your troublesome kernel .config file. >> >> Thanks, >> > Certainly... > Here it is inlined: Thanks. If Len's patch is moved to apply to DRM_I915 instead of to DRM_I915_KMS, it works for me, but I'll leave it to Len to decide if that's the right or wrong thing to do. --- From: Len Brown Subject: [PATCH] ACPI, i915: build fix drivers/built-in.o: In function `intel_opregion_init': (.text+0x9d540): undefined reference to `acpi_video_register' Signed-off-by: Len Brown Signed-off-by: Randy Dunlap --- drivers/gpu/drm/Kconfig | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) --- lnx-2630-rc6.orig/drivers/gpu/drm/Kconfig +++ lnx-2630-rc6/drivers/gpu/drm/Kconfig @@ -67,12 +67,18 @@ config DRM_I830 will load the correct one. config DRM_I915 + tristate "i915 driver" select FB_CFB_FILLRECT select FB_CFB_COPYAREA select FB_CFB_IMAGEBLIT select FB select FRAMEBUFFER_CONSOLE if !EMBEDDED - tristate "i915 driver" + # i915 depends on ACPI_VIDEO when ACPI is enabled + # but for select to work, need to select ACPI_VIDEO's dependencies, ick + select VIDEO_OUTPUT_CONTROL if ACPI + select BACKLIGHT_CLASS_DEVICE if ACPI + select INPUT if ACPI + select ACPI_VIDEO if ACPI help Choose this option if you have a system that has Intel 830M, 845G, 852GM, 855GM 865G or 915G integrated graphics. If M is selected, the