| Message ID | 20210715060021.43249-1-xiyou.wangcong@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 851f36e40962408309ad2665bf0056c19a97881c |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net-next,resend,v2] net_sched: use %px to print skb address in trace_qdisc_dequeue() | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Link |
| netdev/fixes_present | success | Link |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Clearly marked for net-next |
| netdev/subject_prefix | success | Link |
| netdev/cc_maintainers | warning | 2 maintainers not CCed: mingo@redhat.com rostedt@goodmis.org |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Link |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 6 this patch: 6 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | Link |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 6 this patch: 6 |
| netdev/header_inline | success | Link |
On 2021/7/15 14:00, Cong Wang wrote: > From: Qitao Xu <qitao.xu@bytedance.com> > > Print format of skbaddr is changed to %px from %p, because we want > to use skb address as a quick way to identify a packet. > > Note, trace ring buffer is only accessible to privileged users, > it is safe to use a real kernel address here. Does it make more sense to use %pK? see: https://lwn.net/Articles/420403/ > > Reviewed-by: Cong Wang <cong.wang@bytedance.com> > Signed-off-by: Qitao Xu <qitao.xu@bytedance.com> > --- > include/trace/events/qdisc.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/trace/events/qdisc.h b/include/trace/events/qdisc.h > index 330d32d84485..58209557cb3a 100644 > --- a/include/trace/events/qdisc.h > +++ b/include/trace/events/qdisc.h > @@ -41,7 +41,7 @@ TRACE_EVENT(qdisc_dequeue, > __entry->txq_state = txq->state; > ), > > - TP_printk("dequeue ifindex=%d qdisc handle=0x%X parent=0x%X txq_state=0x%lX packets=%d skbaddr=%p", > + TP_printk("dequeue ifindex=%d qdisc handle=0x%X parent=0x%X txq_state=0x%lX packets=%d skbaddr=%px", > __entry->ifindex, __entry->handle, __entry->parent, > __entry->txq_state, __entry->packets, __entry->skbaddr ) > ); >
On Thu, Jul 15, 2021 at 4:26 AM Yunsheng Lin <linyunsheng@huawei.com> wrote: > > On 2021/7/15 14:00, Cong Wang wrote: > > From: Qitao Xu <qitao.xu@bytedance.com> > > > > Print format of skbaddr is changed to %px from %p, because we want > > to use skb address as a quick way to identify a packet. > > > > Note, trace ring buffer is only accessible to privileged users, > > it is safe to use a real kernel address here. > > Does it make more sense to use %pK? > > see: https://lwn.net/Articles/420403/ I think you have the answer: + * %pK cannot be used in IRQ context because it tests + * CAP_SYSLOG. ;) So, no. Thanks.
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Wed, 14 Jul 2021 23:00:21 -0700 you wrote: > From: Qitao Xu <qitao.xu@bytedance.com> > > Print format of skbaddr is changed to %px from %p, because we want > to use skb address as a quick way to identify a packet. > > Note, trace ring buffer is only accessible to privileged users, > it is safe to use a real kernel address here. > > [...] Here is the summary with links: - [net-next,resend,v2] net_sched: use %px to print skb address in trace_qdisc_dequeue() https://git.kernel.org/netdev/net/c/851f36e40962 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/include/trace/events/qdisc.h b/include/trace/events/qdisc.h index 330d32d84485..58209557cb3a 100644 --- a/include/trace/events/qdisc.h +++ b/include/trace/events/qdisc.h @@ -41,7 +41,7 @@ TRACE_EVENT(qdisc_dequeue, __entry->txq_state = txq->state; ), - TP_printk("dequeue ifindex=%d qdisc handle=0x%X parent=0x%X txq_state=0x%lX packets=%d skbaddr=%p", + TP_printk("dequeue ifindex=%d qdisc handle=0x%X parent=0x%X txq_state=0x%lX packets=%d skbaddr=%px", __entry->ifindex, __entry->handle, __entry->parent, __entry->txq_state, __entry->packets, __entry->skbaddr ) );