Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1676471 > unrolled thread

[PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"

Started byVladimir Murzin <vladimir.murzin@arm.com>
First post2017-06-28 11:20 +0200
Last post2017-06-30 14:10 +0200
Articles 6 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU" Vladimir Murzin <vladimir.murzin@arm.com> - 2017-06-28 11:20 +0200
    Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and         friends for NOMMU" Christoph Hellwig <hch@lst.de> - 2017-06-28 16:00 +0200
      Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and  friends for NOMMU" Vladimir Murzin <vladimir.murzin@arm.com> - 2017-07-03 11:00 +0200
      Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and  friends for NOMMU" Vladimir Murzin <vladimir.murzin@arm.com> - 2017-07-03 11:00 +0200
    Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and  friends for NOMMU" Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2017-06-30 13:40 +0200
      Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and  friends for NOMMU" Vladimir Murzin <vladimir.murzin@arm.com> - 2017-06-30 14:10 +0200

#1676471 — [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"

FromVladimir Murzin <vladimir.murzin@arm.com>
Date2017-06-28 11:20 +0200
Subject[PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"
Message-ID<tXhiG-4WZ-15@gated-at.bofh.it>
These two patches are folloups on "[PATCH v6 0/7] ARM: Fix
dma_alloc_coherent() and friends for NOMMU" which has been partly
merged in [1]. PATCH 1/2 is done per Christoph's suggestion [2] to
eliminate duplication in dma-noop.c.  PATCH 2/2 is fixup for ARM's dma
operations for NOMMU due to aforementioned change.

@Benjamin, your Tested-by for ARM bits would be very appretiated!

@Christoph, in case you decide to queue PATCH 1/2, can you put it
before "ARM: NOMMU: Introduce dma operations for noMMU", so transition
to the new dma operations won't break bisectability.

[1] git://git.infradead.org/users/hch/dma-mapping.git for-next
[2] https://www.spinics.net/lists/arm-kernel/msg590587.html

Cc: Steven Miao <realmz6@gmail.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Michal Simek <monstr@monstr.eu>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Cc: Rich Felker <dalias@libc.org>
Cc: Chris Zankel <chris@zankel.net>
Cc: Max Filippov <jcmvbkbc@gmail.com>

Thanks!

Vladimir Murzin (2):
  drivers: dma-mapping: allow dma_common_mmap() for NOMMU
  fixup: ARM: NOMMU: Introduce dma operations for noMMU

 arch/arm/mm/dma-mapping-nommu.c | 18 ------------------
 arch/blackfin/Kconfig           |  1 +
 arch/m32r/Kconfig               |  1 +
 arch/m68k/Kconfig               |  1 +
 arch/microblaze/Kconfig         |  1 +
 arch/sh/Kconfig                 |  1 +
 arch/xtensa/Kconfig             |  1 +
 drivers/base/dma-mapping.c      |  4 ++--
 8 files changed, 8 insertions(+), 20 deletions(-)

-- 
2.0.0

[toc] | [next] | [standalone]


#1676687 — Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"

FromChristoph Hellwig <hch@lst.de>
Date2017-06-28 16:00 +0200
SubjectRe: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"
Message-ID<tXlFF-7uy-35@gated-at.bofh.it>
In reply to#1676471
Thanks Vlad,

these look fine to me.  I'll wait another day or two for comments
(especially if other nommu archs actually want this code), and
if nothing shows up will apply them.

[toc] | [prev] | [next] | [standalone]


#1679776 — Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"

FromVladimir Murzin <vladimir.murzin@arm.com>
Date2017-07-03 11:00 +0200
SubjectRe: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"
Message-ID<tZ5n3-78c-19@gated-at.bofh.it>
In reply to#1676687
On 03/07/17 09:49, Vladimir Murzin wrote:
> On 28/06/17 14:52, Christoph Hellwig wrote:
>> Thanks Vlad,
>>
>> these look fine to me.  I'll wait another day or two for comments
>> (especially if other nommu archs actually want this code), and
>> if nothing shows up will apply them.
>>
> 
> Gentle remainder to make sure this follow-up is not lost ;)

I should have checked the dma-mapping repo first. I see it was applied two
days ago..

Thanks a lot!.. and sorry for the noise!

Vladimir

> 
> Cheers
> Vladimir
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

[toc] | [prev] | [next] | [standalone]


#1679781 — Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"

FromVladimir Murzin <vladimir.murzin@arm.com>
Date2017-07-03 11:00 +0200
SubjectRe: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"
Message-ID<tZ5n4-78c-21@gated-at.bofh.it>
In reply to#1676687
On 28/06/17 14:52, Christoph Hellwig wrote:
> Thanks Vlad,
> 
> these look fine to me.  I'll wait another day or two for comments
> (especially if other nommu archs actually want this code), and
> if nothing shows up will apply them.
> 

Gentle remainder to make sure this follow-up is not lost ;)

Cheers
Vladimir

[toc] | [prev] | [next] | [standalone]


#1678714 — Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"

FromBenjamin Gaignard <benjamin.gaignard@linaro.org>
Date2017-06-30 13:40 +0200
SubjectRe: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"
Message-ID<tY2rg-4Tg-1@gated-at.bofh.it>
In reply to#1676471
2017-06-28 11:16 GMT+02:00 Vladimir Murzin <vladimir.murzin@arm.com>:
> These two patches are folloups on "[PATCH v6 0/7] ARM: Fix
> dma_alloc_coherent() and friends for NOMMU" which has been partly
> merged in [1]. PATCH 1/2 is done per Christoph's suggestion [2] to
> eliminate duplication in dma-noop.c.  PATCH 2/2 is fixup for ARM's dma
> operations for NOMMU due to aforementioned change.
>
> @Benjamin, your Tested-by for ARM bits would be very appretiated!

I have been able to test on stm32f4 with drm/kms drivers which is
calling dma_alloc_coherent
and modetest doing mmap and it is working fine.

tested-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

>
> @Christoph, in case you decide to queue PATCH 1/2, can you put it
> before "ARM: NOMMU: Introduce dma operations for noMMU", so transition
> to the new dma operations won't break bisectability.
>
> [1] git://git.infradead.org/users/hch/dma-mapping.git for-next
> [2] https://www.spinics.net/lists/arm-kernel/msg590587.html
>
> Cc: Steven Miao <realmz6@gmail.com>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> Cc: Michal Simek <monstr@monstr.eu>
> Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
> Cc: Rich Felker <dalias@libc.org>
> Cc: Chris Zankel <chris@zankel.net>
> Cc: Max Filippov <jcmvbkbc@gmail.com>
>
> Thanks!
>
> Vladimir Murzin (2):
>   drivers: dma-mapping: allow dma_common_mmap() for NOMMU
>   fixup: ARM: NOMMU: Introduce dma operations for noMMU
>
>  arch/arm/mm/dma-mapping-nommu.c | 18 ------------------
>  arch/blackfin/Kconfig           |  1 +
>  arch/m32r/Kconfig               |  1 +
>  arch/m68k/Kconfig               |  1 +
>  arch/microblaze/Kconfig         |  1 +
>  arch/sh/Kconfig                 |  1 +
>  arch/xtensa/Kconfig             |  1 +
>  drivers/base/dma-mapping.c      |  4 ++--
>  8 files changed, 8 insertions(+), 20 deletions(-)
>
> --
> 2.0.0
>



-- 
Benjamin Gaignard

Graphic Study Group

Linaro.org │ Open source software for ARM SoCs

Follow Linaro: Facebook | Twitter | Blog

[toc] | [prev] | [next] | [standalone]


#1678737 — Re: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"

FromVladimir Murzin <vladimir.murzin@arm.com>
Date2017-06-30 14:10 +0200
SubjectRe: [PATCH 0/2] Follow-up on "ARM: Fix dma_alloc_coherent() and friends for NOMMU"
Message-ID<tY2Uh-5iT-11@gated-at.bofh.it>
In reply to#1678714
On 30/06/17 12:36, Benjamin Gaignard wrote:
> 2017-06-28 11:16 GMT+02:00 Vladimir Murzin <vladimir.murzin@arm.com>:
>> These two patches are folloups on "[PATCH v6 0/7] ARM: Fix
>> dma_alloc_coherent() and friends for NOMMU" which has been partly
>> merged in [1]. PATCH 1/2 is done per Christoph's suggestion [2] to
>> eliminate duplication in dma-noop.c.  PATCH 2/2 is fixup for ARM's dma
>> operations for NOMMU due to aforementioned change.
>>
>> @Benjamin, your Tested-by for ARM bits would be very appretiated!
> 
> I have been able to test on stm32f4 with drm/kms drivers which is
> calling dma_alloc_coherent
> and modetest doing mmap and it is working fine.
> 
> tested-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> 

Great thanks!

Vladimir

>>
>> @Christoph, in case you decide to queue PATCH 1/2, can you put it
>> before "ARM: NOMMU: Introduce dma operations for noMMU", so transition
>> to the new dma operations won't break bisectability.
>>
>> [1] git://git.infradead.org/users/hch/dma-mapping.git for-next
>> [2] https://www.spinics.net/lists/arm-kernel/msg590587.html
>>
>> Cc: Steven Miao <realmz6@gmail.com>
>> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
>> Cc: Michal Simek <monstr@monstr.eu>
>> Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
>> Cc: Rich Felker <dalias@libc.org>
>> Cc: Chris Zankel <chris@zankel.net>
>> Cc: Max Filippov <jcmvbkbc@gmail.com>
>>
>> Thanks!
>>
>> Vladimir Murzin (2):
>>   drivers: dma-mapping: allow dma_common_mmap() for NOMMU
>>   fixup: ARM: NOMMU: Introduce dma operations for noMMU
>>
>>  arch/arm/mm/dma-mapping-nommu.c | 18 ------------------
>>  arch/blackfin/Kconfig           |  1 +
>>  arch/m32r/Kconfig               |  1 +
>>  arch/m68k/Kconfig               |  1 +
>>  arch/microblaze/Kconfig         |  1 +
>>  arch/sh/Kconfig                 |  1 +
>>  arch/xtensa/Kconfig             |  1 +
>>  drivers/base/dma-mapping.c      |  4 ++--
>>  8 files changed, 8 insertions(+), 20 deletions(-)
>>
>> --
>> 2.0.0
>>
> 
> 
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web