From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Ahern Subject: Re: [PATCH iproute2 net-next] devlink: Add support for devlink-region access Date: Thu, 19 Jul 2018 10:21:20 -0600 Message-ID: References: <1531816467-15860-1-git-send-email-valex@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit To: Alex Vesker , netdev@vger.kernel.org, jiri@mellanox.com Return-path: Received: from mail-pg1-f196.google.com ([209.85.215.196]:46861 "EHLO mail-pg1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731663AbeGSRFR (ORCPT ); Thu, 19 Jul 2018 13:05:17 -0400 Received: by mail-pg1-f196.google.com with SMTP id p23-v6so4198912pgv.13 for ; Thu, 19 Jul 2018 09:21:23 -0700 (PDT) In-Reply-To: <1531816467-15860-1-git-send-email-valex@mellanox.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 7/17/18 2:34 AM, Alex Vesker wrote: > Devlink region allows access to driver defined address regions. > Each device can create its supported address regions and register > them. A device which exposes a region will allow access to it > using devlink. > > This support allows reading and dumping regions snapshots as well > as presenting information such as region size and current available > snapshots. > > A snapshot represents a memory image of a region taken by the driver. > If a device collects a snapshot of an address region it can be later > exposed using devlink region read or dump commands. > This functionality allows for future analyses on the snapshots. > > The dump command is designed to read the full address space of a > region or of a snapshot unlike the read command which allows > reading only a specific section in a region/snapshot indicated by > an address and a length, current support is for reading and dumping > for a previously taken snapshot ID. > > New commands added: > devlink region show [ DEV/REGION ] > devlink region delete DEV/REGION snapshot SNAPSHOT_ID > devlink region dump DEV/REGION [ snapshot SNAPSHOT_ID ] > devlink region read DEV/REGION [ snapshot SNAPSHOT_ID ] > address ADDRESS length length > > Signed-off-by: Alex Vesker > Signed-off-by: Jiri Pirko > --- > devlink/devlink.c | 485 +++++++++++++++++++++++++++++++++++++++++++++- > man/man8/devlink-region.8 | 131 +++++++++++++ > man/man8/devlink.8 | 1 + > 3 files changed, 616 insertions(+), 1 deletion(-) > create mode 100644 man/man8/devlink-region.8 > applied to iproute2-next. Thanks