Quoting Matt Roper (2018-02-01 19:56:15) > intel_cgroup is used to modify i915 cgroup parameters. At the moment only a > single parameter is supported (GPU priority offset). In the future the driver > may support additional parameters as well (e.g., limits on memory usage). > > Signed-off-by: Matt Roper <matthew.d.ro...@intel.com> > --- > tools/Makefile.sources | 1 + > tools/intel_cgroup.c | 103 > +++++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 104 insertions(+) > create mode 100644 tools/intel_cgroup.c > > diff --git a/tools/Makefile.sources b/tools/Makefile.sources > index abd23a0f..b30216c4 100644 > --- a/tools/Makefile.sources > +++ b/tools/Makefile.sources > @@ -11,6 +11,7 @@ tools_prog_lists = \ > intel_reg \ > intel_backlight \ > intel_bios_dumper \ > + intel_cgroup \ > intel_display_crc \ > intel_display_poller \ > intel_forcewaked \ > diff --git a/tools/intel_cgroup.c b/tools/intel_cgroup.c > new file mode 100644 > index 00000000..ce781b08 > --- /dev/null > +++ b/tools/intel_cgroup.c > @@ -0,0 +1,103 @@ > +/* > + * Copyright © 2018 Intel Corporation > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice shall be included in > + * all copies or substantial portions of the Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > THE > + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER > DEALINGS > + * IN THE SOFTWARE. > + * > + */ > + > +#include <fcntl.h> > +#include <getopt.h> > +#include <i915_drm.h> > +#include <stdio.h> > +#include <stdlib.h> > +#include <unistd.h> > + > +#include "igt.h" > +#include "xf86drm.h" > +#include "xf86drmMode.h" > + > +#define I915_CGROUP_PARAM_PRIORITY_OFFSET 0x1
Hmm. Could we not avoid drm_ioctl + well known param names and use a more generic tool to set cgroup attributes? Just feels wrong that a such a generic interface boils down to a driver specific ioctl. -Chris _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
