Skip to content

nbd-client: drop extra error noise

Eric Blake qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v2 Latest3 patches 3 repliesIn review

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3nbd: publish _lookup functionsinclude/block/nbd.h nbd/nbd-internal.h+5 -50 replies
2/3nbd/client: Trace all server option error messagesnbd/client.c nbd/trace-events+3 -01 reply
3/3block/nbd-client: use traces instead of noisy error_report_errblock/nbd-client.c block/trace-events tests/qemu-iotests/083.out+23 -320 replies

Reviews trailers per patch

2 of 3 patches have a Reviewed-by or Acked-by.

PatchEric BlakeVladimir Sementsov-Ogievskiy
1/3R-b
2/3R-b
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [Qemu-devel] [PATCH v2 0/3] nbd-client: drop extra error noise

This is a counter-proposal to Vladimir's initial post for reducing the noise in iotest 083. The difference is that I dropped error_get_hint, since I couldn't find any useful hint to be traced during transmission phase in the first place, and instead added a trace message for the one place where we do set hints during handshaking phase.

Eric Blake (1):
nbd/client: Trace all server option error messages

Vladimir Sementsov-Ogievskiy (2):
nbd: publish _lookup functions
block/nbd-client: use traces instead of noisy error_report_err

 include/block/nbd.h        |  5 +++++
 nbd/nbd-internal.h         |  5 -----
 block/nbd-client.c         | 23 +++++++++++++++++++----
 nbd/client.c               |  2 ++
 block/trace-events         |  4 ++++
 nbd/trace-events           |  1 +
 tests/qemu-iotests/083.out | 28 ----------------------------
 7 files changed, 31 insertions(+), 37 deletions(-)
Signature · 1 line
2.17.2