On 10/28/20 7:10 PM, Auger Eric wrote: > Hi Philippe, > > On 10/27/20 2:55 PM, Philippe Mathieu-Daudé wrote: >> Add a bit of tracing, clean around to finally fix few bugs. >> In particular, restore NVMe on Aarch64 host. >> >> Eric Auger (4): >> block/nvme: Change size and alignment of IDENTIFY response buffer >> block/nvme: Change size and alignment of queue >> block/nvme: Change size and alignment of prp_list_pages >> block/nvme: Align iov's va and size on host page size> >> Philippe Mathieu-Daudé (21): >> MAINTAINERS: Cover 'block/nvme.h' file >> block/nvme: Use hex format to display offset in trace events >> block/nvme: Report warning with warn_report() >> block/nvme: Trace controller capabilities >> block/nvme: Trace nvme_poll_queue() per queue >> block/nvme: Improve nvme_free_req_queue_wait() trace information >> block/nvme: Trace queue pair creation/deletion >> block/nvme: Simplify device reset >> block/nvme: Move definitions before structure declarations >> block/nvme: Use unsigned integer for queue counter/size >> block/nvme: Make nvme_identify() return boolean indicating error >> block/nvme: Make nvme_init_queue() return boolean indicating error >> block/nvme: Introduce Completion Queue definitions >> block/nvme: Use definitions instead of magic values in add_io_queue() >> block/nvme: Correctly initialize Admin Queue Attributes >> block/nvme: Simplify ADMIN queue access >> block/nvme: Simplify nvme_cmd_sync() >> block/nvme: Pass AioContext argument to nvme_add_io_queue() >> block/nvme: Set request_alignment at initialization >> block/nvme: Correct minimum device page size >> block/nvme: Fix use of write-only doorbells page on Aarch64 arch >> >> include/block/nvme.h | 17 ++-- >> block/nvme.c | 208 ++++++++++++++++++++++++------------------- >> MAINTAINERS | 2 + >> block/trace-events | 30 ++++--- >> 4 files changed, 148 insertions(+), 109 deletions(-) >> > > I have tested the series on ARM with both 4kB and 64kB pages and it > works for me. > > Feel free to add: > Tested-by: Eric Auger <eric.au...@redhat.com>
Thanks! Phil.