[LTP] [PATCH v3] memcg/memcontrol04: Fix judgment for recursive_protection

Li Wang liwang@redhat.com
Wed Jan 15 13:49:34 CET 2025


On Wed, Jan 15, 2025 at 8:17 PM Petr Vorel <pvorel@suse.cz> wrote:

> Hi Li, Jin Guojie all,
>
> > On Wed, Jan 15, 2025 at 2:33 PM Jin Guojie <guojie.jin@gmail.com> wrote:
>
> Jin Guojie, thanks for handling this!
>
> > > V3:
> > > * Fix initialization of root->memory_recursiveprot
> > > * The use of tst_cg_memory_recursiveprot(leaf_cg[F]) is changed to safe
> > > check
> > > * The type of memory_recursiveprot is changed to unsigned int
> > > * Rebase the code to the latest branch
> nit: if you add notes like version changelog (or other notes which should
> not be
> part of the final commit message) below --- (above of the list of changed
> files) we will see them in ML, but it will not be in git commit message.
>
> > Thanks for refining this, code looks good to me, but as I pointed out
> > can you rebase the patch on the latest branch? Otherwise, maintainer
> > gets a conflict when applying your patch.
>
> > Reviewed-by: Li Wang <liwang@redhat.com>
>
> Acked-by: Petr Vorel <pvorel@suse.cz>
>
> I also updated patchwork states for all versions of this.
>
> > @Cyril, @Petr, I vote to merge this one before the release, and I also
> think
>
> +1
>
> > we need an additional tiny fix like below:
>
> +1, Li, could you please send a regular patch for it?
>

Yes, I will send it once Guojie gives patch v4.


-- 
Regards,
Li Wang


More information about the ltp mailing list