Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-09-01 Thread Markus Armbruster
David Hildenbrand writes: > On 25.08.23 11:59, David Hildenbrand wrote: [...] >> ... hopefully I'm done with that error-handling pain in QEMU soon and >> can continue focusing on things that make me feel more productive :P I'm afraid you'll be done with error handling right when you're done wi

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-29 Thread Philippe Mathieu-Daudé
On 25/8/23 12:10, David Hildenbrand wrote: On 25.08.23 11:59, David Hildenbrand wrote: On 25.08.23 11:56, Markus Armbruster wrote: David Hildenbrand writes: On 25.08.23 11:10, Markus Armbruster wrote: David Hildenbrand writes: On 25.08.23 08:57, ThinerLogoer wrote: Hello, At 2023-08-23

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-29 Thread Mario Casquero
This series has been successfully tested by QE. Start a vm using pc.ram id but specifying a different memory-backend from the default one. Check the error message has been improved. Tested-by: Mario Casquero On Wed, Aug 23, 2023 at 5:38 PM David Hildenbrand wrote: > > For migration purposes, u

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-25 Thread David Hildenbrand
On 25.08.23 11:59, David Hildenbrand wrote: On 25.08.23 11:56, Markus Armbruster wrote: David Hildenbrand writes: On 25.08.23 11:10, Markus Armbruster wrote: David Hildenbrand writes: On 25.08.23 08:57, ThinerLogoer wrote: Hello, At 2023-08-23 23:34:11, "David Hildenbrand" wrote: For

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-25 Thread David Hildenbrand
On 25.08.23 11:56, Markus Armbruster wrote: David Hildenbrand writes: On 25.08.23 11:10, Markus Armbruster wrote: David Hildenbrand writes: On 25.08.23 08:57, ThinerLogoer wrote: Hello, At 2023-08-23 23:34:11, "David Hildenbrand" wrote: For migration purposes, users might want to reuse

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-25 Thread Markus Armbruster
David Hildenbrand writes: > On 25.08.23 11:10, Markus Armbruster wrote: >> David Hildenbrand writes: >> >>> On 25.08.23 08:57, ThinerLogoer wrote: Hello, At 2023-08-23 23:34:11, "David Hildenbrand" wrote: > For migration purposes, users might want to reuse the default RAM >>

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-25 Thread David Hildenbrand
On 25.08.23 11:10, Markus Armbruster wrote: David Hildenbrand writes: On 25.08.23 08:57, ThinerLogoer wrote: Hello, At 2023-08-23 23:34:11, "David Hildenbrand" wrote: For migration purposes, users might want to reuse the default RAM backend id, but specify a different memory backend. For

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-25 Thread Markus Armbruster
David Hildenbrand writes: > On 25.08.23 08:57, ThinerLogoer wrote: >> Hello, >> >> At 2023-08-23 23:34:11, "David Hildenbrand" wrote: >>> For migration purposes, users might want to reuse the default RAM >>> backend id, but specify a different memory backend. >>> >>> For example, to reuse "pc.r

Re: [PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-25 Thread David Hildenbrand
On 25.08.23 08:57, ThinerLogoer wrote: Hello, At 2023-08-23 23:34:11, "David Hildenbrand" wrote: For migration purposes, users might want to reuse the default RAM backend id, but specify a different memory backend. For example, to reuse "pc.ram" on q35, one has to set -machine q35,memory-

Re:[PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-24 Thread ThinerLogoer
Hello, At 2023-08-23 23:34:11, "David Hildenbrand" wrote: >For migration purposes, users might want to reuse the default RAM >backend id, but specify a different memory backend. > >For example, to reuse "pc.ram" on q35, one has to set >-machine q35,memory-backend=pc.ram >Only then, can a memo

[PATCH v3 11/11] machine: Improve error message when using default RAM backend id

2023-08-23 Thread David Hildenbrand
For migration purposes, users might want to reuse the default RAM backend id, but specify a different memory backend. For example, to reuse "pc.ram" on q35, one has to set -machine q35,memory-backend=pc.ram Only then, can a memory backend with the id "pc.ram" be created manually. Let's improv