Re: [PATCH 14/16] MIPS64: Add the relization of verify symbol

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

 



-----Original Message-----
> Subject: [PATCH 14/16] MIPS64: Add the relization of verify symbol

nitpicking, "realization" or else?

Thanks,
Kazu

> 
> Add mips64_verify_symbol() implementation, accept or reject a
> symbol from the kernel namelist.
> 
> Signed-off-by: Huacai Chen <chenhuacai@xxxxxxxxxxx>
> Signed-off-by: Youling Tang <tangyouling@xxxxxxxxxxx>
> ---
>  mips64.c | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/mips64.c b/mips64.c
> index 6e8e922..d8a5a0a 100644
> --- a/mips64.c
> +++ b/mips64.c
> @@ -1055,7 +1055,14 @@ mips64_get_elf_notes(void)
>  static int
>  mips64_verify_symbol(const char *name, ulong value, char type)
>  {
> -	return TRUE;
> +	if (CRASHDEBUG(8) && name && strlen(name))
> +		fprintf(fp, "%08lx %s\n", value, name);
> +
> +	if (STREQ(name, "_text") || STREQ(name, "_stext"))
> +		machdep->flags |= KSYMS_START;
> +
> +	return (name && strlen(name) && (machdep->flags & KSYMS_START) &&
> +		!STRNEQ(name, "__func__.") && !STRNEQ(name, "__crc_"));
>  }
> 
>  /*
> --
> 2.1.0


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




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

 

Powered by Linux