Re: [RFC PATCH] docs/system: add a gentle prompt for the complexity to come

[Thomas Huth]

On 01/03/2021 12.27, Alex Bennée wrote:
> We all know the QEMU command line can become a fiendishly complex
> beast. Lets gently prepare our user for the horrors to come by
> referencing where other example command lines can be found in the
> manual.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>   docs/system/quickstart.rst | 8 ++++++++
>   docs/system/targets.rst    | 2 ++
>   2 files changed, 10 insertions(+)
>
> diff --git a/docs/system/quickstart.rst b/docs/system/quickstart.rst
> index 3a3acab5e7..3498c5a29f 100644
> --- a/docs/system/quickstart.rst
> +++ b/docs/system/quickstart.rst
> @@ -11,3 +11,11 @@ Download and uncompress a PC hard disk image with Linux 
> installed (e.g.
>      |qemu_system| linux.img
>   
>   Linux should boot and give you a prompt.
> +
> +Users should be aware the above example elides a lot of the complexity
> +of setting up a VM with x86_64 specific defaults and an assumption the

s/an assumption/assumes/ ?
(I'm not a native speaker, but to me it sounds like a verb is missing here)

> +first non switch argument is a PC compatible disk image with a boot
> +sector. For non-x86 system where we emulate a broad range of machine
> +types the command lines are generally more explicit in defining the
> +machine and boot behaviour. You will find more example command lines
> +in the :ref:`system-targets-ref` section of the manual.
> diff --git a/docs/system/targets.rst b/docs/system/targets.rst
> index 560783644d..145cc64551 100644
> --- a/docs/system/targets.rst
> +++ b/docs/system/targets.rst
> @@ -1,3 +1,5 @@
> +.. _system-targets-ref:
> +
>   QEMU System Emulator Targets
>   ============================

With the first sentence fixed:
Reviewed-by: Thomas Huth <thuth@redhat.com>