Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 11 Nov 2014 18:54:57 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r274391 - head/gnu/usr.bin/gdb/kgdb
Message-ID:  <201411111854.sABIsvD4048474@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Tue Nov 11 18:54:57 2014
New Revision: 274391
URL: https://svnweb.freebsd.org/changeset/base/274391

Log:
  Change kbdb's kthr::cpu field into an int, to avoid gcc warnings about
  comparing it with NOCPU, which became -1 recently.  While here, avoid
  using it for address calculations if it is negative.
  
  Reviewed by:	jhb, adrian
  MFC after:	1 week

Modified:
  head/gnu/usr.bin/gdb/kgdb/kgdb.h
  head/gnu/usr.bin/gdb/kgdb/trgt_i386.c

Modified: head/gnu/usr.bin/gdb/kgdb/kgdb.h
==============================================================================
--- head/gnu/usr.bin/gdb/kgdb/kgdb.h	Tue Nov 11 18:15:05 2014	(r274390)
+++ head/gnu/usr.bin/gdb/kgdb/kgdb.h	Tue Nov 11 18:54:57 2014	(r274391)
@@ -41,7 +41,7 @@ struct kthr {
 	uintptr_t	pcb;
 	int		tid;
 	int		pid;
-	u_char		cpu;
+	int		cpu;
 };
 
 extern struct kthr *curkthr;

Modified: head/gnu/usr.bin/gdb/kgdb/trgt_i386.c
==============================================================================
--- head/gnu/usr.bin/gdb/kgdb/trgt_i386.c	Tue Nov 11 18:15:05 2014	(r274390)
+++ head/gnu/usr.bin/gdb/kgdb/trgt_i386.c	Tue Nov 11 18:54:57 2014	(r274391)
@@ -139,7 +139,7 @@ kgdb_trgt_fetch_tss(void)
 	uintptr_t addr, cpu0prvpage, tss;
 
 	kt = kgdb_thr_lookup_tid(ptid_get_pid(inferior_ptid));
-	if (kt == NULL || kt->cpu == NOCPU)
+	if (kt == NULL || kt->cpu == NOCPU || kt->cpu < 0)
 		return (0);
 
 	addr = kgdb_lookup("gdt");



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201411111854.sABIsvD4048474>