From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58318) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XjP5x-0003cq-4m for qemu-devel@nongnu.org; Wed, 29 Oct 2014 04:59:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XjP5o-0002b4-48 for qemu-devel@nongnu.org; Wed, 29 Oct 2014 04:59:09 -0400 Received: from mail-wi0-x230.google.com ([2a00:1450:400c:c05::230]:58981) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XjP5n-0002au-TI for qemu-devel@nongnu.org; Wed, 29 Oct 2014 04:59:00 -0400 Received: by mail-wi0-f176.google.com with SMTP id h11so1029549wiw.9 for ; Wed, 29 Oct 2014 01:58:59 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <5450AC4E.8050700@redhat.com> Date: Wed, 29 Oct 2014 09:58:54 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1412204151-18117-1-git-send-email-jsnow@redhat.com> <20141028135151.GC22805@stefanha-thinkpad.redhat.com> <54502CA1.1040802@redhat.com> <54502EE4.3090704@redhat.com> <54502F9B.2070901@redhat.com> <54503475.6050500@redhat.com> <545042BA.4040800@redhat.com> In-Reply-To: <545042BA.4040800@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/6] AHCI Device Fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: John Snow , Stefan Hajnoczi Cc: kwolf@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, stefanha@redhat.com, mst@redhat.com On 10/29/2014 02:28 AM, John Snow wrote: > > (1) Update the prepare_buf callback (including the AHCI and BMDMA > implementations) to return, simply, the number of bytes prepared. For > AHCI, the largest this can ever be is something like > > (2) Update uses of the callback or implementations to use this number > directly to determine if the call succeeded, failed, or "succeeded > enough" for our purposes. > > (3) We can reserve the return code of -1 to imply catastrophic failure. Sounds like a plan. > In the meantime: > Patches 1, 2, and 6 are fine and should be merged. Great! Paolo