[rds-devel] [RFC] [PATCH] RDS: Added module parameter to enable/disable RDMA operations support

Or Gerlitz or.gerlitz at gmail.com
Tue Dec 4 13:57:59 PST 2007


On 12/4/07, Vladimir Sokolovsky <vlad at dev.mellanox.co.il> wrote:

> RDS: Added module parameter 'rds_rdma_support' that enables/disables
> RDMA operations support.

Why we want to disable this or that feature? if the feature is ready
to release, then it should be distributed and anyone should be able to
use it. If its not, then it should be in a release only under
CONFIG_EXPERIMENTAL and any sane distribution would not peek this
config directive.

This is how it works in Linux (normally). This habit of adding module
parameters everywhere in OFED was rejected on the spot by Roland et al
from the upstream code. I don't think RDS should be different, unless
you want to --tag-- it with lower standards compared to the rest of
the Linux kernel IB stack and/or any other upstream driver.

Or.



More information about the rds-devel mailing list