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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5B6EBC4332F for ; Sat, 19 Nov 2022 01:58:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232958AbiKSB6Y (ORCPT ); Fri, 18 Nov 2022 20:58:24 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55848 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233511AbiKSB56 (ORCPT ); Fri, 18 Nov 2022 20:57:58 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C75609FF2 for ; Fri, 18 Nov 2022 17:49:59 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 525A462762 for ; Sat, 19 Nov 2022 01:49:59 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8980EC433D6; Sat, 19 Nov 2022 01:49:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1668822598; bh=dGHTvDB6EWcD//eljOzunEgw/G3LOla2CtV1gzVJYC8=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=EhF6TL/7EXiGuAdGPtQZCJalz95HpAHOle+gwL8OwfNeEds7zdOWYGizAnbTo/m24 jU5aR5bzLfCWcuMt15/UVcGZLBBJC1cWJeMQ/6NpP30AaaB7N4CLLtzHzDZKCoWzGA rzTKxSX7qNxc16Du+WQmnROrQA/PMC1egSElPvu6Zvf1nD6ayadUVxA52abKsYkKVq RjcHgo6FD3aGZWOdEzCjTjxo4M6KmIjZ4UinQBy8bVaQaUbR2kEkJ1DbIFb/Mn4B09 Umd6n9b20yHdrBQkZhEq4p39+NzsR1cRlghtUjR87cI17IgkCX9+LIUSa2kB+At2En 86AaLph+vdcQA== Date: Fri, 18 Nov 2022 17:49:57 -0800 From: Jakub Kicinski To: Jacob Keller Cc: netdev@vger.kernel.org, Jiri Pirko Subject: Re: [PATCH net-next 6/8] devlink: support directly reading from region memory Message-ID: <20221118174957.7c672c75@kernel.org> In-Reply-To: <20221117220803.2773887-7-jacob.e.keller@intel.com> References: <20221117220803.2773887-1-jacob.e.keller@intel.com> <20221117220803.2773887-7-jacob.e.keller@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Thu, 17 Nov 2022 14:08:01 -0800 Jacob Keller wrote: > +Regions may optionally allow directly reading from their contents without a > +snapshot. A driver wishing to enable this for a region should implement the > +``.read`` callback in the ``devlink_region_ops`` structure. Perhaps worth adding that direct read has weaker atomicity guarantees than snapshot? User at the CLI level may not expect the read request to be broken up into smaller chunks.