Skip to content

Commit b0a830c

Browse files
committed
[KEP-5234] Clarify limits on ResourceSlice for mixins
1 parent cdcf69e commit b0a830c

File tree

1 file changed

+23
-2
lines changed
  • keps/sig-scheduling/5234-dra-resourceslice-mixins

1 file changed

+23
-2
lines changed

keps/sig-scheduling/5234-dra-resourceslice-mixins/README.md

Lines changed: 23 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -325,6 +325,10 @@ type ResourceSliceMixins struct {
325325
// shared attributes and capacities that an actual device can "include"
326326
// to extend the set of attributes and capacities it already defines.
327327
//
328+
// The maximum number of device mixins in a ResourceSlice is 128
329+
// and the total number of attributes and capacities across device
330+
// mixins and devices in a single ResourceSlice must not exceed 4096.
331+
//
328332
// +optional
329333
// +listType=atomic
330334
Device []DeviceMixin
@@ -333,6 +337,11 @@ type ResourceSliceMixins struct {
333337
// consumption mixins, each of which contains a set of counters
334338
// that a device will consume from a counter set.
335339
//
340+
// The maximum number of device counter consumption mixins in a
341+
// ResourceSlice is 64 the total number of consumed counters across device
342+
// counter consumption mixins and devices in a single
343+
// ResourceSlice must not exceed 2048.
344+
//
336345
// +optional
337346
// +listType=atomic
338347
DeviceCounterConsumption []DeviceCounterConsumptionMixin
@@ -341,6 +350,10 @@ type ResourceSliceMixins struct {
341350
// a collection of counters that a CounterSet can "include"
342351
// to extend the set of counters it already defines.
343352
//
353+
// The maximum number of counter set mixins in a ResourceSlice is
354+
// 32 and the total number of counters across counter set mixins and
355+
// counter sets in a single ResourceSlice must not exceed 256.
356+
//
344357
// +optional
345358
// +listType=atomic
346359
CounterSet []CounterSetMixin
@@ -460,7 +473,7 @@ type DeviceCounterConsumption struct {
460473
// The mixins referenced here must be defined in the same
461474
// ResourceSlice.
462475
//
463-
// The maximum number of includes is 8.
476+
// The maximum number of includes is 4.
464477
//
465478
// +featureGate=DRAResourceSliceMixins
466479
// +optional
@@ -501,10 +514,18 @@ The ResourceSlice-wide limits will be:
501514
* Total combined number of attributes and capacity in a ResourceSlice is 4096 (so with the maximum number of devices, there can be 32 per device).
502515
* Total number of counters is 256.
503516
* Total number of consumed counters is 2048 (so with the maximum number of devices, there can be 16 per device).
517+
* Total number of counter sets is 32.
518+
* Total number of device mixins is 128 (same as max number of devices).
519+
* Total number of counter set mixins is 32 (same as max number of counter sets).
520+
* Total number of device counter consumption mixins is 64.
521+
504522

505523
We will still enforce some per-field limits:
506-
* The number of mixins that can be referenced from each device, counter set or device counter consumption is 8.
524+
* The number of mixins that can be referenced from each device or counter set is 8.
525+
* The number of mixins that can be referenced from each device counter consumption is 4.
507526
* The number of taints per device is 4.
527+
* The number of device counter consumptions per device is 4. This means that each device can only consume counters from a maximum of 4 different counter sets.
528+
508529

509530
We will also enforce one limit on the flattened device:
510531
* The combined number of attributes and capacities for a single device can not exceed 32. We do this

0 commit comments

Comments
 (0)