Commit 60e4dac1 authored by Peter Oberparleiter's avatar Peter Oberparleiter Committed by Martin Schwidefsky

[S390] cio: fix repeat setting of cdev parent association

sch_create_and_recog_new_device() associates a parent subchannel
with its ccw device child even though this is already done by
the subsequently called io_subchannel_recog(). Also make sure
io_subchannel_recog() sets the association under lock.
Signed-off-by: default avatarPeter Oberparleiter <peter.oberparleiter@de.ibm.com>
Signed-off-by: default avatarMartin Schwidefsky <schwidefsky@de.ibm.com>
parent 48e4c385
...@@ -888,9 +888,6 @@ static void sch_create_and_recog_new_device(struct subchannel *sch) ...@@ -888,9 +888,6 @@ static void sch_create_and_recog_new_device(struct subchannel *sch)
css_sch_device_unregister(sch); css_sch_device_unregister(sch);
return; return;
} }
spin_lock_irq(sch->lock);
sch_set_cdev(sch, cdev);
spin_unlock_irq(sch->lock);
/* Start recognition for the new ccw device. */ /* Start recognition for the new ccw device. */
if (io_subchannel_recog(cdev, sch)) { if (io_subchannel_recog(cdev, sch)) {
spin_lock_irq(sch->lock); spin_lock_irq(sch->lock);
...@@ -1107,7 +1104,6 @@ io_subchannel_recog(struct ccw_device *cdev, struct subchannel *sch) ...@@ -1107,7 +1104,6 @@ io_subchannel_recog(struct ccw_device *cdev, struct subchannel *sch)
int rc; int rc;
struct ccw_device_private *priv; struct ccw_device_private *priv;
sch_set_cdev(sch, cdev);
cdev->ccwlock = sch->lock; cdev->ccwlock = sch->lock;
/* Init private data. */ /* Init private data. */
...@@ -1125,6 +1121,7 @@ io_subchannel_recog(struct ccw_device *cdev, struct subchannel *sch) ...@@ -1125,6 +1121,7 @@ io_subchannel_recog(struct ccw_device *cdev, struct subchannel *sch)
/* Start async. device sensing. */ /* Start async. device sensing. */
spin_lock_irq(sch->lock); spin_lock_irq(sch->lock);
sch_set_cdev(sch, cdev);
rc = ccw_device_recognition(cdev); rc = ccw_device_recognition(cdev);
spin_unlock_irq(sch->lock); spin_unlock_irq(sch->lock);
if (rc) { if (rc) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment