From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, T_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=altlinux.org; s=dkim; h=Subject:In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=Wt5gGoeEb6pd4TGitf6a9/OZJlnERtzwQGa2XkC7ww0=; b=qc4qoVdBtz6VlQAdVJWqDKh2pP 3uIyfqAobXWYIe+XlrTxZnFyNcQVdYX8PhjFmsLlq8XCxG402ExbdNq//4Qme5gupPfNAfX9gNkiE T7vKzMZbsRvJBHuYmKJ5nkAH/pzCFTu2CaHt1qr8xl9uaXJqq/77uPl//v37yvm36YjY9c1okAVD7 at7F34ILXcgDf7RurwbEpJP5NuCpArBHXLWSDS1KMNYGZyysMZmMDJdWcx0AY81KHYuALzsT94gsJ zGa9WymB/6WJukbr2PDQ/9A+zJScQGQkBDn+tfIj62FFPxXNpwdWpw7ebIWSnxvi6VsCUlXElJBn+ +QRRE46g==; Date: Thu, 1 Oct 2020 23:23:53 +0300 From: Arseny Maslennikov To: ALT Linux Team development discussions Message-ID: <20201001202353.GC1037402@cello> References: <2dd521b85103ae35347e548c89b6873a80811206.1576183643.git.legion@altlinux.org> <20200917131107.GE286846@cello> <20201001191733.tb5kgjn6ylpxlg5t@comp-core-i7-2640m-0182e6> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="iFRdW5/EC4oqxDHL" Content-Disposition: inline In-Reply-To: <20201001191733.tb5kgjn6ylpxlg5t@comp-core-i7-2640m-0182e6> OpenPGP: url=http://grep.cs.msu.ru/~ar/pgp-key.asc X-SA-Exim-Connect-IP: 37.204.119.143 X-SA-Exim-Mail-From: arseny@altlinux.org X-SA-Exim-Version: 4.2.1 X-SA-Exim-Scanned: Yes (on mail.cs.msu.ru) Cc: ldv@altlinux.org Subject: Re: [devel] [PATCH hasher-priv v1 3/3] Add cgroup support X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 01 Oct 2020 20:23:56 -0000 Archived-At: List-Archive: List-Post: --iFRdW5/EC4oqxDHL Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Oct 01, 2020 at 09:17:33PM +0200, Alexey Gladkov wrote: > On Thu, Sep 17, 2020 at 04:11:07PM +0300, Arseny Maslennikov wrote: > > On Fri, Dec 13, 2019 at 12:42:05PM +0100, Alex Gladkov wrote: > > > From: Alexey Gladkov > > >=20 > >=20 > > Could you please explain what you're trying to do with this patch? > > Even if it's obvious from the source itself, we still must have an > > opportunity to discuss, and a decent explanation should stay in the > > project history. >=20 > I think this patch is simple enough. There's a misunderstanding here. I'm not asking to explain the semantics (what this patch does) =E2=80=94 I repeat, it's rather obvious fr= om the source itself, the patch is indeed simple. I'm trying to get how the patch's author would describe the pragmatic value of this patch. IOW: we see this patch does XXX. What, in Alexey's view, are we trying to achieve by implementing XXX? Descriptive commit messages are done (and are enforced in successful communities, e. g. LKML) for a reason. The above essentially is my previous comment here, reworded and clarified. If for some reason you believe it's shameful or rude to the community to "waste time" on textual explanations, fair enough =E2=80=94 I'll maybe writ= e a commit message myself (with my take on why this might be useful) and then most likely ACK the same patch, with authorship reattributed to you via From: in the patch body and the new commit message. Or else NAK this particular revision with an empty commit message and leave it up to ldv@. If it were up to me, I would not approve of empty commit messages in a lasting, crucial project like hasher-privd. People are forgetful, and commit messages exist to help. >=20 > > Most likely, it'll turn out we _at least_ have to pass Delegate=3Dyes to > > the systemd service: > >=20 > > Delegate=3D > > Turns on delegation of further resource control > > partitioning to processes of the unit. Units where > > this is enabled may create and manage their own > > private subhierarchy of control groups below the > > control group of the unit itself. > > Manual page systemd.resource-control(5): lines 786-791 >=20 > I'm pretty sure the hasher-priv shouldn't be tied to systemd. I agree. > I'm also > convinced that the server will not be tied. The hasher-privd must be able > to run on systems without systemd. Sure it must. No one is trying to drop support for anything-but-systemd =66rom hasher-privd. Here I was talking about the operational details of the daemon running _under_ systemd, not without systemd. The insight about Delegate=3Dyes does not interfere with non-systemd installations. >=20 > > Do we only support cgroup2 and ignore cgroup1? If yes, great, but > > perhaps then we might want to have a setting to not fiddle with cgroup > > trees, to support the unfortunate users that have to run Docker and > > other garbage. >=20 > Yeah, I didn't plan on supporting legacy version of cgroups. Docker > already can work with cgroupsv2. Oh, I heard they were just recently working on cgroup2 support. Okay. --iFRdW5/EC4oqxDHL Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE56JD3UKTLEu/ddrm9dQjyAYL01AFAl92OtQACgkQ9dQjyAYL 01CQqxAAqT1z5VzMeVOjMpYRlwiXaj0sv09EAUnx6eLONuC62ap10fLSiu1qtVXE jb1dQFoTmydxmC0vNDDQ4aDdcIAdCD4pm9HwmSd9K6ZVUUBKhMfeftsS0zqiEcSO GV8fFpzNaePCO/WlldgZC4l7xg86swF9CuA3h31UJSJc8wBmLKZbG5A/DAoyHzWB wOAmaOJI8gYFXeR7jQswWS/bskABn8TyRFu7aXuwI7q78LU33QmSkmPfrug7uH7j OGG0NGAc7A4/zbvRssh/61f3dwpcu/2w5CHkVPum+rLfkoq+LCb62gyYULDN8XqD C0pyGTfCnT2I9deAw9lBC7vFKODuhPsLXbSqsUdAp+hCTHIG7T/wSfiE3dygG8Q2 /9gfJiw4y2RQq3ZUVzmkPIEkHgSpx/YLM27ECZN2NgshnL4Tjemei9lbl/8idjYR 9/T+3GVIOQOE9haA+5WG43B+yZTUPPmI/VEVMfiH/Tq7ZIXQH9Bf0WQAhofR46N+ F2R2SxEakLPvdMuhq8neru3yANSe/T0/I3/qSW51DvWlPgXaGM2yLdoYHqbYG2Ju FTXpuPzO4t5JLVYL0jwaH0cmMV36JC0LhXWX0SA7/UsaOQIW0YgweODSoGPiXxVF PDH4T8qVh1WFK98lb7hmzwv6pAOlYh73+Ffzy4o1cclbSJ7Pyp0= =JaUT -----END PGP SIGNATURE----- --iFRdW5/EC4oqxDHL--