[ofa-general] Re: Further 2.6.23 merge plans...

Michael S. Tsirkin mst at dev.mellanox.co.il
Tue Jul 17 14:44:17 PDT 2007


> Quoting Roland Dreier <rdreier at cisco.com>:
> Subject: Re: Further 2.6.23 merge plans...
> 
>  >  - Take a look at Sean's local SA caching patches.  I merged
>  >    everything else from Sean's tree, but I'm still undecided about
>  >    these.  I haven't read them carefully yet, but even aside from that
>  >    I don't have a good feeling about whether there's consensus about
>  >    this yet.  Any opinions about merging, for or against, would be
>  >    appreciated here.
> 
> Does anyone other than Sean have an opinion here?  If you want this
> feature, if you've tested it, if you don't think it's ready yet,
> whatever, please speak up -- I don't feel comfortable making a
> decision on my own here (although I will if I have to).

We have the patches applied in ofed 1.2.c with default module parameter set to
caching disabled (ofed 1.2 had a different version of the patches, but caching
is disabled by default there, too). At least in this configuration
(caching disabled), all issues I've seen seem to be fixed now, and tests seem to
be running smoothly.

So I think it's safe to merge it up if the module parameter
is set to cache disabled by default.
No idea what happens if it's enabled though :)

-- 
MST



More information about the general mailing list