Skip to content

Commit ee7ba6c

Browse files
marquizkolyshkin
authored andcommitted
configs/validate: looser validation for RDT
Don't require CAT or MBA because we don't detect those correctly (we don't support L2 or L3DATA/L3CODE for example, and in the future possibly even more). With plain "ClosId mode" we don't really care: we assign the container to a pre-configured CLOS without trying to do anything smarter. Moreover, this was a duplicate/redundant check anyway, as for CAT and MBA there is another specific sanity check that is done if L3 or MB is specified in the config. Signed-off-by: Markus Lehtonen <[email protected]> (cherry picked from commit 1d5c331) Signed-off-by: Kir Kolyshkin <[email protected]>
1 parent a3765fb commit ee7ba6c

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

libcontainer/configs/validate/validator.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -229,10 +229,6 @@ func (v *ConfigValidator) sysctl(config *configs.Config) error {
229229

230230
func (v *ConfigValidator) intelrdt(config *configs.Config) error {
231231
if config.IntelRdt != nil {
232-
if !intelrdt.IsCATEnabled() && !intelrdt.IsMBAEnabled() {
233-
return errors.New("intelRdt is specified in config, but Intel RDT is not supported or enabled")
234-
}
235-
236232
if config.IntelRdt.ClosID == "." || config.IntelRdt.ClosID == ".." || strings.Contains(config.IntelRdt.ClosID, "/") {
237233
return fmt.Errorf("invalid intelRdt.ClosID %q", config.IntelRdt.ClosID)
238234
}

0 commit comments

Comments
 (0)