Re: [PATCH-cpuset v11 0/2] Add Union-Find and use it to optimize cpuset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/4/24 02:24, Xavier wrote:
Hi all,

Based on Michal's suggestion, the following changes were made:
1. Changed Union-Find to union-find in all places except the title.
2. Optimized the logic of the uf_union function.
3. Modified places where csa[i]->node.parent was used directly.

To Longman,
Regarding the modifications for supporting cpuset merging in both cgroup
v1 and v2, do you mean that you will continue to complete them after my
patch is merged?
Yes.

Kindly review.

Xavier (2):
   Union-Find: add a new module in kernel library
   cpuset: use Union-Find to optimize the merging of cpumasks

  Documentation/core-api/union_find.rst         | 102 ++++++++++++++++
  .../zh_CN/core-api/union_find.rst             |  87 +++++++++++++
  MAINTAINERS                                   |   9 ++
  include/linux/union_find.h                    |  41 +++++++
  kernel/cgroup/cpuset.c                        | 114 +++++++-----------
  lib/Makefile                                  |   2 +-
  lib/union_find.c                              |  49 ++++++++
  7 files changed, 333 insertions(+), 71 deletions(-)
  create mode 100644 Documentation/core-api/union_find.rst
  create mode 100644 Documentation/translations/zh_CN/core-api/union_find.rst
  create mode 100644 include/linux/union_find.h
  create mode 100644 lib/union_find.c

The patch series looks good to me. However, it is a still major change in the domain generation algorithm and it is too late for v6.11. I would also like it to spend more time in linux-next as I don't have a good set of cgroup v1 test. I will support merging this for v6.12.

Acked-by: Waiman Long <longman@xxxxxxxxxx>





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [Monitors]

  Powered by Linux