Commit 2099810f authored by Dave Airlie's avatar Dave Airlie

drm/radeon: enable pci bus mastering after card is initialised (v2)

This closes a race seen with kexec where we enable PCI bus mastering
but the card has been reinitialised fully yet.

This was previously fixed by a patch from Jerome, but this should
close the race completely.

v2: add SI support as suggested by Alex.
Reported-and-tested-by: default avatarMarkus Trippelsdorf <markus@trippelsdorf.de>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
parent 00341028
...@@ -1180,6 +1180,10 @@ int r100_cp_init(struct radeon_device *rdev, unsigned ring_size) ...@@ -1180,6 +1180,10 @@ int r100_cp_init(struct radeon_device *rdev, unsigned ring_size)
WREG32(RADEON_CP_RB_WPTR_DELAY, 0); WREG32(RADEON_CP_RB_WPTR_DELAY, 0);
WREG32(RADEON_CP_CSQ_MODE, 0x00004D4D); WREG32(RADEON_CP_CSQ_MODE, 0x00004D4D);
WREG32(RADEON_CP_CSQ_CNTL, RADEON_CSQ_PRIBM_INDBM); WREG32(RADEON_CP_CSQ_CNTL, RADEON_CSQ_PRIBM_INDBM);
/* at this point everything should be setup correctly to enable master */
pci_set_master(rdev->pdev);
radeon_ring_start(rdev, RADEON_RING_TYPE_GFX_INDEX, &rdev->ring[RADEON_RING_TYPE_GFX_INDEX]); radeon_ring_start(rdev, RADEON_RING_TYPE_GFX_INDEX, &rdev->ring[RADEON_RING_TYPE_GFX_INDEX]);
r = radeon_ring_test(rdev, RADEON_RING_TYPE_GFX_INDEX, ring); r = radeon_ring_test(rdev, RADEON_RING_TYPE_GFX_INDEX, ring);
if (r) { if (r) {
......
...@@ -3047,6 +3047,9 @@ int r600_irq_init(struct radeon_device *rdev) ...@@ -3047,6 +3047,9 @@ int r600_irq_init(struct radeon_device *rdev)
else else
r600_disable_interrupt_state(rdev); r600_disable_interrupt_state(rdev);
/* at this point everything should be setup correctly to enable master */
pci_set_master(rdev->pdev);
/* enable irqs */ /* enable irqs */
r600_enable_interrupts(rdev); r600_enable_interrupts(rdev);
......
...@@ -955,7 +955,6 @@ int radeon_resume_kms(struct drm_device *dev) ...@@ -955,7 +955,6 @@ int radeon_resume_kms(struct drm_device *dev)
console_unlock(); console_unlock();
return -1; return -1;
} }
pci_set_master(dev->pdev);
/* resume AGP if in use */ /* resume AGP if in use */
radeon_agp_resume(rdev); radeon_agp_resume(rdev);
radeon_resume(rdev); radeon_resume(rdev);
......
...@@ -57,8 +57,6 @@ int radeon_driver_load_kms(struct drm_device *dev, unsigned long flags) ...@@ -57,8 +57,6 @@ int radeon_driver_load_kms(struct drm_device *dev, unsigned long flags)
} }
dev->dev_private = (void *)rdev; dev->dev_private = (void *)rdev;
pci_set_master(dev->pdev);
/* update BUS flag */ /* update BUS flag */
if (drm_pci_device_is_agp(dev)) { if (drm_pci_device_is_agp(dev)) {
flags |= RADEON_IS_AGP; flags |= RADEON_IS_AGP;
......
...@@ -3217,6 +3217,8 @@ static int si_irq_init(struct radeon_device *rdev) ...@@ -3217,6 +3217,8 @@ static int si_irq_init(struct radeon_device *rdev)
/* force the active interrupt state to all disabled */ /* force the active interrupt state to all disabled */
si_disable_interrupt_state(rdev); si_disable_interrupt_state(rdev);
pci_set_master(rdev->pdev);
/* enable irqs */ /* enable irqs */
si_enable_interrupts(rdev); si_enable_interrupts(rdev);
......
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