All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jesper Dangaard Brouer <brouer@redhat.com>
To: intel-wired-lan@osuosl.org
Subject: [Intel-wired-lan] [net-next PATCH 00/27] Add support for DMA writable pages being writable by the network stack
Date: Wed, 26 Oct 2016 17:45:42 +0200	[thread overview]
Message-ID: <20161026174542.334798db@redhat.com> (raw)
In-Reply-To: <20161025153220.4815.61239.stgit@ahduyck-blue-test.jf.intel.com>

On Tue, 25 Oct 2016 11:36:48 -0400
Alexander Duyck <alexander.h.duyck@intel.com> wrote:

> The first 22 patches in the set add support for the DMA attribute
> DMA_ATTR_SKIP_CPU_SYNC on multiple platforms/architectures.  This is needed
> so that we can flag the calls to dma_map/unmap_page so that we do not
> invalidate cache lines that do not currently belong to the device.  Instead
> we have to take care of this in the driver via a call to
> sync_single_range_for_cpu prior to freeing the Rx page.
> 
> Patch 23 adds support for dma_map_page_attrs and dma_unmap_page_attrs so
> that we can unmap and map a page using the DMA_ATTR_SKIP_CPU_SYNC
> attribute.
> 
> Patch 24 adds support for freeing a page that has multiple references being
> held by a single caller.  This way we can free page fragments that were
> allocated by a given driver.
> 
> The last 3 patches use these updates in the igb driver to allow for us to
> reimpelement the use of build_skb.
> 
> My hope is to get the series accepted into the net-next tree as I have a
> number of other Intel drivers I could then begin updating once these
> patches are accepted.
> 
> v1: Split out changes DMA_ERROR_CODE fix for swiotlb-xen
>     Minor fixes based on issues found by kernel build bot
>     Few minor changes for issues found on code review
>     Added Acked-by for patches that were acked and not changed

I really appreciate you are doing this work Alex, thanks! And I do
think it fits into my page pool plans. Thanks!

-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Principal Kernel Engineer at Red Hat
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer

WARNING: multiple messages have this Message-ID (diff)
From: Jesper Dangaard Brouer <brouer@redhat.com>
To: Alexander Duyck <alexander.h.duyck@intel.com>
Cc: netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	davem@davemloft.net, brouer@redhat.com
Subject: Re: [net-next PATCH 00/27] Add support for DMA writable pages being writable by the network stack
Date: Wed, 26 Oct 2016 17:45:42 +0200	[thread overview]
Message-ID: <20161026174542.334798db@redhat.com> (raw)
In-Reply-To: <20161025153220.4815.61239.stgit@ahduyck-blue-test.jf.intel.com>

On Tue, 25 Oct 2016 11:36:48 -0400
Alexander Duyck <alexander.h.duyck@intel.com> wrote:

> The first 22 patches in the set add support for the DMA attribute
> DMA_ATTR_SKIP_CPU_SYNC on multiple platforms/architectures.  This is needed
> so that we can flag the calls to dma_map/unmap_page so that we do not
> invalidate cache lines that do not currently belong to the device.  Instead
> we have to take care of this in the driver via a call to
> sync_single_range_for_cpu prior to freeing the Rx page.
> 
> Patch 23 adds support for dma_map_page_attrs and dma_unmap_page_attrs so
> that we can unmap and map a page using the DMA_ATTR_SKIP_CPU_SYNC
> attribute.
> 
> Patch 24 adds support for freeing a page that has multiple references being
> held by a single caller.  This way we can free page fragments that were
> allocated by a given driver.
> 
> The last 3 patches use these updates in the igb driver to allow for us to
> reimpelement the use of build_skb.
> 
> My hope is to get the series accepted into the net-next tree as I have a
> number of other Intel drivers I could then begin updating once these
> patches are accepted.
> 
> v1: Split out changes DMA_ERROR_CODE fix for swiotlb-xen
>     Minor fixes based on issues found by kernel build bot
>     Few minor changes for issues found on code review
>     Added Acked-by for patches that were acked and not changed

I really appreciate you are doing this work Alex, thanks! And I do
think it fits into my page pool plans. Thanks!

-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Principal Kernel Engineer at Red Hat
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Jesper Dangaard Brouer <brouer@redhat.com>
To: Alexander Duyck <alexander.h.duyck@intel.com>
Cc: netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	davem@davemloft.net, brouer@redhat.com
Subject: Re: [net-next PATCH 00/27] Add support for DMA writable pages being writable by the network stack
Date: Wed, 26 Oct 2016 17:45:42 +0200	[thread overview]
Message-ID: <20161026174542.334798db@redhat.com> (raw)
In-Reply-To: <20161025153220.4815.61239.stgit@ahduyck-blue-test.jf.intel.com>

On Tue, 25 Oct 2016 11:36:48 -0400
Alexander Duyck <alexander.h.duyck@intel.com> wrote:

> The first 22 patches in the set add support for the DMA attribute
> DMA_ATTR_SKIP_CPU_SYNC on multiple platforms/architectures.  This is needed
> so that we can flag the calls to dma_map/unmap_page so that we do not
> invalidate cache lines that do not currently belong to the device.  Instead
> we have to take care of this in the driver via a call to
> sync_single_range_for_cpu prior to freeing the Rx page.
> 
> Patch 23 adds support for dma_map_page_attrs and dma_unmap_page_attrs so
> that we can unmap and map a page using the DMA_ATTR_SKIP_CPU_SYNC
> attribute.
> 
> Patch 24 adds support for freeing a page that has multiple references being
> held by a single caller.  This way we can free page fragments that were
> allocated by a given driver.
> 
> The last 3 patches use these updates in the igb driver to allow for us to
> reimpelement the use of build_skb.
> 
> My hope is to get the series accepted into the net-next tree as I have a
> number of other Intel drivers I could then begin updating once these
> patches are accepted.
> 
> v1: Split out changes DMA_ERROR_CODE fix for swiotlb-xen
>     Minor fixes based on issues found by kernel build bot
>     Few minor changes for issues found on code review
>     Added Acked-by for patches that were acked and not changed

I really appreciate you are doing this work Alex, thanks! And I do
think it fits into my page pool plans. Thanks!

-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Principal Kernel Engineer at Red Hat
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer

  parent reply	other threads:[~2016-10-26 15:45 UTC|newest]

Thread overview: 116+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-25 15:36 [Intel-wired-lan] [net-next PATCH 00/27] Add support for DMA writable pages being writable by the network stack Alexander Duyck
2016-10-25 15:36 ` Alexander Duyck
2016-10-25 15:36 ` Alexander Duyck
2016-10-25 15:36 ` [Intel-wired-lan] [net-next PATCH 01/27] swiotlb: Drop unused function swiotlb_map_sg Alexander Duyck
2016-10-25 15:36   ` Alexander Duyck
2016-10-25 15:36   ` Alexander Duyck
2016-10-25 15:36 ` [Intel-wired-lan] [net-next PATCH 02/27] swiotlb-xen: Enforce return of DMA_ERROR_CODE in mapping function Alexander Duyck
2016-10-25 15:36   ` Alexander Duyck
2016-10-25 15:36   ` Alexander Duyck
2016-10-28 17:35   ` [Intel-wired-lan] " Konrad Rzeszutek Wilk
2016-10-28 17:35     ` Konrad Rzeszutek Wilk
2016-10-28 17:35     ` Konrad Rzeszutek Wilk
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 03/27] swiotlb: Add support for DMA_ATTR_SKIP_CPU_SYNC Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-28 17:34   ` [Intel-wired-lan] " Konrad Rzeszutek Wilk
2016-10-28 17:34     ` Konrad Rzeszutek Wilk
2016-10-28 17:34     ` Konrad Rzeszutek Wilk
2016-10-28 18:09     ` [Intel-wired-lan] " Alexander Duyck
2016-10-28 18:09       ` Alexander Duyck
2016-10-28 18:09       ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 04/27] arch/arc: Add option to skip sync on DMA mapping Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 22:00   ` [Intel-wired-lan] " Vineet Gupta
2016-10-25 22:00     ` Vineet Gupta
2016-10-25 22:00     ` Vineet Gupta
2016-10-25 22:00     ` Vineet Gupta
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 05/27] arch/arm: Add option to skip sync on DMA map and unmap Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 06/27] arch/avr32: Add option to skip sync on DMA map Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 07/27] arch/blackfin: " Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 08/27] arch/c6x: Add option to skip sync on DMA map and unmap Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 09/27] arch/frv: Add option to skip sync on DMA map Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 10/27] arch/hexagon: Add option to skip DMA sync as a part of mapping Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 11/27] arch/m68k: " Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 12/27] arch/metag: Add option to skip DMA sync as a part of map and unmap Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37 ` [Intel-wired-lan] [net-next PATCH 13/27] arch/microblaze: " Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:37   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 14/27] arch/mips: " Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 15/27] arch/nios2: " Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 16/27] arch/openrisc: Add option to skip DMA sync as a part of mapping Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 17/27] arch/parisc: Add option to skip DMA sync as a part of map and unmap Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 18/27] arch/powerpc: Add option to skip DMA sync as a part of mapping Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 19/27] arch/sh: " Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 20/27] arch/sparc: Add option to skip DMA sync as a part of map and unmap Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 21/27] arch/tile: " Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 22/27] arch/xtensa: Add option to skip DMA sync as a part of mapping Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 23/27] dma: Add calls for dma_map_page_attrs and dma_unmap_page_attrs Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38 ` [Intel-wired-lan] [net-next PATCH 24/27] mm: Add support for releasing multiple instances of a page Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:38   ` Alexander Duyck
2016-10-25 15:39 ` [Intel-wired-lan] [net-next PATCH 25/27] igb: Update driver to make use of DMA_ATTR_SKIP_CPU_SYNC Alexander Duyck
2016-10-25 15:39   ` Alexander Duyck
2016-10-25 15:39   ` Alexander Duyck
2016-10-26 17:21   ` [Intel-wired-lan] " Jeff Kirsher
2016-10-26 17:21     ` Jeff Kirsher
2016-10-25 15:39 ` [Intel-wired-lan] [net-next PATCH 26/27] igb: Update code to better handle incrementing page count Alexander Duyck
2016-10-25 15:39   ` Alexander Duyck
2016-10-25 15:39   ` Alexander Duyck
2016-10-26 17:21   ` [Intel-wired-lan] " Jeff Kirsher
2016-10-26 17:21     ` Jeff Kirsher
2016-10-25 15:39 ` [Intel-wired-lan] [net-next PATCH 27/27] igb: Revert "igb: Revert support for build_skb in igb" Alexander Duyck
2016-10-25 15:39   ` Alexander Duyck
2016-10-25 15:39   ` Alexander Duyck
2016-10-26 17:22   ` [Intel-wired-lan] " Jeff Kirsher
2016-10-26 17:22     ` Jeff Kirsher
2016-10-26 15:45 ` Jesper Dangaard Brouer [this message]
2016-10-26 15:45   ` [net-next PATCH 00/27] Add support for DMA writable pages being writable by the network stack Jesper Dangaard Brouer
2016-10-26 15:45   ` Jesper Dangaard Brouer
2016-10-28 15:48 ` [Intel-wired-lan] " Alexander Duyck
2016-10-28 15:48   ` Alexander Duyck
2016-10-28 15:48   ` Alexander Duyck
2016-10-28 17:06   ` [Intel-wired-lan] " David Miller
2016-10-28 17:06     ` David Miller
2016-10-28 17:06     ` David Miller

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=20161026174542.334798db@redhat.com \
    --to=brouer@redhat.com \
    --cc=intel-wired-lan@osuosl.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.