Re: [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING

qemu-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISAB

From:	Juan Quintela
Subject:	Re: [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISABLED
Date:	Tue, 07 Feb 2023 09:53:42 +0100
User-agent:	Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)

Markus Armbruster <armbru@redhat.com> wrote:
> slog()'s function comment advises to use QERR_QGA_LOGGING_DISABLED.
> This macro never existed.  The reference got added in commit
> e3d4d25206a "guest agent: add guest agent RPCs/commands" along with
> QERR_QGA_LOGGING_FAILED, so maybe that one was meant.  However,
> QERR_QGA_LOGGING_FAILED was never actually used, and was removed in
> commit d73f0beadb5 "qerror.h: Remove unused error classes".
>
> Drop the dangling reference.
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Reviewed-by: Juan Quintela <quintela@redhat.com>

[Prev in Thread]

Current Thread

[Next in Thread]

Re: [PATCH 11/12] migration/colo: Improve an x-colo-lost-heartbeat error message, (continued)
- [PATCH 09/12] replay: Simplify setting replay blockers, Markus Armbruster, 2023/02/07
  - Re: [PATCH 09/12] replay: Simplify setting replay blockers, Philippe Mathieu-Daudé, 2023/02/07
    - Re: [PATCH 09/12] replay: Simplify setting replay blockers, Markus Armbruster, 2023/02/07
- [PATCH 10/12] hw/core: Improve the query-hotpluggable-cpus error message, Markus Armbruster, 2023/02/07
  - Re: [PATCH 10/12] hw/core: Improve the query-hotpluggable-cpus error message, Philippe Mathieu-Daudé, 2023/02/07
    - Re: [PATCH 10/12] hw/core: Improve the query-hotpluggable-cpus error message, Markus Armbruster, 2023/02/07
- [PATCH 06/12] hw/acpi: Dumb down acpi_table_add() stub, Markus Armbruster, 2023/02/07
  - Re: [PATCH 06/12] hw/acpi: Dumb down acpi_table_add() stub, Juan Quintela, 2023/02/07
- [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISABLED, Markus Armbruster, 2023/02/07
  - Re: [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISABLED, Juan Quintela <=
    - Re: [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISABLED, Konstantin Kostiuk, 2023/02/07
- [PATCH 01/12] error: Drop superfluous #include "qapi/qmp/qerror.h", Markus Armbruster, 2023/02/07
  - Re: [PATCH 01/12] error: Drop superfluous #include "qapi/qmp/qerror.h", Juan Quintela, 2023/02/07
    - Re: [PATCH 01/12] error: Drop superfluous #include "qapi/qmp/qerror.h", Konstantin Kostiuk, 2023/02/07
- [PATCH 03/12] dump: Assert cpu_get_note_size() can't fail, Markus Armbruster, 2023/02/07
  - Re: [PATCH 03/12] dump: Assert cpu_get_note_size() can't fail, Juan Quintela, 2023/02/07
- [PATCH 07/12] hw/acpi: Move QMP command to hw/core/, Markus Armbruster, 2023/02/07
  - Re: [PATCH 07/12] hw/acpi: Move QMP command to hw/core/, Juan Quintela, 2023/02/07
- Re: [PATCH 00/12] error: Reduce qerror.h usage a bit more, Philippe Mathieu-Daudé, 2023/02/07

Prev by Date: Re: [PATCH 07/12] hw/acpi: Move QMP command to hw/core/
Next by Date: [PATCH] MAINTAINERS: add myself to ui/ and audio/
Previous by thread: [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISABLED
Next by thread: Re: [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISABLED
Index(es):
- Date
- Thread