Re: git: 783d3ff6d7fa - main - zfs: merge openzfs/zfs@39be46f43

From: Alexander Motin <mav_at_FreeBSD.org>
Date: Mon, 15 Apr 2024 13:57:15 UTC
On 12.04.2024 15:42, Bjoern A. Zeeb wrote:
> On Sat, 30 Mar 2024, Martin Matuska wrote:
>> The branch main has been updated by mm:
>>
>> URL: 
>> https://cgit.FreeBSD.org/src/commit/?id=783d3ff6d7fae619db8a7990b8a6387de0c677b5
>>
>> commit 783d3ff6d7fae619db8a7990b8a6387de0c677b5
>> Merge: e0388a906ca7 39be46f43f96
>> Author:     Martin Matuska <mm@FreeBSD.org>
>> AuthorDate: 2024-03-30 21:14:52 +0000
>> Commit:     Martin Matuska <mm@FreeBSD.org>
>> CommitDate: 2024-03-30 22:54:24 +0000
>>
>>    zfs: merge openzfs/zfs@39be46f43
>>
>>    Notable upstream pull request merges:
>>     #15509 b1e46f869 Add ashift validation when adding devices to a pool
>>     #15927 45e23abed Update resume token at object receive
>>     #15941 bf8f72359 BRT: Skip duplicate BRT prefetches
>>     #15950 8cd8ccca5 BRT: Skip getting length in brt_entry_lookup()
>>     #15951 80cc51629 ZAP: Massively switch to _by_dnode() interfaces
>>     #15954 2c01cae8b BRT: Change brt_pending_tree sorting order
>>     #15955 4616b96a6 BRT: Relax brt_pending_apply() locking
>>     #15959 5c4a4f82c zio: update ZIO type x stage documentation
>>     #15962 493fcce9b Provide macros for setting and getting blkptr 
>> birth times
>>     #15963 90ff73235 freebsd: fix missing headers in distribution tarball
>>     #15967 f68bde723 BRT: Make BRT block sizes configurable
>>     #15976 c28f94f32 ZAP: Some cleanups/micro-optimizations
>>     #15995 cfb96c772 vdev_disk: clean up spa/bdev mode conversion
>>     #16006 c0aab8b8f zvols: prevent overflow of minor device numbers
>>     #16007 a89d209bb BRT: Fix holes cloning
>>     #16008 c9d8f6c59 Fix option string, adding -e and fixing order
>>
>>    Obtained from:  OpenZFS
>>    OpenZFS commit: 39be46f43f96fb7420386d03751b01f5cb376d6b
> 
> I see this during builds:
> 
> /usr/src/src.git/sys/contrib/openzfs/module/zfs/arc.c:1362:1: warning: 
> function 'arc_buf_is_shared' is not needed and will not be emitted 
> [-Wunneeded-internal-declaration]

It is used only for assertions in debug builds now.  I've tried to wrap 
it with ifdef, but with that on non-debug builds compiler is not happy 
either.

-- 
Alexander Motin