From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steve Wise Subject: Re: iproute2 won't compile without AF_VSOCK Date: Tue, 19 Jun 2018 15:41:14 -0500 Message-ID: <941f29dd-331f-66a2-fe87-d84e2ab481d1@opengridcomputing.com> References: <20180619084732.0de6d75d@xeon-e3> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: netdev To: David Ahern , Stephen Hemminger Return-path: Received: from 72-48-214-68.dyn.grandenetworks.net ([72.48.214.68]:40536 "EHLO smtp.opengridcomputing.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755430AbeFSUlL (ORCPT ); Tue, 19 Jun 2018 16:41:11 -0400 In-Reply-To: Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 6/19/2018 3:29 PM, David Ahern wrote: > On 6/19/18 2:27 PM, David Ahern wrote: >> On 6/19/18 9:47 AM, Stephen Hemminger wrote: >>> On Tue, 19 Jun 2018 10:17:45 -0500 >>> Steve Wise wrote: >>> >>>> Hey David, >>>> >>>> I'm trying to compile the latest iproute2 on an RHEL-7.3 distro, and it >>>> fails to compile because AF_VSOCK is not defined.  Should this >>>> functionality be a configure option to disable it on older distros? >>>> >>>> >>>> Thanks, >>>> >>>> Steve. >>>> >>>> ---- >>>> >>>> misc >>>>     CC       ss.o >>>> ss.c:301:27: error: ‘AF_VSOCK’ undeclared here (not in a function) >>>>    .families = FAMILY_MASK(AF_VSOCK), >>>>                            ^ >>>> ss.c:252:46: note: in definition of macro ‘FAMILY_MASK’ >>>>  #define FAMILY_MASK(family) ((uint64_t)1 << (family)) >>>>                                               ^ >>>> ss.c:334:2: error: array index in initializer not of integer type >>>>   [AF_VSOCK] = { >>>>   ^ >>>> ss.c:334:2: error: (near initialization for ‘default_afs’) >>>> make[1]: *** [ss.o] Error 1 >>>> make: *** [all] Error 2 >>>> >>> Probably should just add an #ifdef to takeout that if not present >>> >> Most userspace tools have a compat header for cases like this. >> >> #ifndef AF_VSOCK >> #define AF_VSOCK 40 >> #endif >> > Add the above to include//utils.h; AF_MPLS is already there. I'll send out a patch. Thanks, Steve.