fix warnings (#133)

* Removes unused variable. * Removes unused --shared from msrsave target
LLNL · Jan 31, 2023 · b020802 · b020802
1 parent 9e7c5fa
commit b020802
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/Makefile b/Makefile
@@ -29,10 +29,10 @@ check: msrsave/msrsave_test
 CURRENT_VERSION := -DVERSION=\"1.6.0\"
 
 msrsave/msrsave.o: msrsave/msrsave.c msrsave/msrsave.h
-	$(CC) $(CFLAGS) $(CURRENT_VERSION) -fPIC -shared -c msrsave/msrsave.c -o $@
+	$(CC) $(CFLAGS) $(CURRENT_VERSION) -fPIC -c msrsave/msrsave.c -o $@
 
 msrsave/msrsave_main.o: msrsave/msrsave_main.c msrsave/msrsave.h
-	$(CC) $(CFLAGS) $(CURRENT_VERSION) -fPIC -shared -c msrsave/msrsave_main.c -o $@
+	$(CC) $(CFLAGS) $(CURRENT_VERSION) -fPIC -c msrsave/msrsave_main.c -o $@
 
 msrsave/msrsave: msrsave/msrsave_main.o msrsave/msrsave.o
 	$(CC) $(CFLAGS) $(CURRENT_VERSION) $^ -o $@

diff --git a/msr_entry.c b/msr_entry.c
@@ -178,7 +178,6 @@ static ssize_t msr_write(struct file *file, const char __user *buf, size_t count
 static int msr_open(struct inode *inode, struct file *file)
 {
     unsigned int cpu = iminor(file->f_path.dentry->d_inode);
-    struct cpuinfo_x86 *c;
 
     if (cpu >= nr_cpu_ids || !cpu_online(cpu))
     {