| Summary: | s390x: memcheck/tests/s390x/{cds,cs,csg} failures | ||
|---|---|---|---|
| Product: | [Developer tools] valgrind | Reporter: | Mark Wielaard <mark> |
| Component: | general | Assignee: | Andreas Arnez <arnez> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | arnez, jseward |
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: | Remove memcheck test cases for cs, cds, and csg | ||
|
Description
Mark Wielaard
2021-03-02 14:11:44 UTC
(In reply to Mark Wielaard from comment #0) > After the fix some issues (in these tests) can no longer be detected. It is > unclear if that can be fixed. So maybe these tests simply need to be deleted. Right. I don't see an easy way to preserve the behavior that is verified by these tests without re-introducing false positives. So I'll go ahead and remove them. Created attachment 137982 [details]
Remove memcheck test cases for cs, cds, and csg
This is the removal patch. Let me know if I'm missing something.
(In reply to Andreas Arnez from comment #2) > Created attachment 137982 [details] > Remove memcheck test cases for cs, cds, and csg Any objections/comments? Otherwise I'm planning to apply the patch early next week. (In reply to Andreas Arnez from comment #3) > (In reply to Andreas Arnez from comment #2) > > Created attachment 137982 [details] > > Remove memcheck test cases for cs, cds, and csg > Any objections/comments? Otherwise I'm planning to apply the patch early > next week. This seems reasonable to me. (In reply to Julian Seward from comment #4) > This seems reasonable to me. Thanks! Applied as d74a637206ef5532ccd2ccb2e31ee2762f184e60. |