Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 26 Sep 2022 10:54:49 -0300
From:      Renato Botelho <garga@FreeBSD.org>
To:        John Baldwin <jhb@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: 6bddde307e21 - main - Bump __FreeBSD_version for pmap_unmap*() and kmem_*() API changes.
Message-ID:  <9b11ed8e-1e32-f290-8a5d-997d46e58ac4@FreeBSD.org>
In-Reply-To: <202209222210.28MMApTd093914@gitrepo.freebsd.org>
References:  <202209222210.28MMApTd093914@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 22/09/22 19:10, John Baldwin wrote:
> The branch main has been updated by jhb:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=6bddde307e21eba297ac3f3e534b4cf3be81dfe2
> 
> commit 6bddde307e21eba297ac3f3e534b4cf3be81dfe2
> Author:     John Baldwin <jhb@FreeBSD.org>
> AuthorDate: 2022-09-22 22:10:03 +0000
> Commit:     John Baldwin <jhb@FreeBSD.org>
> CommitDate: 2022-09-22 22:10:03 +0000
> 
>      Bump __FreeBSD_version for pmap_unmap*() and kmem_*() API changes.

Hello John,

emulators/open-vm-tools is broken after these changes:

--- vmmemctl ---
os.c:445:14: error: incompatible integer to pointer conversion passing 
'vm_offset_t' (aka 'unsigned int') to parameter of type 'void *' 
[-Werror,-Wint-conversion]
    KMEM_FREE((vm_offset_t)p->bitmap, p->size);
              ^~~~~~~~~~~~~~~~~~~~~~
os.c:143:46: note: expanded from macro 'KMEM_FREE'
    #define KMEM_FREE(offset, size) kmem_free(offset, size)
                                              ^~~~~~
/usr/src/sys/vm/vm_extern.h:73:22: note: passing argument to parameter 
'addr' here
void kmem_free(void *addr, vm_size_t size);
                      ^
1 error generated.

I can take a look at it during the week, any advice is appreciated.

-- 
Renato Botelho




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?9b11ed8e-1e32-f290-8a5d-997d46e58ac4>