| Message ID | 20240308081239.3281710-1-liuhangbin@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 44208f59362e7099877f9ba79e1472f6bd321e2e |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [PATCHv2,net-next] netlink: specs: support unterminated-ok | expand |
On Fri, 8 Mar 2024 16:12:39 +0800 Hangbin Liu wrote: > ynl-gen-c.py supports check unterminated-ok, but the yaml schemas don't > have this key. Add this to the yaml files. > > Signed-off-by: Hangbin Liu <liuhangbin@gmail.com> Reviewed-by: Jakub Kicinski <kuba@kernel.org>
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski <kuba@kernel.org>: On Fri, 8 Mar 2024 16:12:39 +0800 you wrote: > ynl-gen-c.py supports check unterminated-ok, but the yaml schemas don't > have this key. Add this to the yaml files. > > Signed-off-by: Hangbin Liu <liuhangbin@gmail.com> > --- > v2: update subject, expand the doc and leave the change out of > genetlink spec. (Jakub Kicinski) > > [...] Here is the summary with links: - [PATCHv2,net-next] netlink: specs: support unterminated-ok https://git.kernel.org/netdev/net-next/c/44208f59362e You are awesome, thank you!
diff --git a/Documentation/netlink/genetlink-c.yaml b/Documentation/netlink/genetlink-c.yaml index c58f7153fcf8..3b5e910f3606 100644 --- a/Documentation/netlink/genetlink-c.yaml +++ b/Documentation/netlink/genetlink-c.yaml @@ -208,6 +208,11 @@ properties: exact-len: description: Exact length for a string or a binary attribute. $ref: '#/$defs/len-or-define' + unterminated-ok: + description: | + For string attributes, do not check whether attribute + contains the terminating null character. + type: boolean sub-type: *attr-type display-hint: &display-hint description: | diff --git a/Documentation/netlink/genetlink-legacy.yaml b/Documentation/netlink/genetlink-legacy.yaml index 938703088306..0f48c14a33d2 100644 --- a/Documentation/netlink/genetlink-legacy.yaml +++ b/Documentation/netlink/genetlink-legacy.yaml @@ -251,6 +251,11 @@ properties: exact-len: description: Exact length for a string or a binary attribute. $ref: '#/$defs/len-or-define' + unterminated-ok: + description: | + For string attributes, do not check whether attribute + contains the terminating null character. + type: boolean sub-type: *attr-type display-hint: *display-hint # Start genetlink-c diff --git a/Documentation/netlink/netlink-raw.yaml b/Documentation/netlink/netlink-raw.yaml index ac4e05415f2f..aa37af9f8a8c 100644 --- a/Documentation/netlink/netlink-raw.yaml +++ b/Documentation/netlink/netlink-raw.yaml @@ -270,6 +270,11 @@ properties: exact-len: description: Exact length for a string or a binary attribute. $ref: '#/$defs/len-or-define' + unterminated-ok: + description: | + For string attributes, do not check whether attribute + contains the terminating null character. + type: boolean sub-type: *attr-type display-hint: *display-hint # Start genetlink-c
ynl-gen-c.py supports check unterminated-ok, but the yaml schemas don't have this key. Add this to the yaml files. Signed-off-by: Hangbin Liu <liuhangbin@gmail.com> --- v2: update subject, expand the doc and leave the change out of genetlink spec. (Jakub Kicinski) --- Documentation/netlink/genetlink-c.yaml | 5 +++++ Documentation/netlink/genetlink-legacy.yaml | 5 +++++ Documentation/netlink/netlink-raw.yaml | 5 +++++ 3 files changed, 15 insertions(+)