Skip to content

Have the kernel conditionally disable SMT if vulnerable#95

Merged
dustymabe merged 1 commit intocoreos:masterfrom
bgilbert:nosmt
May 17, 2019
Merged

Have the kernel conditionally disable SMT if vulnerable#95
dustymabe merged 1 commit intocoreos:masterfrom
bgilbert:nosmt

Conversation

@bgilbert
Copy link
Contributor

Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So related here is ostreedev/ostree#479. I.e. with this, we'll be able to choose whether we want to affect machines on upgrade, or only new installs. In the former case, e.g. we'd maintain a list of default kargs in our overlay/.

For nosmt specifically my understanding is that we only want this on new installs (nevermind the fact that FCOS is still in pre-preview :)) so this WFM.

Just bringing this up so we keep it in mind when discussing "default" kargs.

image.yaml Outdated
size: 8

# Disable SMT on systems vulnerable to MDS or any similar future issue.
extra-kargs: mitigations=auto,nosmt
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be nicer to have this be a list instead to make it cleaner to e.g. update and maintain context comments.

@bgilbert
Copy link
Contributor Author

Updated.

@dustymabe dustymabe merged commit 4a8e334 into coreos:master May 17, 2019
@bgilbert bgilbert deleted the nosmt branch May 17, 2019 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments