Re: [PATCH v7 09/15] x86/sgx: Charge mem_cgroup for per-cgroup reclamation

From: Haitao Huang
Date: Fri Jan 26 2024 - 11:21:40 EST


On Fri, 26 Jan 2024 08:37:25 -0600, Huang, Kai <kai.huang@xxxxxxxxx> wrote:



Signed-off-by: Haitao Huang <haitao.huang@xxxxxxxxxxxxxxx>
Reported-by: Mikko Ylinen <mikko.ylinen@xxxxxxxxxxxxxxx>
---

Non-technical staff:

I believe checkpatch requires you to have a "Closes" tag after "Reported-by"
otherwise it complains something like this:

WARNING: Reported-by: should be immediately followed by Closes: with a URL
to the report

Not sure how strict this rule is, but seems you forgot to run checkpatch so just
a reminder.

Yeah I did remember run checkpatch this time and last time :-)
Here is what I see from documentation[1]:"The tag should be followed by a Closes: tag pointing to the report, unless the report is not available on the web". So it seems to allow exceptions.
Mikko mentioned this issue to me in private IM so this is not available on web.

Thanks
Haitao

https://docs.kernel.org/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes