New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release-1.1] configs/validate: looser validation for RDT #3406
Conversation
@kolyshkin @AkihiroSuda please take a look |
I'm going to try to use github feature to rebase this. If this will work, I'll enable this feature for the main branch as well. |
Alas, the feature is not working the way I hoped for ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@dqminh @AkihiroSuda you have to re-LGTM; my apologies. |
@kolyshkin I guess we need your Signed-off-by line too |
4316e83
to
094e402
Compare
Added. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
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 <markus.lehtonen@intel.com> (cherry picked from commit 1d5c331) Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
094e402
to
ee7ba6c
Compare
Backport of #3382.
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 markus.lehtonen@intel.com
(cherry picked from commit 1d5c331)