mbox series

[iproute2,0/7] Add support for devlink resource abstraction

Message ID 1518598522-27713-1-git-send-email-arkadis@mellanox.com
Headers show
Series Add support for devlink resource abstraction | expand

Message

Arkadi Sharshevsky Feb. 14, 2018, 8:55 a.m. UTC 
Add support for devlink resource abstraction.

Arkadi Sharshevsky (7):
  devlink: Change empty line indication with indentations
  devlink: mnlg: Add support for extended ack
  devlink: Add support for devlink resource abstraction
  devlink: Add support for hot reload
  devlink: Move dpipe context from heap to stack
  devlink: Add support for resource/dpipe relation
  devlink: Update man pages and add resource man

 devlink/devlink.c           | 774 ++++++++++++++++++++++++++++++++++++++++----
 devlink/mnlg.c              |  53 ++-
 include/libnetlink.h        |   1 +
 include/list.h              |   5 +
 lib/libnetlink.c            |   4 +-
 man/man8/devlink-dev.8      |  15 +
 man/man8/devlink-resource.8 |  78 +++++
 man/man8/devlink.8          |   1 +
 8 files changed, 871 insertions(+), 60 deletions(-)
 create mode 100644 man/man8/devlink-resource.8

Comments

David Ahern Feb. 15, 2018, 3:41 a.m. UTC | #1 
On 2/14/18 1:55 AM, Arkadi Sharshevsky wrote:
> Add support for devlink resource abstraction.
> 
> Arkadi Sharshevsky (7):
>   devlink: Change empty line indication with indentations
>   devlink: mnlg: Add support for extended ack
>   devlink: Add support for devlink resource abstraction
>   devlink: Add support for hot reload
>   devlink: Move dpipe context from heap to stack
>   devlink: Add support for resource/dpipe relation
>   devlink: Update man pages and add resource man
> 
>  devlink/devlink.c           | 774 ++++++++++++++++++++++++++++++++++++++++----
>  devlink/mnlg.c              |  53 ++-
>  include/libnetlink.h        |   1 +
>  include/list.h              |   5 +
>  lib/libnetlink.c            |   4 +-
>  man/man8/devlink-dev.8      |  15 +
>  man/man8/devlink-resource.8 |  78 +++++
>  man/man8/devlink.8          |   1 +
>  8 files changed, 871 insertions(+), 60 deletions(-)
>  create mode 100644 man/man8/devlink-resource.8
> 

Looks ok to me.
Arkadi Sharshevsky Feb. 22, 2018, 10:19 a.m. UTC | #2 
On 02/15/2018 05:41 AM, David Ahern wrote:
> On 2/14/18 1:55 AM, Arkadi Sharshevsky wrote:
>> Add support for devlink resource abstraction.
>>
>> Arkadi Sharshevsky (7):
>>   devlink: Change empty line indication with indentations
>>   devlink: mnlg: Add support for extended ack
>>   devlink: Add support for devlink resource abstraction
>>   devlink: Add support for hot reload
>>   devlink: Move dpipe context from heap to stack
>>   devlink: Add support for resource/dpipe relation
>>   devlink: Update man pages and add resource man
>>
>>  devlink/devlink.c           | 774 ++++++++++++++++++++++++++++++++++++++++----
>>  devlink/mnlg.c              |  53 ++-
>>  include/libnetlink.h        |   1 +
>>  include/list.h              |   5 +
>>  lib/libnetlink.c            |   4 +-
>>  man/man8/devlink-dev.8      |  15 +
>>  man/man8/devlink-resource.8 |  78 +++++
>>  man/man8/devlink.8          |   1 +
>>  8 files changed, 871 insertions(+), 60 deletions(-)
>>  create mode 100644 man/man8/devlink-resource.8
>>
> 
> Looks ok to me.
> 

Hi David, noticed it wasn't applied yet.
David Ahern Feb. 22, 2018, 3:02 p.m. UTC | #3 
On 2/22/18 3:19 AM, Arkadi Sharshevsky wrote:
> 
> 
> On 02/15/2018 05:41 AM, David Ahern wrote:
>> On 2/14/18 1:55 AM, Arkadi Sharshevsky wrote:
>>> Add support for devlink resource abstraction.
>>>
>>> Arkadi Sharshevsky (7):
>>>   devlink: Change empty line indication with indentations
>>>   devlink: mnlg: Add support for extended ack
>>>   devlink: Add support for devlink resource abstraction
>>>   devlink: Add support for hot reload
>>>   devlink: Move dpipe context from heap to stack
>>>   devlink: Add support for resource/dpipe relation
>>>   devlink: Update man pages and add resource man
>>>
>>>  devlink/devlink.c           | 774 ++++++++++++++++++++++++++++++++++++++++----
>>>  devlink/mnlg.c              |  53 ++-
>>>  include/libnetlink.h        |   1 +
>>>  include/list.h              |   5 +
>>>  lib/libnetlink.c            |   4 +-
>>>  man/man8/devlink-dev.8      |  15 +
>>>  man/man8/devlink-resource.8 |  78 +++++
>>>  man/man8/devlink.8          |   1 +
>>>  8 files changed, 871 insertions(+), 60 deletions(-)
>>>  create mode 100644 man/man8/devlink-resource.8
>>>
>>
>> Looks ok to me.
>>
> 
> Hi David, noticed it wasn't applied yet.
> 

The feature is in 4.16, so seems like it should be applied to the master
branch. Patchworks should show them assigned to Stephen.
Stephen Hemminger Feb. 23, 2018, 4:41 p.m. UTC | #4 
On Wed, 14 Feb 2018 10:55:15 +0200
Arkadi Sharshevsky <arkadis@mellanox.com> wrote:

> Add support for devlink resource abstraction.
> 
> Arkadi Sharshevsky (7):
>   devlink: Change empty line indication with indentations
>   devlink: mnlg: Add support for extended ack
>   devlink: Add support for devlink resource abstraction
>   devlink: Add support for hot reload
>   devlink: Move dpipe context from heap to stack
>   devlink: Add support for resource/dpipe relation
>   devlink: Update man pages and add resource man
> 
>  devlink/devlink.c           | 774 ++++++++++++++++++++++++++++++++++++++++----
>  devlink/mnlg.c              |  53 ++-
>  include/libnetlink.h        |   1 +
>  include/list.h              |   5 +
>  lib/libnetlink.c            |   4 +-
>  man/man8/devlink-dev.8      |  15 +
>  man/man8/devlink-resource.8 |  78 +++++
>  man/man8/devlink.8          |   1 +
>  8 files changed, 871 insertions(+), 60 deletions(-)
>  create mode 100644 man/man8/devlink-resource.8
> 

I applied all these, but please try and make devlink less wordy.