Re: [PATCH v4 0/4] ibacm: Replace ioctl with netlink and fix inablity to resurrect an interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> On 1 Feb 2019, at 20:00, Jason Gunthorpe <jgg@xxxxxxxx> wrote:
> 
> On Fri, Feb 01, 2019 at 07:50:45PM +0100, Håkon Bugge wrote:
>> The first commit in this series re-factors the code which iterates
>> through the system interfaces. The code used the ioctl SIOCGIFCONF,
>> which only returns "running" interfaces. Restricting the enumeration
>> to only "running" interfaces poses several problems. The problems are
>> not solved by the first commit, but the third commit, "ibacm: Unable
>> to resurrect an interface".
>> 
>> As such, the first commit does not alter any functionality.
>> 
>> The second commit is small fix in a debug print.
>> 
>> The third commit is the new implementation of
>> https://patchwork.kernel.org/patch/10748357, which was NAKed. This
>> commit is now based on the netlink re-factoring from commit one.
>> 
>> The forth commit aligns the use of interface names vs. interface
>> aliases (label) with how the ip utility works.
>> 
>> v1 -> v2:
>>   * Added r-b for 2nd and 4th commit
>>   * Incorporated review comments from Jason and Ira
>>   * Minor nits fixed by the author
>> 
>> v2 -> v3:
>>   * Added Ira's r-b
>> 
>> v3 -> v4:
>>   * Fixed CMake checks for netlink (Jason)
>> 
>> Håkon Bugge (4):
>>  ibacm: Replace ioctl with netlink
>>  ibacm: Print correct pkey
>>  ibacm: Unable to resurrect an interface
>>  ibacm: Use alias name when possible
> 
> You may as well send this as a github PR, lets see the CI results

Sure, let me do that, but that's not tonight ;-)


Thxs, Håkon





[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux