Re: [PATCH v2] backports: add ktime_get_seconds()

[Arend van Spriel <arend@broadcom.com>]

On 12/08/2015 10:14 AM, Johannes Berg wrote:
> On Tue, 2015-12-08 at 10:04 +0100, Johannes Berg wrote:
>>
>> That's not a bad idea. We could have a script that creates some kind
>> of
>> header file with #define HAVE_KTIME_GET_SECONDS and then we could use
>> that. We could even just do that for every single exported functions,
>> that will just be a few hundred lines but shouldn't matter much? :)
>>
>
> Ok maybe not - on a very small test configuration for me that's
> actually something like 6.5k lines...
>
> I guess then we could ship a list of functions we want to look at, like
> a file that simply lists one function per line:

Basically the list would be the exported symbols from the backports 
module, ie. compat.ko, right?

Gr. AvS

> symbols-check:
>>>>
> ktime_get_seconds
> <<<
>
> and then we can do something like
>
> grep -f symbols-check /path/to/Module.symvers | \
> 	cut -f2 | \
> 	tr 'a-z' 'A-Z' | \
> 	sed 's/^/#define HAVE_/;s/$/ 1/' \
> 	> backport-include/backport/existing-symbols.h
>
> and include that in the right place?
>
> johannes
>

--
To unsubscribe from this list: send the line "unsubscribe backports" in