💬 dergoegge commented on pull request "rpc, bugfix: Enforce maximum value for setmocktime":
(https://github.com/bitcoin/bitcoin/pull/29869#issuecomment-2056268268)
Fixed `rpc_uptime` functional test

⚠️ maflcko opened an issue: "Intermittent issue in feature_proxy.py AssertionError: not(bytearray(b'node.noumenon') == b'fc00:1:2:3:4:5:6:7')"
(https://github.com/bitcoin/bitcoin/issues/29871)
https://drahtbot.space/temp_scratch/feature_proxy_179.tar.zstd

```
test 2024-04-13T16:24:13.025000Z TestFramework (DEBUG): Test: outgoing CJDNS connection through node 4 for address [fc00:1:2:3:4:5:6:7]:8888
node4 2024-04-13T16:24:13.089818Z [http] [httpserver.cpp:306] [http_request_cb] [http] Received a POST request for / from 127.0.0.1:53136
node0 2024-04-13T16:24:13.090252Z [net] [net.cpp:2124] [SocketHandlerConnected] [net] socket recv error for peer=5: Connection reset by peer (
...

👍 dergoegge approved a pull request: "rpc: Reword SighashFromStr error message"
(https://github.com/bitcoin/bitcoin/pull/29870#pullrequestreview-2000394515)
ACK fa6ab0d020d0b1492203f7eb2ccb8051812de086

👍 stickies-v approved a pull request: "[27.x] More backports and finalize"
(https://github.com/bitcoin/bitcoin/pull/29780#pullrequestreview-2000395203)
ACK 910e3e8728b258a38d38f8f9ddf6b23406e8d5ce - no changes except addressing nits

💬 maflcko commented on pull request "rpc, bugfix: Enforce maximum value for setmocktime":
(https://github.com/bitcoin/bitcoin/pull/29869#issuecomment-2056303807)
re-ACK c2e0489b7125cceaeef355fc274dd8988822fff4

✅ fanquake closed an issue: "fuzz: minisketch: Undefined-shift in std::__1::vector<Field<unsigned int, 32, 141u, RecLinTrans<unsigned int, 6, 6, 5"
(https://github.com/bitcoin/bitcoin/issues/29799)

🚀 fanquake merged a pull request: "minisketch: update subtree to 3472e2f5ec75ace39ce9243af6b3fee233a67492"
(https://github.com/bitcoin/bitcoin/pull/29823)

👍 fanquake approved a pull request: "ci: Bump s390x to ubuntu:24.04"
(https://github.com/bitcoin/bitcoin/pull/29856#pullrequestreview-2000409530)
ACK fadf7e90dc10e212e59bb294209bab2e73c47800 - finished running after ~5 hours.

🚀 fanquake merged a pull request: "ci: Bump s390x to ubuntu:24.04"
(https://github.com/bitcoin/bitcoin/pull/29856)

💬 Sjors commented on pull request "[DO NOT MERGE] testnet4 including PoW difficulty adjustment fix":
(https://github.com/bitcoin/bitcoin/pull/29775#issuecomment-2056345622)
I spun up `seed.testnet4.bitcoin.sprovoost.nl` and set it to use the magic bytes and port number.

💬 glozow commented on pull request "test: p2p: add test for rejected tx request logic (`m_recent_rejects` filter)":
(https://github.com/bitcoin/bitcoin/pull/29827#discussion_r1565368377)
Use `node.bumpmocktime(MAX_GETDATA_INBOUND_WAIT)` instead?

💬 glozow commented on pull request "test: p2p: add test for rejected tx request logic (`m_recent_rejects` filter)":
(https://github.com/bitcoin/bitcoin/pull/29827#discussion_r1565443465)
Nice catch! I wonder if a more wholistic way to avoid unintentionally creating tx packages is to gather all the (confirmed only) UTXOs at the beginning? e.g. [this](https://github.com/glozow/bitcoin/commits/2024-04-fill-mempool-29827/), but feel free to ignore

🤔 glozow reviewed a pull request: "test: p2p: add test for rejected tx request logic (`m_recent_rejects` filter)"
(https://github.com/bitcoin/bitcoin/pull/29827#pullrequestreview-2000325181)
CI failure looks related to the lines added here (I haven't debugged though)

💬 glozow commented on pull request "test: p2p: add test for rejected tx request logic (`m_recent_rejects` filter)":
(https://github.com/bitcoin/bitcoin/pull/29827#discussion_r1565446681)
`peer.wait_for_getdata()` ?

💬 maflcko commented on pull request "build: Fix false positive `CHECK_ATOMIC` test":
(https://github.com/bitcoin/bitcoin/pull/29859#issuecomment-2056362145)
For reference, on a fresh Ubuntu 24.04 Noble (which should be identical to the current CI config), it fails:

```
apt update && apt install clang-18 g++-multilib -y
clang++-18 -std=c++20 test.cpp -m32

/usr/bin/ld: /tmp/test-9da16c.o: in function `std::__atomic_float<double>::store(double, std::memory_order)':
test.cpp:(.text._ZNSt14__atomic_floatIdE5storeEdSt12memory_order[_ZNSt14__atomic_floatIdE5storeEdSt12memory_order]+0x5a): undefined reference to `__atomic_store'
/usr/bin/ld: /tmp
...

💬 laanwj commented on pull request "guix: replace GCC unaligned VMOV patch with binutils patch":
(https://github.com/bitcoin/bitcoin/pull/29846#issuecomment-2056372622)
> ```
> objdump -D bin/*.exe | rg "vmova|vmovdqa|vmovaps|vmovapd|vmovdqa64|vmovdqa32"
> 141b8be20: c5 f8 28 1a vmovaps (%rdx), %xmm3
> 1420564b3: c5 79 29 36 vmovapd %xmm14, (%rsi)
> 1403060f3: c5 79 29 36 vmovapd %xmm14, (%rsi)
> 140792b13: c5 79 29 36 vmovapd %xmm14, (%rsi)
> 140cb0693: c5 79 29 36 vmovapd %xmm14, (%rsi)
> 1415ea0f3: c5 79 29 36 vmovapd %xmm14, (%rsi)>
```
i've re-r
...

✅ maflcko closed an issue: "Errors in block header at FlatFilePos"
(https://github.com/bitcoin/bitcoin/issues/29861)

👍 brunoerg approved a pull request: "rpc, bugfix: Enforce maximum value for setmocktime"
(https://github.com/bitcoin/bitcoin/pull/29869#pullrequestreview-2000550197)
crACK c2e0489b7125cceaeef355fc274dd8988822fff4

💬 glozow commented on pull request "p2p: opportunistically accept 1-parent-1-child packages":
(https://github.com/bitcoin/bitcoin/pull/28970#issuecomment-2056422724)
Update since my last comment https://github.com/bitcoin/bitcoin/pull/28970#issuecomment-2025812846
Running for 20 days:
- My node made 5218 package evaluation attempts (x2 = 10,436 transactions). That's ~260 attempts per day.
- Of those attempts, 2260 were successful (x2 = 4520 transactions). That's ~43% acceptance rate.
- Some were repeats. I have 4202 unique txids accepted through package evaluation. I don't know if they were repeated packages or not.
- Of the accepted transactions, 3232
...

🤔 glozow reviewed a pull request: "rpc, bugfix: Enforce maximum value for setmocktime"
(https://github.com/bitcoin/bitcoin/pull/29869#pullrequestreview-2000639795)
ACK c2e0489b7125cceaeef355fc274dd8988822fff4