[ofa-general] Re: [PATCH] ib_core: Use weak ordering for data registered memory

Arnd Bergmann arnd at arndb.de
Thu Oct 23 04:47:53 PDT 2008


On Thursday 23 October 2008, Eli Cohen wrote:
> On Wed, Oct 22, 2008 at 04:41:16PM +0200, Arnd Bergmann wrote:
> > 
> > As a minor detail, I think we should make this module parameter
> > writable in order to allow switching the behaviour without
> > reloading the infiniband drivers.
> 
> Although changing the value of the parameter at runtime should not
> cause any inconsistencies in the driver, it might affect other
> applications running so I personally prefer to have this parameter as
> read only and let the system administrator decide on that before
> loading the driver.

ok, that makes sense.

> > As mentioned before, I would personally also prefer to make
> > this attribute '1' by default instead of zero, but I trust
> > your judgement if you think the default should be '0'.
> > 
> 
> Again, I prefer not to change the default that used to be till now, so
> I'd rather have the default as strong ordering.

ok.

Roland, please use the version that Eli posted yesterday.

	Arnd <><



More information about the general mailing list