On Mon, Jun 05, 2023 at 02:43:46PM +0000, Niklas Cassel wrote: (snip) > I assume that the regression report is related to the review comment > that I gave, i.e. that ata_dump_status() is now called unconditionally: > > %stddev %change %stddev > pp.vprintk_emit._printk.ata_dump_status.ata_scsi_qc_complete.ata_qc_complete_multiple > 0.00 +22.4 22.41 ± 8% perf-profile.calltrace.cycles- Perhaps unconditionally is not exactly the right word here, but since the regular case is that a QC does not have flag ATA_QCFLAG_QUIET set, you probably understood what I meant :) Kind regards, Niklas
