[dm-devel] [PATCH v2] hex2bin: make the function hex_to_bin constant-time

Linus Torvalds torvalds at linux-foundation.org
Wed May 4 20:10:03 UTC 2022


On Wed, May 4, 2022 at 12:58 PM Stafford Horne <shorne at gmail.com> wrote:
>
> I have uploaded a diff I created here:
>   https://gist.github.com/54334556f2907104cd12374872a0597c
>
> It shows the same output.

In hex_to_bin itself it seems to only be a difference due to some
register allocation (r19 and r3 switched around).

But then it gets inlined into hex2bin and there changes there seem to
be about instruction and basic block scheduling, so it's a lot harder
to see what's going on.

And a lot of constant changes, which honestly look just like code code
moved around by 16 bytes and offsets changed due to that.

So I doubt it's hex_to_bin() that is causing problems, I think it's
purely code movement. Which explains why adding a nop or a fake printk
fixes things.

Some alignment assumption that got broken?

               Linus



More information about the dm-devel mailing list