On 09/20/2018 08:37 PM, Peter Zijlstra wrote:
On Thu, Sep 20, 2018 at 06:05:59PM +0800, Wei Wang wrote:
/**
+ * lbr_select_user_callstack - check if the user callstack mode is set
+ *
+ * @lbr_select: the lbr select msr
+ *
+ * Returns: true if the msr is configured to the user callstack mode.
+ * Otherwise, false.
+ *
+ */
+bool lbr_select_user_callstack(u64 lbr_select)
+{
+ return !!(lbr_select & LBR_USER_CALLSTACK);
+}
+EXPORT_SYMBOL_GPL(lbr_select_user_callstack);
That function is pure and tiny, wth is that an exported symbol and not
an inline?
Thanks Peter for the comments.
Because this function uses the LBR_ macros which are defined in this
lbr.c file.
Do you think it would be OK to move all the above LBR_ macros to
asm/perf_event.h?
Best,
Wei
