[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 2/2] intel-iommu: start to use error_report_o
Re: [Qemu-devel] [PATCH v4 2/2] intel-iommu: start to use error_report_once
Thu, 14 Jun 2018 14:51:30 +0200
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
Auger Eric <address@hidden> writes:
> On 06/13/2018 10:05 AM, Markus Armbruster wrote:
>> Peter Xu <address@hidden> writes:
>>> Replace existing trace_vtd_err() with error_report_once() then stderr
>>> will capture something if any of the error happens, meanwhile we don't
>>> suffer from any DDOS. Then remove the trace point. Since at it,
>>> provide more information where proper (now we can pass parameters into
>>> the report function).
>>> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
>>> Signed-off-by: Peter Xu <address@hidden>
>>> hw/i386/intel_iommu.c | 59 ++++++++++++++++++++++++-------------------
>>> hw/i386/trace-events | 1 -
>>> 2 files changed, 33 insertions(+), 27 deletions(-)
>> Michael, would you give your Reviewed-by or Acked-by? I'd take the
>> series through my tree then.
> Sorry to enter this thread at this late stage. Just one question: on the
> smmuv3 emulation code, Peter (Maydell) urged me to use
> qemu_log_mask(LOG_GUEST_ERROR, ...) whenever the error was triggered by
> a guest bad behavior. So what is the final guidance to avoid the DOS you
Does the user need to know about the error condition?
If yes, we should tell him. Logging is not telling.
Peter, what do you think?