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;
	}