Page MenuHomeFreeBSD
Differential D54675

libexecinfo: tests: Expect failure on aarch64
ClosedPublic
Actions

Authored by jlduran on Mon, Jan 12, 8:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 20, 2:27 AM
Unknown Object (File)
Mon, Jan 19, 2:06 PM
Unknown Object (File)
Sun, Jan 18, 12:34 PM
Unknown Object (File)
Sat, Jan 17, 8:10 AM
Unknown Object (File)
Sat, Jan 17, 5:03 AM
Unknown Object (File)
Thu, Jan 15, 6:30 AM
Unknown Object (File)
Thu, Jan 15, 1:06 AM
Unknown Object (File)
Thu, Jan 15, 12:32 AM
View All 11 Files
Subscribers
arichardson
imp

Details

Reviewers
dchagin
emaste
siva
Group Reviewers
tests
Commits
rG8e49c6c84cae: libexecinfo: tests: Expect failure on aarch64
rG9f90e5a3ce9d: libexecinfo: tests: Expect failure on aarch64
rGfef84fd8ae84: libexecinfo: tests: Expect failure on aarch64
Summary

Add a guard that expects a failure of the test on aarch64.

Fixes: df1ea5887326 ("tests: Test libexecinfo backtrace call througth signal trampoline")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 69835
Build 66718: arc lint + arc unit

Event Timeline

jlduran created this revision.Mon, Jan 12, 8:57 PM
Herald added a subscriber: imp. · View Herald TranscriptMon, Jan 12, 8:57 PM
jlduran requested review of this revision.Mon, Jan 12, 8:57 PM
jlduran added reviewers: tests, dchagin, emaste.Mon, Jan 12, 8:58 PM
jlduran added a comment.Mon, Jan 12, 9:02 PM

Reference: https://ci.freebsd.org/view/Test/job/FreeBSD-main-aarch64-test/1040/testReport/junit/lib.libexecinfo/sigtramp_test/test_backtrace_sigtramp/

jlduran updated this revision to Diff 169571.Tue, Jan 13, 5:52 PM
jlduran retitled this revision from libexecinfo: tests: Skip on aarch64 to libexecinfo: tests: Expect failure on aarch64.
jlduran edited the summary of this revision. (Show Details)
  • Do not skip, but expect fail
  • Terse message
Harbormaster completed remote builds in B69835: Diff 169541.Wed, Jan 14, 12:50 AM
jlduran added a reviewer: siva.Wed, Jan 14, 2:20 PM
emaste accepted this revision.Wed, Jan 14, 2:48 PM
emaste added a subscriber: arichardson.

I think it's worth adding a comment as well -- https://reviews.llvm.org is deprecated and this review is never going to be updated or completed. I'm not sure if @dchagin or @arichardson have a rebased version of it but someone will need to pick it up.

But adding the xfail is certainly good.

This revision is now accepted and ready to land.Wed, Jan 14, 2:48 PM
jlduran added a comment.Wed, Jan 14, 4:48 PM

I'll add the comment upon commit:

/* https://reviews.llvm.org is deprecated and this review is never going to be updated or completed */

Thanks.

Closed by commit rGfef84fd8ae84: libexecinfo: tests: Expect failure on aarch64 (authored by jlduran). · Explain WhyWed, Jan 14, 5:08 PM
This revision was automatically updated to reflect the committed changes.
jlduran added a commit: rGfef84fd8ae84: libexecinfo: tests: Expect failure on aarch64.
jlduran added a commit: rG9f90e5a3ce9d: libexecinfo: tests: Expect failure on aarch64.Wed, Jan 21, 1:53 AM
jlduran added a commit: rG8e49c6c84cae: libexecinfo: tests: Expect failure on aarch64.

Revision Contents
Changeset List

PathSize
lib/
libexecinfo/
tests/
4 lines

Diff 169541

View Options

lib/libexecinfo/tests/sigtramp_test.c

Loading...