Penchala Narasimha Reddy Chilakala, TLS-Chennai wrote: > Yes, you are right that the up() is used in 3 places and down_trylock, > down_interruptible in one place each, but we need to do other changes > like when we return a value ERESTATSYS from that particular function, > the value will be checked in many places to take appropriate action > and fibptr->done also checked in some other places. Considering all > these, I mentioned that we need to modify the code in lot places. I disagree with this analysis. I am only talking about the type of struct fib.event_wait which very probably should be a struct completion or wait_queue_head_t. > I hope you understand. If and why you as author and submitter of the short term fix are also interested in a real long term fix or not is _entirely_ up to you, or anybody else who works with the aacraid driver. If I had this hardware myself I would send a proposed conversion patch instead of comments what other people could do. :-) -- Stefan Richter -=====-==--= =-== --=-- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
