All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Fabio M. De Francesco" <fmdefrancesco@gmail.com>
To: netdev@vger.kernel.org,
	Anirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Cc: Ira Weiny <ira.weiny@intel.com>,
	Anirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Subject: Re: [PATCH v2 net-next 0/6] Remove uses of kmap_atomic()
Date: Fri, 25 Nov 2022 16:47:55 +0100	[thread overview]
Message-ID: <2325231.NG923GbCHz@suse> (raw)
In-Reply-To: <20221123205219.31748-1-anirudh.venkataramanan@intel.com>

On mercoledì 23 novembre 2022 21:52:13 CET Anirudh Venkataramanan wrote:
> kmap_atomic() is being deprecated. This little series replaces the last
> few uses of kmap_atomic() in the networking subsystem.
> 
> This series triggered a suggestion [1] that perhaps the Sun Cassini,
> LDOM Virtual Switch Driver and the LDOM virtual network drivers should be
> removed completely. I plan to do this in a follow up patchset. For
> completeness, this series still includes kmap_atomic() conversions that
> apply to the above referenced drivers. If for some reason we choose to not
> remove these drivers, at least they won't be using kmap_atomic() anymore.
> 
> Also, the following maintainer entries for the Chelsio driver seem to be
> defunct:
> 
>   Vinay Kumar Yadav <vinay.yadav@chelsio.com>
>   Rohit Maheshwari <rohitm@chelsio.com>
> 
> I can submit a follow up patch to remove these entries, but thought
> maybe the folks over at Chelsio would want to look into this first.
> 
> Changes v1 -> v2:
>   Use memcpy_from_page() in patches 2/6 and 4/6
>   Add new patch for the thunderbolt driver
>   Update commit messages and cover letter
> 
> [1]
> https://lore.kernel.org/netdev/99629223-ac1b-0f82-50b8-ea307b3b0197@intel.com
> /T/#m3da3759652a48f958ab852fa5499009b43ff8fdd
> 
> Anirudh Venkataramanan (6):
>   ch_ktls: Use memcpy_from_page() instead of k[un]map_atomic()
>   sfc: Use kmap_local_page() instead of kmap_atomic()
>   cassini: Use page_address() instead of kmap_atomic()
>   cassini: Use memcpy_from_page() instead of k[un]map_atomic()
>   sunvnet: Use kmap_local_page() instead of kmap_atomic()
>   net: thunderbolt: Use kmap_local_page() instead of kmap_atomic()
> 
>  .../chelsio/inline_crypto/ch_ktls/chcr_ktls.c | 26 +++++-----
>  drivers/net/ethernet/sfc/tx.c                 |  4 +-
>  drivers/net/ethernet/sun/cassini.c            | 48 ++++++-------------
>  drivers/net/ethernet/sun/sunvnet_common.c     |  4 +-
>  drivers/net/thunderbolt.c                     |  8 ++--
>  5 files changed, 35 insertions(+), 55 deletions(-)
> 
> 
> base-commit: e80bd08fd75a644e2337fb535c1afdb6417357ff
> --
> 2.37.2

I noticed too late that your patches were already applied. The message from 
the patchwork bot was the latest. I'm sorry for the noise: my tags were 
unnecessary but I didn't yet know :-(

However, again thanks for helping with these conversions,

Fabio




      parent reply	other threads:[~2022-11-25 15:48 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-23 20:52 [PATCH v2 net-next 0/6] Remove uses of kmap_atomic() Anirudh Venkataramanan
2022-11-23 20:52 ` [PATCH v2 net-next 1/6] ch_ktls: Use memcpy_from_page() instead of k[un]map_atomic() Anirudh Venkataramanan
2022-11-24 10:56   ` Ayush Sawal
2022-11-28 20:22     ` Anirudh Venkataramanan
2022-11-25 15:34   ` Fabio M. De Francesco
2022-11-23 20:52 ` [PATCH v2 net-next 2/6] sfc: Use kmap_local_page() instead of kmap_atomic() Anirudh Venkataramanan
2022-11-24  8:34   ` Martin Habets
2022-11-25 15:37   ` Fabio M. De Francesco
2022-11-23 20:52 ` [PATCH v2 net-next 3/6] cassini: Use page_address() " Anirudh Venkataramanan
2022-11-25 15:38   ` Fabio M. De Francesco
2022-11-23 20:52 ` [PATCH v2 net-next 4/6] cassini: Use memcpy_from_page() instead of k[un]map_atomic() Anirudh Venkataramanan
2022-11-25 15:39   ` Fabio M. De Francesco
2022-11-23 20:52 ` [PATCH v2 net-next 5/6] sunvnet: Use kmap_local_page() instead of kmap_atomic() Anirudh Venkataramanan
2022-11-25 15:40   ` Fabio M. De Francesco
2022-11-23 20:52 ` [PATCH v2 net-next 6/6] net: thunderbolt: " Anirudh Venkataramanan
2022-11-24  9:34   ` Mika Westerberg
2022-11-25 15:41   ` Fabio M. De Francesco
2022-11-25 10:50 ` [PATCH v2 net-next 0/6] Remove uses " patchwork-bot+netdevbpf
2022-11-25 15:47 ` Fabio M. De Francesco [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=2325231.NG923GbCHz@suse \
    --to=fmdefrancesco@gmail.com \
    --cc=anirudh.venkataramanan@intel.com \
    --cc=ira.weiny@intel.com \
    --cc=netdev@vger.kernel.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.