mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2025-01-18 09:16:24 +07:00
86c4e7c350
aty128_set_suspend() is never called in atomic context. The call chains ending up at aty128_set_suspend() are: [1] aty128_set_suspend() <- aty128_pci_suspend() [2] aty128_set_suspend() <- aty128_do_resume() <- aty128_pci_resume() [3] aty128_set_suspend() <- aty128_do_resume() <- aty128_early_resume() aty128_pci_suspend() is set as ".suspend" in struct pci_driver. aty128_pci_resume() is set as ".resume" in struct pci_driver. aty128_early_resume() is never called. These functions are not called in atomic context. Despite never getting called from atomic context, aty128_set_suspend() calls mdelay() to busily wait. This is not necessary and can be replaced with msleep() to avoid busy waiting. This is found by a static analysis tool named DCNS written by myself. And I also manually check it. Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com> Cc: Paul Mackerras <paulus@samba.org> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> |
||
|---|---|---|
| .. | ||
| ati_ids.h | ||
| aty128fb.c | ||
| atyfb_base.c | ||
| atyfb.h | ||
| mach64_accel.c | ||
| mach64_ct.c | ||
| mach64_cursor.c | ||
| mach64_gx.c | ||
| Makefile | ||
| radeon_accel.c | ||
| radeon_backlight.c | ||
| radeon_base.c | ||
| radeon_i2c.c | ||
| radeon_monitor.c | ||
| radeon_pm.c | ||
| radeonfb.h | ||