All of lore.kernel.org
 help / color / mirror / Atom feed
From: Leon Romanovsky <leon@kernel.org>
To: Stefan Metzmacher <metze@samba.org>
Cc: linux-rdma@vger.kernel.org, Bernard Metzler <bmt@zurich.ibm.com>
Subject: Re: [PATCH 0/8] RDMA/siw: [re-]introduce module parameters and add MPA V1
Date: Sun, 6 Jul 2025 10:56:24 +0300	[thread overview]
Message-ID: <20250706075624.GS6278@unreal> (raw)
In-Reply-To: <cover.1751561826.git.metze@samba.org>

On Thu, Jul 03, 2025 at 07:26:11PM +0200, Stefan Metzmacher wrote:
> While working on smbdirect cleanup I'm using siw.ko for
> testing against Windows using Chelsio T404-BT (only supporting MPA V1)
> and T520-BT (supporting MPA V2) cards.
> 
> Up to now I used old patches to add MPA V1 support
> to siw.ko and compiled against the running kernel
> each time. I don't want to do that forever, so
> I re-introduced module parameters in order to
> avoid patching and compiling.
> 
> For my testing I'm using something like this:
> 
>   echo 1 > /sys/module/siw/parameters/mpa_crc_required
>   echo 1 > /sys/module/siw/parameters/mpa_crc_strict
>   echo 0 > /sys/module/siw/parameters/mpa_rdma_write_rtr
>   echo 1 > /sys/module/siw/parameters/mpa_peer_to_peer
> 
>   echo 1 > /sys/module/siw/parameters/mpa_version
>   rdma link add siw_v1_eth0 type siw netdev eth0
> 
>   echo 2 > /sys/module/siw/parameters/mpa_version
>   rdma link add siw_v2_eth1 type siw netdev eth1
> 
> The global parameters are copied at 'rdma link add' time
> and will stay as is for the lifetime of the specific device.
> 
> So I can testing against the T520-BT card using siw_v2_eth1
> and against the T404-BT card using siw_v1_eth0.
> 
> It would be great if this can go upstream.

Sorry no, there is no benefit in adding complexity for such an old device,
like Chelsio T404-BT.

Thanks

      parent reply	other threads:[~2025-07-06  7:56 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-03 17:26 [PATCH 0/8] RDMA/siw: [re-]introduce module parameters and add MPA V1 Stefan Metzmacher
2025-07-03 17:26 ` [PATCH 1/8] RDMA/siw: make mpa_version = MPA_REVISION_2 const Stefan Metzmacher
2025-07-03 17:26 ` [PATCH 2/8] RDMA/siw: remove unused loopback_enabled = true Stefan Metzmacher
2025-07-04  1:14   ` yanjun.zhu
2025-07-03 17:26 ` [PATCH 3/8] RDMA/siw: add and remember siw_device_options per device Stefan Metzmacher
2025-07-03 17:26 ` [PATCH 4/8] RDMA/siw: make use of sdev->options.* and avoid globals Stefan Metzmacher
2025-07-03 17:26 ` [PATCH 5/8] RDMA/siw: combine global options into siw_default_options Stefan Metzmacher
2025-07-03 17:26 ` [PATCH 6/8] RDMA/siw: move rtr_type to siw_device_options Stefan Metzmacher
2025-07-03 17:26 ` [PATCH 7/8] RDMA/siw: [re-]introduce module parameters to alter the behavior at runtime Stefan Metzmacher
2025-07-03 17:26 ` [PATCH 8/8] RDMA/siw: add support for MPA V1 and IRD/ORD negotiation based on [MS-SMBD] Stefan Metzmacher
2025-07-03 19:16 ` [PATCH 0/8] RDMA/siw: [re-]introduce module parameters and add MPA V1 Chuck Lever
2025-07-04 14:13   ` Bernard Metzler
2025-07-06  7:56 ` Leon Romanovsky [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250706075624.GS6278@unreal \
    --to=leon@kernel.org \
    --cc=bmt@zurich.ibm.com \
    --cc=linux-rdma@vger.kernel.org \
    --cc=metze@samba.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.