[Xen-changelog] [xen master] x86: silence false log messages for plain "xpti" / "pv-l1tf"

[patchbot@xxxxxxx]

commit 2fb57e4beefeda923446b73f88b392e59b07d847
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Fri Sep 28 17:12:14 2018 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Fri Sep 28 17:12:14 2018 +0200

    x86: silence false log messages for plain "xpti" / "pv-l1tf"
    
    While commit 2a3b34ec47 ("x86/spec-ctrl: Yet more fixes for xpti=
    parsing")  claimed to have got rid of the 'parameter "xpti" has invalid
    value "", rc=-22!' log message for "xpti" alone on the command line,
    this wasn't the case (the option took effect nevertheless).
    
    Fix this there as well as for plain "pv-l1tf".
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
 xen/arch/x86/spec_ctrl.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/xen/arch/x86/spec_ctrl.c b/xen/arch/x86/spec_ctrl.c
index f28a98b2f5..a67353f766 100644
--- a/xen/arch/x86/spec_ctrl.c
+++ b/xen/arch/x86/spec_ctrl.c
@@ -257,7 +257,7 @@ static __init int parse_pv_l1tf(const char *s)
             else if ( (val = parse_boolean("domu", s, ss)) >= 0 )
                 opt_pv_l1tf = ((opt_pv_l1tf & ~OPT_PV_L1TF_DOMU) |
                                (val ? OPT_PV_L1TF_DOMU : 0));
-            else
+            else if ( *s )
                 rc = -EINVAL;
             break;
         }
@@ -715,7 +715,7 @@ static __init int parse_xpti(const char *s)
             else if ( (val = parse_boolean("domu", s, ss)) >= 0 )
                 opt_xpti = (opt_xpti & ~OPT_XPTI_DOMU) |
                            (val ? OPT_XPTI_DOMU : 0);
-            else
+            else if ( *s )
                 rc = -EINVAL;
             break;
         }
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog