From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56163) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1e2f7v-00066p-AQ for qemu-devel@nongnu.org; Thu, 12 Oct 2017 11:10:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1e2f7q-0003sX-U2 for qemu-devel@nongnu.org; Thu, 12 Oct 2017 11:10:23 -0400 Received: from mail-pf0-x22e.google.com ([2607:f8b0:400e:c00::22e]:46247) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1e2f7q-0003s5-NT for qemu-devel@nongnu.org; Thu, 12 Oct 2017 11:10:18 -0400 Received: by mail-pf0-x22e.google.com with SMTP id p87so5235864pfj.3 for ; Thu, 12 Oct 2017 08:10:18 -0700 (PDT) References: <1507655662-28101-1-git-send-email-peter.maydell@linaro.org> From: Richard Henderson Message-ID: Date: Thu, 12 Oct 2017 08:10:15 -0700 MIME-Version: 1.0 In-Reply-To: <1507655662-28101-1-git-send-email-peter.maydell@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3] docs/devel/loads-stores.rst: Document our various load and store APIs List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , qemu-devel@nongnu.org Cc: Paolo Bonzini , Richard Henderson , patches@linaro.org On 10/10/2017 10:14 AM, Peter Maydell wrote: > QEMU has a wide selection of different functions for doing > loads and stores; provide some overview documentation of > what they do and how to pick which one to use. > > Signed-off-by: Peter Maydell > --- > Changes v2->v3: fixed the typos noticed by Eric; > dropped the note about license in favour of using the > default gpl2+. I think this is ready to commit now. > > Changes v1->v2: filled in most of the gaps thanks to the > comments on v1. > > I have left some TODO markers in where they represent > fixes we should ideally make to the API. Since this > is for-developers internals documentation I think it's > OK to leave those in. > > docs/devel/loads-stores.rst | 398 ++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 398 insertions(+) > create mode 100644 docs/devel/loads-stores.rst Modulo the 2 typos that Eric found, Reviewed-by: Richard Henderson What path should this patch take in? r~