netdev - Re: [net-next PATCH v6 01/10] netdev-genl: spec: Extend netdev netlink spec in YAML for queue

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: <d975471a-9eda-4ba6-bd5a-7cd76ca9bb09@intel.com>
Date: Tue, 24 Oct 2023 19:19:26 -0700
From: "Nambiar, Amritha" <amritha.nambiar@...el.com>
To: Jakub Kicinski <kuba@...nel.org>
CC: <netdev@...r.kernel.org>, <pabeni@...hat.com>,
	<sridhar.samudrala@...el.com>
Subject: Re: [net-next PATCH v6 01/10] netdev-genl: spec: Extend netdev
 netlink spec in YAML for queue


On 10/24/2023 3:45 PM, Jakub Kicinski wrote:
> Let's beef up the docs here.

Okay, will fix in v7.

> 
> On Mon, 23 Oct 2023 18:33:24 -0700 Amritha Nambiar wrote:
>> +  -
>> +    name: queue
>> +    attributes:
>> +      -
>> +        name: queue-id
>> +        doc: queue index
> 
> Queue index, most queue types are indexed like a C array,
> with indexes starting at 0 and ending at queue count - 1.
> Queue indexes are scoped to an interface and queue type.
> 
>> +        type: u32
>> +      -
>> +        name: ifindex
>> +        doc: netdev ifindex
> 
> ifindex of the netdevice to which the queue belongs.
> 
>> +        type: u32
>> +        checks:
>> +          min: 1
>> +      -
>> +        name: queue-type
>> +        doc: queue type as rx, tx
>> +        type: u32
>> +        enum: queue-type
>> +      -
>> +        name: napi-id
>> +        doc: napi id
> 
> ID of the NAPI instance which services this queue.
> 
>> +        type: u32
>> +
>>   operations:
>>     list:
>>       -
>> @@ -120,6 +147,27 @@ operations:
>>         doc: Notification about device configuration being changed.
>>         notify: dev-get
>>         mcgrp: mgmt
>> +    -
>> +      name: queue-get
>> +      doc: queue information
> 
> Get queue information from the kernel. Only configured queues will
> be reported (as opposed to all available queues).
> 
>> +      attribute-set: queue
>