Re: [openib-general] kernel VM monitor for memory registration caching

2005-07-31 Thread Gleb Natapov
On Sun, Jul 31, 2005 at 02:57:01PM -0700, Matt Leininger wrote: > FWIW here is a link to the quadrics vm patch that David Addison posted > on LKML a couple of months ago. > > http://lkml.org/lkml/2005/4/26/198 > I remember this thread. David's patch implements different approach to the problem.

[openib-general] [PATCH] remove in_atomic

2005-07-31 Thread Michael S. Tsirkin
in_atomic isnt a reliable way to check that we are in an atomic context. Just schedule work, always, since most cq polling is currently done under a spinlock, anyway. Signed-off-by: Michael S. Tsirkin <[EMAIL PROTECTED]> Index: linux-2.6.12.2/drivers/infiniband/ulp/sdp/sdp_iocb.c

RE: [openib-general] Re: [Rdma-developers] Meeting (07/22) summary:OpenRDMA community development discussion

2005-07-31 Thread Yaron Haviv
> -Original Message- > From: [EMAIL PROTECTED] [mailto:openib-general- > [EMAIL PROTECTED] On Behalf Of Christoph Hellwig > Sent: Friday, July 29, 2005 8:02 AM > To: Tom Duffy > Cc: Venkata Jagana; [EMAIL PROTECTED]; Christoph > Hellwig; openib-general@openib.org > Subject: [openib-general]

Re: [openib-general] kernel VM monitor for memory registration caching

2005-07-31 Thread Matt Leininger
FWIW here is a link to the quadrics vm patch that David Addison posted on LKML a couple of months ago. http://lkml.org/lkml/2005/4/26/198 - Matt On Sun, 2005-07-31 at 13:31 +0300, Gleb Natapov wrote: > Hello Pete, > > On Fri, Jul 29, 2005 at 01:42:25PM -0400, Pete Wyckoff wrote: > > I'll

Re: [openib-general][PATCH][kdapl]: inc/dec module ref count

2005-07-31 Thread Muli Ben-Yehuda
On Sun, Jul 31, 2005 at 06:08:11PM +0300, Guy German wrote: > Hi Muli, > > Wouldn't it be solved by moving the try_module_get call to the > beginning of the dapl_ia_open function ? No. Even if it's theoretically the first line in the function, the compiler can and will create a function prologu

RE: [openib-general][PATCH][kdapl]: inc/dec module ref count

2005-07-31 Thread Guy German
Hi Muli, Muli Ben-Yehuda wrote: > On Thu, Jul 28, 2005 at 10:14:15AM +0300, Guy German wrote: > >> [kdapl]: increments the module ref count on ia_open, thus making >> sure the kdapl_ib module would not be rmmod-ed while in use >> >> Signed-off-by: Guy German <[EMAIL PR

Re: [openib-general][PATCH][kdapl]: inc/dec module ref count

2005-07-31 Thread Muli Ben-Yehuda
On Thu, Jul 28, 2005 at 03:01:24PM -0400, James Lentini wrote: > We've been using the following convention to indicate that we don't > care about the return value: > > (void) try_module_get(THIS_MODULE); This seems counter intuitive to me (the idea, not the syntax) - try_module_get() can and w

Re: [openib-general][PATCH][kdapl]: inc/dec module ref count

2005-07-31 Thread Muli Ben-Yehuda
On Thu, Jul 28, 2005 at 10:14:15AM +0300, Guy German wrote: > [kdapl]: increments the module ref count on ia_open, thus making > sure the kdapl_ib module would not be rmmod-ed while in use > > Signed-off-by: Guy German <[EMAIL PROTECTED]> > > Index: infiniband/ulp/kdapl/ib/dapl_ia.c > ==

Re: [openib-general] kernel VM monitor for memory registration caching

2005-07-31 Thread Gleb Natapov
Hello Pete, On Fri, Jul 29, 2005 at 01:42:25PM -0400, Pete Wyckoff wrote: > I'll be happy to discuss the code with anyone who ends up wanting > to use it or improve upon it. I glanced over the code and I have couple of questions/improvments. First of all, you have one user_delta per mm that use

[openib-general] Re: [PATCH] arch/xx/pci: remap_pfn_range -> io_remap_pfn_range

2005-07-31 Thread Michael S. Tsirkin
Quoting r. Greg KH <[EMAIL PROTECTED]>: > Subject: Re: [openib-general] Re: [PATCH] arch/xx/pci: remap_pfn_range -> > io_remap_pfn_range > If this is a fix for xen, fine, then say so in the changelog information > for the patch, as it is, no such information was given. > > thanks, > > greg k-h >