Re: [PATCH v1 7/7] tools: add userspace linker table sandbox

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Vrabel <david.vrabel@xxxxxxxxxx>,Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>,Michael Brown <mcb30@xxxxxxxx>,Juergen Gross <jgross@xxxxxxxx>,Andrew Cooper <andrew.cooper3@xxxxxxxxxx>,Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>,Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx>,"xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>,Andi Kleen <ak@xxxxxxxxxxxxxxx>,pali.rohar@xxxxxxxxx,dvhart@xxxxxxxxxxxxx,platform-driver-x86@xxxxxxxxxxxxxxx,Michal Marek <mmarek@xxxxxxxx>,Rasmus Villemoes <linux@xxxxxxxxxxxxxxxxxx>,Jiri Kosina <jkosina@xxxxxxx>,=?UTF-8?B?7KGw6rK966+8?= <korea.drzix@xxxxxxxxx>,linux-kbuild <linux-kbuild@xxxxxxxxxxxxxxx>,Tony Luck <tony.luck@xxxxxxxxx>,Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>,linux-ia64@xxxxxxxxxxxxxxx,"linux-arm-kernel@xxxxxxxxxxxxxxxxxxx" <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>,linux-sh <linux-sh@xxxxxxxxxxxxxxx>,sparclinux <sparclinux@xxxxxxxxxxxxxxx>,Catalin Marinas <catalin.marinas@xxxxxxx>,Will Deacon <will.deacon@xxxxxxx>,Ste!
 ven
Rostedt <rostedt@xxxxxxxxxxx>,Jani Nikula <jani.nikula@xxxxxxxxx>,Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx>,markus.heiser@xxxxxxxxxxx,jolsa@xxxxxxxxxx,Mark Salter <msalter@xxxxxxxxxx>,Chris Zankel <chris@xxxxxxxxxx>,Max Filippov <jcmvbkbc@xxxxxxxxx>,linux-xtensa@xxxxxxxxxxxxxxxx,Paul Mackerras <paulus@xxxxxxxxx>,Michael Ellerman <mpe@xxxxxxxxxxxxxx>,James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>
Message-ID: <58F06CEA-AEB3-4ED7-8211-95F402C9D3D3@xxxxxxxxx>

On August 22, 2016 5:07:39 PM PDT, "Luis R. Rodriguez" <mcgrof@xxxxxxxxxx> wrote:
>On Fri, Aug 19, 2016 at 03:31:47PM -0700, Kees Cook wrote:
>> On Fri, Aug 19, 2016 at 2:41 PM,  <mcgrof@xxxxxxxxxx> wrote:
>> > From: "Luis R. Rodriguez" <mcgrof@xxxxxxxxxx>
>> >
>> > Add a userspace sandbox to allow easy experimentation and
>> > test extensions with linker tables, section ranges and the
>> > new section core definitions.
>> >
>> > The userspace sandbox tries to mimic the Linux kernel development
>> > flow as much as possible, it however relies on and uses libc.
>Support
>> > is currently only provided to x86_64.
>> >
>> > v4: this patch is new in this series -- added to the kenrel as
>> >     suggested by Boris, as otherwise it'd be really hard to keep
>> >     an external userspace repository in sync.
>> >
>> > Signed-off-by: Luis R. Rodriguez <mcgrof@xxxxxxxxxx>
>> > ---
>> >  Documentation/sections/linker-tables.rst           |   4 +-
>> >  MAINTAINERS                                        |   1 +
>> >  include/linux/tables.h                             |   5 +-
>> >  tools/Makefile                                     |   3 +-
>> >  .../arch/x86/include/generated/asm/section-core.h  |   1 +
>> >  tools/arch/x86/include/generated/ranges.h          |   1 +
>> >  tools/arch/x86/include/generated/tables.h          |   1 +
>> >  tools/include/asm-generic/ranges.h                 | 103 ++++
>> >  tools/include/asm-generic/section-core.h           | 341
>+++++++++++
>> >  tools/include/asm-generic/tables.h                 |  50 ++
>> 
>> Aren't a bunch of these files exact duplicates of the headers in
>include/linux?
>
>Indeed... This a userspace tools/ architecture decision that was made
>long ago,
>so its not up to me, I am just following the strategy devised and
>picked up.
>Refer to 7d7d1bf1d1dabe435ef50efb051724b8664749cb ("perf bench: Copy
>kernel
>files needed to build mem{cpy,set} x86_64 benchmarks") for an example
>of
>previous similar work. By sharing header files this enable more tools/
>to be hacked on.
>
>  Luis

I think this is a legacy from before the uapi change that should really be fixed.  If we need to export additional kernel structures for the tools, we could define a third level of we really need it.
-- 
Sent from my Android device with K-9 Mail. Please excuse brevity and formatting.
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux