From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ramon Fried Date: Mon, 28 May 2018 14:03:32 +0300 Subject: [U-Boot] [PATCH v3 0/6] Iotrace improvements Message-ID: <20180528110338.19783-1-ramon.fried@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de These set of patches add few improvements to iotrace. * Region limiting - allows setting an address and size where only io operations that falls into that address are logged. * Timestamping - Timestamp every iotrace record with current timestamp * dumping - iotrace dump command for dumping all records from buffer in a readable fashion. In terms of backwards compatibility, the timestamp is not backward compatible as it changes the iotrace record. so if one developed an offline parsing tool it will be broken. I though of adding #ifdef specific for that, but eventually I didn't. v2: * fixed printf format * added a fix when the buffer is full v3: * changed the "buffer full" scenario as recommended by Simon Glass. This fix is dependand on a sperate patch that introduces WARN_ONCE definition in bug.h (https://patchwork.ozlabs.org/patch/921391/) Ramon Fried (6): cmd: iotrace: add set region command iotrace: add IO region limit common: iotrace: add timestamp to iotrace records iotrace: move record definitons to header file cmd: iotrace: add dump trace command iotrace: fix behaviour when buffer is full cmd/iotrace.c | 63 ++++++++++++++++++++++++++++++++++++++++++--- common/iotrace.c | 65 +++++++++++++++++++++++++++-------------------- include/iotrace.h | 57 +++++++++++++++++++++++++++++++++++++++-- 3 files changed, 153 insertions(+), 32 deletions(-) -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project