linux-kernel - Re: [PATCH] ACPI / video: update the condition to check if a device is in

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <2890155.GWLpTipCZ8@vostro.rjw.lan>
Date:	Wed, 03 Dec 2014 03:25:36 +0100
From:	"Rafael J. Wysocki" <rjw@...ysocki.net>
To:	Aaron Lu <aaron.lu@...el.com>
Cc:	Brian Norris <computersforpeace@...il.com>, ntrrgc@...il.com,
	ACPI Devel Mailing List <linux-acpi@...r.kernel.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ACPI / video: update the condition to check if a device is in _DOD list

On Sunday, November 30, 2014 09:14:29 AM Aaron Lu wrote:
> On 11/30/2014 01:34 AM, Brian Norris wrote:
> > Tested-by: Brian Norris <computersforpeace@...il.com>
> 
> Thanks for the test! Updated patch with changelog:
> 
> From 0b11bb529245ddceb69eb558477b9b4a81208f4b Mon Sep 17 00:00:00 2001
> From: Aaron Lu <aaron.lu@...el.com>
> Date: Sun, 30 Nov 2014 09:01:15 +0800
> Subject: [PATCH] ACPI / video: update the condition to check if a device is in _DOD list
> 
> Commit 0b8db271f159 ("ACPI / video: check _DOD list when creating
> backlight devices") checks if the video device is in the bind devices
> list to decide if we should create backlight device for it, that causes
> problem for one Dell Latitude E6410, where none of the video output
> devices are properly bound due to the way how we did the comparing
> between its _ADR and the _DOD's values. Solve this problem by comparing
> the lower 12 bits of both the device's _ADR and the _DOD's values instead
> of relying on bind result.
> 
> Fixes: 0b8db271f159 ("ACPI / video: check _DOD list when creating backlight devices")
> Signed-off-by: Aaron Lu <aaron.lu@...el.com>
> Tested-by: Brian Norris <computersforpeace@...il.com>

Queued up as a fix for 3.18.

> ---
>  drivers/acpi/video.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
> index 41e6b977ceb2..185a57d13723 100644
> --- a/drivers/acpi/video.c
> +++ b/drivers/acpi/video.c
> @@ -1164,7 +1164,8 @@ static bool acpi_video_device_in_dod(struct acpi_video_device *device)
>  		return true;
>  
>  	for (i = 0; i < video->attached_count; i++) {
> -		if (video->attached_array[i].bind_info == device)
> +		if ((video->attached_array[i].value.int_val & 0xfff) ==
> +		    (device->device_id & 0xfff))
>  			return true;
>  	}
>  
> 

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/