From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hongyang Yang Subject: Re: [PATCH v14 2/7] remus: add libnl3 dependency for network buffering support Date: Thu, 3 Jul 2014 18:31:35 +0800 Message-ID: <53B53107.6030309@cn.fujitsu.com> References: <1404288557-24817-1-git-send-email-yanghy@cn.fujitsu.com> <1404288557-24817-3-git-send-email-yanghy@cn.fujitsu.com> <1404381809.14865.38.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1404381809.14865.38.camel@kazak.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: eddie.dong@intel.com, wency@cn.fujitsu.com, stefano.stabellini@eu.citrix.com, andrew.cooper3@citrix.com, yunhong.jiang@intel.com, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org, rshriram@cs.ubc.ca, laijs@cn.fujitsu.com, roger.pau@citrix.com List-Id: xen-devel@lists.xenproject.org Hi Ian, On 07/03/2014 06:03 PM, Ian Campbell wrote: > On Wed, 2014-07-02 at 16:09 +0800, Yang Hongyang wrote: >> Libnl3 is required for controlling Remus network buffering. >> This patch adds dependency on libnl3 (>= 3.2.8) to autoconf scripts. >> Also provide ability to configure tools without libnl3 support, that >> is without network buffering support. >> >> when there's no network buffering support,libxl__netbuffer_enabled() >> returns 0, otherwise returns 1. The callers of this api will be >> introduced in the rest of the series. >> >> NOTE: This patch changes tools/configure.ac, please rerun >> autogen.sh while apply the patch. >> Signed-off-by: Shriram Rajagopalan >> Signed-off-by: Lai Jiangshan >> Signed-off-by: Yang Hongyang >> Reviewed-by: Wen Congyang >> Acked-by: Ian Jackson > > I considered applying this but I suppose it isn't all that useful > without the rest of the remus stuff so it might as well wait. Thoughts? Agreed. > > Perhaps someone might like to consider modifying ts-xen-build-prep and > ts-xen-install in http://xenbits.xen.org/gitweb/?p=osstest.git to add > whichever Debian packages provide the development headers and runtime > dependencies respectively? > > (that is, add the libfoo-dev to ts-xen-build-prep and the libfoo to > ts-xen-install) I can take a look at this although I'm not familiar with osstest... > > Ian. > > . > -- Thanks, Yang.