Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-17 Thread Philippe Mathieu-Daudé
On 16/1/24 10:52, Thomas Huth wrote: On 16/01/2024 10.46, Philippe Mathieu-Daudé wrote: Thomas, are you OK to post a v2 with the changes I suggested or do you want me to do it? Since your changes were bigger than mine, I think it's just fair if you take credit for the patch. So yes, please

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-17 Thread Philippe Mathieu-Daudé
On 17/1/24 13:42, Markus Armbruster wrote: Thomas Huth writes: It's been marked as deprecated since QEMU 8.1, so it should be fine to remove this now. Signed-off-by: Thomas Huth Should we drop HMP command singlestep, too? Yes, patch on the way.

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-17 Thread Markus Armbruster
Thomas Huth writes: > It's been marked as deprecated since QEMU 8.1, so it should be fine > to remove this now. > > Signed-off-by: Thomas Huth Should we drop HMP command singlestep, too?

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-16 Thread Thomas Huth
On 16/01/2024 10.46, Philippe Mathieu-Daudé wrote: On 16/1/24 07:27, Markus Armbruster wrote: Daniel P. Berrangé writes: On Mon, Jan 15, 2024 at 05:39:19PM +, Peter Maydell wrote: On Mon, 15 Jan 2024 at 13:54, Thomas Huth wrote: On 12/01/2024 16.39, Philippe Mathieu-Daudé wrote: Hi T

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-16 Thread Philippe Mathieu-Daudé
On 16/1/24 07:27, Markus Armbruster wrote: Daniel P. Berrangé writes: On Mon, Jan 15, 2024 at 05:39:19PM +, Peter Maydell wrote: On Mon, 15 Jan 2024 at 13:54, Thomas Huth wrote: On 12/01/2024 16.39, Philippe Mathieu-Daudé wrote: Hi Thomas +Laurent & Peter On 12/1/24 11:00, Thomas Hu

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-15 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Mon, Jan 15, 2024 at 05:39:19PM +, Peter Maydell wrote: >> On Mon, 15 Jan 2024 at 13:54, Thomas Huth wrote: >> > >> > On 12/01/2024 16.39, Philippe Mathieu-Daudé wrote: >> > > Hi Thomas >> > > >> > > +Laurent & Peter >> > > >> > > On 12/1/24 11:00, Thomas Huth

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-15 Thread Daniel P . Berrangé
On Mon, Jan 15, 2024 at 05:39:19PM +, Peter Maydell wrote: > On Mon, 15 Jan 2024 at 13:54, Thomas Huth wrote: > > > > On 12/01/2024 16.39, Philippe Mathieu-Daudé wrote: > > > Hi Thomas > > > > > > +Laurent & Peter > > > > > > On 12/1/24 11:00, Thomas Huth wrote: > > >> It's been marked as depr

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-15 Thread Peter Maydell
On Mon, 15 Jan 2024 at 13:54, Thomas Huth wrote: > > On 12/01/2024 16.39, Philippe Mathieu-Daudé wrote: > > Hi Thomas > > > > +Laurent & Peter > > > > On 12/1/24 11:00, Thomas Huth wrote: > >> It's been marked as deprecated since QEMU 8.1, so it should be fine > >> to remove this now. > >> > >> Si

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-15 Thread Thomas Huth
On 12/01/2024 16.39, Philippe Mathieu-Daudé wrote: Hi Thomas +Laurent & Peter On 12/1/24 11:00, Thomas Huth wrote: It's been marked as deprecated since QEMU 8.1, so it should be fine to remove this now. Signed-off-by: Thomas Huth ---   docs/about/deprecated.rst   |  6 --   docs/about

Re: [PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-12 Thread Philippe Mathieu-Daudé
Hi Thomas +Laurent & Peter On 12/1/24 11:00, Thomas Huth wrote: It's been marked as deprecated since QEMU 8.1, so it should be fine to remove this now. Signed-off-by: Thomas Huth --- docs/about/deprecated.rst | 6 -- docs/about/removed-features.rst | 7 +++ system/vl.c

[PATCH 5/5] qemu-options: Remove the deprecated -singlestep option

2024-01-12 Thread Thomas Huth
It's been marked as deprecated since QEMU 8.1, so it should be fine to remove this now. Signed-off-by: Thomas Huth --- docs/about/deprecated.rst | 6 -- docs/about/removed-features.rst | 7 +++ system/vl.c | 18 +- qemu-options.hx