svn commit: r284107 - head/sys/cddl/compat/opensolaris/sys

Andriy Gapon avg at FreeBSD.org
Sun Jun 7 08:54:26 UTC 2015


Author: avg
Date: Sun Jun  7 08:54:25 2015
New Revision: 284107
URL: https://svnweb.freebsd.org/changeset/base/284107

Log:
  compat nvpair.h: make sure that the names are mangled only for kernel
  
  Currently there is no good reason to mangle the userland API.
  The change was introduced in eac1d566b46edef765754203bef22c75c1699966,
  r279437.  Also see https://reviews.freebsd.org/D1881.
  
  I am still convinced that nv should not have introduced intentionally
  conflicting API.
  
  Discussed with:	rstone
  X-MFC with:	r279437
  Sponsored by:	ClusterHQ

Modified:
  head/sys/cddl/compat/opensolaris/sys/nvpair.h

Modified: head/sys/cddl/compat/opensolaris/sys/nvpair.h
==============================================================================
--- head/sys/cddl/compat/opensolaris/sys/nvpair.h	Sun Jun  7 06:30:25 2015	(r284106)
+++ head/sys/cddl/compat/opensolaris/sys/nvpair.h	Sun Jun  7 08:54:25 2015	(r284107)
@@ -29,6 +29,8 @@
 #ifndef _OPENSOLARIS_SYS_NVPAIR_H_
 #define _OPENSOLARIS_SYS_NVPAIR_H_
 
+#ifdef _KERNEL
+
 /*
  * Some of the symbols in the Illumos nvpair library conflict with symbols
  * provided by nv(9), so we use this preprocessor hack to avoid the conflict.
@@ -254,6 +256,8 @@
 #define nvpair_unpack illumos_nvpair_unpack
 #define nvpair_unpack_descriptor illumos_nvpair_unpack_descriptor
 
+#endif /* _KERNEL */
+
 #include_next <sys/nvpair.h>
 
 #endif


More information about the svn-src-all mailing list