diff options
author | Damien Le Moal <[email protected]> | 2021-08-16 10:44:52 +0900 |
---|---|---|
committer | Jens Axboe <[email protected]> | 2021-08-18 07:19:39 -0600 |
commit | fc5c8aa7bc4977205e0ceb93425075f8a8f49501 (patch) | |
tree | 5d5564c7fc3af911c5aeca20a9eca88c65b03bc1 /tools/perf/util/trace-event-scripting.c | |
parent | 2360fa1812cd77e1de13d3cca789fbd23462b651 (diff) |
libata: fix ata_read_log_page() warning
Support for the READ LOG PAGE DMA EXT command is indicated by words 119
and 120 of a device identify data. This is tested in
ata_read_log_page() with ata_id_has_read_log_dma_ext() and the
READ LOG PAGE DMA command used if the device reports supports for it.
However, some devices lie about this support and using the DMA version
of the command fails, generating the warning message "READ LOG DMA EXT
failed, trying PIO". Since READ LOG PAGE DMA EXT is an optional command,
this warning is not at all important but may be scary for the user.
Change ata_read_log_page() to suppres this warning and to print an
error message if both DMA and PIO attempts failed.
With this change, there is no need to print again an error message when
ata_read_log_page() returns an error. So simplify the users of this
function.
Signed-off-by: Damien Le Moal <[email protected]>
Reviewed-by: Hannes Reinecke <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jens Axboe <[email protected]>
Diffstat (limited to 'tools/perf/util/trace-event-scripting.c')
0 files changed, 0 insertions, 0 deletions