[Qemu-devel] [PATCH 0/3] qemu-error: advanced report_once handling

Cornelia Huck posted 3 patches 7 years, 2 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180828123346.17548-1-cohuck@redhat.com
Test docker-clang@ubuntu failed
Test checkpatch passed
There is a newer version of this series
hw/vfio/ccw.c               | 18 +++---------------
include/qemu/error-report.h | 15 +++++++--------
util/qemu-error.c           | 44 ++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 54 insertions(+), 23 deletions(-)
[Qemu-devel] [PATCH 0/3] qemu-error: advanced report_once handling
Posted by Cornelia Huck 7 years, 2 months ago
Based on previous discussions, I wanted to enhance the recently
merged report_once infrastucture with a way to print a message
once based on a variable instead of globally-once, similar to
what vfio-ccw uses today.

Not really tested, mainly wanted to get this out before my vacation
to get the ball rolling.

Cornelia Huck (3):
  qemu-error: add {error,warn}_report_once_cond
  qemu-error: make use of {error,warn}_report_once_cond
  vfio-ccw: switch to warn_report_once_cond()

 hw/vfio/ccw.c               | 18 +++---------------
 include/qemu/error-report.h | 15 +++++++--------
 util/qemu-error.c           | 44 ++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 54 insertions(+), 23 deletions(-)

-- 
2.14.4


Re: [Qemu-devel] [PATCH 0/3] qemu-error: advanced report_once handling
Posted by Markus Armbruster 7 years, 2 months ago
Cornelia Huck <cohuck@redhat.com> writes:

> Based on previous discussions, I wanted to enhance the recently
> merged report_once infrastucture with a way to print a message
> once based on a variable instead of globally-once, similar to
> what vfio-ccw uses today.
>
> Not really tested, mainly wanted to get this out before my vacation
> to get the ball rolling.

I had a few comments, but found nothing serious enough to withhold my
Reviewed-by: Markus Armbruster <armbru@redhat.com>