Re: [PATCH 5/6] objtool: Add UACCESS validation

From: Linus Torvalds
Date: Mon Feb 25 2019 - 12:34:40 EST


On Mon, Feb 25, 2019 at 9:15 AM Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
>
> Ah, so the thing we need for inline-asm is %=, but Linus didn't like it:

Well, to be fair, I also didn't like the random big numbers.

I think that inline asm that has local labels should just use simple
enumeration, and always use the direction suffix to make those local
labels be unambiguous.

Now, there are cases where we may not be able to do that, because we
have internal macros within the asm that _also_ wants to do its own
numbers. This isn't such a case.

For that case, you might actually want to use a fancy "get me a unique
number and prefix", but it should be seen as the exception rather than
the first solution, because the code starts looking really random.

Linus