search

google / safeside

Understand and mitigate software-observable side-channels
BSD 3-Clause "New" or "Revised" License
494 stars 54 forks source link

procfs is for processes, not random things #108

Open gregkh opened 4 years ago

gregkh commented 4 years ago

Can we please switch to using debugfs for the kernel module filesystem interface? It's simpler, cleaner, and that is explicitly what it is there for.

procfs is for processes, and no new things that are not processes should be added to it.

I'll be glad to write the patch if you all want :)

ssbr commented 4 years ago

Aha, nice suggestion! Assuming at least one of @asteinha or @mmdriley are OK with the idea in the abstract, I'd be happy to review a PR implementing this, as at the very least it'd give me an opportunity to learn how to run/test that demo.

(But I am slightly too lazy to actually implement the change, if that makes sense.)

gregkh commented 4 years ago

I will be glad to do so if the license is put back to GPLv2.