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 X-Spam-Level: X-Spam-Status: No, score=-12.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,INCLUDES_PULL_REQUEST,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_HELO_NONE,SPF_PASS,T_DKIMWL_WL_HIGH,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F08DDC04AB5 for ; Thu, 6 Jun 2019 07:53:36 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id C729120868 for ; Thu, 6 Jun 2019 07:53:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1559807616; bh=2fvCUpnAwuedIO39GLzQOmNGpCsFXtR857PReVMe/OM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=mdOHXCDEEsKXOonqng8/6zaaHnx3qCbfQWCGy9r6wnPJAlF7kLsbfBFBMTltt0/dQ 6TjLBJPuDyp0wNGqioW8AcR1N3DAUVtikp8f7uRLGjW4K1xkQKvGPKijJHXOguliRH q8VIN0AE02xysLZ6xmsBRUELiiyhUKv1T54yfFv0= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726795AbfFFHxg (ORCPT ); Thu, 6 Jun 2019 03:53:36 -0400 Received: from mail.kernel.org ([198.145.29.99]:45696 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725267AbfFFHxf (ORCPT ); Thu, 6 Jun 2019 03:53:35 -0400 Received: from localhost (unknown [193.47.165.251]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id EB8B220866; Thu, 6 Jun 2019 07:53:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1559807614; bh=2fvCUpnAwuedIO39GLzQOmNGpCsFXtR857PReVMe/OM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WZ5Q5xB0OkTIONuwVnkaxebnVOsvgH0CW0W36YfqhkWUVFS2cxTrZZeaIldbN4yGH RW8hVWRYETYtNPfKH+Wlge3DqGdXZP8mMAVRhCRl3Re3naaAdV3eUgaCUqGVuNybp6 xAMr8c8y9W1oy3dwcSUI9bJvbtknCM7SPOFExgEQ= Date: Thu, 6 Jun 2019 10:53:31 +0300 From: Leon Romanovsky To: Max Gurtovoy Cc: Saeed Mahameed , "David S. Miller" , Jason Gunthorpe , Doug Ledford , Michael Chan , Andy Gospodarek , Tal Gilboa , "linux-rdma@vger.kernel.org" , "netdev@vger.kernel.org" Subject: Re: [pull request][for-next 0/9] Generic DIM lib for netdev and RDMA Message-ID: <20190606075331.GW5261@mtr-leonro.mtl.com> References: <20190605232348.6452-1-saeedm@mellanox.com> <20190606071427.GU5261@mtr-leonro.mtl.com> <898e0df0-b73c-c6d7-9cbe-084163643236@mellanox.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <898e0df0-b73c-c6d7-9cbe-084163643236@mellanox.com> User-Agent: Mutt/1.11.4 (2019-03-13) Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Thu, Jun 06, 2019 at 10:19:41AM +0300, Max Gurtovoy wrote: > > On 6/6/2019 10:14 AM, Leon Romanovsky wrote: > > On Wed, Jun 05, 2019 at 11:24:31PM +0000, Saeed Mahameed wrote: > > > Hi Dave, Doug & Jason > > > > > > This series improves DIM - Dynamically-tuned Interrupt > > > Moderation- to be generic for netdev and RDMA use-cases. > > > > > > From Tal and Yamin: > > > The first 7 patches provide the necessary refactoring to current net_dim > > > library which affect some net drivers who are using the API. > > > > > > The last 2 patches provide the RDMA implementation for DIM. > > > > > > For more information please see tag log below. > > > > > > Once we are all happy with the series, please pull to net-next and > > > rdma-next trees. > > > > > > Thanks, > > > Saeed. > > > > > > --- > > > The following changes since commit cd6c84d8f0cdc911df435bb075ba22ce3c605b07: > > > > > > Linux 5.2-rc2 (2019-05-26 16:49:19 -0700) > > > > > > are available in the Git repository at: > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/saeed/linux.git tags/dim-updates-2019-06-05 > > > > > > for you to fetch changes up to 1ec9974e75e7a58bff1ab17c4fcda17b180ed3bb: > > > > > > RDMA/core: Provide RDMA DIM support for ULPs (2019-06-05 16:09:02 -0700) > > > > > > ---------------------------------------------------------------- > > > dim-updates-2019-06-05 > > > > > > From: Tal Gilboa > > > > > > Implement net DIM over a generic DIM library > > > > > > net_dim.h lib exposes an implementation of the DIM algorithm for > > > dynamically-tuned interrupt moderation for networking interfaces. > > > > > > We want a similar functionality for other protocols, which might need to > > > optimize interrupts differently. Main motivation here is DIM for NVMf > > > storage protocol. > > > > > > Current DIM implementation prioritizes reducing interrupt overhead over > > > latency. Also, in order to reduce DIM's own overhead, the algorithm might > > > take some time to identify it needs to change profiles. While this is > > > acceptable for networking, it might not work well on other scenarios. > > > > > > Here I propose a new structure to DIM. The idea is to allow a slightly > > > modified functionality without the risk of breaking Net DIM behavior for > > > netdev. I verified there are no degradations in current DIM behavior with > > > the modified solution. > > > > > > Solution: > > > - Common logic is declared in include/linux/dim.h and implemented in > > > lib/dim/dim.c > > > - Net DIM (existing) logic is declared in include/linux/net_dim.h and > > > implemented in lib/dim/net_dim.c, which uses the common logic from dim.h > > > - Any new DIM logic will be declared in "/include/linux/new_dim.h" and > > > implemented in "lib/dim/new_dim.c". > > > - This new implementation will expose modified versions of profiles, > > > dim_step() and dim_decision(). > > > > > > Pros for this solution are: > > > - Zero impact on existing net_dim implementation and usage > > > - Relatively more code reuse (compared to two separate solutions) > > > - Increased extensibility > > > > > > ---------------------------------------------------------------- > > > Tal Gilboa (6): > > > linux/dim: Move logic to dim.h > > > linux/dim: Remove "net" prefix from internal DIM members > > > linux/dim: Rename externally exposed macros > > > linux/dim: Rename net_dim_sample() to net_dim_update_sample() > > > linux/dim: Rename externally used net_dim members > > > linux/dim: Move implementation to .c files > > > > > > Yamin Friedman (3): > > > linux/dim: Add completions count to dim_sample > > > linux/dim: Implement rdma_dim > > > RDMA/core: Provide RDMA DIM support for ULPs > > Saeed, > > > > No, for the RDMA patches. > > We need to see usage of those APIs before merging. > > I've asked Yamin to prepare patches for NVMeoF initiator and target for > review, so I guess he has it on his plate (this is how he tested it..). > > It might cause conflict with NVMe/blk branch maintained by Sagi, Christoph > and Jens. > > So we need a plan here. We are on top of it and will handle. Thanks > > > > > > Thanks