Re: [PATCH 2/3] ide: add ide_[un]lock_hwgroup() helpers

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Michael Schmitz
Date: Tuesday, November 18, 2008 - 7:33 pm

> -	while (!hwgroup->busy) {

Something I've run into while working on the locking stuff: what happens if the 
above ide_lock_hwgroup(hwgroup) sleeps for long enough to trigger the request 
timer? 

I'll think about the ramifications of your patch in the context of what I tested 
WRT unlocking whenever hwgroup->busy is cleared, and get back to you. 

	Michael
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 2/3] ide: add ide_[un]lock_hwgroup() helpers, Bartlomiej Zolnierki ..., (Tue Nov 18, 1:16 pm)
Re: [PATCH 2/3] ide: add ide_[un]lock_hwgroup() helpers, Michael Schmitz, (Tue Nov 18, 7:33 pm)
Re: [PATCH 2/3] ide: add ide_[un]lock_hwgroup() helpers, Bartlomiej Zolnierki ..., (Sat Dec 13, 4:15 pm)
Re: [PATCH 2/3] ide: add ide_[un]lock_hwgroup() helpers, Michael Schmitz, (Sat Dec 13, 4:52 pm)