Johannes Thumshirn
2017-10-04 10:06:37 UTC
According to fcoe_ctlr.c, the driver may sleep under a RCU lock,
fcoe_ctlr_disc_stop_locked (acquire the RCU lock)
fc_rport_logoff
mutex_lock --> may sleep
fcoe_ctlr_vn_disc
fc_rport_login
mutex_lock --> may sleep
fcoe_ctlr_vn_age
fc_rport_logoff
mutex_lock --> may sleep
These bugs are found by my static analysis tool and my code review.
Thanks,
Jia-Ju Bai
Thanks for the report, coincidentally I have a bug report for this as well (infcoe_ctlr_disc_stop_locked (acquire the RCU lock)
fc_rport_logoff
mutex_lock --> may sleep
fcoe_ctlr_vn_disc
fc_rport_login
mutex_lock --> may sleep
fcoe_ctlr_vn_age
fc_rport_logoff
mutex_lock --> may sleep
These bugs are found by my static analysis tool and my code review.
Thanks,
Jia-Ju Bai
a real setup not just static analysis) and gave the reporter a patch to test.
Once it's verified I'll post it here.
Thanks,
Johannes
--
Johannes Thumshirn Storage
***@suse.de +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
Johannes Thumshirn Storage
***@suse.de +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850