From nobody Sat May 18 11:46:47 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1594621687; cv=none; d=zohomail.com; s=zohoarc; b=SN5PcajIYAhuWdRc6DnG+DpXxFJdbkjRkyvFBVWGRuwOuJoRoCjgo2c+XNpLT8jjUbPjJeHoap9LgYvWPudbOolm4Mi8AOQjCKz23HGN5NreYUZ/Ed1aLX9UuvrSDrYr9PJgbJYbj7ADv3O2M279vVcWF49utBbekKlWnk8hKPA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1594621687; h=Content-Type:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:Message-ID:Sender:Subject:To; bh=+Upczu3Rs9WLUwjB58ywX0Dq7e9Io2yZlsRFZtjOSs8=; b=ARUcXcWl5H0h8Vy8vjr8ME31FESeRQ9YcZl6vkGib9S9EqZpeY3Ie3+pRbVqKUYx2hdif2O/iHYQ6ycX/4a5EXTiDsxFOzu71c9HD40ecjk+UtAfBimNZJLN+eqQKqDx64oGDZLMz9y0Wmj2V5oNuWw0KGw+Yvx66d8xaerkbtI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1594621687182587.2817345210653; Sun, 12 Jul 2020 23:28:07 -0700 (PDT) Received: from localhost ([::1]:44010 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jurwb-0007tn-5W for importer@patchew.org; Mon, 13 Jul 2020 02:28:05 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:46030) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jurvv-0007UZ-BK for qemu-devel@nongnu.org; Mon, 13 Jul 2020 02:27:23 -0400 Received: from us-smtp-1.mimecast.com ([205.139.110.61]:34664 helo=us-smtp-delivery-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1jurvt-00089w-3H for qemu-devel@nongnu.org; Mon, 13 Jul 2020 02:27:23 -0400 Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-200-iIrb5cf0ODKedR7qA7Mebw-1; Mon, 13 Jul 2020 02:27:17 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id C586B107ACCA for ; Mon, 13 Jul 2020 06:27:16 +0000 (UTC) Received: from sirius.home.kraxel.org (ovpn-115-89.ams2.redhat.com [10.36.115.89]) by smtp.corp.redhat.com (Postfix) with ESMTP id BA63C2B6F0; Mon, 13 Jul 2020 06:27:13 +0000 (UTC) Received: by sirius.home.kraxel.org (Postfix, from userid 1000) id 84E649AB2; Mon, 13 Jul 2020 08:27:12 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1594621639; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:content-type:content-type; bh=+Upczu3Rs9WLUwjB58ywX0Dq7e9Io2yZlsRFZtjOSs8=; b=Lzvjb/zcxgQM77OfOI7mfHbeDExnwgslLf+Z0iHLACtvcdyeX35e4EDadnLh0NWe4QJXhd TNamd+FPL7Yte665lt3q4k1/1QvK2xRAOWkbkZEUavdxZpDrnXEzcyXfB9Fmf7mdjDhrEX UmklR3oECsemmsss7ITXrUeaWgPCjno= X-MC-Unique: iIrb5cf0ODKedR7qA7Mebw-1 From: Gerd Hoffmann To: qemu-devel@nongnu.org Subject: [PATCH] usb: fix storage regression Date: Mon, 13 Jul 2020 08:27:12 +0200 Message-Id: <20200713062712.1476-1-kraxel@redhat.com> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=205.139.110.61; envelope-from=kraxel@redhat.com; helo=us-smtp-delivery-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/07/13 01:36:29 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -40 X-Spam_score: -4.1 X-Spam_bar: ---- X-Spam_report: (-4.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-1, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Gerd Hoffmann Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @redhat.com) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Fix the contition to figure whenever we need to wait for more data or not. Simply check the mode, if we are not in DATAIN state any more we are done already and don't need to go ASYNC. Fixes: 7ad3d51ebb8a ("usb: add short-packet handling to usb-storage driver") Reported-by: Sai Pavan Boddu Tested-by: Paul Zimmerman Signed-off-by: Gerd Hoffmann --- hw/usb/dev-storage.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/hw/usb/dev-storage.c b/hw/usb/dev-storage.c index 2ed6a8df2413..405a4ccfe700 100644 --- a/hw/usb/dev-storage.c +++ b/hw/usb/dev-storage.c @@ -546,8 +546,7 @@ static void usb_msd_handle_data(USBDevice *dev, USBPack= et *p) } } } - if (p->actual_length < p->iov.size && (p->short_not_ok || - s->scsi_len >=3D p->ep->max_packet_size)) { + if (p->actual_length < p->iov.size && s->mode =3D=3D USB_MSDM_= DATAIN) { DPRINTF("Deferring packet %p [wait data-in]\n", p); s->packet =3D p; p->status =3D USB_RET_ASYNC; --=20 2.18.4