From: Greg Kurz <groug@kaod.org>
To: Markus Armbruster <armbru@redhat.com>
Cc: farosas@linux.ibm.com, richard.henderson@linaro.org,
qemu-devel@nongnu.org,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
lucas.araujo@eldorado.org.br, fernando.valle@eldorado.org.br,
qemu-ppc@nongnu.org, matheus.ferst@eldorado.org.br,
luis.pires@eldorado.org.br, david@gibson.dropbear.id.au
Subject: Re: [PATCH 4/5] monitor: removed cpustats command
Date: Tue, 8 Jun 2021 17:15:40 +0200 [thread overview]
Message-ID: <20210608171540.6c67bcc3@bahia.lan> (raw)
In-Reply-To: <8735tsxuhj.fsf@dusky.pond.sub.org>
On Tue, 08 Jun 2021 17:10:32 +0200
Markus Armbruster <armbru@redhat.com> wrote:
> Greg Kurz <groug@kaod.org> writes:
>
> > On Wed, 26 May 2021 17:21:03 -0300
> > "Bruno Larsen (billionai)" <bruno.larsen@eldorado.org.br> wrote:
> >
> >> Since ppc was the last architecture to collect these statistics and
> >> it is currently phasing this collection out, the command that would query
> >> this information is being removed.
> >>
> >
> > So this is removing an obviously user visible feature. This should be
> > mentioned in docs/system/removed-features.rst... but, wait, I don't
> > see anything for it in docs/system/deprecated.rst. This is dropping
> > a feature without following the usual deprecation policy, i.e.
> > marking the feature as deprecated and only remove it 2 QEMU versions
> > later. Any justification for that ?
>
> Our deprecation policy applies to stable interfaces, which HMP is not.
>
> We don't break things there just because. But dropping a command right
> away when it is no longer useful is just fine. No need to deprecate and
> wait for the grace period.
>
> [...]
>
Ah, good to know.
Thanks!
--
Greg
next prev parent reply other threads:[~2021-06-08 15:17 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-26 20:20 [PATCH 0/5] stop collection of instruction usage statistics Bruno Larsen (billionai)
2021-05-26 20:21 ` [PATCH 1/5] target/ppc: fixed GEN_OPCODE behavior when PPC_DUMP_CPU is set Bruno Larsen (billionai)
2021-05-26 21:13 ` Luis Fernando Fujita Pires
2021-05-26 21:24 ` Richard Henderson
2021-05-27 1:18 ` david
2021-05-26 20:21 ` [PATCH 2/5] target/ppc: remove ppc_cpu_dump_statistics Bruno Larsen (billionai)
2021-05-26 21:25 ` Richard Henderson
2021-05-26 21:27 ` Luis Fernando Fujita Pires
2021-05-27 1:20 ` David Gibson
2021-05-27 4:35 ` David Gibson
2021-05-27 13:22 ` Bruno Piazera Larsen
2021-05-27 14:01 ` Greg Kurz
2021-05-29 5:46 ` David Gibson
2021-05-27 6:01 ` Greg Kurz
2021-05-29 5:47 ` David Gibson
2021-05-31 14:26 ` Bruno Piazera Larsen
2021-05-26 20:21 ` [PATCH 3/5] target/ppc: removed mentions to DO_PPC_STATISTICS Bruno Larsen (billionai)
2021-05-26 21:26 ` Richard Henderson
2021-05-26 21:35 ` Luis Fernando Fujita Pires
2021-05-27 4:37 ` David Gibson
2021-05-26 20:21 ` [PATCH 4/5] monitor: removed cpustats command Bruno Larsen (billionai)
2021-05-26 21:28 ` Richard Henderson
2021-05-27 4:40 ` David Gibson
2021-05-26 21:35 ` Luis Fernando Fujita Pires
2021-05-27 6:40 ` Greg Kurz
2021-05-27 8:09 ` Dr. David Alan Gilbert
2021-05-27 8:30 ` Greg Kurz
2021-05-27 11:24 ` Bruno Piazera Larsen
2021-05-27 14:57 ` Greg Kurz
2021-06-08 15:10 ` Markus Armbruster
2021-06-08 15:15 ` Greg Kurz [this message]
2021-05-27 8:08 ` Dr. David Alan Gilbert
2021-05-26 20:21 ` [PATCH 5/5] hw/core/cpu: removed cpu_dump_statistics function Bruno Larsen (billionai)
2021-05-26 21:29 ` Richard Henderson
2021-05-26 21:35 ` Luis Fernando Fujita Pires
2021-05-27 1:21 ` David Gibson
2021-05-27 21:05 ` Eduardo Habkost
2021-05-27 13:57 ` [PATCH 0/5] stop collection of instruction usage statistics Alex Bennée
2021-05-27 14:23 ` Bruno Piazera Larsen
2021-05-27 14:25 ` Luis Fernando Fujita Pires
2021-05-27 14:56 ` Alex Bennée
2021-05-27 15:39 ` Luis Fernando Fujita Pires
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=20210608171540.6c67bcc3@bahia.lan \
--to=groug@kaod.org \
--cc=armbru@redhat.com \
--cc=david@gibson.dropbear.id.au \
--cc=dgilbert@redhat.com \
--cc=farosas@linux.ibm.com \
--cc=fernando.valle@eldorado.org.br \
--cc=lucas.araujo@eldorado.org.br \
--cc=luis.pires@eldorado.org.br \
--cc=matheus.ferst@eldorado.org.br \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=richard.henderson@linaro.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.