Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753554AbcLBRXf (ORCPT ); Fri, 2 Dec 2016 12:23:35 -0500 Received: from lelnx193.ext.ti.com ([198.47.27.77]:20405 "EHLO lelnx193.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751554AbcLBRXK (ORCPT ); Fri, 2 Dec 2016 12:23:10 -0500 Subject: Re: [PATCH 5/7] Documentation: DT: net: cpsw: allow to specify descriptors pool size To: Ivan Khoronzhuk References: <20161201233432.6182-1-grygorii.strashko@ti.com> <20161201233432.6182-6-grygorii.strashko@ti.com> <20161202112832.GB1213@khorivan> CC: "David S. Miller" , , Mugunthan V N , Sekhar Nori , , From: Grygorii Strashko Message-ID: <31a048c2-74e4-772e-ab63-9eacc2f49953@ti.com> Date: Fri, 2 Dec 2016 11:22:28 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <20161202112832.GB1213@khorivan> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [128.247.83.173] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2648 Lines: 65 On 12/02/2016 05:28 AM, Ivan Khoronzhuk wrote: > On Thu, Dec 01, 2016 at 05:34:30PM -0600, Grygorii Strashko wrote: >> Add optional property "descs_pool_size" to specify buffer descriptor's >> pool size. The "descs_pool_size" should define total number of CPDMA >> CPPI descriptors to be used for both ingress/egress packets >> processing. If not specified - the default value 256 will be used >> which will allow to place descriptor's pool into the internal CPPI >> RAM on most of TI SoC. >> >> Signed-off-by: Grygorii Strashko >> --- >> Documentation/devicetree/bindings/net/cpsw.txt | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/net/cpsw.txt b/Documentation/devicetree/bindings/net/cpsw.txt >> index 5ad439f..b99d196 100644 >> --- a/Documentation/devicetree/bindings/net/cpsw.txt >> +++ b/Documentation/devicetree/bindings/net/cpsw.txt >> @@ -35,6 +35,11 @@ Optional properties: >> For example in dra72x-evm, pcf gpio has to be >> driven low so that cpsw slave 0 and phy data >> lines are connected via mux. >> +- descs_pool_size : total number of CPDMA CPPI descriptors to be used for >> + both ingress/egress packets processing. if not >> + specified the default value 256 will be used which >> + will allow to place descriptors pool into the >> + internal CPPI RAM. > Does it describe h/w? Why now module parameter? or even smth like ethtool num > ring entries? > It can be module parameter too. in general this is expected to be one-time boot setting only. ----- OR So, do you propose to use ethtool -g ethX ethtool -G ethX [rx N] [tx N] ? Now cpdma has one pool for all RX/TX channels, so changing this settings by ethtool will require: pause interfaces, reallocate cpdma pool, re-arrange buffers between channels, resume interface. Correct? How do you think - we can move forward with one pool or better to have two (Rx and Tx)? Wouldn't it be reasonable to still have DT (or module) parameter to avoid cpdma reconfiguration on system startup (pause/resume interfaces) (faster boot)? How about cpdma re-allocation policy (with expectation that is shouldn't happen too often)? - increasing of Rx, Tx will grow total number of physically allocated buffers (total_desc_num) - decreasing of Rx, Tx will just change number of available buffers (no memory re-allocation) ----- OR ---- Can we move forward with current patch (total number of CPDMA CPPI descriptors defined in DT) and add ethtool -G ethX [rx N] [tx N] which will allow to re-split descs between RX and TX? -- regards, -grygorii