linux-kernel - Re: [PATCH] ahci: unregister acpi notify handler when a ZPODD is unbound

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: <5368900F.6050504@intel.com>
Date:	Tue, 06 May 2014 15:32:31 +0800
From:	Aaron Lu <aaron.lu@...el.com>
To:	Levente Kurusa <levex@...ux.com>, Tejun Heo <tj@...nel.org>
CC:	LKML <linux-kernel@...r.kernel.org>,
	Linux ATA/IDE <linux-ide@...r.kernel.org>,
	Joe Thomas <Joe.Thomas@...hill.com>
Subject: Re: [PATCH] ahci: unregister acpi notify handler when a ZPODD is
 unbound

On 05/06/2014 03:14 PM, Levente Kurusa wrote:
> Hi,
> 
> On 05/06/2014 08:07 AM, Aaron Lu wrote:
>> On 05/06/2014 02:02 PM, Levente Kurusa wrote:
>>> Hi,
>>>
>>> On 05/06/2014 05:16 AM, Aaron Lu wrote:
>>>> On 05/01/2014 12:04 AM, Levente Kurusa wrote:
>>>>> When a ZPODD device is unbound via sysfs, the acpi notify handler
>>>>> is not removed. This causes panics as observed in Bug #74601. The
>>>>
>>>> Ah...too bad, I forgot to consider this situation, thanks for tracking
>>>> this.
>>>>
>>>>> panic only happens when the wake happens from outside the kernel
>>>>> (i.e. inserting media or pressing a button). Implement a new
>>>>> ahci_remove_one function which causes zpodd_exit to be called for all
>>>>> ZPODD devices on the unbound PCI device.
>>>>>
>>>>> Signed-off-by: Levente Kurusa <levex@...ux.com>
>>>>> ---
>>>>>
>>>>> Hi,
>>>>>
>>>>> I am not sure if the loop below is correct. Maybe there is a better
>>>>> solution to loop through all the devices which might use ZPODD?
>>>>
>>>> I didn't find a proper place either. For hotplug, we did the zpodd_exit
>>>> at ata_scsi_handle_link_detach. But for host controller pci device
>>>> removal, we used scsi_remove_host in ata_port_detach and there is no
>>>> place to add the zpodd_exit for a to-be-removed scsi device...
>>>>
>>>> Looks like we can only iterate the ata devices and call zpodd_exit
>>>> explicitly for them if they are zpodd devices. Instead of adding a new
>>>> remove callback, what about just embed that into the ata_port_detach
>>>> like the following example?
>>>
>>> Yes, this makes more sense as this doesn't tinker with exports and such...
>>> However this will throw unused variable compiler warnings if we add the
>>> required #ifdefs... Maybe a new function? ata_zpodd_detach_port?
>>
>> I think we can omit the #ifdefs as the loop is not called frequently and
>> thus doesn't cost much. We already have stubs for zpodd_dev_enabled and
>> zpodd_exit.
>>
> 
> Ah, I see. Shall I send V2? Any tags I should add for you?

Yes please. Also, this seems to be stable candidate.
You can add my reviewed-by tag, thanks.
--
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/