Re: [Xen-devel] [PATCH] kexec: Provide user friendly option for memory limit

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 27.04.17 at 14:22, <simon.crowe@xxxxxxxxxx> wrote:
> @@ -162,6 +166,10 @@ static void __init parse_crashkernel(const char *str)
>              kexec_crash_area.start = parse_size_and_unit(cur = str + 1, 
> &str);
>          else if ( *str == '<' )
>              kexec_crash_area_limit = parse_size_and_unit(cur = str + 1, 
> &str);
> +        else if ( !strncmp(str, ",below=", 7) )
> +        {
> +            kexec_crash_area_limit = parse_size_and_unit(cur = str + 7, 
> &str);
> +        }
>          else
>              printk(XENLOG_WARNING "crashkernel: '%s' ignored\n", str);

The braces you add are inconsistent with surrounding code. With
them dropped
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>

You should also have Cc-ed the maintainer of the code.

Jan


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