diff options
| author | Marc-André Lureau | 2021-07-28 13:34:43 +0200 |
|---|---|---|
| committer | Marc-André Lureau | 2021-08-05 14:15:33 +0200 |
| commit | 64195b0d365712c05ae922b7ff22cf136aac1400 (patch) | |
| tree | 10be11affd1eecbd41fb3f9e0104a39a39e8aa8c /hw/core/machine-hmp-cmds.c | |
| parent | chardev: fix qemu_chr_open_fd() with fd_in==fd_out (diff) | |
| download | qemu-64195b0d365712c05ae922b7ff22cf136aac1400.tar.gz qemu-64195b0d365712c05ae922b7ff22cf136aac1400.tar.xz qemu-64195b0d365712c05ae922b7ff22cf136aac1400.zip | |
chardev: give some context on chardev-add error
Description from Daniel P. Berrangé:
> The original code reported:
>
> "attempt to add duplicate property 'char2' to object (type 'container')"
>
> Since adding yank support, the current code reports
>
> "duplicate yank instance"
>
> With this patch applied it now reports:
>
> "Failed to add chardev 'char2': duplicate yank instance"
>
> This is marginally better, but still not great, not that the original
> error was great either.
>
> It would be nice if we could report
>
> "chardev with id 'char2' already exists"
Related to:
https://bugzilla.redhat.com/show_bug.cgi?id=1984721
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Diffstat (limited to 'hw/core/machine-hmp-cmds.c')
0 files changed, 0 insertions, 0 deletions