RE: [PATCH v3 00/13] Request for Comments on SoftiWarp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> This patch set contributes a new version of the SoftiWarp
> driver, as originally introduced Oct 6th, 2017.
> 
> Thank you all for the commments on my last submissiom. It
> was vey helpful and encouraging. I hereby re-send a SoftiWarp
> patch after taking all comments into account, I hope. Without
> introducing SoftiWarp (siw) again, let me fast forward
> to the changes made to the last submission:
> 
> 
> 1. Removed all kernel module parameters, as requested.
>    I am not completely happy yet with what came out of
>    that, since parameters like the name of interfaces
>    siw shall attach to,

Can configfs be used for this? I don't remember if that was discussed during
v2 review?  OR, the new rdma command that is part of iproute2 can be
extended.  It would use netlink messages to do the configuration.  Perhaps
Leon can comment on that when he gets time.  It could be extended to support
both rxe and siw.

> or if we want to use TCP_NODELAY
>    for low ping-pong delay, or disable it for higher
>    message rates for multiple outstanding operations,
>    were quite handy. I understand I have to find a better
>    solution for that. Ideally, we would be able to
>    distinguish between global siw driver control (like
>    interfaces it attaches to), and per-QP parameters
>    (somehow similar to socket options) which allow for
>    example to tune delay, setting CRC on/off, or control
>    GSO usage.
> 

Per connection settings could be done by extending the rdma_cm API.  See
rdma_set_option() in librdmacm from rdma-core, and kernel rdmacm API
functions like rdma_set_service_type() and rdma_set_reuseaddr().  We really
need the TCP options to be tweakable for iwarp cm_ids...


Steve.

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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux