From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dmarchan@redhat.com>
Received: from mail-ua1-f68.google.com (mail-ua1-f68.google.com
 [209.85.222.68]) by dpdk.org (Postfix) with ESMTP id B74591BBFE
 for <dev@dpdk.org>; Fri, 11 Jan 2019 13:48:24 +0100 (CET)
Received: by mail-ua1-f68.google.com with SMTP id t8so4722176uap.0
 for <dev@dpdk.org>; Fri, 11 Jan 2019 04:48:24 -0800 (PST)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20161025;
 h=x-gm-message-state:mime-version:references:in-reply-to:from:date
 :message-id:subject:to:cc;
 bh=f5UZZuHlpJn9u5J+RBBgqeMUvqgWLbahqkBzIL7cs74=;
 b=aIUqVE4ePa20t/2fHuJN8o5WCso4OSCi1X/XZvCxe2eq/mJU89yHnh7hzmQN5LZMDt
 W8jFuj0Hwsra1KC0Az3pewPlL/hrtllV83CxbwCm1pl05nYbck+dG+9L3RneU8bx0JCw
 9zuwnll5/zHXZB6hoWgUGO6mFJ2JdlLPg/tmtTSUdcPV7vAa/82AoNGkae1hax8IC8Zs
 WC/sNpTMPNLb3QdhfFVEr7mSpJRnUg7o5wDKXgkqJsXDcoBaAfLXPuxHMYI8AsnG98w5
 N5b/l3yf4Dz/lwnInMSh5fnc4MVMh0naSx8erWE/EJYnDC3LkxDX2JVEfstA4aNFdr5l
 5ZRA==
X-Gm-Message-State: AJcUukdrEEka50z43eXnQGtYmQVk/9PobpvRtxuk0fzN8KUXaPRBfe3c
 gKqn1PEmqCqSIuFFOvzfjo8U8LuxAAycofVs0CAdCQ==
X-Google-Smtp-Source: ALg8bN6G7DbIz4TbnXBfFtWUEKACHk8BQnuGnmTDKOgNq5aQKyEKB47Fyfjkzu3hQPkL4Z4mOHmefn7Ahgb8Rgb4la8=
X-Received: by 2002:ab0:2508:: with SMTP id j8mr5170287uan.130.1547210904049; 
 Fri, 11 Jan 2019 04:48:24 -0800 (PST)
MIME-Version: 1.0
References: <20190109085426.39965-1-yskoh@mellanox.com>
 <20190110183528.42503-1-yskoh@mellanox.com>
 <2934bc73-98e6-643a-0d61-cf7804e1535d@solarflare.com>
 <20190111110332.GA8355@minint-98vp2qg>
 <27206464-dcf0-9871-a797-cb0b9f2ff25d@solarflare.com>
 <20190111115701.GB3336@bricha3-MOBL.ger.corp.intel.com>
In-Reply-To: <20190111115701.GB3336@bricha3-MOBL.ger.corp.intel.com>
From: David Marchand <david.marchand@redhat.com>
Date: Fri, 11 Jan 2019 13:48:12 +0100
Message-ID: <CAJFAV8yVnwfwvpRvW1U6XfrF7VsMKW3xndL1q_JyB4qyJXD7AA@mail.gmail.com>
To: Andrew Rybchenko <arybchenko@solarflare.com>
Cc: Yongseok Koh <yskoh@mellanox.com>,
 "olivier.matz@6wind.com" <olivier.matz@6wind.com>, 
 Shahaf Shuler <shahafs@mellanox.com>, "dev@dpdk.org" <dev@dpdk.org>, 
 "roszenrami@gmail.com" <roszenrami@gmail.com>,
 Bruce Richardson <bruce.richardson@intel.com>
Content-Type: text/plain; charset="UTF-8"
X-Content-Filtered-By: Mailman/MimeDel 2.1.15
Subject: Re: [dpdk-dev] [PATCH v3 1/2] mbuf: add function returning default
 buffer address
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Fri, 11 Jan 2019 12:48:24 -0000

On Fri, Jan 11, 2019 at 12:57 PM Bruce Richardson <
bruce.richardson@intel.com> wrote:

> On Fri, Jan 11, 2019 at 02:17:04PM +0300, Andrew Rybchenko wrote:
> > Olivier, David,
> >
> > could you take a look at naming suggested below and share your thoughts.
> > My fear is that rte_mbuf_buf_addr() is too generic and true for direct
> mbuf
> > only. That's why I'd like to highlight it in the function name.
> >
>
> I would tend to agree with that concern.
>

I understand your concern as well.

The only usecase we have so far is for drivers on the rx side, so
implicitely direct mbufs.
But from a api user pov, explicit is always better.

I will let Olivier have the last word :-)


-- 
David Marchand