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 77E6FC6FA8E for ; Thu, 22 Sep 2022 01:02:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229785AbiIVBB7 (ORCPT ); Wed, 21 Sep 2022 21:01:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45620 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229471AbiIVBB4 (ORCPT ); Wed, 21 Sep 2022 21:01:56 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3417686710; Wed, 21 Sep 2022 18:01:56 -0700 (PDT) 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 ams.source.kernel.org (Postfix) with ESMTPS id E1008B8337F; Thu, 22 Sep 2022 01:01:54 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5D280C433D6; Thu, 22 Sep 2022 01:01:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1663808513; bh=mt6Efzdk+la/shXr+HkzyhobECVNYxpLZbiK6qvyBDE=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Z1lnH4+zrROmy34woIUSIwutdzO5u3p06eqOflbF0mlCH3DpRXjNIRNK49zV4yMKh MIsSsK82iSCmCjMuJ9lY7pdmHF4F8XAc+3fd39aj2Z1XGizr0Ly7s/rXkjeFsVunkk uiVdeIdd+kUyrWbEXo38qw1CmdGzJhql0f1+jfeQjxDsBLQ/V3Y5wGCmHVGi2CMYSY XR73fLqmQeP5gg1YFbH6WtxQpx11Ce5QZ18ijstMParmdlwmkqoRjygQjXfP+uSGnA BI0r6XmtSOn6F7kEYDqNNCx290YZ4o0x0JcojfpyVk29tn4GUVkxd8HPTSFWqtApnT i72PCTbOmSTJA== Date: Wed, 21 Sep 2022 18:01:52 -0700 From: Jakub Kicinski To: Gerhard Engleder Cc: davem@davemloft.net, robh+dt@kernel.org, netdev@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH net-next 7/7] tsnep: Rework RX buffer allocation Message-ID: <20220921180152.002dea73@kernel.org> In-Reply-To: <20220915203638.42917-8-gerhard@engleder-embedded.com> References: <20220915203638.42917-1-gerhard@engleder-embedded.com> <20220915203638.42917-8-gerhard@engleder-embedded.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, 15 Sep 2022 22:36:37 +0200 Gerhard Engleder wrote: > Try to refill RX queue continously instead of dropping frames if > allocation fails. This seems to be the more common pattern for network > drivers and makes future XDP support simpler. Is there anything preventing the queue from becoming completely empty and potentially never getting refilled? The lazy refill usually comes with some form of a watchdog / periodic depletion check.