From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cong Wang Subject: Re: [Patch net-next v3 0/3] net_sched: reflect tx_queue_len change for pfifo_fast Date: Mon, 29 Jan 2018 16:12:55 -0800 Message-ID: References: <20180126022624.20442-1-xiyou.wangcong@gmail.com> <20180129.124324.1307163515349322323.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: Linux Kernel Network Developers , John Fastabend To: David Miller Return-path: Received: from mail-pg0-f48.google.com ([74.125.83.48]:38117 "EHLO mail-pg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752302AbeA3ANQ (ORCPT ); Mon, 29 Jan 2018 19:13:16 -0500 Received: by mail-pg0-f48.google.com with SMTP id y27so5774369pgc.5 for ; Mon, 29 Jan 2018 16:13:16 -0800 (PST) In-Reply-To: <20180129.124324.1307163515349322323.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, Jan 29, 2018 at 9:43 AM, David Miller wrote: > > Please follow up with John about making the queue allocation use > a prepare + commit phase. Will do it once net-next is re-open. > > And as for the TX queue state handling on change, I think it's > fine to purge the whole queue. That is definitely better than > allowing the queue length to be visibly over the tx_queue_len > setting. > OK. Thanks.