Page MenuHomeFreeBSD

kevent32: fix the split of data into data1/data2
ClosedPublic

Authored by kevans on Apr 21 2020, 1:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 23, 1:07 PM
Unknown Object (File)
Sat, Feb 22, 3:04 PM
Unknown Object (File)
Feb 21 2025, 7:05 PM
Unknown Object (File)
Feb 21 2025, 1:52 PM
Unknown Object (File)
Feb 21 2025, 7:05 AM
Unknown Object (File)
Feb 12 2025, 1:51 AM
Unknown Object (File)
Jan 21 2025, 4:18 AM
Unknown Object (File)
Jan 17 2025, 12:35 PM
Subscribers

Details

Summary

The current situation results in intermittent breakage if data gets split up with the sign bit set on the data1 half of it, as PAIR32TO64 will then: data1 | (data2 << 32) -> resulting in data1 getting sign-extended when it's implicitly widened and clobbering the result. AFAICT, there's no compelling reason for these to be signed.

This was most exposed by flakiness in the kqueue timer tests under compat32 after the ABSTIME test got switched over to using a better clock and microseconds.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable