Re: Re: [PATCH v9 15/15] selftests/sgx: Add scripts for EPC cgroup testing

From: Michal Koutný
Date: Tue Apr 02 2024 - 13:40:23 EST


On Tue, Apr 02, 2024 at 11:20:21AM -0500, Haitao Huang <haitao.huang@xxxxxxxxxxxxxxx> wrote:
> Do we really want to have it implemented in c?

I only pointed to the available C boilerplate.

> There are much fewer lines of
> code in shell scripts. Note we are not really testing basic cgroup stuff.
> All we needed were creating/deleting cgroups and set limits which I think
> have been demonstrated feasible in the ash scripts now.

I assume you refer to
Message-Id: <20240331174442.51019-1-haitao.huang@xxxxxxxxxxxxxxx>
right?

Could it be even simpler if you didn't stick to cgtools APIs and v1
compatibility?

Reducing ash_cgexec.sh to something like
echo 0 >$R/$1/cgroup.procs
shift
exec "$@"
(with some small builerplate for $R and previous mkdirs)


Thanks,
Michal

Attachment: signature.asc
Description: PGP signature