netdev - Re: [PATCH net-next iproute2 1/2] Update devlink kernel header

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <7cf54485-4757-7140-1b57-264144622b6c@intel.com>
Date:   Thu, 5 Mar 2020 16:44:17 -0800
From:   Jacob Keller <jacob.e.keller@...el.com>
To:     Parav Pandit <parav@...lanox.com>, netdev@...r.kernel.org
Cc:     stephen@...workplumber.org, dsahern@...nel.org, jiri@...lanox.com
Subject: Re: [PATCH net-next iproute2 1/2] Update devlink kernel header



On 3/5/2020 4:42 PM, Jacob Keller wrote:
> 
> 
> On 3/3/2020 8:06 PM, Parav Pandit wrote:
>> Update devlink kernel header to commit:
>> acf1ee44ca5d ("devlink: Introduce devlink port flavour virtual")
>>
>> Signed-off-by: Parav Pandit <parav@...lanox.com>
>> ---
>>  include/uapi/linux/devlink.h | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/include/uapi/linux/devlink.h b/include/uapi/linux/devlink.h
>> index 3f82dedd..1b412281 100644
>> --- a/include/uapi/linux/devlink.h
>> +++ b/include/uapi/linux/devlink.h
>> @@ -187,6 +187,7 @@ enum devlink_port_flavour {
>>  				      * for the PCI VF. It is an internal
>>  				      * port that faces the PCI VF.
>>  				      */
>> +	DEVLINK_PORT_FLAVOUR_VIRTUAL, /* Any virtual port facing the user. */
>>  };
>>  
>>  enum devlink_param_cmode {
>> @@ -252,6 +253,8 @@ enum devlink_trap_type {
>>  enum {
>>  	/* Trap can report input port as metadata */
>>  	DEVLINK_ATTR_TRAP_METADATA_TYPE_IN_PORT,
>> +	/* Trap can report flow action cookie as metadata */
>> +	DEVLINK_ATTR_TRAP_METADATA_TYPE_FA_COOKIE,
>>  };
>>  
> 
> This hunk doesn't seem relevant to the patch and isn't mentioned in the
> subject or description.
> 
> Thanks,
> Jake

Oops, disregard this. This is just the kernel header update, which is
fine. Just hadn't expected to see it, and it didn't grok the first time
I looked at this.

Thanks,
Jake