qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Christian Schoenebeck <qemu_oss@crudebyte.com>
To: qemu-devel@nongnu.org
Cc: Greg Kurz <groug@kaod.org>, Paolo Bonzini <pbonzini@redhat.com>,
	Peter Maydell <peter.maydell@linaro.org>
Subject: [PATCH 3/4] docs/devel/9p: fix references to pictures
Date: Tue, 23 Mar 2021 19:55:46 +0100	[thread overview]
Message-ID: <a1ef93da295669d939df344e9169e1e8db57d9fd.1616528420.git.qemu_oss@crudebyte.com> (raw)
In-Reply-To: <cover.1616528420.git.qemu_oss@crudebyte.com>

The original source https://wiki.qemu.org/Documentation/9p does not use
a prefixed path for pictures. The (auto converted) in-tree .rst version
of the 9p documentation though uses pictures that are placed into the
subdirectory 'img' for not mixing them with .rst files within the same
directory, so prefix references to those pictures with 'img/'.

Also the auto conversion tool 'pandoc' did not replace the image
references by a correct 'image' rst keyword, so correct that as well.

Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
 docs/devel/9p.rst | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/docs/devel/9p.rst b/docs/devel/9p.rst
index e2a57b77b2..f326edc8d7 100644
--- a/docs/devel/9p.rst
+++ b/docs/devel/9p.rst
@@ -64,7 +64,7 @@ Topology
 The following figure shows the basic structure of the 9pfs
 implementation in QEMU.
-`frameless|upright=3.0 <File:9pfs_topology.png>`__
+.. image:: img/9pfs_topology.png
 The implementation consists of 3 modular components: 9p server, 9p
 filesystem drivers and 9p transport drivers. The 9p client on guest OS
@@ -219,7 +219,7 @@ Another important aspect to know is that once a thread leaves the scope
 of a Coroutine, then that thread is back at using its own thread-owned
 stack again.
-`frameless|upright=2.4 <File:Coroutines_stacks.png>`__
+.. image:: img/Coroutines_stacks.png
 Each coroutine instance usually handles a certain "collaborative" task,
 where "collaborative" means that individual parts of the task usually
@@ -253,7 +253,7 @@ Control Flow
 The following figure shows the control flow and relationship of Threads
 and Coroutines of the 9pfs implementation.
-`frameless|upright=3.5 <File:9pfs_control_flow.png>`__
+.. image::  img/9pfs_control_flow.png
 Getting back to 9pfs as concrete user of Coroutines, every 9P client
 request that comes in on 9P server side is a task the 9P server needs to

  parent reply	other threads:[~2021-03-23 20:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-23 19:40 [PATCH 0/4] add in-tree 9pfs developers documentation Christian Schoenebeck
2021-03-23 18:36 ` [PATCH 1/4] docs/devel: add 9p.rst Christian Schoenebeck
2021-03-23 18:52 ` [PATCH 2/4] docs/devel: add directory for pictures Christian Schoenebeck
2021-03-23 18:55 ` Christian Schoenebeck [this message]
2021-03-23 19:31 ` [PATCH 4/4] MAINTAINERS: add responsibility for docs/devel/9p.rst Christian Schoenebeck
2021-04-06 12:27 ` [PATCH 0/4] add in-tree 9pfs developers documentation Christian Schoenebeck
2021-04-07  7:32   ` Greg Kurz
2021-04-07  8:52     ` Christian Schoenebeck

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:

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

  git send-email \
    --in-reply-to=a1ef93da295669d939df344e9169e1e8db57d9fd.1616528420.git.qemu_oss@crudebyte.com \
    --to=qemu_oss@crudebyte.com \
    --cc=groug@kaod.org \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --subject='Re: [PATCH 3/4] docs/devel/9p: fix references to pictures' \


* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).