Re: [PATCH] Add missing spin_unlock_irq to ide.c (244ac16)
On Fri, May 25, 2001 at 11:11:23PM +0200, Jens Axboe wrote: [...] > This isn't right. Granted the locking isn't straight forward here, but > take a look at ide_write_setting -> ide_spin_wait_hwgroup and the > latters return value. Yes, Andre set me straight here. My apologies for being lazy and not checking the call path. -- Rasmus([EMAIL PROTECTED]) You know how dumb the average guy is? Well, by definition, half of them are even dumber than that. -- J.R. "Bob" Dobbs - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Add missing spin_unlock_irq to ide.c (244ac16)
On Fri, May 25 2001, Rasmus Andersen wrote: > (I forgot to cc l-k on this one when it went to andre.) > > Hi. > > This patch adds a spin_unlock_irqsave to ide_spin_wait_hwgroup as > reported by the Stanford team way back. It applies against 244ac16. > > > --- linux-244-ac16-clean/drivers/ide/ide.cFri May 25 21:11:08 2001 > +++ linux-244-ac16/drivers/ide/ide.c Fri May 25 22:46:43 2001 > @@ -2362,6 +2362,8 @@ > __restore_flags(lflags);/* local CPU only */ > spin_lock_irq(_request_lock); > } > + > +spin_unlock_irq(_request_lock); > return 0; > } This isn't right. Granted the locking isn't straight forward here, but take a look at ide_write_setting -> ide_spin_wait_hwgroup and the latters return value. BTW, also try and follow local style when making such changes. -- Jens Axboe - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] Add missing spin_unlock_irq to ide.c (244ac16)
(I forgot to cc l-k on this one when it went to andre.) Hi. This patch adds a spin_unlock_irqsave to ide_spin_wait_hwgroup as reported by the Stanford team way back. It applies against 244ac16. --- linux-244-ac16-clean/drivers/ide/ide.c Fri May 25 21:11:08 2001 +++ linux-244-ac16/drivers/ide/ide.cFri May 25 22:46:43 2001 @@ -2362,6 +2362,8 @@ __restore_flags(lflags);/* local CPU only */ spin_lock_irq(_request_lock); } + +spin_unlock_irq(_request_lock); return 0; } -- Rasmus([EMAIL PROTECTED]) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] Add missing spin_unlock_irq to ide.c (244ac16)
(I forgot to cc l-k on this one when it went to andre.) Hi. This patch adds a spin_unlock_irqsave to ide_spin_wait_hwgroup as reported by the Stanford team way back. It applies against 244ac16. --- linux-244-ac16-clean/drivers/ide/ide.c Fri May 25 21:11:08 2001 +++ linux-244-ac16/drivers/ide/ide.cFri May 25 22:46:43 2001 @@ -2362,6 +2362,8 @@ __restore_flags(lflags);/* local CPU only */ spin_lock_irq(io_request_lock); } + +spin_unlock_irq(io_request_lock); return 0; } -- Rasmus([EMAIL PROTECTED]) - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Add missing spin_unlock_irq to ide.c (244ac16)
On Fri, May 25 2001, Rasmus Andersen wrote: (I forgot to cc l-k on this one when it went to andre.) Hi. This patch adds a spin_unlock_irqsave to ide_spin_wait_hwgroup as reported by the Stanford team way back. It applies against 244ac16. --- linux-244-ac16-clean/drivers/ide/ide.cFri May 25 21:11:08 2001 +++ linux-244-ac16/drivers/ide/ide.c Fri May 25 22:46:43 2001 @@ -2362,6 +2362,8 @@ __restore_flags(lflags);/* local CPU only */ spin_lock_irq(io_request_lock); } + +spin_unlock_irq(io_request_lock); return 0; } This isn't right. Granted the locking isn't straight forward here, but take a look at ide_write_setting - ide_spin_wait_hwgroup and the latters return value. BTW, also try and follow local style when making such changes. -- Jens Axboe - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Add missing spin_unlock_irq to ide.c (244ac16)
On Fri, May 25, 2001 at 11:11:23PM +0200, Jens Axboe wrote: [...] This isn't right. Granted the locking isn't straight forward here, but take a look at ide_write_setting - ide_spin_wait_hwgroup and the latters return value. Yes, Andre set me straight here. My apologies for being lazy and not checking the call path. -- Rasmus([EMAIL PROTECTED]) You know how dumb the average guy is? Well, by definition, half of them are even dumber than that. -- J.R. Bob Dobbs - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/