From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Sender: 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 457C898646D for ; Tue, 1 Mar 2022 12:25:23 +0000 (UTC) Date: Tue, 1 Mar 2022 07:25:16 -0500 From: "Michael S. Tsirkin" Message-ID: <20220301071913-mutt-send-email-mst@kernel.org> References: <20220301083446.55424-1-xuanzhuo@linux.alibaba.com> <20220301054557-mutt-send-email-mst@kernel.org> <1646134343.8491874-1-xuanzhuo@linux.alibaba.com> MIME-Version: 1.0 In-Reply-To: <1646134343.8491874-1-xuanzhuo@linux.alibaba.com> Subject: [virtio-dev] Re: [PATCH v10] virtio-net: support device stats Content-Type: text/plain; charset=us-ascii Content-Disposition: inline To: Xuan Zhuo Cc: virtio-dev@lists.oasis-open.org, jasowang@redhat.com List-ID: On Tue, Mar 01, 2022 at 07:32:23PM +0800, Xuan Zhuo wrote: > > I have to say, this design is really good. > > I think one of the core points of your design is to separate drop stats, gso > stats, etc. Get the stats you want based on negotiation and needs. And covering > the control queue, I am very surprised. > > But does this design solve the problem of expansion? Various stats independent > structures seem to be convenient for expansion. But is this expansion ready for > dynamic negotiation? For negotiation, we can either separate feature flags for each supported type, or do like RSS and add commands for querying supported types and enabling stats collection. Features is probably easier for now ... Also, do we need to enable/disable stat collection? > My earliest solution is dynamic negotiation, that is, the driver tells the > device how many stats it wants to get. And the device negotiates how many stats > it should give to the driver based on this information of the driver. > > But that's not great for migration. So gave up. @Jason Any optional feature is problematic for migration, at least with features device knows ahead of the time what does driver want to use. -- MST --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org