Return-Path: Received: from mail-ig0-f172.google.com ([209.85.213.172]:37700 "EHLO mail-ig0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964923AbbLRSwu (ORCPT ); Fri, 18 Dec 2015 13:52:50 -0500 Received: by mail-ig0-f172.google.com with SMTP id to18so40998154igc.0 for ; Fri, 18 Dec 2015 10:52:50 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: <1448389782-99578-1-git-send-email-kolga@netapp.com> Date: Fri, 18 Dec 2015 13:52:49 -0500 Message-ID: Subject: Re: [PATCH Version 2 0/2] Adding to tracepoints From: Olga Kornievskaia To: Trond Myklebust Cc: linux-nfs Content-Type: text/plain; charset=UTF-8 Sender: linux-nfs-owner@vger.kernel.org List-ID: Trond, I have had wireshark accept the patch to display the crc32 hash of the stateid's other field to match the output in the tracepoints. Can you please provide feedback about the proposed additions to tracepoints? Thank you. On Wed, Dec 9, 2015 at 3:59 PM, Olga Kornievskaia wrote: > Trond, > > Any thoughts on the patches? > > Thank you. > > On Tue, Nov 24, 2015 at 1:29 PM, Olga Kornievskaia wrote: >> First patch adds stateid information to tracepoints (openstateid, delegation >> stateid, and locking stateid). Version 2 of this patch fixes two things from >> the previous version: (1) typo in printout missing ":" in one stateid, (2) >> close tracepoint should take stateid from the args instead of the state passed >> in as state's stateid could be delegation stateid. >> >> Second patch adds a new tracepoint nfs_cached_open() >> >> Olga Kornievskaia (2): >> Adding stateid information to tracepoints >> Adding tracepoint to cached open >> >> fs/nfs/internal.h | 9 ++ >> fs/nfs/nfs4proc.c | 15 +-- >> fs/nfs/nfs4trace.h | 294 ++++++++++++++++++++++++++++++++++++++++++++++++----- >> 3 files changed, 286 insertions(+), 32 deletions(-) >> >> -- >> 1.8.3.1 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html