Page MenuHomeFreeBSD
Differential D31862

Add a test for https://reviews.freebsd.org/D31858 (PR 258310)
ClosedPublic
Actions

Authored by arichardson on Sep 6 2021, 3:20 PM.
Tags
None
Referenced Files
F102165630: D31862.diff
Fri, Nov 8, 10:29 AM
Unknown Object (File)
Tue, Oct 22, 8:32 PM
Unknown Object (File)
Sat, Oct 19, 9:30 AM
Unknown Object (File)
Wed, Oct 16, 3:15 AM
Unknown Object (File)
Sep 21 2024, 8:09 AM
Unknown Object (File)
Sep 17 2024, 1:34 PM
Unknown Object (File)
Sep 9 2024, 5:57 AM
Unknown Object (File)
Sep 5 2024, 2:51 PM
View All 51 Files
Subscribers
imp

Details

Reviewers
kib
imp
Commits
rGb0188ea20071: Add a test for https://reviews.freebsd.org/D31858 (PR 258310)
rGd7d962ead0b6: Add a test for https://reviews.freebsd.org/D31858 (PR 258310)
Summary

This test (based on https://github.com/jiixyj/epoll-shim/pull/32#issuecomment-891276654)
fails reproducibly on QEMU with KVM and -smp 2 prior to D31858 and passes
with the patch applied.

Test Plan

Passes with D31858 (and fails on smp qemu before)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41416
Build 38305: arc lint + arc unit

Event Timeline

arichardson created this revision.Sep 6 2021, 3:20 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 6 2021, 3:20 PM
arichardson requested review of this revision.Sep 6 2021, 3:20 PM
Harbormaster completed remote builds in B41407: Diff 94738.Sep 6 2021, 3:20 PM
arichardson mentioned this in D31858: kqueue: drain kqueue taskqueue if syscall tickled it.Sep 6 2021, 3:22 PM
imp accepted this revision.Sep 6 2021, 3:34 PM
This revision is now accepted and ready to land.Sep 6 2021, 3:34 PM
kib added inline comments.Sep 6 2021, 10:33 PM
tests/sys/kqueue/kqueue_peek_signal.c
32

This tag is definitely extra.

73

No, as I said, the issue is not with the signal visibility. It is the visibility of the nested kqueue activation.

arichardson updated this revision to Diff 94788.Sep 7 2021, 9:33 AM
arichardson marked an inline comment as done.

Correct comment and drop $FreeBSD$ tags

This revision now requires review to proceed.Sep 7 2021, 9:33 AM
Harbormaster completed remote builds in B41416: Diff 94788.Sep 7 2021, 9:33 AM
arichardson marked an inline comment as done.Sep 7 2021, 9:33 AM
kib accepted this revision.Sep 7 2021, 9:54 AM
This revision is now accepted and ready to land.Sep 7 2021, 9:54 AM
Closed by commit rGd7d962ead0b6: Add a test for https://reviews.freebsd.org/D31858 (PR 258310) (authored by arichardson). · Explain WhySep 13 2021, 9:33 AM
This revision was automatically updated to reflect the committed changes.
arichardson added a commit: rGd7d962ead0b6: Add a test for https://reviews.freebsd.org/D31858 (PR 258310).
arichardson added a commit: rGb0188ea20071: Add a test for https://reviews.freebsd.org/D31858 (PR 258310).May 7 2022, 3:57 PM

Revision Contents
Changeset List

PathSize
tests/
sys/
kqueue/
2 lines
106 lines

Diff 94788

View Options

tests/sys/kqueue/Makefile

Loading...
View Options

tests/sys/kqueue/kqueue_peek_signal.c

Loading...