[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/2] x86/VPMU: Disable VPMU when NMI watchdog is on



>>> On 28.01.15 at 20:56, <boris.ostrovsky@xxxxxxxxxx> wrote:
> @@ -59,6 +60,12 @@ static void __init parse_vpmu_param(char *s)
>          }
>          /* fall through */
>      case 1:
> +        if ( opt_watchdog )
> +        {
> +            printk("NMI watchdog is enabled. Disabling VPMU\n");
> +            opt_vpmu_enabled = 0;
> +            break;
> +        }
>          opt_vpmu_enabled |= VPMU_BOOT_ENABLED;
>          break;
>      }

Not only to address Andrew's concerns this needs to be changed:
Logging messages from cmdline argument parsing functions is only
marginally useful - they won't appear on the serial console. But
afaict that'll go away anyway by consolidating the patch into simply
checking opt_watchdog from vPMU code.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.