Re: gdth new set of patches for 2.6.24 stable

2008-02-19 Thread Ralf Baechle
On Mon, Feb 18, 2008 at 12:27:42PM -0800, David Brownell wrote: > > Actually I think David might have been wrong about mips.  afaict its > > dma_free_coherent() is callable under local_irq_disable(), so ARM SMP is > > the sole exception?   > > All I recall at this point was getting some arch-spec

Re: gdth new set of patches for 2.6.24 stable

2008-02-18 Thread Ralf Baechle
On Mon, Feb 18, 2008 at 04:57:36AM -0800, Andrew Morton wrote: > On Sun, 17 Feb 2008 18:46:03 +0200 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > > > ... > > > > All my testers have reported back that with these 5 patches applied they can > > now run with a 2.6.24 kernel the same way they ran before.

Re: gdth new set of patches for 2.6.24 stable

2008-02-18 Thread David Brownell
On Monday 18 February 2008, Andrew Morton wrote: > Russell, Ralf: is there something we can do here to relax this requirement? > > I'm thinking that perhaps we can do some rcu/refcounting tricks: launch the > IPI from within dma_free_coherent(), but don't wait for it to complete. > When all CPUs

Re: gdth new set of patches for 2.6.24 stable

2008-02-18 Thread James Bottomley
On Mon, 2008-02-18 at 14:02 +, Russell King wrote: > Another solution jejb suggested was to make dma_free_coherent() lazy, > but (a) I'm unconvinced that this'll work with drivers which > constantly > alloc+free in IRQ context since there's generally only 2MB of VM space > for such mappings, an

Re: gdth new set of patches for 2.6.24 stable

2008-02-18 Thread Russell King
On Mon, Feb 18, 2008 at 04:57:36AM -0800, Andrew Morton wrote: > But arm and mips require enabled local irqs because their > dma_free_coherent() needs to do a cross-cpu IPI call. Presumably because > of certain unusual TLB protocols. Consider that TLB flushing needs to call a function on another

Re: gdth new set of patches for 2.6.24 stable

2008-02-18 Thread Boaz Harrosh
On Mon, Feb 18 2008 at 14:57 +0200, Andrew Morton <[EMAIL PROTECTED]> wrote: > On Sun, 17 Feb 2008 18:46:03 +0200 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > >> ... >> >> All my testers have reported back that with these 5 patches applied they can >> now run with a 2.6.24 kernel the same way they ra

Re: gdth new set of patches for 2.6.24 stable

2008-02-18 Thread Andrew Morton
On Sun, 17 Feb 2008 18:46:03 +0200 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > > ... > > All my testers have reported back that with these 5 patches applied they can > now run with a 2.6.24 kernel the same way they ran before. However there is > that reported issue, with the dma_free_coherent WARN_

Re: gdth new set of patches for 2.6.24 stable

2008-02-18 Thread Boaz Harrosh
On Sun, Feb 17 2008 at 19:24 +0200, James Bottomley <[EMAIL PROTECTED]> wrote: > On Sun, 2008-02-17 at 18:46 +0200, Boaz Harrosh wrote: >> On Thu, Feb 14 2008 at 20:47 +0200, Boaz Harrosh <[EMAIL PROTECTED]> wrote: >>> Submitted are a new set of patches, that fix lots of problems >>> with the gdth

Re: gdth new set of patches for 2.6.24 stable

2008-02-17 Thread James Bottomley
On Sun, 2008-02-17 at 18:46 +0200, Boaz Harrosh wrote: > On Thu, Feb 14 2008 at 20:47 +0200, Boaz Harrosh <[EMAIL PROTECTED]> wrote: > > Submitted are a new set of patches, that fix lots of problems > > with the gdth driver. > > > > It fixes the following problems: > > - scan for drives on hosts.

Re: gdth new set of patches for 2.6.24 stable

2008-02-17 Thread Boaz Harrosh
On Thu, Feb 14 2008 at 20:47 +0200, Boaz Harrosh <[EMAIL PROTECTED]> wrote: > Submitted are a new set of patches, that fix lots of problems > with the gdth driver. > > It fixes the following problems: > - scan for drives on hosts. (Already in mainline) > - truly fixes the exit/reboot problems but

gdth new set of patches for 2.6.24 stable

2008-02-14 Thread Boaz Harrosh
Submitted are a new set of patches, that fix lots of problems with the gdth driver. It fixes the following problems: - scan for drives on hosts. (Already in mainline) - truly fixes the exit/reboot problems but does call flush() before reboot. - fix crash when accessing array with icpcon manageme