There are some inline functions in sys/arch/x86/x86/intr.c that are only
called if INTRSTACKSIZE is defined but the functions are declared static
in any case which causes compile errors if -Werror is on which happens
if developer is true in mk.conf.

I don't normally work in the kernel so even though it seems like a
no-brainer I thought I should ask before committing the patch at the end
of this message.  Also, should I continue to ask for things this simple
or just go ahead in the future?

I also tripped over this but I am not sure how to fix this one.  I
assume that it is a clang issue.

/usr/src/external/cddl/osnet/dist/uts/common/sys/list_impl.h:30:9: error:
      unknown pragma ignored [-Werror,-Wunknown-pragmas]
#pragma ident   "%Z%%M% %I%     %E% SMI"


Index: sys/arch/x86/x86/intr.c
===================================================================
RCS file: /cvsroot/src/sys/arch/x86/x86/intr.c,v
retrieving revision 1.119
diff -u -u -r1.119 intr.c
--- sys/arch/x86/x86/intr.c     13 Jan 2018 20:36:06 -0000      1.119
+++ sys/arch/x86/x86/intr.c     19 Jan 2018 12:22:03 -0000
@@ -245,8 +245,10 @@
 #endif /* XXX: XEN */
 #endif

+#if defined(INTRSTACKSIZE)
 static inline bool redzone_const_or_false(bool);
 static inline int redzone_const_or_zero(int);
+#endif

 static void intr_redistribute_xc_t(void *, void *);
 static void intr_redistribute_xc_s1(void *, void *);
@@ -1428,6 +1430,7 @@
 static const char *x86_ipi_names[X86_NIPI] = X86_IPI_NAMES;
 #endif

+#if defined(INTRSTACKSIZE)
 static inline bool
 redzone_const_or_false(bool x)
 {
@@ -1443,6 +1446,7 @@
 {
        return redzone_const_or_false(true) ? x : 0;
 }
+#endif

 /*
  * Initialize all handlers that aren't dynamically allocated, and exist

-- 
D'Arcy J.M. Cain <da...@netbsd.org>
http://www.NetBSD.org/ IM:da...@vex.net

Reply via email to