From mboxrd@z Thu Jan  1 00:00:00 1970
From: Vincent JARDIN <vincent.jardin@6wind.com>
Subject: Re: [PATCH v4 02/16] net/mrvl: add mrvl net pmd driver
 skeleton
Date: Thu, 12 Oct 2017 15:57:31 +0200
Message-ID: <f2bd7e3f-0b10-dcf7-f5cc-c3354be4a1b2@6wind.com>
References: <1507031500-11473-1-git-send-email-tdu@semihalf.com>
 <1507561244-20115-1-git-send-email-tdu@semihalf.com>
 <1507561244-20115-3-git-send-email-tdu@semihalf.com> <9041127.34t6OW5FrT@xps>
 <20171012065104.GC19106@tdu> <6bfe47de-c5b4-2b01-7991-2ac48913b2c3@6wind.com>
 <CAD_GOWaNReULvdUUQZxKtcLc_qE31SKn+VvymKm+X8z_n9kaqQ@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Cc: Tomasz Duszynski <tdu@semihalf.com>, Thomas Monjalon
 <thomas@monjalon.net>, dev@dpdk.org, Marcin Wojtas <mw@semihalf.com>,
 Dmitri Epshtein <dima@marvell.com>, Natalie Samsonov <nsamsono@marvell.com>,
 Jianbo.liu@linaro.org
To: Jacek Siuda <jck@semihalf.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mail-wm0-f44.google.com (mail-wm0-f44.google.com [74.125.82.44])
 by dpdk.org (Postfix) with ESMTP id B3A8E1B38B
 for <dev@dpdk.org>; Thu, 12 Oct 2017 15:57:33 +0200 (CEST)
Received: by mail-wm0-f44.google.com with SMTP id b189so13360434wmd.4
 for <dev@dpdk.org>; Thu, 12 Oct 2017 06:57:33 -0700 (PDT)
In-Reply-To: <CAD_GOWaNReULvdUUQZxKtcLc_qE31SKn+VvymKm+X8z_n9kaqQ@mail.gmail.com>
Content-Language: en-US
List-Id: DPDK patches and discussions <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>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

Le 12/10/2017 à 14:14, Jacek Siuda a écrit :
> 
> What we can do, is to add a run-time driver option and oblige user (in 
> documentation) to set the option only for the first mrvl vdev. If the 
> driver doesn't see the option set, it  won't initialize DMA. This, plus 
> meaningful error handling/logs should take care of any potential 
> mishandling by user that we were afraid of. I hope that would be more 
> acceptable.

Yes, it does. And it already matches the current design in fact since 
the return error can be EEXIST and it is ignored.

thank you,
   Vincent