From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <david.marchand@6wind.com>
Received: from mail-ob0-f177.google.com (mail-ob0-f177.google.com
 [209.85.214.177]) by dpdk.org (Postfix) with ESMTP id 39CCC2E83
 for <dev@dpdk.org>; Wed, 10 Feb 2016 12:38:40 +0100 (CET)
Received: by mail-ob0-f177.google.com with SMTP id wb13so23039560obb.1
 for <dev@dpdk.org>; Wed, 10 Feb 2016 03:38:40 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=6wind-com.20150623.gappssmtp.com; s=20150623;
 h=mime-version:in-reply-to:references:from:date:message-id:subject:to
 :cc:content-type;
 bh=ZiNzwAZLaBFhaz4QqqNaeG9KGakZ5RqfO58qFwzeLfc=;
 b=Yzt16hCA9/hpad4cQbnHyBnqMOlMMz5jz8qdItzkFuOcHyUVHKP+2Yj80nTwK6ujVb
 gMBtpiyxtSUJz3brdlrYgH8xbgw1VE5FMFf0G+R8ex6fYtb2+BuN6VydFzdxKIAbgnoG
 n/Q42fBHhcy3gz9hDVvRQbU8fSW/oKSrgalRJ/VgtEWg03lbOq0HdViU+cEWnShFjbZi
 q3R7qMKRiPxnuw5rWB6+2CQ59dv3cvKWWVAIKXbdylNaO8MearvJ3UE6XRfF1fFlHPg6
 iqSutzjNdbuWYQEsPq/pBxMzy1qSW0KtEVmj30wmfSQgplIhHXaSqwTsjWQOSQryGEAs
 ye9Q==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20130820;
 h=x-gm-message-state:mime-version:in-reply-to:references:from:date
 :message-id:subject:to:cc:content-type;
 bh=ZiNzwAZLaBFhaz4QqqNaeG9KGakZ5RqfO58qFwzeLfc=;
 b=cGcu0+73OMfjbE03w3Mms9uYDLNjhjBMqB9tjADduQZoGRt+TZ+d/KJPw45i3jCy2F
 Rr54XPqWFtO62K/lNa03bo5zf/1KcKi3AsBJu5rOOm5u8/OmSWMHdFF1FgER1kI6n9an
 aPu8xniXkskFhnIDeSpxx+ntY28cd1kMOYV4NeXiiRMaeH+1bh9G/+cZi/xfbXuGKzHj
 ogv49Es+lOySKCefZ89qZELJdZ+rUX0Qa8WFWpytOh87E6Sgxh/FDs7e81VtDn4Zj3Eg
 H0V5xgWH2J96YbklSs1fQcf+IW1UODuZHRo+ZbA7PUBd359qSArGSmB27jyZMGJo2phw
 L6Kw==
X-Gm-Message-State: AG10YOR64S0BdW+fuJX8I0zQx/3IyvfS0Qk+8gRSMPJl6d88epjoVc1PihXtnQ3XX9R6rWh7eH0FsgIDRsZ5h1vI
X-Received: by 10.60.59.37 with SMTP id w5mr17516609oeq.44.1455104319556; Wed,
 10 Feb 2016 03:38:39 -0800 (PST)
MIME-Version: 1.0
Received: by 10.76.180.72 with HTTP; Wed, 10 Feb 2016 03:38:20 -0800 (PST)
In-Reply-To: <20160210112028.7d1004a0@pcviktorin.fit.vutbr.cz>
References: <1454076516-21591-1-git-send-email-david.marchand@6wind.com>
 <1454076516-21591-4-git-send-email-david.marchand@6wind.com>
 <20160209180555.3ad379cf@pcviktorin.fit.vutbr.cz>
 <CALwxeUs8zro1cM2zFVRmmEViwY8P=+s8L_6TOCOqf=w-ohscOw@mail.gmail.com>
 <CALwxeUuqiib7_KjNjvP9e=qGgWPyuZa7ZDXOOL=VHq4i=x8usQ@mail.gmail.com>
 <20160210112028.7d1004a0@pcviktorin.fit.vutbr.cz>
From: David Marchand <david.marchand@6wind.com>
Date: Wed, 10 Feb 2016 12:38:20 +0100
Message-ID: <CALwxeUvUVSXafp_ykUcjftq=4WNUeasXrcmbaiWM8g9+N6EKng@mail.gmail.com>
To: Jan Viktorin <viktorin@rehivetech.com>
Content-Type: text/plain; charset=UTF-8
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Wed, 10 Feb 2016 11:38:40 -0000

On Wed, Feb 10, 2016 at 11:20 AM, Jan Viktorin <viktorin@rehivetech.com> wrote:
> On Wed, 10 Feb 2016 10:27:14 +0100
> David Marchand <david.marchand@6wind.com> wrote:
>> #define RTE_EAL_PCI_REGISTER(name, d)\
>> void pciinitfn_ ##name(void);\
>> void __attribute__((constructor, used)) pciinitfn_ ##name(void)\
>> {\
>>         rte_eal_pci_register(d);\
>
> I meant
>         rte_eal_pci_register(&(d)->pci_drv);\
>
> Perhaps, my assumption that a PCI driver is always referred as pci_drv is wrong...

Well, I suppose we will always have a pci driver embedded in some
other internal pmd structure.
So we can always expect it to be called pci_drv ...

Btw, for drivers like mlx or virtio that need to do some more stuff in
their constructor (the iopl stuff for virtio is the most interesting
case, since the pci register happens only if iopl succeeded), we might
need some RTE_MODULE_INIT for those.

But in such a case, I think having RTE_MODULE_INIT in all pmds would
make more sense, and RTE_EAL_PCI_REGISTER looks unneeded ?

-- 
David Marchand