Skip to content

kdbus cleanups

Sergei Zviagintsev lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest44 patches 29 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/44Documentation/kdbus: Document new name registry flagsDocumentation/kdbus/kdbus.name.xml+39 -30 replies
2/44uapi: kdbus.h: Kernel-doc fixesinclude/uapi/linux/kdbus.h+23 -201 reply
3/44kdbus: Kernel-docs and comments trivial fixesipc/kdbus/connection.c ipc/kdbus/connection.h ipc/kdbus/fs.c +3 more+8 -121 reply
4/44kdbus: Update kernel-doc for struct kdbus_poolipc/kdbus/pool.c+2 -20 replies
5/44kdbus: Add comment on merging free pool slicesipc/kdbus/pool.c+6 -02 replies
6/44kdbus: Fix kernel-doc for struct kdbus_gapsipc/kdbus/message.h+5 -40 replies
7/44kdbus: Fix comment on translation of caps between namespacesipc/kdbus/metadata.c+15 -90 replies
8/44kdbus: Rename var in kdbus_meta_export_caps()ipc/kdbus/metadata.c+3 -30 replies
9/44kdbus: Remove unused KDBUS_MSG_MAX_SIZE constantipc/kdbus/limits.h+0 -30 replies
10/44kdbus: Use conditional operatoripc/kdbus/names.c+1 -40 replies
11/44kdbus: Cosmetic fix of kdbus_name_is_valid()ipc/kdbus/names.c+2 -20 replies
12/44kdbus: Use conventional list macros in __kdbus_pool_slice_release()ipc/kdbus/pool.c+2 -40 replies
13/44kdbus: Use list_next_entry() in kdbus_queue_entry_unlink()ipc/kdbus/queue.c+1 -21 reply
14/44kdbus: Simplify expression in kdbus_get_memfd()ipc/kdbus/message.c+1 -11 reply
15/44kdbus: Simplify bitwise expression in kdbus_meta_get_mask()ipc/kdbus/metadata.c+1 -12 replies
16/44kdbus: Drop redundant code from kdbus_name_acquire()ipc/kdbus/names.c+0 -20 replies
17/44kdbus: Drop duplicated code from kdbus_pool_slice_alloc()ipc/kdbus/pool.c+1 -40 replies
18/44kdbus: Add var initialization to kdbus_conn_entry_insert()ipc/kdbus/connection.c+1 -32 replies
19/44kdbus: Drop useless initialization from kdbus_conn_reply()ipc/kdbus/connection.c+1 -10 replies
20/44kdbus: Drop useless initialization from kdbus_cmd_hello()ipc/kdbus/connection.c+1 -10 replies
21/44kdbus: Cleanup tests in kdbus_cmd_send()ipc/kdbus/connection.c+5 -42 replies
22/44kdbus: Cleanup error path in kdbus_staging_new_user()ipc/kdbus/message.c+1 -20 replies
23/44kdbus: Cleanup kdbus_conn_call()ipc/kdbus/connection.c+2 -82 replies
24/44kdbus: Cleanup kdbus_conn_unicast()ipc/kdbus/connection.c+4 -62 replies
25/44kdbus: Cleanup kdbus_cmd_conn_info()ipc/kdbus/connection.c+7 -92 replies
26/44kdbus: Cleanup kdbus_pin_dst()ipc/kdbus/connection.c+6 -112 replies
27/44kdbus: Cleanup kdbus_conn_new()ipc/kdbus/connection.c+2 -70 replies
28/44kdbus: Cleanup kdbus_queue_entry_new()ipc/kdbus/queue.c+6 -100 replies
29/44kdbus: Improve tests on incrementing quotaipc/kdbus/connection.c+3 -30 replies
30/44kdbus: Cleanup kdbus_meta_proc_mask()ipc/kdbus/metadata.c+12 -201 reply
31/44kdbus: Cleanup kdbus_conn_move_messages()ipc/kdbus/connection.c+7 -72 replies
32/44kdbus: Remove duplicated code from kdbus_conn_lock2()ipc/kdbus/connection.h+7 -110 replies
33/44kdbus: Improve kdbus_staging_reserve()ipc/kdbus/message.c+8 -140 replies
34/44kdbus: Improve kdbus_conn_entry_sync_attach()ipc/kdbus/connection.c+21 -310 replies
35/44kdbus: Drop goto from kdbus_queue_entry_link()ipc/kdbus/queue.c+16 -160 replies
36/44kdbus: Improve kdbus_name_release()ipc/kdbus/names.c+13 -80 replies
37/44kdbus: Fix error path in kdbus_meta_proc_collect_cgroup()ipc/kdbus/metadata.c+9 -60 replies
38/44kdbus: Fix error path in kdbus_user_lookup()ipc/kdbus/domain.c+10 -102 replies
39/44kdbus: Cleanup kdbus_user_lookup()ipc/kdbus/domain.c+12 -140 replies
40/44kdbus: Cleanup kdbus_item_validate_name()ipc/kdbus/item.c+10 -160 replies
41/44kdbus: Fix memfd install algorithmipc/kdbus/message.c+1 -00 replies
42/44kdbus: Check if fd is allocated before trying to free itipc/kdbus/message.c+2 -10 replies
43/44kdbus: Give up on failed fd allocationipc/kdbus/message.c+24 -182 replies
44/44kdbus: Cleanup kdbus_gaps_install()ipc/kdbus/message.c+5 -110 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 00/44] kdbus cleanups

Hi all,

This is a set of various kdbus code cleanups. Patches are ordered by increasing complexity, starting with docs and comments fixes and one-liners.

Patch 29 is the revised version of
http://lkml.kernel.org/g/[email protected]

Feel free to ask to change layout of this, split/join, etc if necessary.

Thanks, Sergei

Sergei Zviagintsev (44):
Documentation/kdbus: Document new name registry flags
uapi: kdbus.h: Kernel-doc fixes
kdbus: Kernel-docs and comments trivial fixes
kdbus: Update kernel-doc for struct kdbus_pool
kdbus: Add comment on merging free pool slices
kdbus: Fix kernel-doc for struct kdbus_gaps
kdbus: Fix comment on translation of caps between namespaces
kdbus: Rename var in kdbus_meta_export_caps()
kdbus: Remove unused KDBUS_MSG_MAX_SIZE constant
kdbus: Use conditional operator
kdbus: Cosmetic fix of kdbus_name_is_valid()
kdbus: Use conventional list macros in __kdbus_pool_slice_release()
kdbus: Use list_next_entry() in kdbus_queue_entry_unlink()
kdbus: Simplify expression in kdbus_get_memfd()
kdbus: Simplify bitwise expression in kdbus_meta_get_mask()
kdbus: Drop redundant code from kdbus_name_acquire()
kdbus: Drop duplicated code from kdbus_pool_slice_alloc()
kdbus: Add var initialization to kdbus_conn_entry_insert()
kdbus: Drop useless initialization from kdbus_conn_reply()
kdbus: Drop useless initialization from kdbus_cmd_hello()
kdbus: Cleanup tests in kdbus_cmd_send()
kdbus: Cleanup error path in kdbus_staging_new_user()
kdbus: Cleanup kdbus_conn_call()
kdbus: Cleanup kdbus_conn_unicast()
kdbus: Cleanup kdbus_cmd_conn_info()
kdbus: Cleanup kdbus_pin_dst()
kdbus: Cleanup kdbus_conn_new()
kdbus: Cleanup kdbus_queue_entry_new()
kdbus: Improve tests on incrementing quota
kdbus: Cleanup kdbus_meta_proc_mask()
kdbus: Cleanup kdbus_conn_move_messages()
kdbus: Remove duplicated code from kdbus_conn_lock2()
kdbus: Improve kdbus_staging_reserve()
kdbus: Improve kdbus_conn_entry_sync_attach()
kdbus: Drop goto from kdbus_queue_entry_link()
kdbus: Improve kdbus_name_release()
kdbus: Fix error path in kdbus_meta_proc_collect_cgroup()
kdbus: Fix error path in kdbus_user_lookup()
kdbus: Cleanup kdbus_user_lookup()
kdbus: Cleanup kdbus_item_validate_name()
kdbus: Fix memfd install algorithm
kdbus: Check if fd is allocated before trying to free it
kdbus: Give up on failed fd allocation
kdbus: Cleanup kdbus_gaps_install()

 Documentation/kdbus/kdbus.name.xml |  42 +++++++++-
 include/uapi/linux/kdbus.h         |  43 +++++-----
 ipc/kdbus/connection.c             | 157 +++++++++++++++----------------------
 ipc/kdbus/connection.h             |  19 ++---
 ipc/kdbus/domain.c                 |  38 +++++----
 ipc/kdbus/fs.c                     |   2 +-
 ipc/kdbus/item.c                   |  26 +++---
 ipc/kdbus/limits.h                 |   3 -
 ipc/kdbus/message.c                |  81 +++++++++----------
 ipc/kdbus/message.h                |   9 ++-
 ipc/kdbus/metadata.c               |  79 ++++++++++---------
 ipc/kdbus/names.c                  |  32 ++++----
 ipc/kdbus/node.c                   |   4 +-
 ipc/kdbus/pool.c                   |  26 +++---
 ipc/kdbus/queue.c                  |  51 ++++++------
 ipc/kdbus/queue.h                  |   2 +-
 16 files changed, 298 insertions(+), 316 deletions(-)
Signature · 1 line
1.8.3.1