lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20190123114452.GE2191@nanopsycho> Date: Wed, 23 Jan 2019 12:44:52 +0100 From: Jiri Pirko <jiri@...nulli.us> To: Eran Ben Elisha <eranbe@...lanox.com> Cc: netdev@...r.kernel.org, Jiri Pirko <jiri@...lanox.com>, "David S. Miller" <davem@...emloft.net>, Saeed Mahameed <saeedm@...lanox.com>, Moshe Shemesh <moshe@...lanox.com> Subject: Re: [PATCH net-next 0/7] Devlink health updates Tue, Jan 22, 2019 at 04:57:17PM CET, eranbe@...lanox.com wrote: >This patchset fixes some comments that were received for the devlink >health series, mostly around the devlink health buffers API. > >It offers a new devlink<->driver API for passing health dump and diagnose info. >As part of this patchset, the new API is developed and integrated into the >devlink health and mlx5e TX reporter. >Also, added some helpers together with the new API, which reduce the code >required by the driver to fill dump and diagnose significantly. > >Eventually, it also deletes the old API. > >In addition, it includes some small fixes in the devlink and mlx5e TX reporter. We are (re-)defining UAPI here. You need to present some examples of devlink tool output, both in json and stdout. Again, much more convenient to be done for the whole patchset, not this "fixup-one" :/ > > >Eran Ben Elisha (7): > devlink: Add devlink msg API > net/mlx5e: Move driver to use devlink msg API > devlink: move devlink health reporter to use devlink msg API > devlink: Delete depracated health buffers API > devlink: Remove spaces around "=" in the logger print > devlink: Fix use-after-free at reporter destroy > net/mlx5e: Add RTNL lock to TX recover flow > > .../mellanox/mlx5/core/en/reporter_tx.c | 124 +--- > include/net/devlink.h | 79 +-- > include/trace/events/devlink.h | 2 +- > include/uapi/linux/devlink.h | 14 +- > net/core/devlink.c | 633 ++++++++---------- > 5 files changed, 342 insertions(+), 510 deletions(-) > >-- >2.17.1 >
Powered by blists - more mailing lists