Re: Bogus reports from coverage.postgresql.org

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Stephen Frost <sfrost(at)snowman(dot)net>, Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL WWW <pgsql-www(at)lists(dot)postgresql(dot)org>
Subject: Re: Bogus reports from coverage.postgresql.org
Date: 2018-03-14 19:52:40
Message-ID: 12595.1521057160@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

I wrote:
> I'm still a bit suspicious that we may have a process problem,
> but it's looking like there may be grounds for an lcov bug
> report.

Well, this is darn interesting. I got the Fedora lcov maintainer
to push an update absorbing the upstream "gcc 8" fixes. (Turned
out he'd already done that for rawhide, but forgot to push it into
the F28 branch.) And with that, and gcc 8.0.1, ... no bug. The
lines are marked "lineNoCov" with or without lcov_branch_coverage.

The commit message for said upstream patch is even more interesting:

Subject: [PATCH] geninfo: Add gcc 8 support

Fix errors and incorrect data when trying to collect coverage data
for programs compiled with gcc 8.

Covers the following gcov-related changes in gcc:

.gcov-file format:
- Line coverage data can appear multiple times for the same line
- Line coverage count can be suffixed by '*' to indicated unexecuted
basic blocks in that line

.gcno-file format:
- new header field 'support unexecuted blocks flag'
- new function record fields 'column number', 'ending line number',
and 'compiler-generated entity flag'

Perhaps I'm reading too much into this, but what it sounds like is
that gcc's coverage data output format was incapable of correctly
representing the situation before gcc 8, and thus that it's not
really lcov's fault that we get questionable output.

regards, tom lane

In response to

Responses

Browse pgsql-www by date

  From Date Subject
Next Message Alvaro Herrera 2018-03-14 20:02:40 Re: Bogus reports from coverage.postgresql.org
Previous Message Alvaro Herrera 2018-03-14 17:50:08 Re: Main page of wiki.postgresql.org