* [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter
@ 2024-06-14 8:14 Thomas Huth
2024-06-14 11:34 ` Christian Borntraeger
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Thomas Huth @ 2024-06-14 8:14 UTC (permalink / raw)
To: linux-doc, Christian Borntraeger, Janosch Frank, Claudio Imbrenda,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev, linux-s390
Cc: linux-kernel, Jonathan Corbet, Sven Schnelle
The "swiotlb" kernel parameter is used on s390 for protected virt since
commit 64e1f0c531d1 ("s390/mm: force swiotlb for protected virtualization")
and thus should be marked in kernel-parameters.txt accordingly.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
PS: I wonder whether we could remove IA-64 nowadays...?
Documentation/admin-guide/kernel-parameters.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
index b600df82669d..423427bf6e49 100644
--- a/Documentation/admin-guide/kernel-parameters.txt
+++ b/Documentation/admin-guide/kernel-parameters.txt
@@ -6548,7 +6548,7 @@
This parameter controls use of the Protected
Execution Facility on pSeries.
- swiotlb= [ARM,IA-64,PPC,MIPS,X86,EARLY]
+ swiotlb= [ARM,IA-64,PPC,MIPS,X86,S390,EARLY]
Format: { <int> [,<int>] | force | noforce }
<int> -- Number of I/O TLB slabs
<int> -- Second integer after comma. Number of swiotlb
--
2.45.2
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter
2024-06-14 8:14 [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter Thomas Huth
@ 2024-06-14 11:34 ` Christian Borntraeger
2024-06-14 12:39 ` Claudio Imbrenda
2024-06-17 22:14 ` Jonathan Corbet
2 siblings, 0 replies; 4+ messages in thread
From: Christian Borntraeger @ 2024-06-14 11:34 UTC (permalink / raw)
To: Thomas Huth, linux-doc, Janosch Frank, Claudio Imbrenda,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev, linux-s390
Cc: linux-kernel, Jonathan Corbet, Sven Schnelle
Am 14.06.24 um 10:14 schrieb Thomas Huth:
> The "swiotlb" kernel parameter is used on s390 for protected virt since
> commit 64e1f0c531d1 ("s390/mm: force swiotlb for protected virtualization")
> and thus should be marked in kernel-parameters.txt accordingly.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
Acked-by: Christian Borntraeger <borntraeger@linux.ibm.com>
> ---
> PS: I wonder whether we could remove IA-64 nowadays...?
probably
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter
2024-06-14 8:14 [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter Thomas Huth
2024-06-14 11:34 ` Christian Borntraeger
@ 2024-06-14 12:39 ` Claudio Imbrenda
2024-06-17 22:14 ` Jonathan Corbet
2 siblings, 0 replies; 4+ messages in thread
From: Claudio Imbrenda @ 2024-06-14 12:39 UTC (permalink / raw)
To: Thomas Huth
Cc: linux-doc, Christian Borntraeger, Janosch Frank, Heiko Carstens,
Vasily Gorbik, Alexander Gordeev, linux-s390, linux-kernel,
Jonathan Corbet, Sven Schnelle
On Fri, 14 Jun 2024 10:14:38 +0200
Thomas Huth <thuth@redhat.com> wrote:
> The "swiotlb" kernel parameter is used on s390 for protected virt since
> commit 64e1f0c531d1 ("s390/mm: force swiotlb for protected virtualization")
> and thus should be marked in kernel-parameters.txt accordingly.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
Acked-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
> ---
> PS: I wonder whether we could remove IA-64 nowadays...?
>
> Documentation/admin-guide/kernel-parameters.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index b600df82669d..423427bf6e49 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -6548,7 +6548,7 @@
> This parameter controls use of the Protected
> Execution Facility on pSeries.
>
> - swiotlb= [ARM,IA-64,PPC,MIPS,X86,EARLY]
> + swiotlb= [ARM,IA-64,PPC,MIPS,X86,S390,EARLY]
> Format: { <int> [,<int>] | force | noforce }
> <int> -- Number of I/O TLB slabs
> <int> -- Second integer after comma. Number of swiotlb
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter
2024-06-14 8:14 [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter Thomas Huth
2024-06-14 11:34 ` Christian Borntraeger
2024-06-14 12:39 ` Claudio Imbrenda
@ 2024-06-17 22:14 ` Jonathan Corbet
2 siblings, 0 replies; 4+ messages in thread
From: Jonathan Corbet @ 2024-06-17 22:14 UTC (permalink / raw)
To: Thomas Huth, linux-doc, Christian Borntraeger, Janosch Frank,
Claudio Imbrenda, Heiko Carstens, Vasily Gorbik,
Alexander Gordeev, linux-s390
Cc: linux-kernel, Sven Schnelle
Thomas Huth <thuth@redhat.com> writes:
> The "swiotlb" kernel parameter is used on s390 for protected virt since
> commit 64e1f0c531d1 ("s390/mm: force swiotlb for protected virtualization")
> and thus should be marked in kernel-parameters.txt accordingly.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
Applied, thanks.
For the record, this is a great set of cleanups, thanks for doing this.
> PS: I wonder whether we could remove IA-64 nowadays...?
Yes, references to IA-64 should come out at this point. If you want to
do that, I'd suggest doing it as one big patch... in theory, that
architecture could come back if enough maintenance initiative is shown.
I don't think it will happen but, if it does, it would be nice to be
able to undo the docs change with a single revert.
Thanks,
jon
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-06-17 22:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-14 8:14 [PATCH] Documentation: Add "S390" to the swiotlb kernel parameter Thomas Huth
2024-06-14 11:34 ` Christian Borntraeger
2024-06-14 12:39 ` Claudio Imbrenda
2024-06-17 22:14 ` Jonathan Corbet
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox