On Fri, Jul 17, 2020 at 03:20:50PM +1000, Herbert Xu wrote: > On Fri, Jul 17, 2020 at 07:01:34AM +0200, Sven Auhagen wrote: > > > > Alright, that makes sense, thank you. > > > > As I said in my second email yesterday, it is just a hint and not binding. > > I run some tests and here is what happens when I disable CPU3 on my 4 Core MCBin: > > I don't think we should be adding policy logic like this into > individual drivers. If the kernel should be doing this at all > it should be done in the IRQ layer. The alternative is to do > it in user-space through irqbalance. I disagree as this is common practice among other kernel drivers like ethernet. Also this is also beeing done in other crypto drivers not to say that the speed improvements are pretty significant. irqbalance can of course also do the job but there is no downside of adding the irq hint in the driver. Best Sven > > Cheers, > -- > Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> > Home Page: https://eur03.safelinks.protection.outlook.com/?url=http:%2F%2Fgondor.apana.org.au%2F~herbert%2F&data=02%7C01%7Csven.auhagen%40voleatech.de%7C85a3fd0bef964ac07a1d08d82a112f12%7Cb82a99f679814a7295344d35298f847b%7C0%7C0%7C637305600595365880&sdata=E%2FnccG%2FNnIivbW0A2mE%2B9k89tWEWA%2B%2FcljshtLi29TI%3D&reserved=0 > PGP Key: https://eur03.safelinks.protection.outlook.com/?url=http:%2F%2Fgondor.apana.org.au%2F~herbert%2Fpubkey.txt&data=02%7C01%7Csven.auhagen%40voleatech.de%7C85a3fd0bef964ac07a1d08d82a112f12%7Cb82a99f679814a7295344d35298f847b%7C0%7C0%7C637305600595365880&sdata=e3f%2FXrlr0k9c1Cdv5kBo6zp5gtkPtkBNMNTJhB2Dg8c%3D&reserved=0
