From nobody Mon Mar 06 21:01:33 2023 X-Original-To: dev-commits-ports-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4PVrbh3yQ5z3wNk6; Mon, 6 Mar 2023 21:01:36 +0000 (UTC) (envelope-from jrm@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4PVrbh3XGNz4ZkP; Mon, 6 Mar 2023 21:01:36 +0000 (UTC) (envelope-from jrm@freebsd.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1678136496; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=y63QBZ/GeTdPd2dH3We1r5Rf4bvnWdxbnlPzHSb6s+c=; b=vm1f4jkk+BlpkBt19A5WW65BXBjQL75tKTXKHRroa9XPoDn0XMbR/BtrRVbxwvJZC9hU8c k5kv9bbNyq0LT92BgAGhRJdhvLG7yqeSoTVetq7tHk5786EUoTITnd9utd4GlSv0SQgL60 K21Ylubv/yt1MGqdp6SXva67zCLpP04P5L2HkwFri1zMlb7jgLEtT6b/taPc8zhW0dUP8l oAoRmLlcS5UbUZvdKX8V8nqe3eMhXYHNhCjeNqfnkA+DyS7Sysmu0JRf3OQw8rWMn1/TaU KRjnlNFGq8ZzmrRRfOObuMjTg2lbX9tThFgdApoW0sJoia/4O4ae/gBpt8JITA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1678136496; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=y63QBZ/GeTdPd2dH3We1r5Rf4bvnWdxbnlPzHSb6s+c=; b=AojVTC/pz7S0xcV7eeya3TEkjL95IwPLWvzob0zCrRNx5piEATx6LVSlvW3O0uk4Xrj/UD XvyX7Q1hEwkRyj/8uwqq846ks1r3/TTbTwMH0/m4rLbfReOF9Iyk6eGewXyRGc+qvthNG0 mNxbFrJssTiAaEuNNOhL/B6kJ4F0sLjJMhdwlTwjGTN6i+bgzSXaK4/yu/B2WjOgwT7ppC ZpQ62vR447VES7XIlWPnxPoq4dzGr9W8YCOSoh4GxhzFR4G2IsIzbYoG4meE1JOGeG9rnV KzBX4kBu1J3ME3S3JsqN/Yq0m/qs4f6ght4OMYoIIpRhUGOToEAb5SClBNoQ/A== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1678136496; a=rsa-sha256; cv=none; b=wcD06KDQ5tDv3tNTUakS7eUYUqDF7D0YZYZ4fVGn4/qt0C6SC6rubNmp3QwaG1Xans71yJ w6CqnYExx2FnR2DNppTP0JdVJvVesZMo0IkG7l6wO5wm3YznVekOVsxLrcgk2GIt9zlNLx YMnLSADXDmexZF/rn4rGtcMW1Y0soUTYh8XPa9/qEt56XXmh9FyqQuecV/b70WuSSi5Rjy He+IvU8MzefuCX6JJNvLyNvGteq7uX5ImCXGYvj+9YkW1YlIztQWEfNIhMQEwNgMtFDeTr wgATVoNqW5aZpzO4N0Z7pbuSLqvzgvDgIxDgSb343LsgWLnYAYx8/7Ey1VOpKg== Received: from phe.ftfl.ca.ftfl.ca (drmons0544w-156-34-187-123.dhcp-dynamic.fibreop.ns.bellaliant.net [156.34.187.123]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) (Authenticated sender: jrm/mail) by smtp.freebsd.org (Postfix) with ESMTPSA id 4PVrbh0LjGzFX7; Mon, 6 Mar 2023 21:01:35 +0000 (UTC) (envelope-from jrm@freebsd.org) From: Joseph Mingrone To: Jung-uk Kim Cc: ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org Subject: Re: git: f4a869a6424a - main - emulators/virtualbox-ose-kmod: Fix build with LLVM 15 In-Reply-To: (Jung-uk Kim's message of "Mon, 6 Mar 2023 15:54:21 -0500") References: <202303061317.326DHUiq095917@gitrepo.freebsd.org> Date: Mon, 06 Mar 2023 17:01:33 -0400 Message-ID: <86lek94ms2.fsf@phe.ftfl.ca> User-Agent: Gnus/5.13 (Gnus v5.13) List-Id: Commit messages for all branches of the ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-all@freebsd.org X-BeenThere: dev-commits-ports-all@freebsd.org MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" X-ThisMailContainsUnwantedMimeParts: N --=-=-= Content-Type: text/plain On Mon, 2023-03-06 at 15:54, Jung-uk Kim wrote: > On 23. 3. 6., Joseph Mingrone wrote: >> The branch main has been updated by jrm: >> URL: https://cgit.FreeBSD.org/ports/commit/?id=f4a869a6424aa8bc32d50642762d9f294c7ce3da >> commit f4a869a6424aa8bc32d50642762d9f294c7ce3da >> Author: Joseph Mingrone >> AuthorDate: 2023-03-04 04:31:00 +0000 >> Commit: Joseph Mingrone >> CommitDate: 2023-03-06 13:15:34 +0000 >> emulators/virtualbox-ose-kmod: Fix build with LLVM 15 >> Starting with clang 15, the -Wint-conversion warning diagnostic for >> implicit int <-> pointer conversions defaults to an error. Use a cast to >> fix such an error here. >> See also: https://lists.freebsd.org/archives/freebsd-emulation/2023-February/000431.html >> https://lists.freebsd.org/archives/freebsd-emulation/2023-February/000460.html >> PR: 269721 >> Reported by: fbl@aoek.com, ohartmann@walstatt.org, rkoberman@gmail.com >> Differential Revision: https://reviews.freebsd.org/D38906 >> Sponsored by: The FreeBSD Foundation >> --- >> ...ch-src_VBox_HostDrivers_Support_freebsd_SUPDrv-freebsd.c | 13 +++++++------ >> 1 file changed, 7 insertions(+), 6 deletions(-) >> diff --git a/emulators/virtualbox-ose/files/patch-src_VBox_HostDrivers_Support_freebsd_SUPDrv-freebsd.c b/emulators/virtualbox-ose/files/patch-src_VBox_HostDrivers_Support_freebsd_SUPDrv-freebsd.c >> index 2e2c292d7ee8..b7e262dd4ba2 100644 >> --- a/emulators/virtualbox-ose/files/patch-src_VBox_HostDrivers_Support_freebsd_SUPDrv-freebsd.c >> +++ b/emulators/virtualbox-ose/files/patch-src_VBox_HostDrivers_Support_freebsd_SUPDrv-freebsd.c >> @@ -1,5 +1,5 @@ >> ---- src/VBox/HostDrivers/Support/freebsd/SUPDrv-freebsd.c.orig 2021-10-18 10:58:03.000000000 -0700 >> -+++ src/VBox/HostDrivers/Support/freebsd/SUPDrv-freebsd.c 2021-11-19 15:31:33.152269000 -0800 >> +--- src/VBox/HostDrivers/Support/freebsd/SUPDrv-freebsd.c.orig 2022-07-19 20:58:42 UTC >> ++++ src/VBox/HostDrivers/Support/freebsd/SUPDrv-freebsd.c >> @@ -44,8 +44,10 @@ >> #include >> #include >> @@ -26,7 +26,7 @@ >> #ifdef VBOX_WITH_HARDENING >> # define VBOXDRV_PERM 0600 >> #else >> -@@ -76,7 +85,9 @@ static d_open_t VBoxDrvFreeBSDOpenUsr; >> +@@ -76,7 +85,9 @@ static d_ioctl_t VBoxDrvFreeBSDIOCtl; >> static d_open_t VBoxDrvFreeBSDOpenSys; >> static void vboxdrvFreeBSDDtr(void *pvData); >> static d_ioctl_t VBoxDrvFreeBSDIOCtl; >> @@ -112,7 +112,7 @@ >> pvUser = *(void **)pvData; >> - int rc = copyin(pvUser, &Hdr, sizeof(Hdr)); >> - if (RT_UNLIKELY(rc)) >> -+ if (RT_FAILURE(RTR0MemUserCopyFrom(&Hdr, pvUser, sizeof(Hdr)))) >> ++ if (RT_FAILURE(RTR0MemUserCopyFrom(&Hdr, (uintptr_t)pvUser, sizeof(Hdr)))) >> { >> - OSDBGPRINT(("VBoxDrvFreeBSDIOCtlSlow: copyin(%p,Hdr,) -> %#x; ulCmd=%#lx\n", pvUser, rc, ulCmd)); >> - return rc; >> @@ -151,7 +151,7 @@ >> Log(("VBoxDrvFreeBSDIOCtlSlow: returns %d / %d ulCmd=%lx\n", 0, pHdr->rc, ulCmd)); >> -@@ -540,8 +595,7 @@ bool VBOXCALL supdrvOSGetForcedAsyncTscMode(PSUPDRVDE >> +@@ -540,8 +595,7 @@ bool VBOXCALL supdrvOSAreCpusOfflinedOnSuspend(void) >> bool VBOXCALL supdrvOSAreCpusOfflinedOnSuspend(void) >> { >> @@ -161,7 +161,7 @@ >> } >> -@@ -624,19 +678,43 @@ int VBOXCALL supdrvOSMsrProberModify(RTCPUID idCpu, >> +@@ -624,20 +678,44 @@ int VBOXCALL supdrvOSMsrProberModify(RTCPUID idCpu, >> #endif /* SUPDRV_WITH_MSR_PROBER */ >> @@ -206,3 +206,4 @@ >> + return fFlags; >> } >> + > This fix was incomplete. > /usr/ports/emulators/virtualbox-ose-kmod/work/VirtualBox-6.1.36/out/freebsd.amd64/release/bin/src/vboxdrv/freebsd/SUPDrv-freebsd.c:461:50: error: incompatible pointer to integer conversion passing 'void *' to parameter of type 'RTR3PTR' (aka 'unsigned long') > [-Wint-conversion] > if (RT_FAILURE(RTR0MemUserCopyFrom(pHdr, pvUser, Hdr.cbIn))) > ^~~~~~ > include/iprt/errcore.h:99:58: note: expanded from macro 'RT_FAILURE' > #define RT_FAILURE(rc) ( RT_UNLIKELY(!RT_SUCCESS_NP(rc)) ) > ^~ > include/iprt/errcore.h:80:38: note: expanded from macro 'RT_SUCCESS_NP' > # define RT_SUCCESS_NP(rc) ( (int)(rc) >= VINF_SUCCESS ) > ^~ > include/iprt/cdefs.h:1826:53: note: expanded from macro 'RT_UNLIKELY' > # define RT_UNLIKELY(expr) __builtin_expect(!!(expr), 0) > ^~~~ > include/iprt/mem.h:756:56: note: passing argument to parameter 'R3PtrSrc' here > RTR0DECL(int) RTR0MemUserCopyFrom(void *pvDst, RTR3PTR R3PtrSrc, size_t cb); > ^ > /usr/ports/emulators/virtualbox-ose-kmod/work/VirtualBox-6.1.36/out/freebsd.amd64/release/bin/src/vboxdrv/freebsd/SUPDrv-freebsd.c:494:46: error: incompatible pointer to integer conversion passing 'void *' to parameter of type 'RTR3PTR' (aka 'unsigned long') > [-Wint-conversion] > if (RT_FAILURE(RTR0MemUserCopyTo(pvUser, pHdr, cbOut))) > ^~~~~~ > include/iprt/errcore.h:99:58: note: expanded from macro 'RT_FAILURE' > #define RT_FAILURE(rc) ( RT_UNLIKELY(!RT_SUCCESS_NP(rc)) ) > ^~ > include/iprt/errcore.h:80:38: note: expanded from macro 'RT_SUCCESS_NP' > # define RT_SUCCESS_NP(rc) ( (int)(rc) >= VINF_SUCCESS ) > ^~ > include/iprt/cdefs.h:1826:53: note: expanded from macro 'RT_UNLIKELY' > # define RT_UNLIKELY(expr) __builtin_expect(!!(expr), 0) > ^~~~ > include/iprt/mem.h:768:41: note: passing argument to parameter 'R3PtrDst' here > RTR0DECL(int) RTR0MemUserCopyTo(RTR3PTR R3PtrDst, void const *pvSrc, size_t cb); > ^ > 2 errors generated. > I'll commit a fix. > Jung-uk Kim Sorry. I didn't see those errors when testing and `poudriere testport` built fine for me. http://pkg.ftfl.ca/data/14amd64-default/2023-03-06_16h57m27s/logs/virtualbox-ose-kmod-6.1.36.log --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQKkBAEBCgCOFiEEVbCTpybDiFVxIrrVNqQMg7DW754FAmQGVK5fFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDU1 QjA5M0E3MjZDMzg4NTU3MTIyQkFENTM2QTQwQzgzQjBENkVGOUUQHGpybUBmcmVl YnNkLm9yZwAKCRA2pAyDsNbvnkG5D/sGyRFwK+92UW5dsf6FujdyUpiHDZ8c6F1J ++19QFMlcqXwM+uMI3BIfCHEZ28rK+A2VhLgZwhbeFxkVvgEYyGCSt02mR00crdl xfFkOTL8l/4Jsxp73S3iVw6eIU0Xtcj+CQ6ZV0zpUMizlA1t7jpcxrKEmgYjuCPe NIdlGbGU+Z1gi2E4UrsM7X3kv55dNWJLx47xTHJ04tyEQoAX+G841Qg/I1mFCUcG 8hI21+V+Y/p5zzXbrfAdbVol2RqIKJuFcl1kV3mDUGVnrPBE3L9YA0i61KLa+u1g Hqx3vsM1KX5gGy+YtW5CXVAyu3O68A2+bKrz7ZV1a5iZgV73Mv9dUF44UgmvT+ve zd1WavzjOzGp2IMSUjpBB0oPphZb9qjcOLSvBPHeF9K/PIFg/o45SLWFCCNHNw43 OV506/HeAdoiLCNJizKHnh7pyk/pWK4OZI4UY87A/Em+QKrkkCtxTtKwAZKD6Q/k OYO9a/Y59QtzXroHLrocjcNovRjGDC3TBflpca/Xeg5Z4N0/KgVljACSTOWOaWGU XUY8lStktN+n8HVujA5mWqel5B50BuR2m5QvTPztXjCw/j68JY1hgYZNSxPz4wXp BmrZDQtxytiHxVPAnkeJjyHpAmH9ZdwauZgLN588/4rTnYb4MRHQx6VcUNStrPNd Le5nPy/OmQ== =KjO0 -----END PGP SIGNATURE----- --=-=-=--