From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f41.google.com (mail-pa0-f41.google.com [209.85.220.41]) by dpdk.org (Postfix) with ESMTP id AC70F12A8 for ; Sun, 31 May 2015 18:48:34 +0200 (CEST) Received: by padj3 with SMTP id j3so26967725pad.0 for ; Sun, 31 May 2015 09:48:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-type:content-transfer-encoding; bh=V+qN0TvW/Zcp7TNqfNTxeWY1QZQRCe8FiIJpNwfj29s=; b=kYsguT6LveOKpNIusQiHnnjupUbFHrJuNiXcfAzTjYrEq/Z8OkYeP9mg42kkOmpixq BrLOrDca/2WI3Ez3PnWoN8sdrgIroxm6PuytybpOIQbpj65E4uK2JwK6pEMpCW+C4Z/L skGQK2JdcBZVIlXQjbIihqqg19B0yANnNE+dYo6wUt7/G6HX6FLA6JE0X5SGF+pLnTNe 6NsuakaUcwr0giF5trdooikMo/uKZ7WGOXWZDkrbyFyySSA2hYlWrQV7lzzlKdgBT35U l0NhTEDcHVcfeTyLyL384Z503W77To9N//7x3Ri2mpzqJoIBTw+pIrnEqPSQYvhvLVqo g3ag== X-Gm-Message-State: ALoCoQl1MCqjWg/CZld1tOVAN9Hya12PwQGl0ulvCyE1AiQU2fNAb+78IDTdxnTz4FdHW2DfeUPL X-Received: by 10.70.96.35 with SMTP id dp3mr30683665pdb.44.1433090913964; Sun, 31 May 2015 09:48:33 -0700 (PDT) Received: from urahara (static-50-53-82-155.bvtn.or.frontiernet.net. [50.53.82.155]) by mx.google.com with ESMTPSA id b10sm11708693pdo.84.2015.05.31.09.48.33 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 31 May 2015 09:48:33 -0700 (PDT) Date: Sun, 31 May 2015 09:48:28 -0700 From: Stephen Hemminger To: "Wang, Liang-min" Message-ID: <20150531094828.371263fe@urahara> In-Reply-To: References: <1432946276-9424-1-git-send-email-liang-min.wang@intel.com> <1432946276-9424-3-git-send-email-liang-min.wang@intel.com> <20150530084800.56e3585e@urahara> <20150530122617.1bf2cd18@urahara> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH 2/2] ethtool: add new library to provide ethtool-alike APIs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 31 May 2015 16:48:35 -0000 On Sat, 30 May 2015 19:40:46 +0000 "Wang, Liang-min" wrote: > > On Sat, 30 May 2015 16:16:01 +0000 > "Wang, Liang-min" wrote: > > > >The design decision is to keep ethdev as THE interface for all the external API, so ethtool APIs are designed based upon ethdev API. At the meantime, the ethtool APIs are designed to enable users to migrate designs based upon kernel-space ethtool. The open/close/start are put in place to enable quick migration. > > > >But there is no open/close/start in ethtool in kernel. > >Anyway ethtool is currently on the disfavored list from kernel developers. > >What about netlink or something better? > > > >Remember each new API creates more long term compatiablity and ABI issues. > >So I am against introducing any new API that does the same thing as existing API's. > > Just to clarify APIs supported by this ethtool api: there are net_open and net_stop and no net_start. Both functions are put in place to support net_device_ops::ndo_open and net_device_ops::ndo_close as defined in linux/netdevice.h I get the feeling there is some use case you are not telling the list about. What kind of application would use this api only. Why or how would DPDK application be involved in net_device_ops. If you are planning on putting DPDK in the kernel there are lots of other issues including kernel ABI stability and licensing that need to be dealt with.