Index | Thread | Search

From:
David Leadbeater <dgl@dgl.cx>
Subject:
Re: dead code tells no tales, 2/16
To:
Miod Vallat <miod@online.fr>
Cc:
tech@openbsd.org
Date:
Tue, 22 Sep 2026 04:29:06 +1000

Download raw body.

Thread
On Mon, Sep 21, 2026 at 05:59:58PM +0000, Miod Vallat wrote:
> > > -const char sccs[] =
> > > -    "    @(#)${ost} ${osr}" STATUS " (${id}) #${v}: ${t}\n";
> > 
> > This one is read by what(1).
> 
> Right; although I don't think many people run what(1) on kernel binaries
> anymore (and you can get that information with strings(1) or, more
> easily, with config -e).
> 
> But let's drop this chunk, and the diff becomes:

Can also drop the sys/utsname.h include. ok dgl with this.

index f7fc6e979c5..bff856bf70e 100644
--- sys/conf/param.c
+++ sys/conf/param.c
@@ -40,7 +40,6 @@
 #include <sys/param.h>
 #include <sys/systm.h>
 #include <sys/kernel.h>
-#include <sys/utsname.h>
 #ifdef SYSVSHM
 #include <machine/vmparam.h>
 #include <sys/shm.h>
@@ -124,10 +123,3 @@ struct	seminfo seminfo = {
 	SEMAEM		/* adjust on exit max value */
 };
 #endif
-
-/*
- * This has to be allocated somewhere; allocating
- * them here forces loader errors if this file is omitted
- * (if they've been externed everywhere else; hah!).
- */
-struct	utsname utsname;