Re: [RFC PATCH 1/9] kernel: add support for patchable function pointers

From: Andy Lutomirski
Date: Fri Oct 05 2018 - 13:44:24 EST


On Fri, Oct 5, 2018 at 10:38 AM Jason A. Donenfeld <Jason@xxxxxxxxx> wrote:
>
> On Fri, Oct 5, 2018 at 7:29 PM Andy Lutomirski <luto@xxxxxxxxxx> wrote:
> > (None of this is to say that I disagree with Jason, though -- I'm not
> > entirely convinced that this makes sense for Zinc. But maybe it can
> > be done in a way that makes everyone happy.)
>
> Zinc indeed will continue to push in the simpler and more minimal
> direction. Down the line I'm open to trying and benching a few
> different ways of going about it with dynamic patching -- something
> that will be pretty easy to experiment with given the lean structure
> of Zinc -- but for the initial merge I intend to do it the way it is,
> which is super fast and pretty straightforward to follow.
>

I *think* the only change to Zinc per se would be that the calls like
chacha20_simd() would be static calls or patchable functions or
whatever we want to call them. And there could be a debugfs to
override the default selection.

Ard, I don't think that sticking this in udev rules makes sense. The
kernel has bascially complete information as to what the right choice
is, and that will change over time as the implementation gets tuned,
and the udev rules will never get updated in sync.