[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <a8589894-e6ee-47c8-af76-4b636dd1edd9@mellanox.com>
Date: Sun, 29 Oct 2017 09:49:39 +0200
From: Arkadi Sharshevsky <arkadis@...lanox.com>
To: David Ahern <dsa@...ulusnetworks.com>,
Jiri Pirko <jiri@...nulli.us>, netdev@...r.kernel.org
Cc: davem@...emloft.net, mlxsw@...lanox.com, andrew@...n.ch,
vivien.didelot@...oirfairelinux.com, f.fainelli@...il.com,
michael.chan@...adcom.com, ganeshgr@...lsio.com,
saeedm@...lanox.com, matanb@...lanox.com, leonro@...lanox.com,
idosch@...lanox.com, jakub.kicinski@...ronome.com, ast@...nel.org,
daniel@...earbox.net, simon.horman@...ronome.com,
pieter.jansenvanvuuren@...ronome.com, john.hurley@...ronome.com,
alexander.h.duyck@...el.com, linville@...driver.com,
gospo@...adcom.com, steven.lin1@...adcom.com, yuvalm@...lanox.com,
ogerlitz@...lanox.com
Subject: Re: [patch net-next RFC 1/7] devlink: Add support for resource
abstraction
On 10/25/2017 06:26 PM, David Ahern wrote:
> On 10/24/17 3:22 AM, Jiri Pirko wrote:
>> diff --git a/include/uapi/linux/devlink.h b/include/uapi/linux/devlink.h
>> index 0cbca96..9db1d70 100644
>> --- a/include/uapi/linux/devlink.h
>> +++ b/include/uapi/linux/devlink.h
>> @@ -69,6 +69,8 @@ enum devlink_command {
>> DEVLINK_CMD_DPIPE_ENTRIES_GET,
>> DEVLINK_CMD_DPIPE_HEADERS_GET,
>> DEVLINK_CMD_DPIPE_TABLE_COUNTERS_SET,
>> + DEVLINK_CMD_RESOURCE_SET,
>> + DEVLINK_CMD_RESOURCE_DUMP,
>>
>> /* add new commands above here */
>> __DEVLINK_CMD_MAX,
>> @@ -201,6 +203,12 @@ enum devlink_attr {
>> DEVLINK_ATTR_PAD,
>>
>> DEVLINK_ATTR_ESWITCH_ENCAP_MODE, /* u8 */
>> + DEVLINK_ATTR_RESOURCES, /* nested */
>> + DEVLINK_ATTR_RESOURCE, /* nested */
>> + DEVLINK_ATTR_RESOURCE_NAME, /* string */
>> + DEVLINK_ATTR_RESOURCE_SIZE, /* u64 */
>> + DEVLINK_ATTR_RESOURCE_SIZE_NEW, /* u64 */
>> + DEVLINK_ATTR_RESOURCE_ID, /* u64 */
>>
>> /* add new attributes above here, update the policy in devlink.c */
>>
>
> Where can I find the userspace patch to try out the RFC?
>
Hi David,
Thanks for the review. I will send the updated version this week
with the userspace part.
Powered by blists - more mailing lists