public inbox for cgroups@vger.kernel.org
 help / color / mirror / Atom feed
From: Tejun Heo <tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Haitao Huang <haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Cc: jarkko-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-sgx-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org,
	mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
	bp-Gina5bIWoIWzQB+pC5nmwQ@public.gmane.org,
	hpa-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org,
	sohil.mehta-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	zhiquan1.li-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	kristen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
	seanjc-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org,
	zhanb-0li6OtcxBFHby3iVrkZq2A@public.gmane.org,
	anakrish-0li6OtcxBFHby3iVrkZq2A@public.gmane.org,
	mikko.ylinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
	yangjie-0li6OtcxBFHby3iVrkZq2A@public.gmane.org
Subject: Re: [PATCH v5 02/18] cgroup/misc: Add SGX EPC resource type and export APIs for SGX driver
Date: Mon, 25 Sep 2023 08:50:17 -1000	[thread overview]
Message-ID: <ZRHWaZB5BeXt2EcY@slm.duckdns.org> (raw)
In-Reply-To: <20230923030657.16148-3-haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>

On Fri, Sep 22, 2023 at 08:06:41PM -0700, Haitao Huang wrote:
> From: Kristen Carlson Accardi <kristen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> 
> Add SGX EPC memory, MISC_CG_RES_SGX_EPC, to be a valid resource type
> for the misc controller.
> 
> Add per resource type private data so that SGX can store additional per
> cgroup data in misc_cg->misc_cg_res[MISC_CG_RES_SGX_EPC].
> 
> Export misc_cg_root() so the SGX driver can initialize and add those
> additional structures to the root misc cgroup as part of initialization
> for EPC cgroup support. This bootstraps the same additional
> initialization for non-root cgroups in the 'alloc()' callback added in the
> previous patch.
> 
> The SGX driver, as the EPC memory provider, will have a background
> worker to reclaim EPC pages to make room for new allocations in the same
> cgroup when its usage counter reaches near the limit controlled by the
> cgroup and its ancestors. Therefore it needs to do a walk from the
> current cgroup up to the root. To enable this walk, move parent_misc()
> into misc_cgroup.h and make inline to make this function available to
> SGX, rename it to misc_cg_parent(), and update kernel/cgroup/misc.c to
> use the new name.
> 
> Signed-off-by: Kristen Carlson Accardi <kristen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> Signed-off-by: Haitao Huang <haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>

Acked-by: Tejun Heo <tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>

Thanks.

-- 
tejun

WARNING: multiple messages have this Message-ID (diff)
From: Tejun Heo <tj@kernel.org>
To: Haitao Huang <haitao.huang@linux.intel.com>
Cc: jarkko@kernel.org, dave.hansen@linux.intel.com,
	linux-kernel@vger.kernel.org, linux-sgx@vger.kernel.org,
	x86@kernel.org, cgroups@vger.kernel.org, tglx@linutronix.de,
	mingo@redhat.com, bp@alien8.de, hpa@zytor.com,
	sohil.mehta@intel.com, zhiquan1.li@intel.com,
	kristen@linux.intel.com, seanjc@google.com, zhanb@microsoft.com,
	anakrish@microsoft.com, mikko.ylinen@linux.intel.com,
	yangjie@microsoft.com
Subject: Re: [PATCH v5 02/18] cgroup/misc: Add SGX EPC resource type and export APIs for SGX driver
Date: Mon, 25 Sep 2023 08:50:17 -1000	[thread overview]
Message-ID: <ZRHWaZB5BeXt2EcY@slm.duckdns.org> (raw)
Message-ID: <20230925185017.Z9JQcx8FHfM9Fz5bvZloYoa9dFY7-pbcrp52gO4G-wE@z> (raw)
In-Reply-To: <20230923030657.16148-3-haitao.huang@linux.intel.com>

On Fri, Sep 22, 2023 at 08:06:41PM -0700, Haitao Huang wrote:
> From: Kristen Carlson Accardi <kristen@linux.intel.com>
> 
> Add SGX EPC memory, MISC_CG_RES_SGX_EPC, to be a valid resource type
> for the misc controller.
> 
> Add per resource type private data so that SGX can store additional per
> cgroup data in misc_cg->misc_cg_res[MISC_CG_RES_SGX_EPC].
> 
> Export misc_cg_root() so the SGX driver can initialize and add those
> additional structures to the root misc cgroup as part of initialization
> for EPC cgroup support. This bootstraps the same additional
> initialization for non-root cgroups in the 'alloc()' callback added in the
> previous patch.
> 
> The SGX driver, as the EPC memory provider, will have a background
> worker to reclaim EPC pages to make room for new allocations in the same
> cgroup when its usage counter reaches near the limit controlled by the
> cgroup and its ancestors. Therefore it needs to do a walk from the
> current cgroup up to the root. To enable this walk, move parent_misc()
> into misc_cgroup.h and make inline to make this function available to
> SGX, rename it to misc_cg_parent(), and update kernel/cgroup/misc.c to
> use the new name.
> 
> Signed-off-by: Kristen Carlson Accardi <kristen@linux.intel.com>
> Signed-off-by: Haitao Huang <haitao.huang@linux.intel.com>

Acked-by: Tejun Heo <tj@kernel.org>

Thanks.

-- 
tejun

  parent reply	other threads:[~2023-09-25 18:50 UTC|newest]

Thread overview: 126+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-23  3:06 [PATCH v5 00/18] Add Cgroup support for SGX EPC memory Haitao Huang
2023-09-23  3:06 ` [PATCH v5 03/18] x86/sgx: Add sgx_epc_lru_lists to encapsulate LRU lists Haitao Huang
     [not found] ` <20230923030657.16148-1-haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2023-09-23  3:06   ` [PATCH v5 01/18] cgroup/misc: Add per resource callbacks for CSS events Haitao Huang
     [not found]     ` <20230923030657.16148-2-haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2023-09-25 17:09       ` Jarkko Sakkinen
2023-09-25 17:09         ` Jarkko Sakkinen
2023-09-26  3:04         ` Haitao Huang
     [not found]           ` <op.2buytfetwjvjmi-yDQzE4XY+yVaPPhiJ6yCxLKMmGWinSIL2HeeBUIffwg@public.gmane.org>
2023-09-26 13:10             ` Jarkko Sakkinen
2023-09-26 13:10               ` Jarkko Sakkinen
2023-09-26 13:13               ` Jarkko Sakkinen
2023-09-26 13:13                 ` Jarkko Sakkinen
2023-09-27  1:56                 ` Haitao Huang
2023-10-02 22:47                   ` Jarkko Sakkinen
2023-10-02 22:55                     ` Jarkko Sakkinen
2023-10-04 15:45                       ` Haitao Huang
2023-10-04 17:18                         ` Tejun Heo
2023-09-27  9:20     ` Huang, Kai
2023-10-03 14:29       ` Haitao Huang
2023-10-17 18:55     ` Michal Koutný
2023-09-23  3:06   ` [PATCH v5 02/18] cgroup/misc: Add SGX EPC resource type and export APIs for SGX driver Haitao Huang
     [not found]     ` <20230923030657.16148-3-haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2023-09-25 18:50       ` Tejun Heo [this message]
2023-09-25 18:50         ` Tejun Heo
2023-09-28  3:59     ` Huang, Kai
2023-10-03  7:00       ` Haitao Huang
2023-10-03 19:33         ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 04/18] x86/sgx: Use sgx_epc_lru_lists for existing active page list Haitao Huang
2023-09-23  3:06   ` [PATCH v5 05/18] x86/sgx: Store reclaimable EPC pages in sgx_epc_lru_lists Haitao Huang
2023-09-27 10:14     ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 06/18] x86/sgx: Introduce EPC page states Haitao Huang
     [not found]     ` <20230923030657.16148-7-haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2023-09-25 17:11       ` Jarkko Sakkinen
2023-09-25 17:11         ` Jarkko Sakkinen
2023-09-27 10:28     ` Huang, Kai
2023-10-03  4:49       ` Haitao Huang
2023-10-03 20:03         ` Huang, Kai
2023-10-04 15:24           ` Haitao Huang
2023-10-04 21:05             ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 07/18] x86/sgx: Introduce RECLAIM_IN_PROGRESS state Haitao Huang
2023-09-25 17:13     ` Jarkko Sakkinen
2023-09-25 17:13       ` Jarkko Sakkinen
2023-09-27 10:42     ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 08/18] x86/sgx: Use a list to track to-be-reclaimed pages Haitao Huang
2023-09-28  9:28     ` Huang, Kai
2023-10-03  5:09       ` Haitao Huang
2023-09-23  3:06   ` [PATCH v5 09/18] x86/sgx: Store struct sgx_encl when allocating new VA pages Haitao Huang
2023-09-27 11:14     ` Huang, Kai
2023-09-27 15:35       ` Haitao Huang
2023-09-27 21:21         ` Huang, Kai
2023-09-29 15:06           ` Haitao Huang
2023-10-02 11:05             ` Huang, Kai
2023-09-27 11:35     ` Huang, Kai
2023-10-03  6:45       ` Haitao Huang
2023-10-03 20:07         ` Huang, Kai
2023-10-04 15:03           ` Haitao Huang
2023-10-04 21:13             ` Huang, Kai
2023-10-05  4:22               ` Haitao Huang
2023-10-05  6:49                 ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 10/18] x86/sgx: Add EPC page flags to identify owner types Haitao Huang
2023-09-23  3:06   ` [PATCH v5 11/18] x86/sgx: store unreclaimable pages in LRU lists Haitao Huang
2023-09-27 11:57     ` Huang, Kai
2023-10-03  5:42       ` Haitao Huang
2023-09-28  9:41     ` Huang, Kai
2023-10-03  5:15       ` Haitao Huang
2023-10-03 20:12         ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC Haitao Huang
2023-10-09 23:45     ` Huang, Kai
2023-10-10  0:23       ` Sean Christopherson
2023-10-10  0:50         ` Huang, Kai
2023-10-10  1:34           ` Huang, Kai
2023-10-10 16:49             ` Haitao Huang
2023-10-11  0:51               ` Huang, Kai
2023-10-12 13:27                 ` Haitao Huang
2023-10-16 10:57                   ` Huang, Kai
2023-10-16 19:52                     ` Haitao Huang
2023-10-16 21:09                       ` Huang, Kai
2023-10-17  0:10                         ` Haitao Huang
2023-10-17  1:34                           ` Huang, Kai
2023-10-17 12:58                             ` Haitao Huang
2023-10-17 18:54                               ` Michal Koutný
2023-10-17 19:13                                 ` Michal Koutný
2023-10-18  4:39                                   ` Haitao Huang
2023-10-18  4:37                                 ` Haitao Huang
2023-10-18 13:55                                   ` Dave Hansen
2023-10-18 15:26                                     ` Haitao Huang
2023-10-18 15:37                                       ` Dave Hansen
2023-10-18 15:52                                         ` Michal Koutný
2023-10-18 16:25                                           ` Haitao Huang
2023-10-16 21:32                       ` Sean Christopherson
2023-10-17  0:09                         ` Haitao Huang
2023-10-17 15:43                           ` Sean Christopherson
2023-10-17 11:49                         ` Mikko Ylinen
2023-10-11  1:14               ` Huang, Kai
2023-10-16 11:02                 ` Huang, Kai
2023-10-10  1:42         ` Haitao Huang
2023-10-10  2:23           ` Huang, Kai
2023-10-10 13:26             ` Haitao Huang
2023-10-11  0:01               ` Sean Christopherson
2023-10-11 15:02                 ` Haitao Huang
2023-10-10  1:04       ` Haitao Huang
2023-10-10  1:18         ` Huang, Kai
2023-10-10  1:38           ` Haitao Huang
2023-10-10  2:12             ` Huang, Kai
2023-10-10 17:05               ` Haitao Huang
2023-10-11  0:31                 ` Huang, Kai
2023-10-11 16:04                   ` Haitao Huang
2023-09-23  3:06   ` [PATCH v5 13/18] x86/sgx: Expose sgx_reclaim_pages() for use by EPC cgroup Haitao Huang
2023-10-05 12:24     ` Huang, Kai
2023-10-05 19:23       ` Haitao Huang
2023-10-05 20:25         ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 14/18] x86/sgx: Add helper to grab pages from an arbitrary EPC LRU Haitao Huang
2023-09-23  3:06   ` [PATCH v5 15/18] x86/sgx: Prepare for multiple LRUs Haitao Huang
2023-10-05 12:30     ` Huang, Kai
2023-10-05 19:33       ` Haitao Huang
2023-10-05 20:38         ` Huang, Kai
2023-09-23  3:06   ` [PATCH v5 16/18] x86/sgx: Limit process EPC usage with misc cgroup controller Haitao Huang
     [not found]     ` <20230923030657.16148-17-haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2023-09-25 17:15       ` Jarkko Sakkinen
2023-09-25 17:15         ` Jarkko Sakkinen
2023-10-05 21:01     ` Huang, Kai
2023-10-10  0:12     ` Huang, Kai
2023-10-10  0:16     ` Huang, Kai
2023-10-10  0:26     ` Huang, Kai
2023-10-22 18:26       ` Haitao Huang
2023-10-10  9:19     ` Huang, Kai
2023-10-10  9:32     ` Huang, Kai
2023-10-17 18:54     ` Michal Koutný
2023-10-19 16:05       ` Haitao Huang
2023-09-23  3:06   ` [PATCH v5 17/18] Docs/x86/sgx: Add description for cgroup support Haitao Huang
2023-09-23  3:06   ` [PATCH v5 18/18] selftests/sgx: Add scripts for EPC cgroup testing Haitao Huang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZRHWaZB5BeXt2EcY@slm.duckdns.org \
    --to=tj-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
    --cc=anakrish-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \
    --cc=bp-Gina5bIWoIWzQB+pC5nmwQ@public.gmane.org \
    --cc=cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=haitao.huang-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=hpa-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org \
    --cc=jarkko-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=kristen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-sgx-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mikko.ylinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=seanjc-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
    --cc=sohil.mehta-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org \
    --cc=x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=yangjie-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \
    --cc=zhanb-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \
    --cc=zhiquan1.li-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox