On Fri Feb 2, 2024 at 6:34 PM EET, Haitao Huang wrote:

On Thu, 01 Feb 2024 17:24:40 -0600, Jarkko Sakkinen [off-list ref]  
wrote:

quoted

On Tue Jan 30, 2024 at 4:09 AM EET, Haitao Huang wrote:

quoted

From: Kristen Carlson Accardi <redacted>

The misc cgroup controller (subsystem) currently does not perform
resource type specific action for Cgroups Subsystem State (CSS) events:
the 'css_alloc' event when a cgroup is created and the 'css_free' event
when a cgroup is destroyed.

Define callbacks for those events and allow resource providers to
register the callbacks per resource type as needed. This will be
utilized later by the EPC misc cgroup support implemented in the SGX
driver.

Signed-off-by: Kristen Carlson Accardi <redacted>
Co-developed-by: Haitao Huang <redacted>
Signed-off-by: Haitao Huang <redacted>
---
V8:
- Abstract out _misc_cg_res_free() and _misc_cg_res_alloc() (Jarkko)
V7:
- Make ops one per resource type and store them in array (Michal)
- Rename the ops struct to misc_res_ops, and enforce the constraints of  
required callback
functions (Jarkko)
- Moved addition of priv field to patch 4 where it was used first.  
(Jarkko)

V6:
- Create ops struct for per resource callbacks (Jarkko)
- Drop max_write callback (Dave, Michal)
- Style fixes (Kai)

This version looks nice and smooth:

Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org>

BR, Jarkko

Thanks Jarkko!

@tj, I had your Reviewed-by tags previously but dropped it due to some  
changes for refactoring suggested by other reviewers. Could you take a  
look at patches 1-2 for MiscController and confirm if I can have your  
Reviewed-by tags again?
Thanks

Haitao

you can add tags

BR, Jarkko