Open MPI logo

Hardware Locality Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Hardware Locality Development mailing list

Subject: Re: [hwloc-devel] Compiling hwloc 1.7 with NV support
From: Brice Goglin (Brice.Goglin_at_[hidden])
Date: 2013-04-20 03:53:31


Le 20/04/2013 01:35, Brice Goglin a écrit :
> Le 20/04/2013 01:11, Brice Goglin a écrit :
>> Le 20/04/2013 00:56, Jiri Hladky a écrit :
>>> topology-gl.c: In function 'hwloc_gl_query_devices':
>>> topology-gl.c:91:41: error: 'NV_CTRL_PCI_DOMAIN' undeclared (first use
>>> in this function)
>>> topology-gl.c:91:41: note: each undeclared identifier is reported only
>>> once for each function it appears in
>>> make[2]: *** [topology-gl.lo] Error 1
>>> make[2]: Leaving directory `/tmp/hwloc-1.7/src'
>>> make[1]: *** [all-recursive] Error 1
>>> make[1]: Leaving directory `/tmp/hwloc-1.7/src'
>>> make: *** [all-recursive] Error 1
>>>
>>> Indeed, there is no NV_CTRL_PCI_DOMAIN MACRO defined in NVCtrl header
>>> files:
>> Old CUDA versions didn't have a similar domain macro either, maybe
>> NVIDIA did the same mistake with old nvml.
> I meant nvctrl, not nvml.
>
> Indeed, it was added in release 190.18.5
>

I just fixed this in trunk and v1.7 branch after testing for real. We
also needed to remove a useless duplicated call to CTRL_PCI_DOMAIN. See
the attached patch.
Thanks for the report
Brice