linux-kernel - Re: [PATCH 00/21] liblockdep fixes for v4.12

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]

Date:   Fri, 26 May 2017 08:52:52 +0200
From:   Ingo Molnar <mingo@...nel.org>
To:     "Levin, Alexander (Sasha Levin)" <alexander.levin@...izon.com>
Cc:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "ben@...adent.org.uk" <ben@...adent.org.uk>,
        "a.p.zijlstra@...llo.nl" <a.p.zijlstra@...llo.nl>,
        "tglx@...utronix.de" <tglx@...utronix.de>
Subject: Re: [PATCH 00/21] liblockdep fixes for v4.12


* Levin, Alexander (Sasha Levin) <alexander.levin@...izon.com> wrote:

>  MAINTAINERS                                    |   2 +-
>  tools/Makefile                                 |   8 +++--
>  tools/include/linux/bitops.h                   |  10 ++++++
>  tools/include/linux/err.h                      |   5 +++
>  tools/include/linux/jhash.h                    | 175 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  tools/include/linux/sched/clock.h              |   0
>  tools/include/linux/sched/mm.h                 |   0
>  tools/include/linux/sched/task.h               |   0
>  tools/include/linux/unaligned/packed_struct.h  |  46 +++++++++++++++++++++++++++
>  tools/lib/lockdep/Makefile                     |   9 +++---
>  tools/lib/lockdep/lockdep.c                    |  17 ++++++++++
>  tools/lib/lockdep/preload.c                    |   3 +-
>  tools/lib/lockdep/run_tests.sh                 |   8 ++---
>  tools/lib/lockdep/uinclude/linux/bitops.h      |   3 --
>  tools/lib/lockdep/uinclude/linux/compiler.h    |   2 ++
>  tools/lib/lockdep/uinclude/linux/debug_locks.h |   2 +-
>  tools/lib/lockdep/uinclude/linux/irqflags.h    |   8 ++---
>  tools/lib/lockdep/uinclude/linux/kallsyms.h    |   3 +-
>  tools/lib/lockdep/uinclude/linux/kernel.h      |  20 ++++++++++--
>  tools/lib/lockdep/uinclude/linux/lockdep.h     |  26 ++++++++--------
>  tools/lib/lockdep/uinclude/linux/module.h      |   5 +++
>  tools/lib/lockdep/uinclude/linux/rcu.h         |   2 ++
>  22 files changed, 317 insertions(+), 37 deletions(-)
>  create mode 100644 tools/include/linux/jhash.h
>  create mode 100644 tools/include/linux/sched/clock.h
>  create mode 100644 tools/include/linux/sched/mm.h
>  create mode 100644 tools/include/linux/sched/task.h
>  create mode 100644 tools/include/linux/unaligned/packed_struct.h
>  delete mode 100644 tools/lib/lockdep/uinclude/linux/bitops.h

Yeah, so what needs to be fixed as well is for liblockdep to exclusively use 
tools/include/ (and extend those headers where required).

perf already uses that method and it works well.

liblockdep already uses tools/include/ in part, but even after I apply all your 
patches, there's still lib/lockdep/uinclude/ which appears to duplicate a number 
of headers.

I did a quick check - for example WARN_ON() et al is duplicated in 
tools/lib/lockdep/uinclude/linux/kernel.h.

Thanks,

	Ingo