]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
MFC r343748:
authordim <dim@FreeBSD.org>
Thu, 7 Feb 2019 06:55:26 +0000 (06:55 +0000)
committerdim <dim@FreeBSD.org>
Thu, 7 Feb 2019 06:55:26 +0000 (06:55 +0000)
commitaf20fb97392a780d011209d29849b761bfd7ccff
tree7b967cbbe56b4a0bab179d41a47943e39a0449f4
parent3d1f39178fe872cc1d2cfec7ecb6481be8a3c3fd
MFC r343748:

Use NLDT to get number of LDTs on i386

Compiling a GENERIC kernel for i386 with clang 8.0 results in the
following warning:

/usr/src/sys/i386/i386/sys_machdep.c:542:40: error: 'sizeof ((ldt))' will return the size of the pointer, not the array itself [-Werror,-Wsizeof-pointer-div]
        nldt = pldt != NULL ? pldt->ldt_len : nitems(ldt);
                                              ^~~~~~~~~~~
/usr/src/sys/sys/param.h:299:32: note: expanded from macro 'nitems'
#define nitems(x)       (sizeof((x)) / sizeof((x)[0]))
                         ~~~~~~~~~~~ ^

Indeed, 'ldt' is declared as 'union descriptor *', so nitems() is not
the right way to determine the number of LDTs.  Instead, the NLDT define
from sys/x86/include/segments.h should be used.

Reviewed by: kib
Differential Revision: https://reviews.freebsd.org/D19074
sys/i386/i386/sys_machdep.c