Re: [PATCH v10 09/14] x86/sgx: Implement async reclamation for cgroup

[Huang, Kai]

On Wed, 2024-03-27 at 17:22 -0700, Haitao Huang wrote:
>  
>  void sgx_cgroup_init(void)
>  {
> +     sgx_cg_wq = alloc_workqueue("sgx_cg_wq", WQ_UNBOUND | WQ_FREEZABLE, 
> WQ_UNBOUND_MAX_ACTIVE);
> +
> +     /* All Cgroups functionalities are disabled. */
> +     if (WARN_ON(!sgx_cg_wq))
> +             return;
> +

I don't think you should WARN(), because it's not a kernel bug or similar.  Just
print a message saying EPC cgroup is disabled and move on.

        if (!sgx_cg_wq) {
                pr_err("SGX EPC cgroup disabled: alloc_workqueue() failed.\n");
                return;
        }