On Fri Feb 2, 2024 at 6:34 PM EET, Haitao Huang wrote: > On Thu, 01 Feb 2024 17:24:40 -0600, Jarkko Sakkinen <jarkko@xxxxxxxxxx> > wrote: > > > On Tue Jan 30, 2024 at 4:09 AM EET, Haitao Huang wrote: > >> From: Kristen Carlson Accardi <kristen@xxxxxxxxxxxxxxx> > >> > >> 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 <kristen@xxxxxxxxxxxxxxx> > >> Co-developed-by: Haitao Huang <haitao.huang@xxxxxxxxxxxxxxx> > >> Signed-off-by: Haitao Huang <haitao.huang@xxxxxxxxxxxxxxx> > >> --- > >> 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@xxxxxxxxxx> > > > > 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
