Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 5 Oct 2017 11:56:04 +0000 (UTC)
From:      Konstantin Belousov <kib@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r324304 - head/sys/amd64/amd64
Message-ID:  <201710051156.v95Bu4PE027340@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kib
Date: Thu Oct  5 11:56:04 2017
New Revision: 324304
URL: https://svnweb.freebsd.org/changeset/base/324304

Log:
  Remove useless comments.
  
  Requested by:	bde
  Sponsored by:	The FreeBSD Foundation
  MFC after:	1 week

Modified:
  head/sys/amd64/amd64/sys_machdep.c

Modified: head/sys/amd64/amd64/sys_machdep.c
==============================================================================
--- head/sys/amd64/amd64/sys_machdep.c	Thu Oct  5 11:50:01 2017	(r324303)
+++ head/sys/amd64/amd64/sys_machdep.c	Thu Oct  5 11:56:04 2017	(r324304)
@@ -655,12 +655,7 @@ amd64_set_ldt(struct thread *td, struct i386_ldt_args 
 		case SDT_SYSNULL4:
 		case SDT_SYSIGT:
 		case SDT_SYSTGT:
-			/* I can't think of any reason to allow a user proc
-			 * to create a segment of these types.  They are
-			 * for OS use only.
-			 */
 			return (EACCES);
-			/*NOTREACHED*/
 
 		/* memory segment types */
 		case SDT_MEMEC:   /* memory execute only conforming */
@@ -686,7 +681,6 @@ amd64_set_ldt(struct thread *td, struct i386_ldt_args 
 			break;
 		default:
 			return(EINVAL);
-			/*NOTREACHED*/
 		}
 
 		/* Only user (ring-3) descriptors may be present. */



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