RE: [PATCH 1/1] makedumpfile: make use of 'uts_namespace.name' offset in VMCOREINFO

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

 



Hi Alex,

-----Original Message-----
> The offset of the field 'init_uts_ns.name' has changed
> since linux-next commit 9a56493f6942c0e2df1579986128721da96e00d8.
> Make use of the offset 'uts_namespace.name' if available in VMCOREINFO.
> 
> Signed-off-by: Alexander Egorenkov <egorenar@xxxxxxxxxxxxx>

Now I see the following patches in mainline, so I've merged this.
Thank you for the work.

ca4a9241cc5e kdump: append uts_namespace.name offset to VMCOREINFO
9a56493f6942 uts: Use generic ns_common::count

https://github.com/makedumpfile/makedumpfile/commit/54aec3878b3f91341e6bc735eda158cca5c54ec9

Thanks,
Kazu

> ---
>  makedumpfile.c | 17 +++++++++++++++--
>  makedumpfile.h |  6 ++++++
>  2 files changed, 21 insertions(+), 2 deletions(-)
> 
> diff --git a/makedumpfile.c b/makedumpfile.c
> index 4c4251e..5b8b90c 100644
> --- a/makedumpfile.c
> +++ b/makedumpfile.c
> @@ -1159,7 +1159,10 @@ check_release(void)
>  	if (SYMBOL(system_utsname) != NOT_FOUND_SYMBOL) {
>  		utsname = SYMBOL(system_utsname);
>  	} else if (SYMBOL(init_uts_ns) != NOT_FOUND_SYMBOL) {
> -		utsname = SYMBOL(init_uts_ns) + sizeof(int);
> +		if (OFFSET(uts_namespace.name) != NOT_FOUND_STRUCTURE)
> +			utsname = SYMBOL(init_uts_ns) + OFFSET(uts_namespace.name);
> +		else
> +			utsname = SYMBOL(init_uts_ns) + sizeof(int);
>  	} else {
>  		ERRMSG("Can't get the symbol of system_utsname.\n");
>  		return FALSE;
> @@ -2008,6 +2011,11 @@ get_structure_info(void)
>  	SIZE_INIT(cpu_spec, "cpu_spec");
>  	OFFSET_INIT(cpu_spec.mmu_features, "cpu_spec", "mmu_features");
> 
> +	/*
> +	 * Get offsets of the uts_namespace's members.
> +	 */
> +	OFFSET_INIT(uts_namespace.name, "uts_namespace", "name");
> +
>  	return TRUE;
>  }
> 
> @@ -2077,7 +2085,10 @@ get_str_osrelease_from_vmlinux(void)
>  	if (SYMBOL(system_utsname) != NOT_FOUND_SYMBOL) {
>  		utsname = SYMBOL(system_utsname);
>  	} else if (SYMBOL(init_uts_ns) != NOT_FOUND_SYMBOL) {
> -		utsname = SYMBOL(init_uts_ns) + sizeof(int);
> +		if (OFFSET(uts_namespace.name) != NOT_FOUND_STRUCTURE)
> +			utsname = SYMBOL(init_uts_ns) + OFFSET(uts_namespace.name);
> +		else
> +			utsname = SYMBOL(init_uts_ns) + sizeof(int);
>  	} else {
>  		ERRMSG("Can't get the symbol of system_utsname.\n");
>  		return FALSE;
> @@ -2284,6 +2295,7 @@ write_vmcoreinfo_data(void)
>  	WRITE_MEMBER_OFFSET("vmemmap_backing.list", vmemmap_backing.list);
>  	WRITE_MEMBER_OFFSET("mmu_psize_def.shift", mmu_psize_def.shift);
>  	WRITE_MEMBER_OFFSET("cpu_spec.mmu_features", cpu_spec.mmu_features);
> +	WRITE_MEMBER_OFFSET("uts_namespace.name", uts_namespace.name);
> 
>  	if (SYMBOL(node_data) != NOT_FOUND_SYMBOL)
>  		WRITE_ARRAY_LENGTH("node_data", node_data);
> @@ -2682,6 +2694,7 @@ read_vmcoreinfo(void)
>  	READ_MEMBER_OFFSET("vmemmap_backing.list", vmemmap_backing.list);
>  	READ_MEMBER_OFFSET("mmu_psize_def.shift", mmu_psize_def.shift);
>  	READ_MEMBER_OFFSET("cpu_spec.mmu_features", cpu_spec.mmu_features);
> +	READ_MEMBER_OFFSET("uts_namespace.name", uts_namespace.name);
> 
>  	READ_STRUCTURE_SIZE("printk_log", printk_log);
>  	if (SIZE(printk_log) != NOT_FOUND_STRUCTURE) {
> diff --git a/makedumpfile.h b/makedumpfile.h
> index 03fb4ce..9088f1f 100644
> --- a/makedumpfile.h
> +++ b/makedumpfile.h
> @@ -1719,6 +1719,8 @@ struct size_table {
>  	long	cpu_spec;
> 
>  	long	pageflags;
> +
> +	long	uts_namespace;
>  };
> 
>  struct offset_table {
> @@ -1880,6 +1882,10 @@ struct offset_table {
>  	struct cpu_spec_s {
>  		long	mmu_features;
>  	} cpu_spec;
> +
> +	struct uts_namespace_s {
> +		long	name;
> +	} uts_namespace;
>  };
> 
>  /*
> --
> 2.26.2
> 
> 
> _______________________________________________
> kexec mailing list
> kexec@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/kexec

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



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux