aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2022-01-05ata: sata_sx4: Drop pointless VPRINTK() calls and convert the remaining onesHannes Reinecke1-71/+34
Drop pointless VPRINTK() calls for setting up SG tables and convert the remaining calls to structured logging. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_sil: Drop pointless VPRINTK() callsHannes Reinecke1-1/+0
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_fsl: convert VPRINTK() calls to ata_port_dbg()Hannes Reinecke1-47/+33
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_nv: drop pointless VPRINTK() calls and convert remaining onesHannes Reinecke1-24/+14
Quite some information from the VPRINTK() is already covered by tracepoints, so remove the pointless calls and convert the remaining ones to structured logging. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_mv: Drop pointless VPRINTK() call and convert the remaining oneHannes Reinecke1-6/+1
Drop pointless VPRINTK() call and convert the remaining one to dev_dbg(). Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_inic162x: Drop pointless VPRINTK() callsHannes Reinecke1-2/+0
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_rcar: Drop pointless VPRINTK() callsHannes Reinecke1-16/+1
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_qstor: Drop pointless VPRINTK() callsHannes Reinecke1-11/+0
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_promise: Drop pointless VPRINTK() calls and convert the remaining onesHannes Reinecke1-23/+8
Drop pointless VPRINTK() calls for entering and existing interrupt routines and convert the remaining calls to ata_port_dbg(). Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: pata_via: Drop pointless VPRINTK() callsHannes Reinecke1-12/+0
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: pata_octeon_cf: Drop pointless VPRINTK() calls and convert the ↵Hannes Reinecke1-21/+1
remaining one Drop pointless VPRINTK() calls and convert the remaining calls to the existing bmdma tracepoint. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: pdc_adma: Drop pointless VPRINTK() calls and remove disabled NCQ debuggingHannes Reinecke1-29/+2
Drop pointless VPRINTK() calls for entering routines and setting up sg tables. And while we're at it, remove the disabled debugging messages. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: ahci: Drop pointless VPRINTK() calls and convert the remaining onesHannes Reinecke3-21/+5
Drop pointless VPRINTK() calls for entering and existing interrupt routines and convert the remaining calls to dev_dbg(). Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata: remove pointless VPRINTK() callsHannes Reinecke5-52/+0
Most of the information is already covered by tracepoints (if not downright pointless), so remove the VPRINTK() calls. And while we're at it, remove ata_scsi_dump_cdb(), too, as this information can be retrieved from scsi tracing. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: pata_pdc2027x: Replace PDPRINTK() with standard ata loggingHannes Reinecke1-43/+28
Use standard ata logging macros instead of the hand-crafted PDPRINTK and remove duplicate logging messages. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_qstor: replace DPRINTK() with dev_dbg()Hannes Reinecke1-2/+2
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_rcar: replace DPRINTK() with ata_port_dbg()Hannes Reinecke1-1/+1
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_fsl: move DPRINTK to ata debuggingHannes Reinecke1-44/+39
Replace all DPRINTK calls with the ata_XXX_dbg functions. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: pdc_adma: Remove DPRINTK callHannes Reinecke1-2/+0
The DPRINTK call doesn't print information which isn't already covered by tracepoints later on. Remove it. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: pata_octeon_cf: remove DPRINTK() macro in interrupt contextHannes Reinecke1-2/+0
There is only so much information to be glanced when the interrupt routine is called and exited, so remove these DPRINTK() calls. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_mv: replace DPRINTK with dynamic debuggingHannes Reinecke1-38/+38
Move the DPRINTK calls over to dynamic debugging. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: sata_mv: kill 'port' argument in mv_dump_all_regs()Hannes Reinecke1-13/+6
Always '-1', so drop it and simplify the function. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata: move DPRINTK to ata debuggingHannes Reinecke1-9/+6
Replace all DPRINTK calls with ata_dev_dbg(). Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata: revamp ata_get_cmd_descript()Hannes Reinecke3-17/+11
Rename ata_get_cmd_descrip() to ata_get_cmd_name() and simplify it to return "unknown" instead of NULL. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata: move ata_{port,link,dev}_dbg to standard pr_XXX() macrosHannes Reinecke4-117/+67
Use standard pr_{debug,info,notice,warn,err} macros instead of the hand-crafted printk helpers. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata: add tracepoints for ATA error handlingHannes Reinecke4-30/+67
Add tracepoints for ATA error handling. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata-scsi: drop DPRINTK calls for cdb translationHannes Reinecke1-19/+1
Drop DPRINTK calls for cdb translation as they are already covered by other traces, and also drop the DPRINTK calls in ata_scsi_hotplug(). Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata-sff: tracepoints for HSM state machineHannes Reinecke3-13/+161
Add tracepoints for the HSM state machine and drop DPRINTK calls Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-05ata: libata: tracepoints for bus-master DMAHannes Reinecke12-97/+201
Add tracepoints for bus-master DMA and taskfile related functions. That allows us to drop the relevant DPRINTK() calls. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: add qc_prep tracepointHannes Reinecke2-1/+10
Convert the existing ata_qc_issue() tracepoint into a template, and add tracepoints for ata_qc_prep() and ata_qc_issue() based on that template. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: add reset tracepointsHannes Reinecke11-57/+115
To follow the flow of control we should be using tracepoints, as they will tie in with the actual I/O flow and deliver a better overview about what it happening. This patch adds tracepoints for hard reset, soft reset, and postreset and adds them in the libata-eh control flow. With that we can drop the reset DPRINTK calls in the various drivers. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: sanitize ATA_HORKAGE_DUMP_IDHannes Reinecke1-2/+2
With moving ata_dev_dbg() over to dynamic debugging ATA_HORKAGE_DUMP_ID will now print out the raw IDENTIFY data without a header unless explicitly enable via dyndebug. So move the logging level up to INFO and have the header printed always. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: move ata_dump_id() to dynamic debuggingHannes Reinecke1-28/+10
Use ata_dev_dbg() to print out the information in ata_dump_id() and remove the ata_msg_probe() conditional. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: Add ata_port_classify() helperHannes Reinecke8-21/+42
Add an ata_port_classify() helper to print out the results from the device classification and remove the debugging statements from ata_dev_classify(). Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: whitespace cleanupHannes Reinecke2-10/+10
Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: remove pointless debugging messagesHannes Reinecke7-29/+0
Debugging messages in pci init functions or sg setup are pretty much pointless, as the workflow pretty much decides what happened. So drop them. Signed-off-by: Hannes Reinecke <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata: use min() to make code cleanerChangcheng Deng1-8/+2
Use min() in order to make code cleaner. Reported-by: Zeal Robot <[email protected]> Signed-off-by: Changcheng Deng <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libahci_platform: Get rid of dup message when IRQ can't be retrievedAndy Shevchenko1-4/+1
platform_get_irq() will print a message when it fails. No need to repeat this. Signed-off-by: Andy Shevchenko <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libahci_platform: Remove bogus 32-bit DMA mask attemptAndy Shevchenko1-7/+2
If 64-bit mask attempt fails, the 32-bit will fail by the very same reason. Don't even try the latter. It's a continuation of the changes that contains, e.g. dcc02c19cc06 ("sata_sil24: use dma_set_mask_and_coherent"). Signed-off-by: Andy Shevchenko <[email protected]> Reviewed-by: Hans de Goede <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: sata_dwc_460ex: Remove unused forward declarationAndy Shevchenko1-1/+0
sata_dwc_port_stop() is not used before being defined, remove redundant forward declaration. Signed-off-by: Andy Shevchenko <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: sata_dwc_460ex: Use temporary variable for struct deviceAndy Shevchenko1-18/+19
Use temporary variable for struct device to make code neater. Signed-off-by: Andy Shevchenko <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: sata_dwc_460ex: Use devm_platform_*ioremap_resource() APIsAndy Shevchenko1-5/+2
Use devm_platform_get_and_ioremap_resource() and devm_platform_ioremap_resource() APIs instead of their open coded analogues. Signed-off-by: Andy Shevchenko <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: sata_fsl: use sysfs_emit()Damien Le Moal1-3/+3
Use sysfs_emit() instead of sprintf() in fsl_sata_intr_coalescing_show() and fsl_sata_rx_watermark_show(). Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: ahci: use sysfs_emit()Damien Le Moal1-1/+1
Use sysfs_emit() instead of sprintf in remapped_nvme_show(). Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata-scsi: use sysfs_emit()Damien Le Moal1-1/+1
Use sysfs_emit() instead of snprintf() in ata_scsi_park_show(). Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: libata-sata: use sysfs_emit()Damien Le Moal1-2/+2
Use sysfs_emit() instead of snprintf() in sysfs attibute show() functions. Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: sata_fsl: Use struct_group() for memcpy() regionKees Cook1-4/+6
In preparation for FORTIFY_SOURCE performing compile-time and run-time field bounds checking for memcpy(), memmove(), and memset(), avoid intentionally writing across neighboring fields. Use struct_group() in struct command_desc around members acmd and fill, so they can be referenced together. This will allow memset(), memcpy(), and sizeof() to more easily reason about sizes, improve readability, and avoid future warnings about writing beyond the end of acmd: In function 'fortify_memset_chk', inlined from 'sata_fsl_qc_prep' at drivers/ata/sata_fsl.c:534:3: ./include/linux/fortify-string.h:199:4: warning: call to '__write_overflow_field' declared with attribute warning: detected write beyond size of field (1st parameter); maybe use struct_group()? [-Wattribute-warning] 199 | __write_overflow_field(); | ^~~~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Kees Cook <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-04ata: pata_ali: no need to initialise statics to 0Jason Wang1-1/+1
Static variables do not need to be initialized to 0. Signed-off-by: Jason Wang <[email protected]> Signed-off-by: Damien Le Moal <[email protected]>
2022-01-02Linux 5.16-rc8Linus Torvalds1-1/+1
2022-01-02Merge tag 'perf-tools-fixes-for-v5.16-2022-01-02' of ↵Linus Torvalds6-21/+43
git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux Pull perf tools fixes from Arnaldo Carvalho de Melo: - Fix TUI exit screen refresh race condition in 'perf top'. - Fix parsing of Intel PT VM time correlation arguments. - Honour CPU filtering command line request of a script's switch events in 'perf script'. - Fix printing of switch events in Intel PT python script. - Fix duplicate alias events list printing in 'perf list', noticed on heterogeneous arm64 systems. - Fix return value of ids__new(), users expect NULL for failure, not ERR_PTR(-ENOMEM). * tag 'perf-tools-fixes-for-v5.16-2022-01-02' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux: perf top: Fix TUI exit screen refresh race condition perf pmu: Fix alias events list perf scripts python: intel-pt-events.py: Fix printing of switch events perf script: Fix CPU filtering of a script's switch events perf intel-pt: Fix parsing of VM time correlation arguments perf expr: Fix return value of ids__new()
Powered by cgit, Gruvboxed by Blaster4385.