From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59640) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z8ZHd-0007IQ-H3 for qemu-devel@nongnu.org; Fri, 26 Jun 2015 15:27:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z8ZHc-0005Ko-92 for qemu-devel@nongnu.org; Fri, 26 Jun 2015 15:27:29 -0400 Message-ID: <558DA79A.6060502@redhat.com> Date: Fri, 26 Jun 2015 15:27:22 -0400 From: John Snow MIME-Version: 1.0 References: <1435018875-22527-1-git-send-email-jsnow@redhat.com> <20150626161142.GH31186@stefanha-thinkpad.redhat.com> In-Reply-To: <20150626161142.GH31186@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 00/16] ahci: ncq cleanup, part 2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: kwolf@redhat.com, pbonzini@redhat.com, qemu-devel@nongnu.org, qemu-block@nongnu.org On 06/26/2015 12:11 PM, Stefan Hajnoczi wrote: > On Mon, Jun 22, 2015 at 08:20:59PM -0400, John Snow wrote: >> requires: 1434470575-21625-1-git-send-email-jsnow@redhat.com >> 1435016308-6150-1-git-send-email-jsnow@redhat.com [PATCH v2 0/4] >> ahci: misc fixes/tests for 2.4 [PATCH v2 00/16] ahci: ncq >> cleanup, part 1 >> >> This chunk gets NCQ migration and and resume support working. >> >> There's still some left to do, particularly around error >> handling and FIS semantics, but this should get us most of the >> way there. >> >> There is one RFC bit in this patch, inside of Patch #1, >> concerning how to handle truncating PRDTs that are too large -- >> it looks like we have attempted to address it in the past, and I >> accidentally bumped up against it now. By actually trying to >> consume the PRDs but ignoring any extra space they describe, I >> would break ide-test. >> >> I'll post logs later, but judging by the test text itself, we >> don't seem to know what the right behavior is. >> >> ________________________________________________________________________________ >> >> >> For convenience, this branch is available at: >> https://github.com/jnsnow/qemu.git branch ahci-ncq-s2 >> https://github.com/jnsnow/qemu/tree/ahci-ncq-s2 >> >> This version is tagged ahci-ncq-s2-v1: >> https://github.com/jnsnow/qemu/releases/tag/ahci-ncq-s2-v1 >> >> John Snow (16): ide: add limit to .prepare_buf() ahci: stash ncq >> command ahci: assert is_ncq for process_ncq ahci: refactor >> process_ncq_command ahci: factor ncq_finish out of ncq_cb ahci: >> record ncq failures ahci: kick NCQ queue ahci: correct types in >> NCQTransferState ahci: correct ncq sector count qtest/ahci: >> halted NCQ test ahci: add cmd header to ncq transfer state ahci: >> ncq migration ahci: add get_cmd_header helper ahci: Do not map >> cmd_fis to generate response qtest/ahci: halted ncq migration >> test ahci: fix sdb fis semantics >> >> hw/ide/ahci.c | 330 >> ++++++++++++++++++++++++++++++++---------------------- >> hw/ide/ahci.h | 9 +- hw/ide/core.c | 12 +- >> hw/ide/internal.h | 4 +- hw/ide/macio.c | 2 +- >> hw/ide/pci.c | 5 +- tests/ahci-test.c | 38 +++++-- 7 >> files changed, 252 insertions(+), 148 deletions(-) > > I posted comments on a few patches. The rest looks good. > OK, I'm smooshing 6/7 and 12/13, and correcting issues where appropriate. Will wait to feedback-feedback before I hit the send button. --js