Re: [PATCH] makedumpfile: reverse -c and -p if using snappy compression

[Baoquan He <bhe@redhat.com>]

USESNAPPY is a switch which the snappy support has been built into.
With this flag, it means snappy compression algorithm is available.
Which compression algorithm is taken need be decided by -c/-l/-p. 

If I make a build with USELZO and USESNAPPY, then I want to test
zlib/lzo/snappy separately, then this patch will confuse me very
much. Current action and configuration are enough for me. So it's
better to leave the current action alone.

Baoquan
Thanks

On 08/28/13 at 05:08pm, Cliff Wickman wrote:
> From: Cliff Wickman <cpw@sgi.com>
> 
> Reverse the meanings of -c (compression) and -p (snappy compression) if
> USESNAPPY is defined.
> 
> The distro kdump scripts seem to only support -c for compression.
> So make -c mean snappy compression if it is supported.
> 
> Diffed against makedumpfile-1.5.4
> Signed-off-by: Cliff Wickman <cpw@sgi.com>
> ---
>  makedumpfile.c |   10 ++++++++++
>  print_info.c   |   13 +++++++++----
>  2 files changed, 19 insertions(+), 4 deletions(-)
> 
> Index: makedumpfile-1.5.4/makedumpfile.c
> ===================================================================
> --- makedumpfile-1.5.4.orig/makedumpfile.c
> +++ makedumpfile-1.5.4/makedumpfile.c
> @@ -8607,7 +8607,12 @@ main(int argc, char *argv[])
>  			info->name_filterconfig = optarg;
>  			break;
>  		case 'c':
> +#ifdef USESNAPPY
> +			/* if snappy is available make it the default */
> +			info->flag_compress = DUMP_DH_COMPRESSED_SNAPPY;
> +#else
>  			info->flag_compress = DUMP_DH_COMPRESSED_ZLIB;
> +#endif
>  			break;
>  		case 'D':
>  			flag_debug = TRUE;
> @@ -8656,7 +8661,12 @@ main(int argc, char *argv[])
>  			info->flag_dmesg = 1;
>  			break;
>  		case 'p':
> +#ifdef USESNAPPY
> +			/* is snappy is available only use zlib with -p */
> +			info->flag_compress = DUMP_DH_COMPRESSED_ZLIB;
> +#else
>  			info->flag_compress = DUMP_DH_COMPRESSED_SNAPPY;
> +#endif
>  			break;
>  		case 'P':
>  			info->xen_phys_start = strtoul(optarg, NULL, 0);
> Index: makedumpfile-1.5.4/print_info.c
> ===================================================================
> --- makedumpfile-1.5.4.orig/print_info.c
> +++ makedumpfile-1.5.4/print_info.c
> @@ -41,9 +41,9 @@ print_usage(void)
>  #endif
>  	MSG("snappy support:\n");
>  #ifdef USESNAPPY
> -	MSG("  enabled\n");
> +	MSG("  enabled using -c\n");
>  #else
> -	MSG("  disabled ('-p' option will be ignored.)\n");
> +	MSG("  disabled\n");
>  #endif
>  	MSG("\n");
>  	MSG("Usage:\n");
> @@ -90,8 +90,13 @@ print_usage(void)
>  	MSG("\n");
>  	MSG("Available options:\n");
>  	MSG("  [-c|-l|-p]:\n");
> -	MSG("      Compress dump data by each page using zlib for -c option, lzo for -l option\n");
> -	MSG("      or snappy for -p option. A user cannot specify either of these options with\n");
> +#ifdef USESNAPPY
> +	MSG("      Compress dump data by each page using zlib for -p option, lzo for -l option\n");
> +	MSG("      or snappy for -c option. A user cannot specify any of these options with\n");
> +#else
> +	MSG("      Compress dump data by each page using zlib for -c option or lzo for -l option\n");
> +	MSG("      A user cannot specify either of these options with\n");
> +#endif
>  	MSG("      -E option, because the ELF format does not support compressed data.\n");
>  	MSG("      THIS IS ONLY FOR THE CRASH UTILITY.\n");
>  	MSG("\n");
> 
> _______________________________________________
> kexec mailing list
> kexec@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/kexec

_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec