From: Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
To: "Vincent Stehlé"
<vincent.stehle-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
Cc: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
linux-next-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH linux-next] iommu: Enable LPAE pagetable on ARM only
Date: Mon, 23 Feb 2015 18:52:41 +0200 [thread overview]
Message-ID: <2868840.GyWYJOzqWH@avalon> (raw)
In-Reply-To: <1424621346-7822-1-git-send-email-vincent.stehle-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
Hi Vincent,
Thank you for the patch.
On Sunday 22 February 2015 17:09:06 Vincent Stehlé wrote:
> LPAE iommu page table makes sense only for ARM architecture. Add the
> corresponding dependency in Kconfig.
>
> This fixes the following runtime warnings on PA-RISC architecture:
I suppose this is caused by the selftest code ?
> WARNING: at drivers/iommu/io-pgtable-arm.c:413
> Modules linked in:
> CPU: 0 PID: 1 Comm: swapper/0 Not tainted
> 3.19.0-next-20150222-00002-gc420256 #2 task: 17ca0000 ti: 17ca8000 task.ti:
> 17ca8000
>
> YZrvWESTHLNXBCVMcbcbcbcbOGFRQPDI
> PSW: 00000000000001001111111100001111 Not tainted
> r00-03 0004ff0f 0000001b 104871b0 17ca83c0
> r04-07 00001000 00000000 00000000 00000000
> r08-11 00000000 40001000 1013b36c 17ca8248
> r12-15 00000080 17e712c0 00000000 17c5e100
> r16-19 0000000c 105d8eb0 40001000 ffffffff
> r20-23 00000000 00800020 1048717c 00000000
> r24-27 00001000 00000000 00000009 10774ce0
> r28-31 00000000 00000027 17ca8480 10486d44
> sr00-03 00000000 00000000 00000000 00000000
> sr04-07 00000000 00000000 00000000 00000000
>
> IASQ: 00000000 00000000 IAOQ: 10486df4 10486df8
> IIR: 03ffe01f ISR: 00000000 IOR: 17c86e54
> CPU: 0 CR30: 17ca8000 CR31: 00000002
> ORIG_R28: 00000001
> IAOQ[0]: __arm_lpae_unmap+0x144/0x4cc
> IAOQ[1]: __arm_lpae_unmap+0x148/0x4cc
> RP(r2): arm_lpae_unmap+0x34/0x6c
> Backtrace:
> [<104871b0>] arm_lpae_unmap+0x34/0x6c
> [<10127944>] 0x10127944
> [<10168ee0>] do_one_initcall+0x128/0x1dc
> [<101028fc>] 0x101028fc
> [<10161b48>] kernel_init+0x1c/0x118
> [<1015d01c>] end_fault_vector+0x1c/0x24
> [<101762ec>] save_stack_trace+0x28/0x60
>
> Signed-off-by: Vincent Stehlé <vincent.stehle-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
> Cc: Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
> Cc: Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>
> ---
>
>
> Hi,
>
> This can be seen with e.g. Linux next-20150222 on parisc with allmodconfig.
>
> Best regards,
>
> V.
>
>
> drivers/iommu/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/iommu/Kconfig b/drivers/iommu/Kconfig
> index baa0d97..c73fd72 100644
> --- a/drivers/iommu/Kconfig
> +++ b/drivers/iommu/Kconfig
> @@ -22,6 +22,7 @@ config IOMMU_IO_PGTABLE
>
> config IOMMU_IO_PGTABLE_LPAE
> bool "ARMv7/v8 Long Descriptor Format"
> + depends on ARM
How about ARM || COMPILE_TEST ? It can still be useful to compile the module
on other architectures.
> select IOMMU_IO_PGTABLE
> help
> Enable support for the ARM long descriptor pagetable format.
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: laurent.pinchart@ideasonboard.com (Laurent Pinchart)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH linux-next] iommu: Enable LPAE pagetable on ARM only
Date: Mon, 23 Feb 2015 18:52:41 +0200 [thread overview]
Message-ID: <2868840.GyWYJOzqWH@avalon> (raw)
In-Reply-To: <1424621346-7822-1-git-send-email-vincent.stehle@laposte.net>
Hi Vincent,
Thank you for the patch.
On Sunday 22 February 2015 17:09:06 Vincent Stehl? wrote:
> LPAE iommu page table makes sense only for ARM architecture. Add the
> corresponding dependency in Kconfig.
>
> This fixes the following runtime warnings on PA-RISC architecture:
I suppose this is caused by the selftest code ?
> WARNING: at drivers/iommu/io-pgtable-arm.c:413
> Modules linked in:
> CPU: 0 PID: 1 Comm: swapper/0 Not tainted
> 3.19.0-next-20150222-00002-gc420256 #2 task: 17ca0000 ti: 17ca8000 task.ti:
> 17ca8000
>
> YZrvWESTHLNXBCVMcbcbcbcbOGFRQPDI
> PSW: 00000000000001001111111100001111 Not tainted
> r00-03 0004ff0f 0000001b 104871b0 17ca83c0
> r04-07 00001000 00000000 00000000 00000000
> r08-11 00000000 40001000 1013b36c 17ca8248
> r12-15 00000080 17e712c0 00000000 17c5e100
> r16-19 0000000c 105d8eb0 40001000 ffffffff
> r20-23 00000000 00800020 1048717c 00000000
> r24-27 00001000 00000000 00000009 10774ce0
> r28-31 00000000 00000027 17ca8480 10486d44
> sr00-03 00000000 00000000 00000000 00000000
> sr04-07 00000000 00000000 00000000 00000000
>
> IASQ: 00000000 00000000 IAOQ: 10486df4 10486df8
> IIR: 03ffe01f ISR: 00000000 IOR: 17c86e54
> CPU: 0 CR30: 17ca8000 CR31: 00000002
> ORIG_R28: 00000001
> IAOQ[0]: __arm_lpae_unmap+0x144/0x4cc
> IAOQ[1]: __arm_lpae_unmap+0x148/0x4cc
> RP(r2): arm_lpae_unmap+0x34/0x6c
> Backtrace:
> [<104871b0>] arm_lpae_unmap+0x34/0x6c
> [<10127944>] 0x10127944
> [<10168ee0>] do_one_initcall+0x128/0x1dc
> [<101028fc>] 0x101028fc
> [<10161b48>] kernel_init+0x1c/0x118
> [<1015d01c>] end_fault_vector+0x1c/0x24
> [<101762ec>] save_stack_trace+0x28/0x60
>
> Signed-off-by: Vincent Stehl? <vincent.stehle@laposte.net>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Will Deacon <will.deacon@arm.com>
> ---
>
>
> Hi,
>
> This can be seen with e.g. Linux next-20150222 on parisc with allmodconfig.
>
> Best regards,
>
> V.
>
>
> drivers/iommu/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/iommu/Kconfig b/drivers/iommu/Kconfig
> index baa0d97..c73fd72 100644
> --- a/drivers/iommu/Kconfig
> +++ b/drivers/iommu/Kconfig
> @@ -22,6 +22,7 @@ config IOMMU_IO_PGTABLE
>
> config IOMMU_IO_PGTABLE_LPAE
> bool "ARMv7/v8 Long Descriptor Format"
> + depends on ARM
How about ARM || COMPILE_TEST ? It can still be useful to compile the module
on other architectures.
> select IOMMU_IO_PGTABLE
> help
> Enable support for the ARM long descriptor pagetable format.
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: "Vincent Stehlé" <vincent.stehle@laposte.net>
Cc: iommu@lists.linux-foundation.org, linux-next@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
Will Deacon <will.deacon@arm.com>
Subject: Re: [PATCH linux-next] iommu: Enable LPAE pagetable on ARM only
Date: Mon, 23 Feb 2015 18:52:41 +0200 [thread overview]
Message-ID: <2868840.GyWYJOzqWH@avalon> (raw)
In-Reply-To: <1424621346-7822-1-git-send-email-vincent.stehle@laposte.net>
Hi Vincent,
Thank you for the patch.
On Sunday 22 February 2015 17:09:06 Vincent Stehlé wrote:
> LPAE iommu page table makes sense only for ARM architecture. Add the
> corresponding dependency in Kconfig.
>
> This fixes the following runtime warnings on PA-RISC architecture:
I suppose this is caused by the selftest code ?
> WARNING: at drivers/iommu/io-pgtable-arm.c:413
> Modules linked in:
> CPU: 0 PID: 1 Comm: swapper/0 Not tainted
> 3.19.0-next-20150222-00002-gc420256 #2 task: 17ca0000 ti: 17ca8000 task.ti:
> 17ca8000
>
> YZrvWESTHLNXBCVMcbcbcbcbOGFRQPDI
> PSW: 00000000000001001111111100001111 Not tainted
> r00-03 0004ff0f 0000001b 104871b0 17ca83c0
> r04-07 00001000 00000000 00000000 00000000
> r08-11 00000000 40001000 1013b36c 17ca8248
> r12-15 00000080 17e712c0 00000000 17c5e100
> r16-19 0000000c 105d8eb0 40001000 ffffffff
> r20-23 00000000 00800020 1048717c 00000000
> r24-27 00001000 00000000 00000009 10774ce0
> r28-31 00000000 00000027 17ca8480 10486d44
> sr00-03 00000000 00000000 00000000 00000000
> sr04-07 00000000 00000000 00000000 00000000
>
> IASQ: 00000000 00000000 IAOQ: 10486df4 10486df8
> IIR: 03ffe01f ISR: 00000000 IOR: 17c86e54
> CPU: 0 CR30: 17ca8000 CR31: 00000002
> ORIG_R28: 00000001
> IAOQ[0]: __arm_lpae_unmap+0x144/0x4cc
> IAOQ[1]: __arm_lpae_unmap+0x148/0x4cc
> RP(r2): arm_lpae_unmap+0x34/0x6c
> Backtrace:
> [<104871b0>] arm_lpae_unmap+0x34/0x6c
> [<10127944>] 0x10127944
> [<10168ee0>] do_one_initcall+0x128/0x1dc
> [<101028fc>] 0x101028fc
> [<10161b48>] kernel_init+0x1c/0x118
> [<1015d01c>] end_fault_vector+0x1c/0x24
> [<101762ec>] save_stack_trace+0x28/0x60
>
> Signed-off-by: Vincent Stehlé <vincent.stehle@laposte.net>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Will Deacon <will.deacon@arm.com>
> ---
>
>
> Hi,
>
> This can be seen with e.g. Linux next-20150222 on parisc with allmodconfig.
>
> Best regards,
>
> V.
>
>
> drivers/iommu/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/iommu/Kconfig b/drivers/iommu/Kconfig
> index baa0d97..c73fd72 100644
> --- a/drivers/iommu/Kconfig
> +++ b/drivers/iommu/Kconfig
> @@ -22,6 +22,7 @@ config IOMMU_IO_PGTABLE
>
> config IOMMU_IO_PGTABLE_LPAE
> bool "ARMv7/v8 Long Descriptor Format"
> + depends on ARM
How about ARM || COMPILE_TEST ? It can still be useful to compile the module
on other architectures.
> select IOMMU_IO_PGTABLE
> help
> Enable support for the ARM long descriptor pagetable format.
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2015-02-23 16:52 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-22 16:09 [PATCH linux-next] iommu: Enable LPAE pagetable on ARM only Vincent Stehlé
2015-02-22 16:09 ` Vincent Stehlé
[not found] ` <1424621346-7822-1-git-send-email-vincent.stehle-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
2015-02-23 16:52 ` Laurent Pinchart [this message]
2015-02-23 16:52 ` Laurent Pinchart
2015-02-23 16:52 ` Laurent Pinchart
2015-02-23 23:36 ` Vincent Stehlé
2015-02-23 23:36 ` Vincent Stehlé
2015-02-23 23:55 ` [PATCH v2 " Vincent Stehlé
2015-02-23 23:55 ` Vincent Stehlé
2015-02-23 23:55 ` Vincent Stehlé
2015-02-23 23:58 ` Laurent Pinchart
2015-02-23 23:58 ` Laurent Pinchart
2015-02-24 0:02 ` Laurent Pinchart
2015-02-24 0:02 ` Laurent Pinchart
2015-02-24 0:02 ` Laurent Pinchart
2015-02-24 0:20 ` [PATCH v3 linux-next] iommu: Enable LPAE pagetable on ARM and ARM64 only Vincent Stehlé
2015-02-24 0:20 ` Vincent Stehlé
2015-02-24 0:23 ` Laurent Pinchart
2015-02-24 0:23 ` Laurent Pinchart
2015-02-25 9:59 ` Will Deacon
2015-02-25 9:59 ` Will Deacon
2015-02-24 10:33 ` [PATCH linux-next] iommu: Enable LPAE pagetable on ARM only Geert Uytterhoeven
2015-02-24 10:33 ` Geert Uytterhoeven
2015-02-24 10:33 ` Geert Uytterhoeven
2015-02-25 10:00 ` Will Deacon
2015-02-25 10:00 ` Will Deacon
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=2868840.GyWYJOzqWH@avalon \
--to=laurent.pinchart-rylnwiuwjnjg/c1bvhzhaw@public.gmane.org \
--cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-next-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=vincent.stehle-QFKgK+z4sOrR7s880joybQ@public.gmane.org \
--cc=will.deacon-5wv7dgnIgG8@public.gmane.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.