kern/167671: [libkvm] [patch] libkvm doesn't initialize vnet

David Marker dmarker at gmail.com
Tue Sep 4 14:40:06 UTC 2012


The following reply was made to PR kern/167671; it has been noted by GNATS.

From: David Marker <dmarker at gmail.com>
To: bug-followup at FreeBSD.org, dmarker at gmail.com
Cc:  
Subject: Re: kern/167671: [libkvm] [patch] libkvm doesn&#39;t initialize
 vnet
Date: Tue, 4 Sep 2012 08:36:34 -0600

 After further investigation, the problem is that clang(1) optimized the symbol dumptid out of kern_shutdown.c. 
 To reproduce you need "options VIMAGE" and you need to be building with clang(1) not gcc(1).
 
 I can verify this with a simple program to look for the "dumptid" symbol in the same way _kvm_nlist() does:
 
 --------------------------------------------------------------------------------
 #include <errno.h>
 #include <stdio.h>
 #include <sys/param.h>
 #include <sys/linker.h>
 int
 main()
 {
 int rc;
 char *symname = "dumptid";
 struct kld_sym_lookup lookup = {
 .version = sizeof(struct kld_sym_lookup),
 .symname = symname,
 .symvalue = 0,
 .symsize = 0
 };
 if (0 == (rc = kldsym(0, KLDSYM_LOOKUP, &lookup))) {
 (void) fprintf(stdout, "kldsym(...) -> %d\n", rc);
 (void) fprintf(stdout, "found symbol \"%s\"\n", symname);
 } else {
 (void) fprintf(stdout, "kldsym(...) -> %d, errno = %d\n", rc, errno);
 perror(NULL);
 }
 return (0);
 }
 
 --------------------------------------------------------------------------------
 
 The symbol is missing. So the fix is much simpler, make dumptid volatile so clang(1) doesn't feel free to optimize it away.
 After making dumptid volatile the test program above finds the symbol and the original patch isn't needed.
 
 
 $ svn diff
 Index: sys/kern/kern_shutdown.c
 ===================================================================
 --- sys/kern/kern_shutdown.c (revision 240074)
 +++ sys/kern/kern_shutdown.c (working copy)
 @@ -148,7 +148,7 @@
 
 /* Context information for dump-debuggers. */
 static struct pcb dumppcb; /* Registers. */
 -static lwpid_t dumptid; /* Thread ID. */
 +static volatile lwpid_t dumptid; /* Thread ID. */
 
 static void poweroff_wait(void *, int);
 static void shutdown_halt(void *junk, int howto);
 
 
 
 
 


More information about the freebsd-bugs mailing list