[PATCH RT] - Mellanox IB driver patch




Hi Ingo,

RT driver patch to eliminate in_atomic stack dump.

The problem code was identified by Michael S. Tsirkin, and he suggested
the fix.

I adapted to use RT's _nort primitives- should work correctly in all
configs.

Thanks,

Sven


Fixes in_atomic stack-dump, when Mellanox module
is loaded into the RT Kernel.

From: Michael S. Tsirkin <mst@xxxxxxxxxxxxxxxxxx>

"Basically, if you just make spin_lock_irqsave (and spin_lock_irq) not disable
interrupts for non-raw spinlocks, I think all of infiniband will be fine without
changes."

signed-off-by: Sven-Thorsten Dietrich <sven@xxxxxxxxxxxxxxxxxxxxx>


Index: linux-2.6.21/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
===================================================================
--- linux-2.6.21.orig/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
+++ linux-2.6.21/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
@@ -788,7 +788,7 @@ void ipoib_mcast_restart_task(struct wor

ipoib_mcast_stop_thread(dev, 0);

- local_irq_save(flags);
+ local_irq_save_nort(flags);
netif_tx_lock(dev);
spin_lock(&priv->lock);

@@ -863,7 +863,7 @@ void ipoib_mcast_restart_task(struct wor

spin_unlock(&priv->lock);
netif_tx_unlock(dev);
- local_irq_restore(flags);
+ local_irq_restore_nort(flags);

/* We have to cancel outside of the spinlock */
list_for_each_entry_safe(mcast, tmcast, &remove_list, list) {


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • RE: Drill Through for Multiple Cells
    ... I'm having a similar issue to what Michael describes in his 3b example. ... hoping someone can fill me in on if there is a fix for this. ... running SQL server 05 and Excel 2007 and working with a data cube. ... I appreciated some more info about this subject "Drill Through for Multiple ...
    (microsoft.public.sqlserver.olap)
  • RE: IIS Stops working only solution is server reboot
    ... Thanks for the suggestion. ... "Michael C." wrote: ... In IIS I created a new application pool and put the application in that ... It didn't fix the problem, but limited the site being unavailable to ...
    (microsoft.public.windows.server.sbs)
  • Re: Event Procedures Stopped Working
    ... Choose Compile from the Debug menu. ... Access will highlight the problem code. ... Fix the problem, and repeat until the code compiles. ... time I do something which will start an event procedure I get this message: ...
    (microsoft.public.access.formscoding)
  • infiniband patch series (was Re: ipath patch series a-comin, but no IB maintainer to shepherd th
    ... Michael S. Tsirkin: ... am I using the right format to send things upstream to you? ... There's really a set of independent patches, ...
    (Linux-Kernel)
  • Re: kernel lockup ?
    ... The solution to fix the problem is, ... buggy old kernel running on your EOL distro. ... Next you need to plan on upgrading to a version with patches ... Michael Heiming ...
    (comp.os.linux.development.system)