On Thu, 5 Nov 2020 14:41:12 -0800 Andrii Nakryiko wrote: > On Thu, Nov 5, 2020 at 1:53 PM Jakub Kicinski <kuba@xxxxxxxxxx> wrote: > > On Thu, 5 Nov 2020 13:22:12 -0800 Andrii Nakryiko wrote: > > > Should we just drop check_verifier_log() checks? > > > > Drivers only print error messages when something goes wrong, so the > > messages are high priority. IIUC this change was just supposed to > > decrease verbosity, right? > > Seems like check_verifier_log() in test_offline.py is only called for > successful cases. This patch truncates parts of the verifier log that > correspond to successfully validated code paths, so that in case if > verification fails, only relevant parts are left. So for completely > successful verification the log will be almost empty, with only final > stats available. If you're saying the driver message would still be there if verification or translation failed that's perfectly fine, we can definitely adjust the test. But some check that driver message reporting is working is needed, don't just remove it. Sorry, don't have cycles to look closely :(
