From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50785) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZrUOy-0001sv-BR for qemu-devel@nongnu.org; Wed, 28 Oct 2015 13:20:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZrUOu-0000zO-5R for qemu-devel@nongnu.org; Wed, 28 Oct 2015 13:20:44 -0400 Received: from relay-05.andrew.cmu.edu ([128.2.157.12]:44024 helo=relay.andrew.cmu.edu) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZrUOt-0000yv-MN for qemu-devel@nongnu.org; Wed, 28 Oct 2015 13:20:39 -0400 From: "Gabriel L. Somlo" Date: Wed, 28 Oct 2015 13:20:32 -0400 Message-Id: <1446052836-31737-1-git-send-email-somlo@cmu.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PATCH v2 0/4] fw_cfg: spec update, read optimization, misc. cleanup List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, jordan.l.justen@intel.com, kraxel@redhat.com, pbonzini@redhat.com, markmb@redhat.com, lersek@redhat.com This series' main purpose is to update (and simplify) the specified read callback behavior. An earlier standalone patch to move qemu function call API documentation into fw_cfg.h should logically be part of the series. Here's the summary of what each patch does: - Patch 1/4 is an updated version of the standalone v1 patch I sent out earlier; it moves all the qemu-internal host-side function call api documentation out of docs/specs/fw_cfg.txt, and into the fw_cfg.h header file, next to the prototype of each documented api function. - Patch 2/4 modifies the specified behavior of read callbacks (from being invoked once per byte read, to being invoked once, before ANY data is read, specifically once each time an item is selected). - Patch 3/4 additionally removes the now-redundant offset argument from the read callback prototype. - Finally, 4/4 consolidates (non-DMA) reads, minimizing the number of times redundant sanity checks are performed, particularly for wide (> byte) sized reads. Comments, reviews, etc. much appreciated! Thanks, Gabriel Gabriel L. Somlo (4): fw_cfg: move internal function call docs to header file fw_cfg: amend callback behavior spec to once per select fw_cfg: remove offset argument from callback prototype fw_cfg: streamline (non-DMA) read operations docs/specs/fw_cfg.txt | 85 +------------------------------ hw/arm/virt-acpi-build.c | 2 +- hw/i386/acpi-build.c | 2 +- hw/nvram/fw_cfg.c | 53 +++++++------------ include/hw/nvram/fw_cfg.h | 127 +++++++++++++++++++++++++++++++++++++++++++++- trace-events | 2 +- 6 files changed, 147 insertions(+), 124 deletions(-) -- 2.4.3