linux-kernel - Re: [PATCH v3 1/3] driver core: Add ability for arch code to setup pdev

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: <5F42C27F-25F0-4E3C-AFED-4662D57B1A88@kernel.crashing.org>
Date:	Thu, 7 Jul 2011 23:03:44 -0500
From:	Kumar Gala <galak@...nel.crashing.org>
To:	Greg KH <greg@...ah.com>
Cc:	linux-kernel@...r.kernel.org, linuxppc-dev@...abs.org
Subject: Re: [PATCH v3 1/3] driver core: Add ability for arch code to setup pdev_archdata


On Jul 7, 2011, at 9:11 AM, Greg KH wrote:

> On Thu, Jul 07, 2011 at 08:55:44AM -0500, Kumar Gala wrote:
>> 
>> On Jul 1, 2011, at 5:09 PM, Greg KH wrote:
>> 
>>> On Thu, Jun 23, 2011 at 06:33:57AM -0500, Kumar Gala wrote:
>>>> On some architectures we need to setup pdev_archdata before we add the
>>>> device.  Waiting til a bus_notifier is too late since we might need the
>>>> pdev_archdata in the bus notifier.  One example is setting up of dma_mask
>>>> pointers such that it can be used in a bus_notifier.
>>>> 
>>>> We add noop version of arch_setup_pdev_archdata() in
>>>> <asm-generic/platform_device.h> and allow the arch code to override with
>>>> access the full definitions of struct device, struct platform_device, and
>>>> struct pdev_archdata.
>>> 
>>> Isn't there some way to use "weak" symbols to keep us from having to
>>> create this .h file in every single arch and then if the arch wants to
>>> define it, it does so?
>>> 
>>> That should make this patch simpler, right?
>>> 
>>> thanks,
>>> 
>>> greg k-h
>> 
>> Changing to weak should be doable if we are ok with the additional
>> function call always happening.
> 
> Given that this is not a fast path, and the function call for the
> "normal" case would do nothing, it should be fine, right?
> 
> greg k-h

Sounds good, new version sent that uses weak instead.  If this looks good let me know if you just want to ACK and send via powerpc.git tree or if you'll pick these up.

- k


--
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/