Re: [PATCH] deduplicate kernel_version open-coded parser

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

 



On Mon, Aug 14, 2023 at 09:41:12AM -0400, Rafael Aquini wrote:
> The code that parses kernel version from OSRELEASE/UTSRELEASE strings
> and populates the global kernel table is duplicated across the codebase
> for no good reason. This commit consolidates all the duplicated parsing
> code into a single method to remove the unnecessary duplicated code.
>

this changeset is also at https://github.com/crash-utility/crash/pull/151
 
> Signed-off-by: Rafael Aquini <aquini@xxxxxxxxxx>
> ---
>  arm64.c   | 27 +++----------------
>  defs.h    |  2 ++
>  kernel.c  | 77 ++++++++++++++++++++++++++-----------------------------
>  riscv64.c | 25 ++----------------
>  4 files changed, 43 insertions(+), 88 deletions(-)
> 
> diff --git a/arm64.c b/arm64.c
> index 67b1a22..39d5f04 100644
> --- a/arm64.c
> +++ b/arm64.c
> @@ -834,35 +834,14 @@ static struct kernel_va_range_handler kernel_va_range_handlers[] = {
>  static unsigned long arm64_get_kernel_version(void)
>  {
>  	char *string;
> -	char buf[BUFSIZE];
> -	char *p1, *p2;
>  
>  	if (THIS_KERNEL_VERSION)
>  		return THIS_KERNEL_VERSION;
>  
> -	string = pc->read_vmcoreinfo("OSRELEASE");
> -	if (string) {
> -		strcpy(buf, string);
> -
> -		p1 = p2 = buf;
> -		while (*p2 != '.')
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[0] = atoi(p1);
> -
> -		p1 = ++p2;
> -		while (*p2 != '.')
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[1] = atoi(p1);
> -
> -		p1 = ++p2;
> -		while ((*p2 >= '0') && (*p2 <= '9'))
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[2] = atoi(p1);
> +	if ((string = pc->read_vmcoreinfo("OSRELEASE"))) {
> +		parse_kernel_version(string);
> +		free(string);
>  	}
> -	free(string);
>  	return THIS_KERNEL_VERSION;
>  }
>  
> diff --git a/defs.h b/defs.h
> index 5ee60f1..1925148 100644
> --- a/defs.h
> +++ b/defs.h
> @@ -6032,6 +6032,8 @@ void clone_bt_info(struct bt_info *, struct bt_info *, struct task_context *);
>  void dump_kernel_table(int);
>  void dump_bt_info(struct bt_info *, char *where);
>  void dump_log(int);
> +void parse_kernel_version(char *);
> +
>  #define LOG_LEVEL(v) ((v) & 0x07)
>  #define SHOW_LOG_LEVEL (0x1)
>  #define SHOW_LOG_DICT  (0x2)
> diff --git a/kernel.c b/kernel.c
> index 2114700..988206b 100644
> --- a/kernel.c
> +++ b/kernel.c
> @@ -104,6 +104,38 @@ static void check_vmcoreinfo(void);
>  static int is_pvops_xen(void);
>  static int get_linux_banner_from_vmlinux(char *, size_t);
>  
> +/*
> + * popuplate the global kernel table (kt) with kernel version
> + * information parsed from UTSNAME/OSRELEASE string
> + */
> +void
> +parse_kernel_version(char *str)
> +{
> +	char *p1, *p2, separator;
> +
> +	p1 = p2 = str;
> +	while (*p2 != '.' && *p2 != '\0')
> +		p2++;
> +
> +	*p2 = NULLCHAR;
> +	kt->kernel_version[0] = atoi(p1);
> +	p1 = ++p2;
> +	while (*p2 != '.' && *p2 != '-' && *p2 != '\0')
> +		p2++;
> +
> +	separator = *p2;
> +	*p2 = NULLCHAR;
> +	kt->kernel_version[1] = atoi(p1);
> +
> +	if (separator == '.') {
> +		p1 = ++p2;
> +		while ((*p2 >= '0') && (*p2 <= '9'))
> +			p2++;
> +
> +		*p2 = NULLCHAR;
> +		kt->kernel_version[2] = atoi(p1);
> +	}
> +}
>  
>  /*
>   *  Gather a few kernel basics.
> @@ -112,7 +144,7 @@ void
>  kernel_init()
>  {
>  	int i, c;
> -	char *p1, *p2, buf[BUFSIZE];
> +	char buf[BUFSIZE];
>  	struct syment *sp1, *sp2;
>  	char *rqstruct;
>  	char *rq_timestamp_name = NULL;
> @@ -270,28 +302,7 @@ kernel_init()
>  	if (buf[64])
>  		buf[64] = NULLCHAR;
>  	if (ascii_string(kt->utsname.release)) {
> -		char separator;
> -
> -		p1 = p2 = buf;
> -		while (*p2 != '.')
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[0] = atoi(p1);
> -		p1 = ++p2;
> -		while (*p2 != '.' && *p2 != '-' && *p2 != '\0')
> -			p2++;
> -		separator = *p2;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[1] = atoi(p1);
> -		*p2 = separator;
> -		if (*p2 == '.') {
> -			p1 = ++p2;
> -			while ((*p2 >= '0') && (*p2 <= '9'))
> -				p2++;
> -			*p2 = NULLCHAR;
> -			kt->kernel_version[2] = atoi(p1);
> -		} else
> -			kt->kernel_version[2] = 0;
> +		parse_kernel_version(buf);
>  
>  		if (CRASHDEBUG(1))
>  			fprintf(fp, "base kernel version: %d.%d.%d\n",
> @@ -10973,8 +10984,6 @@ void
>  get_log_from_vmcoreinfo(char *file)
>  {
>  	char *string;
> -	char buf[BUFSIZE];
> -	char *p1, *p2;
>  	struct vmcoreinfo_data *vmc = &kt->vmcoreinfo;
>  
>  	if (!(pc->flags2 & VMCOREINFO))
> @@ -10986,22 +10995,8 @@ get_log_from_vmcoreinfo(char *file)
>  	if ((string = pc->read_vmcoreinfo("OSRELEASE"))) {
>  		if (CRASHDEBUG(1))
>  			fprintf(fp, "OSRELEASE: %s\n", string);
> -		strcpy(buf, string);
> -		p1 = p2 = buf;
> -		while (*p2 != '.')
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[0] = atoi(p1);
> -		p1 = ++p2;
> -		while (*p2 != '.')
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[1] = atoi(p1);
> -		p1 = ++p2;
> -		while ((*p2 >= '0') && (*p2 <= '9'))
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[2] = atoi(p1);
> +
> +		parse_kernel_version(string);
>  
>  		if (CRASHDEBUG(1))
>  			fprintf(fp, "base kernel version: %d.%d.%d\n",
> diff --git a/riscv64.c b/riscv64.c
> index 6b9a688..066cf2c 100644
> --- a/riscv64.c
> +++ b/riscv64.c
> @@ -259,33 +259,12 @@ riscv64_processor_speed(void)
>  static unsigned long riscv64_get_kernel_version(void)
>  {
>  	char *string;
> -	char buf[BUFSIZE];
> -	char *p1, *p2;
>  
>  	if (THIS_KERNEL_VERSION)
>  		return THIS_KERNEL_VERSION;
>  
> -	string = pc->read_vmcoreinfo("OSRELEASE");
> -	if (string) {
> -		strcpy(buf, string);
> -
> -		p1 = p2 = buf;
> -		while (*p2 != '.')
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[0] = atoi(p1);
> -
> -		p1 = ++p2;
> -		while (*p2 != '.')
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[1] = atoi(p1);
> -
> -		p1 = ++p2;
> -		while ((*p2 >= '0') && (*p2 <= '9'))
> -			p2++;
> -		*p2 = NULLCHAR;
> -		kt->kernel_version[2] = atoi(p1);
> +	if ((string = pc->read_vmcoreinfo("OSRELEASE"))) {
> +		parse_kernel_version(string);
>  		free(string);
>  	}
>  	return THIS_KERNEL_VERSION;
> -- 
> 2.41.0
> 

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility
Contribution Guidelines: https://github.com/crash-utility/crash/wiki




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux