Le 08/01/2019 à 12:11, Breno Leitao a écrit :
Hi Michael,

On 1/8/19 7:20 AM, Michael Ellerman wrote:
Breno Leitao <lei...@debian.org> writes:

hi Christophe,

On 1/3/19 3:19 PM, LEROY Christophe wrote:
Breno Leitao <lei...@debian.org> a écrit :

This patch simply adds definitions for the MSR bits and some macros to
test for MSR TM bits.

This was copied from arch/powerpc/include/asm/reg.h generic MSR part.

Can't we find a way to avoid duplicating such defines ?

I think there are three possible ways, but none of them respect the premises
we are used too. These are the possible ways I can think of:

1) Including arch/powerpc/include/asm as part of the selftest compilation
process.
    Problem: This might break the selftest independence of the kbuild system.

2) Generate a temporary header file inside selftests/include which contains
these macros at compilation time.
    Problem: The problem as above.

3) Define MSR fields at userspace headers (/usr/include).
    Problem: I am not sure userspace should have MSR bits information.

Do you suggest me to investigate any other way?

In this case I think we can probably just copy the few #defines we need.

Cool. That is the very same thing as suggested by Christophe. I sent the v4
yesterday, with this change already.

It didn't get through. Latest is still v3 it seems, look https://patchwork.ozlabs.org/project/linuxppc-dev/list/?submitter=71262

Christophe


Thanks for looking at it.

Reply via email to