From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48483) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zhgth-000873-SP for qemu-devel@nongnu.org; Thu, 01 Oct 2015 12:40:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zhgte-0000Iw-K4 for qemu-devel@nongnu.org; Thu, 01 Oct 2015 12:39:57 -0400 References: <1442868929-17777-1-git-send-email-jsnow@redhat.com> From: John Snow Message-ID: <560D61D9.3030609@redhat.com> Date: Thu, 1 Oct 2015 12:39:53 -0400 MIME-Version: 1.0 In-Reply-To: <1442868929-17777-1-git-send-email-jsnow@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] qtest/ahci: fix redundant assertion List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: qemu-trivial@nongnu.org, mjt@tls.msk.ru On 09/21/2015 04:55 PM, John Snow wrote: > Fixes https://bugs.launchpad.net/qemu/+bug/1497711 > > (!ncq || (ncq && lba48)) is the same as > (!ncq || lba48). > > The intention is simply: "If a command is NCQ, > it must also be LBA48." > > Signed-off-by: John Snow > --- > tests/libqos/ahci.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/libqos/ahci.c b/tests/libqos/ahci.c > index cf66b3e..adb2665 100644 > --- a/tests/libqos/ahci.c > +++ b/tests/libqos/ahci.c > @@ -742,7 +742,7 @@ AHCICommand *ahci_command_create(uint8_t command_name) > g_assert(!(props->lba28 && props->lba48)); > g_assert(!(props->read && props->write)); > g_assert(!props->size || props->data); > - g_assert(!props->ncq || (props->ncq && props->lba48)); > + g_assert(!props->ncq || props->lba48); > > /* Defaults and book-keeping */ > cmd->props = props; > Tentatively staged. Barring objections, I will send a pull request for this tomorrow, thanks. https://github.com/jnsnow/qemu/commits/ide https://github.com/jnsnow/qemu.git --js