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: <88f6c9ea-3ac1-ba78-aeb3-03d37d5a0bb1@gmail.com>
Date:   Tue, 27 Feb 2018 21:34:56 +0200
From:   Serhey Popovych <serhe.popovych@...il.com>
To:     Stephen Hemminger <stephen@...workplumber.org>
Cc:     netdev@...r.kernel.org, thomas.de_schampheleire@...ia.com
Subject: Re: [PATCH iproute2 1/3] ip: Fix compilation with kernel headers <
 3.4

Stephen Hemminger wrote:
> On Tue, 27 Feb 2018 14:06:50 +0200
> Serhey Popovych <serhe.popovych@...il.com> wrote:
> 
>> Since commit 596b1c94aa38 ("iproute: build more easily on Android"),
>> iproute2 uses types __kernel_long_t and __kernel_ulong_t but does not
>> provide internal definitions for it.
>>
>> This means that compilation using kernel headers that are older than 3.4
>> (where these types were added) will fail. This situation may be uncommon
>> for native compilation, but not uncommon for cross compilation where the
>> toolchains may be a bit older.
>>
>> Provide the necessary types internally if not provided by the kernel
>> headers to fix compilation in such cases.
>>
>> Co-Developed-by: Serhii Popovych <serhe.popovych@...il.com>
>> Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@...ia.com>
>> Signed-off-by: Serhey Popovych <serhe.popovych@...il.com>
>> ---
>>  Makefile                              |    5 ++++-
>>  include/compat/kernel/linux/sysinfo.h |   14 ++++++++++++++
> 
> Why not just start a single file include/compat.h which is what
> other software does.

Yes it is good, but not for our case. We use include_next to define
__kernel_long_t and __kernel_ulong_t types if they not defined. If doing
single <compat.h> we need to include it in nearly all .c files as first
include file.

I also start thinking on single <compat.h> and found it bit complicated
than just adding header, (re)defining functionality and then include_next.

> 
> Doing fine grained kernel and libc per file makes it more painful.

Agree, and we already have <netinet/tcp.h> done using similar schema
that is reverted with this series.

> 




Download attachment "signature.asc" of type "application/pgp-signature" (491 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ