All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ján Tomko" <jtomko@redhat.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: qemu-devel@nongnu.org, marcandre.lureau@redhat.com,
	pbonzini@redhat.com, devel@lists.libvirt.org, eblake@redhat.com
Subject: Re: [PATCH 4/4] qapi/char: Deprecate backend type "memory"
Date: Mon, 5 Feb 2024 10:37:59 +0100	[thread overview]
Message-ID: <ZcCsdw6entrmSDr4@fedora> (raw)
In-Reply-To: <20240203080228.2766159-5-armbru@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 1125 bytes --]

On a Saturday in 2024, Markus Armbruster wrote:
>It's an alias for "ringbuf" we kept for backward compatibility; see
>commit 3a1da42eb35 (qapi: Rename ChardevBackend member "memory" to
>"ringbuf").  Deprecation is long overdue.
>
>Signed-off-by: Markus Armbruster <armbru@redhat.com>
>---
> docs/about/deprecated.rst | 8 ++++++++
> qapi/char.json            | 8 +++++---
> 2 files changed, 13 insertions(+), 3 deletions(-)
>
>diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
>index d4492b9460..ae1a520c26 100644
>--- a/docs/about/deprecated.rst
>+++ b/docs/about/deprecated.rst
>@@ -371,6 +371,14 @@ Specifying the iSCSI password in plain text on the command line using the
> used instead, to refer to a ``--object secret...`` instance that provides
> a password via a file, or encrypted.
>
>+Character device options
>+''''''''''''''''''''''''
>+
>+Backend ``memory`` (since 9.0)
>+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>+
>+``memory`` is a deprecated synonym for ``ringbuf``.
>+

For libvirt:
Reviewed-by: Ján Tomko <jtomko@redhat.com>

(We don't support either of those)

Jano

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

  reply	other threads:[~2024-02-05  9:40 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-03  8:02 [PATCH 0/4] char: Minor fixes, and a tighter QAPI schema Markus Armbruster
2024-02-03  8:02 ` [PATCH 1/4] chardev/parallel: Don't close stdin on inappropriate device Markus Armbruster
2024-02-07 19:15   ` Eric Blake
2024-02-08  6:52     ` Markus Armbruster
2024-02-13 13:58   ` Markus Armbruster
2024-02-13 18:25     ` Marc-André Lureau
2024-02-03  8:02 ` [PATCH 2/4] tests/unit/test-char: Fix qemu_socket(), make_udp_socket() check Markus Armbruster
2024-02-07 19:45   ` Eric Blake
2024-02-08  6:52     ` Markus Armbruster
2024-02-03  8:02 ` [PATCH 3/4] qapi/char: Make backend types properly conditional Markus Armbruster
2024-02-07 19:47   ` Eric Blake
2024-02-03  8:02 ` [PATCH 4/4] qapi/char: Deprecate backend type "memory" Markus Armbruster
2024-02-05  9:37   ` Ján Tomko [this message]
2024-02-07 19:48   ` Eric Blake
2024-02-03 11:07 ` [PATCH 0/4] char: Minor fixes, and a tighter QAPI schema Marc-André Lureau

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZcCsdw6entrmSDr4@fedora \
    --to=jtomko@redhat.com \
    --cc=armbru@redhat.com \
    --cc=devel@lists.libvirt.org \
    --cc=eblake@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.