From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751304AbdLITUD (ORCPT ); Sat, 9 Dec 2017 14:20:03 -0500 Received: from mx0a-00082601.pphosted.com ([67.231.145.42]:36074 "EHLO mx0a-00082601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751073AbdLITUA (ORCPT ); Sat, 9 Dec 2017 14:20:00 -0500 Date: Sat, 9 Dec 2017 19:19:07 +0000 From: Roman Gushchin To: Quentin Monnet CC: , , , , , , , David Ahern Subject: Re: [PATCH v2 net-next 4/4] bpftool: implement cgroup bpf operations Message-ID: <20171209191858.GA20014@castle> References: <20171207183909.16240-1-guro@fb.com> <20171207183909.16240-5-guro@fb.com> <20171208141251.GA9458@castle> <0eed580a-804b-329e-7bfc-1dc5c09a1deb@netronome.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <0eed580a-804b-329e-7bfc-1dc5c09a1deb@netronome.com> User-Agent: Mutt/1.9.1 (2017-09-22) X-Originating-IP: [2620:10d:c092:180::1:2c96] X-ClientProxiedBy: VI1PR09CA0067.eurprd09.prod.outlook.com (2603:10a6:802:29::11) To BL2PR15MB1073.namprd15.prod.outlook.com (2603:10b6:201:17::7) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: ad4cb5ae-39b7-426e-46c1-08d53f39c065 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(5600026)(4604075)(4534020)(4602075)(4627115)(201703031133081)(201702281549075)(2017052603307);SRVR:BL2PR15MB1073; X-Microsoft-Exchange-Diagnostics: 1;BL2PR15MB1073;3:GVIfgcoEeOBmbOsvlej/zmYDxm1cPM1jYX59YqFl+nyTgm09G99I+wVxk8orXEigPwnI/rQmnijY2I3w4aLfg16KcYXzBAhvGYcGezK6kkoGdWWopEjiHZbr38GIwGn79XuYvYEDFR9iVwD7ohV4i6PAvHYJMdCggfk44tkb8OMGQopeYK48m+FjqInn2vdYBBjoARYXg6Busj3C65IWGRk9GIif5+pw1/ZFwMVUFbTboDFTRhCrLL3qdbCaW6fh;25:/C0nnH58LkRNoG91jFt9Gk1LqhwBqIsumR5am+pNd7T3AUriWrFKHL4SLUWP73vtO6MDycVTgNO00ag+6rPGLpwCd/FiTknpMo8ooMCmbimnk2Wboh5oQlFoU54zk3oIiVg2DdY8Y9K0rE+GAm5Zatrk7um+UKXXVjilrgQG8e/I/am2buCt0dwUGp/t3GWIsTANIXVrIGi0BeKo3rHfROwXvpaC0hsxkQfoDYlEmne6YgVGGNCUH1VrfppJ/9Hc7L61paYy9h91/eIAfnXwKb69F73IEKxQZOro5A3Wv/6xdaTmkRfgJv2JR1uQjxFFfe9h16FC3igi90Q1RzHg2w==;31:pL4SogC+6j8usUV9U7LxX8XW2PfcjulKOyaF6Fjttm2rwtv1KB6aHf54Z/onnRK1nz5qWAV7zTzxRjOERh/mPo8tqPxqQ5iy/JxYD5scqtEtcneeRt1c7+XXkMr2CPxIKPrXDrxyiljJwOw/prxurlXE4lrKtWop/wBc2yQGGeY4Z/hiH+0E4Spc4kyeDKrrg6HCCfndhkCYTPdBH86CkKYlrSpNL8Ie8A3wQeW2eRw= X-MS-TrafficTypeDiagnostic: BL2PR15MB1073: X-Microsoft-Exchange-Diagnostics: 1;BL2PR15MB1073;20:h97xF6HBPgf19+Du9CfNiJDqy4Rb5KQWEgg0gLihs0i7aeISBu2JvLaeXnV6Tc2joGUk0iXJB9JvPscdpcO+SfFxG0AwWr4XQipJl12qX+5o6vgUO87+o0L8OZAQMSjFVcVq/PAREmJwmwq/RmifCr5VfaCwOnEz5bD4t7MNgn1PRk87CyH/ZOOCF/n2DW6iLsgLGJQZcL6sIF37uSh+TUYDz6qx5wpkdvDNWKBxiqtk7AB6LHGuBKg+oYYbjjLlPNlKsHgP1oRlF3QIq0wVpap9+DjrV+srUKScVVbmFMQVvbrO0EGo8yDP7shWVKBMe84Ub1m6IH6JTVPRx+gsX5Non2TMBX9QhAYmi0iCy0rtwHMWYlsnk5C8ab1Y3C4fp2dIA1UaRkSUDBlf1ysRBPqbBrOyIgpAKieEpduItCUz+D9zxFvLPq1h1Azxkk8VIerRL96d6qPf0pphbaY0OdMDm5Cipk3L7nSeQT1B4rKjo8vulQO2CRWeEfrHu5Lh;4:1dltX8QIeu6kxJ2SffQ2gEeVJvlxWxgfOQ9/MOItghKGRMMb2p6+i6W8Cn/npophUv24fIrD0nK5tqIjBC6J3J5VTFvTEhRQYsB28iKWT9uXWPH/oZ+4+ZCb0UhAPJfSQ81mnc4KxcjjnEsnOqs/jRdLcP96pkTg1JHOQqY9jP0R3FhogsvJ09vCTrITY1Ot6f9+jO4Gp5oSZsIwXfBMWq9J6ZQeVSp409muwfxpKKPMHf1b0OhP7qEyCIW32EQqxUonR2hYf29ZAEYFanul6fllGrMpocAtwvdI82XwuYvhjmCTy+9izF/dHHqjd3yn X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(67672495146484); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(11241501159)(6040450)(2401047)(8121501046)(5005006)(93006095)(93001095)(3002001)(10201501046)(3231022)(6041248)(20161123562025)(20161123564025)(20161123560025)(20161123555025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(6072148)(201708071742011);SRVR:BL2PR15MB1073;BCL:0;PCL:0;RULEID:(100000803101)(100110400095);SRVR:BL2PR15MB1073; X-Forefront-PRVS: 05168A3970 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(346002)(376002)(366004)(52314003)(24454002)(377424004)(189003)(199004)(33896004)(25786009)(83506002)(5890100001)(4326008)(9686003)(55016002)(47776003)(316002)(76176011)(93886005)(59450400001)(478600001)(97736004)(6666003)(2950100002)(229853002)(6916009)(16586007)(58126008)(53936002)(6116002)(1076002)(6246003)(6496006)(52396003)(23726003)(52116002)(39060400002)(81166006)(81156014)(8676002)(8936002)(5660300001)(2906002)(105586002)(33656002)(7736002)(68736007)(106356001)(305945005)(50466002)(86362001)(33716001)(18370500001)(42262002);DIR:OUT;SFP:1102;SCL:1;SRVR:BL2PR15MB1073;H:castle;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BL2PR15MB1073;23:dBTlSI8JiXLe+w38SJfkecekl+CFDaPjEe4qjByXI?= =?us-ascii?Q?c6F2m+CxLav2q3LSSn4AUMkAVrI6cSoFcMe1rMMmdxf6stqgUp0mED04LK08?= =?us-ascii?Q?NaMRKSZ8IT8sC7LVxQOuhFbAZM058zZNj7eB5W5ohSEF3E96QvahWGcu1M01?= =?us-ascii?Q?q8ZzZA22+NQXABHZCqLfkIVRg+/t91AqOfYwM7Zi2jRp7YIjkRQ+u9WQ5HTO?= =?us-ascii?Q?0KsgmgK99TgMmSgGM5DcS8QobZD1MvaOo+51/VgCErjQLthgQ0ceUsUu+Pzk?= =?us-ascii?Q?eTBV3OJqjiNYgH2mhn15dFEknVSTuv/wZO4W+BKdYVhw/E3Z9awww1lQooCH?= =?us-ascii?Q?k2o5Rwo90TAKenw+bOV6hWnyxZdnfxeqLI22ecKMWRR5mSQ6mtgrS7zicid/?= =?us-ascii?Q?9y93L34dk1pFuiRpRCx0cqxJOA6vFP2V5+72CwP1RzVoQ+JEDvkc+Gwn+zRG?= =?us-ascii?Q?eoyiiaCngKJh3YcjtyUmmWH1/tb56MbH9MOjKE6Mrg67QraGjWANDkctcgE5?= =?us-ascii?Q?e26PqvoahOjLbcQNBYHdjSsKYEon72gBArJJT2mSUmwqLUvYhyO6rtc3aRXS?= =?us-ascii?Q?yu/0rDs4hUC5a4JWUqk42m2bapLwaQ2I2i/GQOVR626ShwJLonQdo3ym9tUU?= =?us-ascii?Q?o5fh1lGUhEKzmap89bmRwiq8qp+gZm+lEnvkipqgi87BLT22wH6aYAXud+d5?= =?us-ascii?Q?8zutpcSm6KgNDz6ZXORIleEiYc6VOzhnBo/0Ymw1omQH64713MjYofpUgXht?= =?us-ascii?Q?9VRCh0wZhR+c70HqwOtn4CaMtnhaaKhuhmFsCpCD0kBzHl9KmuD4PdtA6dnC?= =?us-ascii?Q?BuDCz50kmXnFS79LXmpRts8itMWdWg6sqcUQrnnpo4Rjvhd5d6TlgVb+V3m6?= =?us-ascii?Q?NZGKjDS1bveiWEtMc4lu1LpZAmOj2Qu+kSlbOmOrY1PWxZsO86Rj1b5fjHv2?= =?us-ascii?Q?xg2NwARKOMnjyBMPBer18EJXV+qldy0sY7YZrQVv8oBpgpD7cj/43urkElWQ?= =?us-ascii?Q?z8q49CizXLkXdvyJsjBhbMDps6TzxKgR1bDgIsu9fCkYJ12xGXGaxuCFOLL8?= =?us-ascii?Q?yuFDbLe1GqB6oCAKBTeJL2V96yLfLpXKbX1x6fWA3CQ0BIT/9sbqClUpRvUC?= =?us-ascii?Q?9lfBmdJX8oL/rWMJpVWyUmQ/Y+L7mSOOnd/jF5t+fenqpvs2xRvzjaDDrg0W?= =?us-ascii?Q?6uYKJSELo2Bm34blXfCC93JUqCJomUoX6Usgff9kQOq4uRmD2fZa1FWoK/uK?= =?us-ascii?Q?Lxg0e/XwaiPsD0wjQsXx5uXlZrBPMszwVw7va+l?= X-Microsoft-Exchange-Diagnostics: 1;BL2PR15MB1073;6:AEHcfkMWa3PodJLEVE16K4zQUf5zNEQ6AuSEk25dajweiPRYX13Bnr/xonCXjqR7i58zWokYTaR2A2aZiMbw2nbtUv/Vf6B9EmrSO411UOczC5eLswFiE2HzbXZRlm5X9RZw4bOYAFv36TNrZTJVJZJVWhanCttOOl4Q2xt74ve9ELhzg5hG6u4mufUnfLzmNnv2anEFS6Edpm0+d/L0277HRMtmlm+cr/diMTPqceXZK7TrswZZlgS1OVdKa1/U6CNjSLA6RicwzeLrnOgFWZfrXXj1Ez1//Bq5iPn7NyyS7zIrHK6ufizJhFjSJR7n6TT73Il/6AjGCXpSZ4ldPt6EDhfC/C4PF5wkroLnZTQ=;5:fe0dMJJW6fpyIFMWHxN696VFa6KSIPFBGRnasePMK99YAYlce7665qLIyq2oDyazUaIlNVlUKLvGue/PfmsVaJHbgeARhkJtybW5tTglezda1vcp6+uISUt9FG+tdtiO2e8qbHJaP2zMqFo9fOkmnfnXUKNi1fkk8Pt8ncz70fY=;24:nxATTRiJO5rfMXjv9DwlnVxSxWWfjSKTI4pUfjenOkFIqfsUVSHQT364pOPQCOv6ysOeMLukJSgbnLoBC0tgS8QWE2/Wz7oIdZBS33OfGGw=;7:zGwIvB2tXoCH/0aMvIP2Nv0i0jUUOorG7yyo4LQnUbX6uK05W866jdxBsXIrFL3Rs5PMtnC2JK36s7vKgHL0W8oaoHsV8+ek6faDudlq6W58edjMoe3/3+9bt1zsfXEbufje/HJjb7n1UkKXUCbWGSKCB+z19uKrG3lYXoGLIPykixNgtaqsd4wctCZXuJAzdQsOXmdPFKA58HtZ6eixdueMkEtO0QmLL3k4TsvRdggvk+dyQ3zmCAPMJk2XUvKW SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;BL2PR15MB1073;20:ja7vKPfKDZ0Ods9/uOrQXj3kZbOwUmW0FPE/G1XYpngz4+IwWsWsuF2dpKKUsUD17MV9DDFOKiwnKK4NPA7NyjuiMN4FfH15/J1jKmOOhCWKy6rlGPk5Q8kjBiq7bj0Td9EJ2M4uIipcpjHKfMGj/UE1E/tsvEbBb33Hlnyr3vU= X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Dec 2017 19:19:19.6028 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: ad4cb5ae-39b7-426e-46c1-08d53f39c065 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 8ae927fe-1255-47a7-a2af-5f3a069daaa2 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR15MB1073 X-OriginatorOrg: fb.com X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-12-09_09:,, signatures=0 X-Proofpoint-Spam-Reason: safe X-FB-Internal: Safe Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Dec 08, 2017 at 03:39:43PM +0000, Quentin Monnet wrote: > 2017-12-08 14:12 UTC+0000 ~ Roman Gushchin > > On Fri, Dec 08, 2017 at 10:34:16AM +0000, Quentin Monnet wrote: > >> 2017-12-07 18:39 UTC+0000 ~ Roman Gushchin > >>> This patch adds basic cgroup bpf operations to bpftool: > >>> cgroup list, attach and detach commands. > >>> > >>> Usage is described in the corresponding man pages, > >>> and examples are provided. > > [...] > >>> +MAP COMMANDS > >>> +============= > >>> + > >>> +| **bpftool** **cgroup list** *CGROUP* > >>> +| **bpftool** **cgroup attach** *CGROUP* *ATTACH_TYPE* *PROG* [*ATTACH_FLAGS*] > >>> +| **bpftool** **cgroup detach** *CGROUP* *ATTACH_TYPE* *PROG* > >>> +| **bpftool** **cgroup help** > >>> +| > >>> +| *PROG* := { **id** *PROG_ID* | **pinned** *FILE* | **tag** *PROG_TAG* } > >> > >> Could you please give the different possible values for ATTACH_TYPE and > >> ATTACH_FLAGS, and provide some documentation for the flags? > > > > I intentionally didn't include the list of possible values, as it depends > > on the exact kernel version, and other bpftool docs are carefully avoiding > > specifying such things. > > Do they? As far as I can tell the only other bpftool command that uses > flags is the `bpftool map update`, and it does specify the possible > values for UPDATE_FLAGS (and document them) in the man page. You are right about UPDATE_FLAGS, but at the same time we do not describe bpf program attributes in prog show: **bpftool prog show** [*PROG*] Show information about loaded programs. If *PROG* is specified show information only about given program, otherwise list all programs currently loaded on the system. Output will start with program ID followed by program type and zero or more named attributes (depending on kernel version). I think, that actually ATTACH_TYPE is similar to PROGRAM_TYPE because it will likely be extended in the following kernel versions. So we should probably support specifying it in a numeric form too. ATTACH_FLAGS are probably less volatile and will unlikely be extended often, so we can describe them in docs and add a note about the kernel version next time when a new flag will be added. Anyway, I don't see any big problem in documenting current ATTACH_FLAG and ATTACH_TYPE sets, if we think that it's a good way forward. Thanks!