Re: [PATCH] docs: kernel-parameters.txt: Fix whitespace

2018-04-16 Thread Thymo van Beers
On Mon, Apr 16, 2018 at 02:08:46PM -0600, Jonathan Corbet wrote:
> On Mon, 16 Apr 2018 17:45:01 +0200
> Thymo van Beers  wrote:
> 
> > Some lines used spaces instead of tabs at line start.
> > This can cause mangled lines in editors due to inconsistency.
> > 
> > Replace spaces for tabs where appropriate.
> 
> Seems like a fine idea.  The patch doesn't apply, though; can you please
> make a version against current docs-next?  (Or against 4.17-rc1 will work
> too).
> 
> Also...
> 
> > domain
> > - Isolate from the general SMP balancing and scheduling
> > - algorithms. Note that performing domain isolation 
> > this way
> > - is irreversible: it's not possible to bring back a 
> > CPU to
> > - the domains once isolated through isolcpus. It's 
> > strongly
> > - advised to use cpusets instead to disable scheduler 
> > load
> > - balancing through the "cpuset.sched_load_balance" 
> > file.
> > - It offers a much more flexible interface where CPUs 
> > can
> > - move in and out of an isolated set anytime.
> > -
> > - You can move a process onto or off an "isolated" CPU 
> > via
> > - the CPU affinity syscalls or cpuset.
> > -  begins at 0 and the maximum value is
> > - "number of CPUs in system - 1".
> > +   Isolate from the general SMP balancing and 
> > scheduling
> > +   algorithms. Note that performing domain 
> > isolation this way
> > +   is irreversible: it's not possible to bring 
> > back a CPU to
> > +   the domains once isolated through isolcpus. 
> > It's strongly
> > +   advised to use cpusets instead to disable 
> > scheduler load
> > +   balancing through the 
> > "cpuset.sched_load_balance" file.
> > +   It offers a much more flexible interface where 
> > CPUs can
> > +   move in and out of an isolated set anytime.
> > +
> > +   You can move a process onto or off an "isolated" CPU via
> > +   the CPU affinity syscalls or cpuset.
> > +begins at 0 and the maximum value is
> > +   "number of CPUs in system - 1".
> 
> This would appear to have changed the indentation of some of the text?
> 
> Thanks,
> 
> jon

Thanks for the comment.

I will rebase against docs-next. Hadn't realised my tree wasn't
up-to-date.

Upon reviewing that section I see that was intentional indentation and
not some tab/space mix up.

Both will be fixed in v2.

- Thymo
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] docs: kernel-parameters.txt: Fix whitespace

2018-04-16 Thread Jonathan Corbet
On Mon, 16 Apr 2018 17:45:01 +0200
Thymo van Beers  wrote:

> Some lines used spaces instead of tabs at line start.
> This can cause mangled lines in editors due to inconsistency.
> 
> Replace spaces for tabs where appropriate.

Seems like a fine idea.  The patch doesn't apply, though; can you please
make a version against current docs-next?  (Or against 4.17-rc1 will work
too).

Also...

>   domain
> -   Isolate from the general SMP balancing and scheduling
> -   algorithms. Note that performing domain isolation 
> this way
> -   is irreversible: it's not possible to bring back a 
> CPU to
> -   the domains once isolated through isolcpus. It's 
> strongly
> -   advised to use cpusets instead to disable scheduler 
> load
> -   balancing through the "cpuset.sched_load_balance" 
> file.
> -   It offers a much more flexible interface where CPUs 
> can
> -   move in and out of an isolated set anytime.
> -
> -   You can move a process onto or off an "isolated" CPU 
> via
> -   the CPU affinity syscalls or cpuset.
> -begins at 0 and the maximum value is
> -   "number of CPUs in system - 1".
> + Isolate from the general SMP balancing and 
> scheduling
> + algorithms. Note that performing domain 
> isolation this way
> + is irreversible: it's not possible to bring 
> back a CPU to
> + the domains once isolated through isolcpus. 
> It's strongly
> + advised to use cpusets instead to disable 
> scheduler load
> + balancing through the 
> "cpuset.sched_load_balance" file.
> + It offers a much more flexible interface where 
> CPUs can
> + move in and out of an isolated set anytime.
> +
> + You can move a process onto or off an "isolated" CPU via
> + the CPU affinity syscalls or cpuset.
> +  begins at 0 and the maximum value is
> + "number of CPUs in system - 1".

This would appear to have changed the indentation of some of the text?

Thanks,

jon
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] docs: kernel-parameters.txt: Fix whitespace

2018-04-16 Thread Thymo van Beers
Some lines used spaces instead of tabs at line start.
This can cause mangled lines in editors due to inconsistency.

Replace spaces for tabs where appropriate.

Signed-off-by: Thymo van Beers 
---
 Documentation/admin-guide/kernel-parameters.txt | 168 
 1 file changed, 84 insertions(+), 84 deletions(-)

diff --git a/Documentation/admin-guide/kernel-parameters.txt 
b/Documentation/admin-guide/kernel-parameters.txt
index 1d1d53f85ddd..3aa8eed85f39 100644
--- a/Documentation/admin-guide/kernel-parameters.txt
+++ b/Documentation/admin-guide/kernel-parameters.txt
@@ -106,11 +106,11 @@
use by PCI
Format: ,...
 
-   acpi_mask_gpe=  [HW,ACPI]
+   acpi_mask_gpe=  [HW,ACPI]
Due to the existence of _Lxx/_Exx, some GPEs triggered
by unsupported hardware/firmware features can result in
-GPE floodings that cannot be automatically disabled by
-the GPE dispatcher.
+   GPE floodings that cannot be automatically disabled by
+   the GPE dispatcher.
This facility can be used to prevent such uncontrolled
GPE floodings.
Format: 
@@ -472,10 +472,10 @@
for platform specific values (SB1, Loongson3 and
others).
 
-   ccw_timeout_log [S390]
+   ccw_timeout_log [S390]
See Documentation/s390/CommonIO for details.
 
-   cgroup_disable= [KNL] Disable a particular controller
+   cgroup_disable= [KNL] Disable a particular controller
Format: {name of the controller(s) to disable}
The effects of cgroup_disable=foo are:
- foo isn't auto-mounted if you mount all cgroups in
@@ -641,8 +641,8 @@
hvc  Use the hypervisor console device . This is for
both Xen and PowerPC hypervisors.
 
-If the device connected to the port is not a TTY but a braille
-device, prepend "brl," before the device type, for instance
+   If the device connected to the port is not a TTY but a braille
+   device, prepend "brl," before the device type, for instance
console=brl,ttyS0
For now, only VisioBraille is supported.
 
@@ -662,7 +662,7 @@
 
consoleblank=   [KNL] The console blank (screen saver) timeout in
seconds. A value of 0 disables the blank timer.
-   Defaults to 0.
+   Defaults to 0.
 
coredump_filter=
[KNL] Change the default value for
@@ -730,7 +730,7 @@
or memory reserved is below 4G.
 
cryptomgr.notests
-[KNL] Disable crypto self-tests
+   [KNL] Disable crypto self-tests
 
cs89x0_dma= [HW,NET]
Format: 
@@ -746,7 +746,7 @@
Format: ,
See also 
Documentation/input/devices/joystick-parport.rst
 
-   ddebug_query=   [KNL,DYNAMIC_DEBUG] Enable debug messages at early boot
+   ddebug_query=   [KNL,DYNAMIC_DEBUG] Enable debug messages at early boot
time. See
Documentation/admin-guide/dynamic-debug-howto.rst for
details.  Deprecated, see dyndbg.
@@ -833,7 +833,7 @@
causing system reset or hang due to sending
INIT from AP to BSP.
 
-   disable_ddw [PPC/PSERIES]
+   disable_ddw [PPC/PSERIES]
Disable Dynamic DMA Window support. Use this if
to workaround buggy firmware.
 
@@ -1188,7 +1188,7 @@
parameter will force ia64_sal_cache_flush to call
ia64_pal_cache_flush instead of SAL_CACHE_FLUSH.
 
-   forcepae [X86-32]
+   forcepae[X86-32]
Forcefully enable Physical Address Extension (PAE).
Many Pentium M systems disable PAE but may have a
functionally usable PAE implementation.
@@ -1247,7 +1247,7 @@
 
gamma=  [HW,DRM]
 
-   gart_fix_e820=  [X86_64] disable the fix e820 for K8 GART
+   gart_fix_e820=  [X86_64] disable the fix e820 for K8 GART
Format: off | on
default: on
 
@@ -1341,14 +1341,14 @@
x86-64 are 2M (when the CPU supports "pse") and 1G
(when the CPU supports the "pdpe1gb" cpuinfo flag).
 
-   hvc_iucv=   [S390] Number of z/VM IUCV hypervisor console (HVC)
-  terminal devices. Valid values: 0..8
-   hvc_iucv_allow= [S390]