Messages in this thread | | | From | Joel Fernandes <> | Date | Thu, 13 Feb 2020 09:25:08 -0500 | Subject | Re: [PATCH v2 2/9] rcu: Mark rcu_dynticks_curr_cpu_in_eqs() inline |
| |
On Wed, Feb 12, 2020 at 8:41 PM Steven Rostedt <rostedt@goodmis.org> wrote: > > On Wed, 12 Feb 2020 17:38:18 -0500 > Joel Fernandes <joel@joelfernandes.org> wrote: > > > I think there are ways to turn off function inlining, such as gcc's: > > -fkeep-inline-functions > > > > And just to be sure weird compilers (clang *cough*) don't screw this up, > > could we make it static inline notrace? > > inline is defined as notrace, so not needed. > > I did that because of surprises when functions marked as inline > suddenly became non inlined and traced, which caused issues with > function tracing (before I finally got recursion protection working). > But even then, I figured, if something is inlined and gcc actually > inlines it, it wont be traced. For consistency, if something is marked > inline, it should not be traced.
Ah I see it, thanks for the clarification Steve! That looks like a good idea. I withdraw my previous comment.
- Joel
| |