Commit ef29888e authored by Matthias Kaehlcke's avatar Matthias Kaehlcke Committed by Bartlomiej Zolnierkiewicz
Browse files

ide: use mutex instead of ide_cfg_sem semaphore in IDE driver 



The IDE driver uses a semaphore as mutex.
Use the mutex API instead of the (binary) semaphore.

Signed-off-by: default avatarMatthias Kaehlcke <matthias.kaehlcke@gmail.com>
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
parent c283f5db

drivers/ide/ide-probe.c

+3 −3
Original line number Diff line number Diff line
@@ -1025,7 +1025,7 @@ static int init_irq (ide_hwif_t *hwif) 
	BUG_ON(irqs_disabled());	

	BUG_ON(hwif == NULL);



	down(&ide_cfg_sem);

	mutex_lock(&ide_cfg_mtx);

	hwif->hwgroup = NULL;

#if MAX_HWIFS > 1

	/*
@@ -1154,7 +1154,7 @@ static int init_irq (ide_hwif_t *hwif) 
		printk(" (%sed with %s)",

			hwif->sharing_irq ? "shar" : "serializ", match->name);

	printk("\n");

	up(&ide_cfg_sem);

	mutex_unlock(&ide_cfg_mtx);

	return 0;

out_unlink:

	spin_lock_irq(&ide_lock);
@@ -1177,7 +1177,7 @@ static int init_irq (ide_hwif_t *hwif) 
	}

	spin_unlock_irq(&ide_lock);

out_up:

	up(&ide_cfg_sem);

	mutex_unlock(&ide_cfg_mtx);

	return 1;

}

drivers/ide/ide.c

+4 −4
Original line number Diff line number Diff line
@@ -169,7 +169,7 @@ static const u8 ide_hwif_to_major[] = { IDE0_MAJOR, IDE1_MAJOR, 
static int idebus_parameter;	/* holds the "idebus=" parameter */

static int system_bus_speed;	/* holds what we think is VESA/PCI bus speed */



DECLARE_MUTEX(ide_cfg_sem);

DEFINE_MUTEX(ide_cfg_mtx);

 __cacheline_aligned_in_smp DEFINE_SPINLOCK(ide_lock);



#ifdef CONFIG_IDEPCI_PCIBUS_ORDER
@@ -564,7 +564,7 @@ void ide_unregister(unsigned int index) 
{

	ide_drive_t *drive;

	ide_hwif_t *hwif, *g;

	static ide_hwif_t tmp_hwif; /* protected by ide_cfg_sem */

	static ide_hwif_t tmp_hwif; /* protected by ide_cfg_mtx */

	ide_hwgroup_t *hwgroup;

	int irq_count = 0, unit;
@@ -572,7 +572,7 @@ void ide_unregister(unsigned int index) 


	BUG_ON(in_interrupt());

	BUG_ON(irqs_disabled());

	down(&ide_cfg_sem);

	mutex_lock(&ide_cfg_mtx);

	spin_lock_irq(&ide_lock);

	hwif = &ide_hwifs[index];

	if (!hwif->present)
@@ -679,7 +679,7 @@ void ide_unregister(unsigned int index) 


abort:

	spin_unlock_irq(&ide_lock);

	up(&ide_cfg_sem);

	mutex_unlock(&ide_cfg_mtx);

}



EXPORT_SYMBOL(ide_unregister);

include/linux/ide.h

+2 −2
Original line number Diff line number Diff line
@@ -1382,11 +1382,11 @@ extern const ide_pio_timings_t ide_pio_timings[6]; 




extern spinlock_t ide_lock;

extern struct semaphore ide_cfg_sem;

extern struct mutex ide_cfg_mtx;

/*

 * Structure locking:

 *

 * ide_cfg_sem and ide_lock together protect changes to

 * ide_cfg_mtx and ide_lock together protect changes to

 * ide_hwif_t->{next,hwgroup}

 * ide_drive_t->next

 *