From patchwork Mon May 28 15:35:52 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yafang Shao X-Patchwork-Id: 921600 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="ROpvqTBG"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 40vgvQ3jqrz9s15 for ; Tue, 29 May 2018 01:36:50 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S968854AbeE1PgY (ORCPT ); Mon, 28 May 2018 11:36:24 -0400 Received: from mail-pg0-f67.google.com ([74.125.83.67]:43057 "EHLO mail-pg0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S968791AbeE1PgS (ORCPT ); Mon, 28 May 2018 11:36:18 -0400 Received: by mail-pg0-f67.google.com with SMTP id p8-v6so5404907pgq.10; Mon, 28 May 2018 08:36:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id; bh=MBvOJol/rKNolXyQVGFIkkejRvAKn3SlHc1WjGwFIhQ=; b=ROpvqTBG0S2kg45bnduhlTSML9GMdeRqvoSVHKVIst1gg0JhAqAMS6FoNw0I87/9/R CB6Zk5/eL/zcPTShjHwOk3EzyMvPHqhre2MscKpZAru8JLfjD93cZYNZtSwn4MMa+XmT 1MQd1Y1SjsL+pMaxKgBLWNpMrdRLRxD8o3zbTXGL+aVjtZI/oN9Z/+iEu8D/NZu8xwlv kdAxtN8glddH2To+ulDjotgEb6DXWPp4Q0mXgOFeUABQRwNL/ZQlGMJjPQN1g49T1Y+A 2JHr1xNt/RvVmowv/oqVxHmCFhQCOTgKypFLCLrwA9LmLIE4WRAqtOV+E28DcGE/fGY6 jTNw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=MBvOJol/rKNolXyQVGFIkkejRvAKn3SlHc1WjGwFIhQ=; b=dezwJlmE2AiXHsh7WfRAK/CZoNKYHiMkuOLy98x6Aa3zBEImr/lYX4vPTYiVJdH0CD 8dhc0mWWl6/VnAtHzjBNBQYVuArDcEO4aXRsAS9PKAc50vY/FbuP8yAqTS4UrAlTvbcx QogkWBpClVW8dkTPXHxLp+XSPY9zvLWPaG7iAkFfmadxCtf4A0RWtsa+Gp6cRu7S6PxM SBWpy9KxA/bjqakrIT8eRxVlxV293I0YsAnuLARSd2Dx3EdkvyD+BE2Z6wQ3EAo56vhP 03+t8ONy2jjkDvlwMMvcBwmpu2cIeeIGvTtG1veCeulRlA0LUL/pFuOeqVDvPsIQMPsz qqyA== X-Gm-Message-State: ALKqPwcXlkMab3OECDytwVvKGP5PFJcdESCgcJIZ4Xhm83zxKKQoafrJ eSvQoadLmLL8n6nrkyOqQoo= X-Google-Smtp-Source: AB8JxZqEaBKQNug7dkm/cLBPvX+s9heDHVMvc68JjKBtNiF3z+lhqOEQdHI6JlvBZ+g4d6bJDaGBlA== X-Received: by 2002:a62:c00e:: with SMTP id x14-v6mr13823569pff.67.1527521778176; Mon, 28 May 2018 08:36:18 -0700 (PDT) Received: from li1588-6.members.linode.com (li1588-6.members.linode.com. [139.162.104.6]) by smtp.gmail.com with ESMTPSA id n10-v6sm71302232pfk.145.2018.05.28.08.36.16 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 28 May 2018 08:36:17 -0700 (PDT) From: Yafang Shao To: songliubraving@fb.com, edumazet@google.com, davem@davemloft.net Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Yafang Shao Subject: [PATCH v3 net-next 1/2] tcp: use data length instead of skb->len in tcp_probe Date: Mon, 28 May 2018 23:35:52 +0800 Message-Id: <1527521753-17963-1-git-send-email-laoar.shao@gmail.com> X-Mailer: git-send-email 1.8.3.1 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org At this point skb->len is including tcp header length, so it is meaningless to user. data length could be more helpful, with which we can easily filter out the packet without payload. Cc: Eric Dumazet Acked-by: Song Liu Signed-off-by: Yafang Shao --- v3: tcp_hdr() is a little expensive than skb->data, so replace it with skb->data. Signed-off-by: Yafang Shao --- include/trace/events/tcp.h | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/include/trace/events/tcp.h b/include/trace/events/tcp.h index c1a5284..7ff0446 100644 --- a/include/trace/events/tcp.h +++ b/include/trace/events/tcp.h @@ -236,7 +236,7 @@ __field(__u16, sport) __field(__u16, dport) __field(__u32, mark) - __field(__u16, length) + __field(__u16, data_len) __field(__u32, snd_nxt) __field(__u32, snd_una) __field(__u32, snd_cwnd) @@ -250,6 +250,7 @@ TP_fast_assign( const struct tcp_sock *tp = tcp_sk(sk); const struct inet_sock *inet = inet_sk(sk); + const struct tcphdr *th = (const struct tcphdr *)skb->data; memset(__entry->saddr, 0, sizeof(struct sockaddr_in6)); memset(__entry->daddr, 0, sizeof(struct sockaddr_in6)); @@ -261,7 +262,7 @@ __entry->dport = ntohs(inet->inet_dport); __entry->mark = skb->mark; - __entry->length = skb->len; + __entry->data_len = skb->len - __tcp_hdrlen(th); __entry->snd_nxt = tp->snd_nxt; __entry->snd_una = tp->snd_una; __entry->snd_cwnd = tp->snd_cwnd; @@ -272,9 +273,9 @@ __entry->sock_cookie = sock_gen_cookie(sk); ), - TP_printk("src=%pISpc dest=%pISpc mark=%#x length=%d snd_nxt=%#x snd_una=%#x snd_cwnd=%u ssthresh=%u snd_wnd=%u srtt=%u rcv_wnd=%u sock_cookie=%llx", + TP_printk("src=%pISpc dest=%pISpc mark=%#x data_len=%d snd_nxt=%#x snd_una=%#x snd_cwnd=%u ssthresh=%u snd_wnd=%u srtt=%u rcv_wnd=%u sock_cookie=%llx", __entry->saddr, __entry->daddr, __entry->mark, - __entry->length, __entry->snd_nxt, __entry->snd_una, + __entry->data_len, __entry->snd_nxt, __entry->snd_una, __entry->snd_cwnd, __entry->ssthresh, __entry->snd_wnd, __entry->srtt, __entry->rcv_wnd, __entry->sock_cookie) );