From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from ws5-mx01.kavi.com (ws5-mx01.kavi.com [34.193.7.191]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B80BAC77B75 for ; Tue, 16 May 2023 21:11:46 +0000 (UTC) Received: from lists.oasis-open.org (oasis.ws5.connectedcommunity.org [10.110.1.242]) by ws5-mx01.kavi.com (Postfix) with ESMTP id 11F466034A for ; Tue, 16 May 2023 21:11:46 +0000 (UTC) Received: from lists.oasis-open.org (oasis-open.org [10.110.1.242]) by lists.oasis-open.org (Postfix) with ESMTP id D5E7D9865B6 for ; Tue, 16 May 2023 21:11:45 +0000 (UTC) Received: from host09.ws5.connectedcommunity.org (host09.ws5.connectedcommunity.org [10.110.1.97]) by lists.oasis-open.org (Postfix) with QMQP id B57329862F6; Tue, 16 May 2023 21:11:45 +0000 (UTC) Mailing-List: contact virtio-dev-help@lists.oasis-open.org; run by ezmlm List-ID: Sender: Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Received: from lists.oasis-open.org (oasis-open.org [10.110.1.242]) by lists.oasis-open.org (Postfix) with ESMTP id A16309862F8 for ; Tue, 16 May 2023 21:11:45 +0000 (UTC) X-Virus-Scanned: amavisd-new at kavi.com X-MC-Unique: jn1myOO3NueF28YTl3bvdA-1 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684271502; x=1686863502; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=N/iiiaceKZsE+IZsDGvi7gmkMiFVQxlV6fBOXalkqU4=; b=F1PBUB3giOXP+eShEhgQAGjUwvhRu0/q4dtpmvG9bblw5+xK9TM4ibhXF0acVtzDRE lNC/QBLS/5N9IEU23hiS0v+fVVbw2gGEXta+Pzu4ZAELpo/XU7NhNxTy8FJ01uaIgedH rczQIfDM8FPsoRSxlEt9/uPspEcA39oPq+Kek/nFHiGTCtGio6X4HpwETxNNrQtYCBA9 SQswDkCcujvy0mDzRYlI60QF/RTs58o7NuGOdL+IXoJsZpInMJjXuWMc+tUYXi8jch4m LaoYS2nZ9wvYHmNrzDjWXPi4PRUaMOBjCiZpents9dF94jaPEj6Y0tZyhmzmQi7BGZos qffw== X-Gm-Message-State: AC+VfDx2IowKKGxqxl6tRt4j27m9nE21lhw0wpecds/au5NQW9TcgjjH IHah8Br54ZSxS4WyGs7ZgJUBpM+in558/+ZCjUpV4Uz4BZrC2bwPCQtOExlOtym0CHp27z7Kg80 cKDnUL6chDNGWRI6NtS3pE/2IyNLT X-Received: by 2002:a7b:ce95:0:b0:3f4:e853:6a1 with SMTP id q21-20020a7bce95000000b003f4e85306a1mr12998562wmj.38.1684271502706; Tue, 16 May 2023 14:11:42 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5+w25qRTjVzMEF3ykfThHxfG6ruu2YQQKYxTrFytJEieSaqmEfEH/peFvXO2Z7YjrwEUOJ0g== X-Received: by 2002:a7b:ce95:0:b0:3f4:e853:6a1 with SMTP id q21-20020a7bce95000000b003f4e85306a1mr12998550wmj.38.1684271502385; Tue, 16 May 2023 14:11:42 -0700 (PDT) Date: Tue, 16 May 2023 17:11:38 -0400 From: "Michael S. Tsirkin" To: Parav Pandit Cc: Jason Wang , "virtio-dev@lists.oasis-open.org" , "cohuck@redhat.com" , "david.edmondson@oracle.com" , "sburla@marvell.com" , Yishai Hadas , Maor Gottlieb , "virtio-comment@lists.oasis-open.org" , Shahaf Shuler Message-ID: <20230516170940-mutt-send-email-mst@kernel.org> References: <20230507093959-mutt-send-email-mst@kernel.org> <20230510014534-mutt-send-email-mst@kernel.org> <20230510033812-mutt-send-email-mst@kernel.org> <83603339-e3fa-fe56-d447-813e73eb1961@redhat.com> MIME-Version: 1.0 In-Reply-To: X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit Subject: [virtio-dev] Re: [virtio-comment] Re: [PATCH v2 0/2] transport-pci: Introduce legacy registers access using AQ On Tue, May 16, 2023 at 07:35:20PM +0000, Parav Pandit wrote: > > > From: virtio-comment@lists.oasis-open.org > open.org> On Behalf Of Jason Wang > > Sent: Monday, May 15, 2023 11:55 PM > > > I don’t see how this is being any different than register-offset interface. > > > > It bisects more things at hypervisor level that makes things hard to add #12th > > entry. > > > > > >> 1) device features > > >> 2) driver features > > >> 3) queue address > > >> 4) queue size > > >> 5) queue select > > >> 6) queue notify > > >> 7) device status > > >> 8) ISR status > > >> 9) config msix > > >> 10) queue msix > > >> 11) device configuration space > > >> > > >> It focuses on the facilities instead of transport specific details like registers > > (we > > >> don't even need legacy registers in this case), I gives more deterministic > > >> behavior so we don't need to care about the cross registers read/write. > > >> > > > 1.x has these registers at raw level and that seems fine. > > > > > > Note that 1.x has more, the above is dumped from the section of "Legacy > > Interfaces: A Note on PCI Device Layout". > > Yeah. > Current two commands proposal has one box to transport legacy registers that follows the legacy semantics. > Above commands in future with legacy commands of this work will be able to work together in future anyway. Nah, we don't need a "break randomly unless it's a full moon and you cross your heart three times" mode. If you are going to implement support for legacy emulation implement it in a way that either predictably works or predictably refuses to load. -- MST --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org