linux-kernel - Re: [PATCH] drivers: base: power: domain: Use 'pm_cpu_data' instead of 'cpu

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <5430A8B9.6040003@gmail.com>
Date:	Sun, 05 Oct 2014 10:11:05 +0800
From:	Chen Gang <gang.chen.5i5j@...il.com>
To:	Greg KH <gregkh@...uxfoundation.org>
CC:	rjw@...ysocki.net, pavel@....cz, len.brown@...el.com,
	ulf.hansson@...aro.org, khilman@...aro.org,
	geert+renesas@...der.be, tomasz.figa@...il.com, richard@....at,
	linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drivers: base: power: domain: Use 'pm_cpu_data' instead
 of 'cpu_data' for compiling break

On 10/5/14 9:23, Greg KH wrote:
> On Sun, Oct 05, 2014 at 09:13:09AM +0800, Chen Gang wrote:
>> On 10/5/14 0:00, Greg KH wrote:
>>> On Sat, Oct 04, 2014 at 10:19:50PM +0800, Chen Gang wrote:
>>>> 'cpu_data' is too common to be already used by some architectures (e.g.
>>>> um, m32r, and mn10300), so need use 'pm_cpu_data' instead of, or cause
>>>> compiling break. The related error (with allmodconfig under um):
>>>>
>>>>     CC      drivers/base/platform.o
>>>>   In file included from ./arch/x86/um/asm/processor.h:31:0,
>>>>                    from ./arch/um/include/asm/uaccess.h:16,
>>>>                    from ./arch/um/include/asm/thread_info.h:13,
>>>>                    from include/linux/thread_info.h:54,
>>>>                    from include/asm-generic/current.h:4,
>>>>                    from arch/um/include/generated/asm/current.h:1,
>>>>                    from include/linux/mutex.h:13,
>>>>                    from include/linux/kernfs.h:13,
>>>>                    from include/linux/sysfs.h:15,
>>>>                    from include/linux/kobject.h:21,
>>>>                    from include/linux/device.h:17,
>>>>                    from include/linux/platform_device.h:14,
>>>>                    from drivers/base/platform.c:14:
>>>>   ./arch/um/include/asm/processor-generic.h:107:19: error: expected identifier or '(' before '&' token
>>>>    #define cpu_data (&boot_cpu_data)
>>>>                      ^
>>>>   include/linux/pm_domain.h:74:23: note: in expansion of macro 'cpu_data'
>>>>     struct gpd_cpu_data *cpu_data;
>>>>                        ^
>>>>
>>>> Also need notice about 80 columns boundary.
>>>
>>> I don't object to this change at all, but it could be easier to solve
>>> this by fixing up 'cpu_data' to be named something a bit less "generic"?
>>> What does x86 use for this data type?
>>>
>>
>> It is for some kinds of arm cpu (I let this patch pass arm s3c600
>> def_config building).  Other architectures did not use it, at present.
> 
> So why not change the name for these archs that have such a generic name
> to something not so generic?
> 

All the code are really generic, I don't know whether it will be used
under other architectures in the future (I guess, it will, or it need
not be put to "drivers/base/power/" and "include/linux").

At present, it is not used by other architectures, but it already has
negative effect with other architectures (I found it during compiling
um architecture).

So for me, this issue is not only within arm itself, but also in common
drivers.


Thanks
-- 
Chen Gang

Open, share, and attitude like air, water, and life which God blessed
--
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/