On Wed, Jul 15, 2020 at 12:13:10PM +0200, Peter Zijlstra wrote: > On Wed, Jul 15, 2020 at 12:08:08PM +0200, Joerg Roedel wrote: > > Right, but the handler calls into various other functions. I actually > > started to annotate them all with noinstr, but that was a can of worms > > when calling into generic kernel functions. And the only problem with > > intrumentation in the #VC handler is the #VC-for-#DB exit-code, so I > > decided to only handle this one with instrumentation forbidden and allow > > it for the rest of the handler. > > OK, then maybe change the comment to something like: > > /* > * Handle #DB before calling any !noinstr code to avoid > * recursive #DB. > */ Sounds good, will do. Thanks, Joerg
