From patchwork Fri Jun 12 00:51:00 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Zhao, Yakui" X-Patchwork-Id: 29666 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 n5C0nmUF010525 for ; Fri, 12 Jun 2009 00:49:49 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751800AbZFLAtp (ORCPT ); Thu, 11 Jun 2009 20:49:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754014AbZFLAtp (ORCPT ); Thu, 11 Jun 2009 20:49:45 -0400 Received: from mga09.intel.com ([134.134.136.24]:61923 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751800AbZFLAto (ORCPT ); Thu, 11 Jun 2009 20:49:44 -0400 Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga102.jf.intel.com with ESMTP; 11 Jun 2009 17:38:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.42,205,1243839600"; d="scan'208";a="523779506" Received: from yakui_zhao.sh.intel.com (HELO [10.239.13.185]) ([10.239.13.185]) by orsmga001.jf.intel.com with ESMTP; 11 Jun 2009 17:49:39 -0700 Subject: [PATCH]: ACPI: Add the reference count to avoid unloading ACPI video bus twice From: yakui_zhao To: lenb@kernel.org Cc: linux-acpi@vger.kernel.org Organization: Intel Open Source Technology Center Date: Fri, 12 Jun 2009 08:51:00 +0800 Message-Id: <1244767860.3618.15.camel@localhost.localdomain> Mime-Version: 1.0 X-Mailer: Evolution 2.22.1 (2.22.1-2.fc9) Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org From: Zhao Yakui Sometimes both acpi video and i915 driver are compiled as modules. And there exists the strict dependency between the two drivers. The acpi video bus will be unloaded in course of unloading the i915 driver. If we unload the acpi video driver, then the kernel oops will be triggered. Add the reference count to avoid unloading the ACPI video bus twice. The reference count should be checked before unregistering the acpi video bus. If the reference count is already zero, it won't unregister it again. And after the acpi video bus is already unregistered, the reference count will be set to zero. At the same time the reference count is also checked when registering the acpi video bus. http://bugzilla.kernel.org/show_bug.cgi?id=13396 Signed-off-by: Zhao Yakui --- drivers/acpi/video.c | 42 ++++++++++++++++++++++++++++------- drivers/gpu/drm/i915/i915_opregion.c | 2 - include/acpi/video.h | 4 +-- 3 files changed, 37 insertions(+), 11 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Index: linux-2.6/drivers/acpi/video.c =================================================================== --- linux-2.6.orig/drivers/acpi/video.c 2009-05-31 09:33:17.000000000 +0800 +++ linux-2.6/drivers/acpi/video.c 2009-06-01 13:29:41.000000000 +0800 @@ -76,6 +76,7 @@ static int brightness_switch_enabled = 1; module_param(brightness_switch_enabled, bool, 0644); +static int video_atomic = 0; static int acpi_video_bus_add(struct acpi_device *device); static int acpi_video_bus_remove(struct acpi_device *device, int type); static int acpi_video_resume(struct acpi_device *device); @@ -2302,7 +2303,13 @@ int acpi_video_register(void) { int result = 0; - + if (video_atomic) { + /* + * if the function of acpi_video_register is already called, + * don't register the acpi_vide_bus again and return no error. + */ + return 0; + } acpi_video_dir = proc_mkdir(ACPI_VIDEO_CLASS, acpi_root_dir); if (!acpi_video_dir) return -ENODEV; @@ -2312,11 +2319,34 @@ remove_proc_entry(ACPI_VIDEO_CLASS, acpi_root_dir); return -ENODEV; } - + /* + * When the acpi_video_bus is loaded successfully, increase + * the counter reference. + */ + video_atomic = 1; return 0; } EXPORT_SYMBOL(acpi_video_register); +void acpi_video_unregister(void) +{ + if (!video_atomic) { + /* + * If the acpi video bus is already unloaded, don't + * unload it again and return directly. + */ + return; + } + acpi_bus_unregister_driver(&acpi_video_bus); + + remove_proc_entry(ACPI_VIDEO_CLASS, acpi_root_dir); + + video_atomic = 0; + + return; +} +EXPORT_SYMBOL(acpi_video_unregister); + /* * This is kind of nasty. Hardware using Intel chipsets may require * the video opregion code to be run first in order to initialise @@ -2334,16 +2364,12 @@ return acpi_video_register(); } -void __exit acpi_video_exit(void) +static void __exit acpi_video_exit(void) { - - acpi_bus_unregister_driver(&acpi_video_bus); - - remove_proc_entry(ACPI_VIDEO_CLASS, acpi_root_dir); + acpi_video_unregister(); return; } -EXPORT_SYMBOL(acpi_video_exit); module_init(acpi_video_init); module_exit(acpi_video_exit); Index: linux-2.6/include/acpi/video.h =================================================================== --- linux-2.6.orig/include/acpi/video.h 2009-04-29 09:32:00.000000000 +0800 +++ linux-2.6/include/acpi/video.h 2009-06-01 13:34:19.000000000 +0800 @@ -3,10 +3,10 @@ #if (defined CONFIG_ACPI_VIDEO || defined CONFIG_ACPI_VIDEO_MODULE) extern int acpi_video_register(void); -extern int acpi_video_exit(void); +extern void acpi_video_unregister(void); #else static inline int acpi_video_register(void) { return 0; } -static inline void acpi_video_exit(void) { return; } +static inline void acpi_video_unregister(void) { return; } #endif #endif Index: linux-2.6/drivers/gpu/drm/i915/i915_opregion.c =================================================================== --- linux-2.6.orig/drivers/gpu/drm/i915/i915_opregion.c 2009-04-29 09:32:00.000000000 +0800 +++ linux-2.6/drivers/gpu/drm/i915/i915_opregion.c 2009-06-01 12:02:00.000000000 +0800 @@ -419,7 +419,7 @@ return; if (!suspend) - acpi_video_exit(); + acpi_video_unregister(); opregion->acpi->drdy = 0;