Re: [PATCH] x86/KASLR: move aligning of minimal address to choose_random_location()

On 01-04-18, Baoquan He wrote: > On 01/04/18 at 03:54pm, Alexander Kuleshov wrote: > > we align minimal possible address during randomization to > > CONFIG_PHYSICAL_ALIGN > > two times: during getting of random physical address and virtual > > address (only for x

Re: [PATCH] x86/KASLR: move aligning of minimal address to choose_random_location()

On 01-04-18, Baoquan He wrote: > On 01/04/18 at 03:54pm, Alexander Kuleshov wrote: > > we align minimal possible address during randomization to > > CONFIG_PHYSICAL_ALIGN > > two times: during getting of random physical address and virtual > > address (only for x

[PATCH] x86/KASLR: move aligning of minimal address to choose_random_location()

(). Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/boot/compressed/kaslr.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/arch/x86/boot/compressed/kaslr.c b/arch/x86/boot/compressed/kaslr.c index 8199a6187251..8c93e399fdfd 100644 --- a/arch/x8

[PATCH] x86/KASLR: move aligning of minimal address to choose_random_location()

(). Signed-off-by: Alexander Kuleshov --- arch/x86/boot/compressed/kaslr.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/arch/x86/boot/compressed/kaslr.c b/arch/x86/boot/compressed/kaslr.c index 8199a6187251..8c93e399fdfd 100644 --- a/arch/x86/boot/compressed/kaslr.c +++ b

[PATCH] assoc_array: fix path to assoc_array documentation

Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- lib/assoc_array.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/assoc_array.c b/lib/assoc_array.c index 59fd7c0b119c..155c55d8db5f 100644 --- a/lib/assoc_array.c +++ b/lib/assoc_array.c @@ -1,

[PATCH] assoc_array: fix path to assoc_array documentation

Signed-off-by: Alexander Kuleshov --- lib/assoc_array.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/assoc_array.c b/lib/assoc_array.c index 59fd7c0b119c..155c55d8db5f 100644 --- a/lib/assoc_array.c +++ b/lib/assoc_array.c @@ -1,6 +1,6 @@ /* Generic associative array

[tip:x86/cleanups] x86/traps: Get rid of unnecessary preempt_disable/preempt_enable_no_resched

Commit-ID: 07d495dae20717b00881798ef812f7aa53ca0eb3 Gitweb: http://git.kernel.org/tip/07d495dae20717b00881798ef812f7aa53ca0eb3 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Mon, 28 Nov 2016 13:50:57 +0600 Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:x86/cleanups] x86/traps: Get rid of unnecessary preempt_disable/preempt_enable_no_resched

Commit-ID: 07d495dae20717b00881798ef812f7aa53ca0eb3 Gitweb: http://git.kernel.org/tip/07d495dae20717b00881798ef812f7aa53ca0eb3 Author: Alexander Kuleshov AuthorDate: Mon, 28 Nov 2016 13:50:57 +0600 Committer: Thomas Gleixner CommitDate: Sat, 4 Feb 2017 09:36:59 +0100 x86/traps: Get

[PATCH] x86/traps: get rid of unnecessary preempt_disable/preempt_enable_no_resched

() and preempt_enable_no_resched() after/before ist_enter()/ist_exit(). Let's get rid from them. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/kernel/traps.c | 5 - 1 file changed, 5 deletions(-) diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c index b

[PATCH] x86/traps: get rid of unnecessary preempt_disable/preempt_enable_no_resched

() and preempt_enable_no_resched() after/before ist_enter()/ist_exit(). Let's get rid from them. Signed-off-by: Alexander Kuleshov --- arch/x86/kernel/traps.c | 5 - 1 file changed, 5 deletions(-) diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c index bd4e3d4..5fbff64 100644 --- a/arch/x86

[tip:x86/asm] x86/entry64: Remove unused audit related macros

Commit-ID: 5e25d5bdf6d407224ad185a3fb8b870ad7d6c627 Gitweb: http://git.kernel.org/tip/5e25d5bdf6d407224ad185a3fb8b870ad7d6c627 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Sun, 23 Oct 2016 19:56:46 +0600 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:x86/asm] x86/entry64: Remove unused audit related macros

Commit-ID: 5e25d5bdf6d407224ad185a3fb8b870ad7d6c627 Gitweb: http://git.kernel.org/tip/5e25d5bdf6d407224ad185a3fb8b870ad7d6c627 Author: Alexander Kuleshov AuthorDate: Sun, 23 Oct 2016 19:56:46 +0600 Committer: Ingo Molnar CommitDate: Tue, 25 Oct 2016 11:43:31 +0200 x86/entry64: Remove

[PATCH] x86/entry64: move comment about informing of context tracking in right place

then the CALL_enter_from_user_mode macro expands to the informing of context tracking about this event. Let's move part of comment to the right place where actual call to context tracing occured. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/entry/entry_64.S | 6 -- 1 file chan

[PATCH] x86/entry64: move comment about informing of context tracking in right place

then the CALL_enter_from_user_mode macro expands to the informing of context tracking about this event. Let's move part of comment to the right place where actual call to context tracing occured. Signed-off-by: Alexander Kuleshov --- arch/x86/entry/entry_64.S | 6 -- 1 file changed, 4 insertions(+), 2

[PATCH v3] x86/entry64: remove unused audit related macros

arch/x86/entry/common.c") commit. syscall_trace_enter() now uses same defines from , so these defines are no longer used anywhere in entry_64.S outside of it and we may remove them. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> Reviewed-by: Borislav Petkov <b...@suse

[PATCH v3] x86/entry64: remove unused audit related macros

arch/x86/entry/common.c") commit. syscall_trace_enter() now uses same defines from , so these defines are no longer used anywhere in entry_64.S outside of it and we may remove them. Signed-off-by: Alexander Kuleshov Reviewed-by: Borislav Petkov --- ChangeLog: v1 -> v2: - Added more detail

Re: [PATCH v2] x86/entry64: remove unused audit related macros

On 10-23-16, Borislav Petkov wrote: > Now, I went and made it a bit more readable and put the commit IDs and > their names on separate lines, like we do in tip: > > "These macros were added in commit > > 86a1c34a929f ("x86_64 syscall audit fast-path"). > > They were used for sycalls auditing,

Re: [PATCH v2] x86/entry64: remove unused audit related macros

On 10-23-16, Borislav Petkov wrote: > Now, I went and made it a bit more readable and put the commit IDs and > their names on separate lines, like we do in tip: > > "These macros were added in commit > > 86a1c34a929f ("x86_64 syscall audit fast-path"). > > They were used for sycalls auditing,

[PATCH v2] x86/entry64: remove unused audit related macros

t (x86/entry: Move C entry and exit code to arch/x86/entry/common.c by Andy Lutomirski <l...@kernel.org>). The syscall_trace_enter() now uses same defines from , so these defines are no longer used anywhere in entry_64.S and we may remove them. Signed-off-by: Alexander Kuleshov <kulesho

[PATCH v2] x86/entry64: remove unused audit related macros

entry and exit code to arch/x86/entry/common.c by Andy Lutomirski ). The syscall_trace_enter() now uses same defines from , so these defines are no longer used anywhere in entry_64.S and we may remove them. Signed-off-by: Alexander Kuleshov --- ChangeLog: v1 -> v2: - Added more details in the com

Re: [PATCH] x86/entry64: remove unused audit related macros

Hello Borislav, On 10-23-16, Borislav Petkov wrote: > I'd prefer it if you said which patch *removed* them and made those > defines unused again. > > What would be even better is if your commit message explained *why* they > were removed in that patch and that because of we > don't need them

Re: [PATCH] x86/entry64: remove unused audit related macros

Hello Borislav, On 10-23-16, Borislav Petkov wrote: > I'd prefer it if you said which patch *removed* them and made those > defines unused again. > > What would be even better is if your commit message explained *why* they > were removed in that patch and that because of we > don't need them

[PATCH] x86/entry64: remove unused audit related macros

which are ocured in the 86a1c34a929f commit (x86_64 syscall audit fast-path by Roland McGrath <rol...@redhat.com>) and now they are not used anymore. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/entry/entry_64.S | 6 -- 1 file changed, 6 deletions(-)

[PATCH] x86/entry64: remove unused audit related macros

which are ocured in the 86a1c34a929f commit (x86_64 syscall audit fast-path by Roland McGrath ) and now they are not used anymore. Signed-off-by: Alexander Kuleshov --- arch/x86/entry/entry_64.S | 6 -- 1 file changed, 6 deletions(-) diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry

[tip:x86/asm] entry/64: Remove unused ZERO_EXTRA_REGS macro

Commit-ID: 29a6d7964d6853f5bcd84dfb92c074fb41d00563 Gitweb: http://git.kernel.org/tip/29a6d7964d6853f5bcd84dfb92c074fb41d00563 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Thu, 20 Oct 2016 18:07:04 +0600 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:x86/asm] entry/64: Remove unused ZERO_EXTRA_REGS macro

Commit-ID: 29a6d7964d6853f5bcd84dfb92c074fb41d00563 Gitweb: http://git.kernel.org/tip/29a6d7964d6853f5bcd84dfb92c074fb41d00563 Author: Alexander Kuleshov AuthorDate: Thu, 20 Oct 2016 18:07:04 +0600 Committer: Ingo Molnar CommitDate: Fri, 21 Oct 2016 07:47:36 +0200 entry/64: Remove

[tip:x86/urgent] entry/64: Remove unused ZERO_EXTRA_REGS macro

Commit-ID: b05d20f9a12fb7795b2bd220e7dd56e29e196ba8 Gitweb: http://git.kernel.org/tip/b05d20f9a12fb7795b2bd220e7dd56e29e196ba8 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Thu, 20 Oct 2016 18:07:04 +0600 Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:x86/urgent] entry/64: Remove unused ZERO_EXTRA_REGS macro

Commit-ID: b05d20f9a12fb7795b2bd220e7dd56e29e196ba8 Gitweb: http://git.kernel.org/tip/b05d20f9a12fb7795b2bd220e7dd56e29e196ba8 Author: Alexander Kuleshov AuthorDate: Thu, 20 Oct 2016 18:07:04 +0600 Committer: Thomas Gleixner CommitDate: Thu, 20 Oct 2016 23:41:15 +0200 entry/64: Remove

[PATCH] entry/64: remove unused ZERO_EXTRA_REGS macro

Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/entry/calling.h | 9 - 1 file changed, 9 deletions(-) diff --git a/arch/x86/entry/calling.h b/arch/x86/entry/calling.h index 9a9e588..e32353a 100644 --- a/arch/x86/entry/calling.h +++ b/arch/x86/entry/cal

[PATCH] entry/64: remove unused ZERO_EXTRA_REGS macro

Signed-off-by: Alexander Kuleshov --- arch/x86/entry/calling.h | 9 - 1 file changed, 9 deletions(-) diff --git a/arch/x86/entry/calling.h b/arch/x86/entry/calling.h index 9a9e588..e32353a 100644 --- a/arch/x86/entry/calling.h +++ b/arch/x86/entry/calling.h @@ -147,15 +147,6 @@ For 32

[tip:x86/asm] x86/entry/64: Remove unused 'addskip' parameter of the ALLOC_PT_GPREGS_ON_STACK macro

Commit-ID: 59df2268311795ee7161776248d5288ee0841d41 Gitweb: http://git.kernel.org/tip/59df2268311795ee7161776248d5288ee0841d41 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Thu, 20 Oct 2016 01:11:08 +0600 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:x86/asm] x86/entry/64: Remove unused 'addskip' parameter of the ALLOC_PT_GPREGS_ON_STACK macro

Commit-ID: 59df2268311795ee7161776248d5288ee0841d41 Gitweb: http://git.kernel.org/tip/59df2268311795ee7161776248d5288ee0841d41 Author: Alexander Kuleshov AuthorDate: Thu, 20 Oct 2016 01:11:08 +0600 Committer: Ingo Molnar CommitDate: Thu, 20 Oct 2016 08:42:36 +0200 x86/entry/64: Remove

[tip:x86/urgent] x86/entry: Remove addskip parameter from ALLOC_PT_GPREGS_ON_STACK

Commit-ID: ac679bf247cf044a5be6ca7d380307540284dd45 Gitweb: http://git.kernel.org/tip/ac679bf247cf044a5be6ca7d380307540284dd45 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Thu, 20 Oct 2016 01:11:08 +0600 Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:x86/urgent] x86/entry: Remove addskip parameter from ALLOC_PT_GPREGS_ON_STACK

Commit-ID: ac679bf247cf044a5be6ca7d380307540284dd45 Gitweb: http://git.kernel.org/tip/ac679bf247cf044a5be6ca7d380307540284dd45 Author: Alexander Kuleshov AuthorDate: Thu, 20 Oct 2016 01:11:08 +0600 Committer: Thomas Gleixner CommitDate: Thu, 20 Oct 2016 10:57:37 +0200 x86/entry

[PATCH] x86/calling.h: remove unused addskip parameter of ALLOC_PT_GPREGS_ON_STACK

as no-one user of ALLOC_PT_GPREGS_ON_STACK macro doesn't pass it. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/entry/calling.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/entry/calling.h b/arch/x86/entry/calling.h index 9

[PATCH] x86/calling.h: remove unused addskip parameter of ALLOC_PT_GPREGS_ON_STACK

as no-one user of ALLOC_PT_GPREGS_ON_STACK macro doesn't pass it. Signed-off-by: Alexander Kuleshov --- arch/x86/entry/calling.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/entry/calling.h b/arch/x86/entry/calling.h index 9a9e588..b493c6a 100644 --- a/arch

[tip:irq/core] genirq: No need to mask non trigger mode flags before __irq_set_trigger()

Commit-ID: 00b992deaa08495ab958da5950c9ebbba27d0ddc Gitweb: http://git.kernel.org/tip/00b992deaa08495ab958da5950c9ebbba27d0ddc Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Tue, 19 Jul 2016 15:54:08 +0600 Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:irq/core] genirq: No need to mask non trigger mode flags before __irq_set_trigger()

Commit-ID: 00b992deaa08495ab958da5950c9ebbba27d0ddc Gitweb: http://git.kernel.org/tip/00b992deaa08495ab958da5950c9ebbba27d0ddc Author: Alexander Kuleshov AuthorDate: Tue, 19 Jul 2016 15:54:08 +0600 Committer: Thomas Gleixner CommitDate: Tue, 6 Sep 2016 12:14:12 +0200 genirq: No need

[PATCH] Documentation/x86_64: Fix a thread stack size

current information about THREAD_SIZE value is obsoleted. This patch fixes this. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Documentation/x86/kernel-stacks | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/x86/kernel-stacks b/Documen

[PATCH] Documentation/x86_64: Fix a thread stack size

current information about THREAD_SIZE value is obsoleted. This patch fixes this. Signed-off-by: Alexander Kuleshov --- Documentation/x86/kernel-stacks | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/x86/kernel-stacks b/Documentation/x86/kernel-stacks index

[PATCH] ACPICA: Remove unnecessary '\n' in the end of ACPI_INFO output

as the ACPI_INFO already prints `\n` in the end itself. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- drivers/acpi/acpica/tbxfload.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c index a

[PATCH] ACPICA: Remove unnecessary '\n' in the end of ACPI_INFO output

as the ACPI_INFO already prints `\n` in the end itself. Signed-off-by: Alexander Kuleshov --- drivers/acpi/acpica/tbxfload.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c index ac71abc..e7119b7 100644

[PATCH] mm/memblock: fix a typo in a comment

s/accomodate/accommodate Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- mm/memblock.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mm/memblock.c b/mm/memblock.c index ff5ff3b..1f065da 100644 --- a/mm/memblock.c +++ b/mm/memblock.c @@ -482,7

[PATCH] mm/memblock: fix a typo in a comment

s/accomodate/accommodate Signed-off-by: Alexander Kuleshov --- mm/memblock.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mm/memblock.c b/mm/memblock.c index ff5ff3b..1f065da 100644 --- a/mm/memblock.c +++ b/mm/memblock.c @@ -482,7 +482,7 @@ static void

[PATCH] genirq: no need to mask non trigger mode flags before __irq_set_trigger()

Some callers of __irq_set_trigger() masks all flags besides trigger mode flags. Actually it is unnecessary operation now, because the __irq_set_trigger() already does this before usage of flags. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- kernel/irq/chip.c | 1 - kern

[PATCH] genirq: no need to mask non trigger mode flags before __irq_set_trigger()

Some callers of __irq_set_trigger() masks all flags besides trigger mode flags. Actually it is unnecessary operation now, because the __irq_set_trigger() already does this before usage of flags. Signed-off-by: Alexander Kuleshov --- kernel/irq/chip.c | 1 - kernel/irq/manage.c | 5 ++--- 2

[PATCH] mm/memblock: move memblock_{add,reserve}_region into memblock_{add,reserve}

little. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- mm/memblock.c | 28 ++-- 1 file changed, 6 insertions(+), 22 deletions(-) diff --git a/mm/memblock.c b/mm/memblock.c index b570ddd..3b93daa 100644 --- a/mm/memblock.c +++ b/mm/memblock.c @@ -606,22 +6

[PATCH] mm/memblock: move memblock_{add,reserve}_region into memblock_{add,reserve}

are not used by anybody outside of memblock.c and the memblock_{add,reserve}() have the same set of flags and nids. Since the memblock_add_region() and memblock_reserve_region() anyway will be inlined, there will not be functional changes, but will improve code readability a little. Signed-off-by: Alexander

[PATCH] clocksource: use clocksource_freq2mult() helper

which is introduced in the 7aca0c072 commit to simplify calculation of the mult and shift in the clocks_calc_mult_shift(). Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- kernel/time/clocksource.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/kerne

[PATCH] clocksource: use clocksource_freq2mult() helper

which is introduced in the 7aca0c072 commit to simplify calculation of the mult and shift in the clocks_calc_mult_shift(). Signed-off-by: Alexander Kuleshov --- kernel/time/clocksource.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/kernel/time/clocksource.c b/kernel

[tip:x86/boot] x86/setup: Calculate ramdisk parameters only once

Commit-ID: 6fc77525f509e16b1e909e5ffd6f9ad9dccc0f82 Gitweb: http://git.kernel.org/tip/6fc77525f509e16b1e909e5ffd6f9ad9dccc0f82 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Fri, 26 Feb 2016 15:04:36 +0600 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:x86/boot] x86/setup: Calculate ramdisk parameters only once

Commit-ID: 6fc77525f509e16b1e909e5ffd6f9ad9dccc0f82 Gitweb: http://git.kernel.org/tip/6fc77525f509e16b1e909e5ffd6f9ad9dccc0f82 Author: Alexander Kuleshov AuthorDate: Fri, 26 Feb 2016 15:04:36 +0600 Committer: Ingo Molnar CommitDate: Sat, 27 Feb 2016 12:52:27 +0100 x86/setup: Calculate

[tip:timers/core] jiffies: Use CLOCKSOURCE_MASK instead of constant

Commit-ID: 232d26373d310a941ef2ab46e53ea62fe076ed13 Gitweb: http://git.kernel.org/tip/232d26373d310a941ef2ab46e53ea62fe076ed13 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Fri, 26 Feb 2016 19:14:14 -0800 Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:timers/core] jiffies: Use CLOCKSOURCE_MASK instead of constant

Commit-ID: 232d26373d310a941ef2ab46e53ea62fe076ed13 Gitweb: http://git.kernel.org/tip/232d26373d310a941ef2ab46e53ea62fe076ed13 Author: Alexander Kuleshov AuthorDate: Fri, 26 Feb 2016 19:14:14 -0800 Committer: Thomas Gleixner CommitDate: Sat, 27 Feb 2016 08:55:31 +0100 jiffies: Use

[tip:timers/core] clocksource: Introduce clocksource_freq2mult()

Commit-ID: 7aca0c07207385cca76025cc85231519935722b9 Gitweb: http://git.kernel.org/tip/7aca0c07207385cca76025cc85231519935722b9 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Fri, 26 Feb 2016 19:14:13 -0800 Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:timers/core] clocksource: Introduce clocksource_freq2mult()

Commit-ID: 7aca0c07207385cca76025cc85231519935722b9 Gitweb: http://git.kernel.org/tip/7aca0c07207385cca76025cc85231519935722b9 Author: Alexander Kuleshov AuthorDate: Fri, 26 Feb 2016 19:14:13 -0800 Committer: Thomas Gleixner CommitDate: Sat, 27 Feb 2016 08:55:30 +0100 clocksource

[PATCH] x86/bitops.h: use IS_IMMEDIATE instead of dirrect call of __builtin_constant_p

The arch/x86/include/asm/bitops.h provides IS_IMMEDIATE macro which expands to the call of the __builtin_constant_p(). Let's use this macro in the test_bit() to be more clear. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/include/asm/bitops.h | 2 +- 1 file chan

[PATCH] x86/bitops.h: use IS_IMMEDIATE instead of dirrect call of __builtin_constant_p

The arch/x86/include/asm/bitops.h provides IS_IMMEDIATE macro which expands to the call of the __builtin_constant_p(). Let's use this macro in the test_bit() to be more clear. Signed-off-by: Alexander Kuleshov --- arch/x86/include/asm/bitops.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v8] x86/setup: get ramdisk parameters only once

() and relocate_initrd() functions. This allows us to not get/check ramdisk parameters from the bootparams every time in the early_reserve_initrd(), reserve_initrd() and relocate_initrd() function. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Tested as with virtual machine as with real ha

[PATCH v8] x86/setup: get ramdisk parameters only once

() and relocate_initrd() functions. This allows us to not get/check ramdisk parameters from the bootparams every time in the early_reserve_initrd(), reserve_initrd() and relocate_initrd() function. Signed-off-by: Alexander Kuleshov --- Tested as with virtual machine as with real hardware. Changelog: v8

[PATCH v7] x86/setup: get ramdisk parameters only once

() and relocate_initrd() functions. This allows us to not get/check ramdisk parameters from the bootparams every time in the early_reserve_initrd(), reserve_initrd() and relocate_initrd() function. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Tested as with virtual machine as with real ha

[PATCH v7] x86/setup: get ramdisk parameters only once

() and relocate_initrd() functions. This allows us to not get/check ramdisk parameters from the bootparams every time in the early_reserve_initrd(), reserve_initrd() and relocate_initrd() function. Signed-off-by: Alexander Kuleshov --- Tested as with virtual machine as with real hardware. Changelog: v7

[PATCH v2] x86/head64: no need to define early_idt_ripmsg only if CONFIG_KALLSYMS=n

We have definition of the early_idt_ripmsg which represents string header for the output of the kernel symbols during early exception. But the header with kernel symbols will be printed in a case if the CONFIG_KALLSYMS=y. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Cha

[PATCH v2] x86/head64: no need to define early_idt_ripmsg only if CONFIG_KALLSYMS=n

We have definition of the early_idt_ripmsg which represents string header for the output of the kernel symbols during early exception. But the header with kernel symbols will be printed in a case if the CONFIG_KALLSYMS=y. Signed-off-by: Alexander Kuleshov --- Changelog: v2: commit message fixed

[PATCH v6] x86/setup: get ramdisk parameters only once

() and relocate_initrd() functions. This allows us to not get/check ramdisk parameters from the bootparams every time in the early_reserve_initrd(), reserve_initrd() and relocate_initrd() function. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Tested as with virtual machine as with real ha

[PATCH v6] x86/setup: get ramdisk parameters only once

() and relocate_initrd() functions. This allows us to not get/check ramdisk parameters from the bootparams every time in the early_reserve_initrd(), reserve_initrd() and relocate_initrd() function. Signed-off-by: Alexander Kuleshov --- Tested as with virtual machine as with real hardware. Changelog: v6

[PATCH v5] x86/setup: get ramdisk parameters only once

29979605 1c973d5 vmlinux text data bss dec hex filename 9292602 5011696 15675392 29979690 1c9742a vmlinux Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Tested as with virtual machine as with real hardware. Changelog: v5: * move

[PATCH v5] x86/setup: get ramdisk parameters only once

29979605 1c973d5 vmlinux text data bss dec hex filename 9292602 5011696 15675392 29979690 1c9742a vmlinux Signed-off-by: Alexander Kuleshov --- Tested as with virtual machine as with real hardware. Changelog: v5: * move check of the reserve_ramdisk to the reserve_initrd

[PATCH v4] x86/setup: get ramdisk parameters only once

29979690 1c9742a vmlinux.orig text data bss dec hex filename 9292517 5011696 15675392 29979605 1c973d5 vmlinux Signed-off-by: Alexander Kuleshov --- Tested as with virtual machine as with real hardware. Changelog: v4: * bool reserve_ramdisk moved to the struct

[PATCH v4] x86/setup: get ramdisk parameters only once

29979690 1c9742a vmlinux.orig text data bss dec hex filename 9292517 5011696 15675392 29979605 1c973d5 vmlinux Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Tested as with virtual machine as with real hardware. Changelog: v4:

[tip:x86/boot] x86/boot: Use proper array element type in memset( ) size calculation

Commit-ID: a91bbe017552b80e12d712c85549b933a62c6ed4 Gitweb: http://git.kernel.org/tip/a91bbe017552b80e12d712c85549b933a62c6ed4 Author: Alexander Kuleshov AuthorDate: Tue, 9 Feb 2016 19:44:54 +0600 Committer: Ingo Molnar CommitDate: Tue, 9 Feb 2016 14:55:48 +0100 x86/boot: Use proper

[PATCH] x86/head64: use pointer type in sizeof

We changed loops with memset in the 5e9ebbd87a99 commit (x86/boot: Micro-optimize reset_early_page_tables()). The base for size of memset was size of pud_p/pmd_p, but the they are actually represented as pointers, although they have the same sizes. Signed-off-by: Alexander Kuleshov --- based

[PATCH v2] x86/head64: micro optimization: use memset instead of for loop

1cc8500 vmlinux.old 9735119 4970776 15474688301805831cc84e7 vmlinux Signed-off-by: Alexander Kuleshov --- Changelog: v2: We got sizeof pud_p/pmd_p in the v1, but actually we need to take sizeof of pud_p/pmd_p pointer, although their sizes are the same. arch/x86/kernel/head64.c

[PATCH v3] x86/setup: get ramdisk parameters only once

bss dechex filename 9281559 5010736 15474688297669831c63547 vmlinux Signed-off-by: Alexander Kuleshov --- Tested as with virtual machine as with real hardware. Changelog: v3: introduced ramdisk setup which is filled in th setup_arch() and passed

Re: [PATCH v2] x86/setup: Merge {early_,}reserve_initrd() to one function

Hello Ingo, On Tue, Feb 9, 2016 at 3:16 PM, Ingo Molnar wrote: > > So I don't think the code got easier to understand - in particular the > memblock_reserve()/free() pattern, depending on a flag value, is confusing. > > The duplication is there - but please factor it out into a helper structure

[PATCH] x86/head64: use pointer type in sizeof

We changed loops with memset in the 5e9ebbd87a99 commit (x86/boot: Micro-optimize reset_early_page_tables()). The base for size of memset was size of pud_p/pmd_p, but the they are actually represented as pointers, although they have the same sizes. Signed-off-by: Alexander Kuleshov <kulesh

Re: [PATCH v2] x86/setup: Merge {early_,}reserve_initrd() to one function

Hello Ingo, On Tue, Feb 9, 2016 at 3:16 PM, Ingo Molnar wrote: > > So I don't think the code got easier to understand - in particular the > memblock_reserve()/free() pattern, depending on a flag value, is confusing. > > The duplication is there - but please factor it out into a

[PATCH v3] x86/setup: get ramdisk parameters only once

bss dechex filename 9281559 5010736 15474688297669831c63547 vmlinux Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Tested as with virtual machine as with real hardware. Changelog: v3: introduced ramdisk setup which is filled

[PATCH v2] x86/head64: micro optimization: use memset instead of for loop

1cc8500 vmlinux.old 9735119 4970776 15474688301805831cc84e7 vmlinux Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- Changelog: v2: We got sizeof pud_p/pmd_p in the v1, but actually we need to take sizeof of pud_p/pmd_p pointer, although their sizes are th

[tip:x86/boot] x86/boot: Use proper array element type in memset( ) size calculation

Commit-ID: a91bbe017552b80e12d712c85549b933a62c6ed4 Gitweb: http://git.kernel.org/tip/a91bbe017552b80e12d712c85549b933a62c6ed4 Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Tue, 9 Feb 2016 19:44:54 +0600 Committer: Ingo Molnar <mi...@kernel.org> CommitDate

[PATCH v2] x86/setup: Merge {early_,}reserve_initrd() to one function

data bss dec hex filename 9281618 5010736 15474688 29767042 1c63582 vmlinux.orig textdata bss dec hex filename 9281222 5010672 15474688 29766582 1c633b6 vmlinux.new Signed-off-by: Alexander Kuleshov --- Changelog: v2: parameter

Re: [PATCH] x86/setup: refactor initrd reservation

On Mon, Feb 8, 2016 at 9:56 PM, Andy Shevchenko wrote: > On Fri, Feb 5, 2016 at 7:38 PM, Alexander Kuleshov > wrote: >> The check and definitions related to ramdisk are similar in the >> early_reserve_initrd() and reserve_initrd(). So we can get rid of >> early_reserve_i

[PATCH v2] x86/setup: Merge {early_,}reserve_initrd() to one function

data bss dec hex filename 9281618 5010736 15474688 29767042 1c63582 vmlinux.orig textdata bss dec hex filename 9281222 5010672 15474688 29766582 1c633b6 vmlinux.new Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.

Re: [PATCH] x86/setup: refactor initrd reservation

On Mon, Feb 8, 2016 at 9:56 PM, Andy Shevchenko <andy.shevche...@gmail.com> wrote: > On Fri, Feb 5, 2016 at 7:38 PM, Alexander Kuleshov > <kuleshovm...@gmail.com> wrote: >> The check and definitions related to ramdisk are similar in the >> early_reserve_initrd() and r

[PATCH v2] jiffies: use CLOCKSOURCE_MASK instead of constant

The CLOCKSOURCE_MASK(32) macro expands to the same value, but makes code more readable. Signed-off-by: Alexander Kuleshov --- v2: typo fixed in commit message kernel/time/jiffies.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/time/jiffies.c b/kernel/time

[PATCH v2] jiffies: use CLOCKSOURCE_MASK instead of constant

The CLOCKSOURCE_MASK(32) macro expands to the same value, but makes code more readable. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- v2: typo fixed in commit message kernel/time/jiffies.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kerne

[PATCH] jiffies: use CLOSKOURCE_MASK instead of constant

The CLOCKSOURCE_MASK(32) macro expands to the same value, but makes code more readable. Signed-off-by: Alexander Kuleshov --- kernel/time/jiffies.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/time/jiffies.c b/kernel/time/jiffies.c index 347fecf..555e21f 100644

[PATCH] clocksource: introduce clocksource_freq2mult()

code duplication. Signed-off-by: Alexander Kuleshov --- include/linux/clocksource.h | 45 +++-- 1 file changed, 19 insertions(+), 26 deletions(-) diff --git a/include/linux/clocksource.h b/include/linux/clocksource.h index 6013021..a307bf6 100644

[PATCH] x86/setup: refactor initrd reservation

The check and definitions related to ramdisk are similar in the early_reserve_initrd() and reserve_initrd(). So we can get rid of early_reserve_initrd() and and use late or early algorithm for initrd reservation depends on reserve_initrd() parameter value. Signed-off-by: Alexander Kuleshov

[PATCH] jiffies: use CLOSKOURCE_MASK instead of constant

The CLOCKSOURCE_MASK(32) macro expands to the same value, but makes code more readable. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- kernel/time/jiffies.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/time/jiffies.c b/kernel/time/jiffies.c

[PATCH] clocksource: introduce clocksource_freq2mult()

code duplication. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- include/linux/clocksource.h | 45 +++-- 1 file changed, 19 insertions(+), 26 deletions(-) diff --git a/include/linux/clocksource.h b/include/linux/clocksource.h index 6

[PATCH] x86/setup: refactor initrd reservation

The check and definitions related to ramdisk are similar in the early_reserve_initrd() and reserve_initrd(). So we can get rid of early_reserve_initrd() and and use late or early algorithm for initrd reservation depends on reserve_initrd() parameter value. Signed-off-by: Alexander Kuleshov

[PATCH] mm/memblock: remove unnecessary memblock_type variable

We define struct memblock_type *type in the memblock_add_region() and memblock_reserve_region() functions only for passing it to the memlock_add_range() and memblock_reserve_range() functions. Let's remove these variables and will pass a type directly. Signed-off-by: Alexander Kuleshov --- mm

[PATCH] mm/memblock: remove unnecessary memblock_type variable

We define struct memblock_type *type in the memblock_add_region() and memblock_reserve_region() functions only for passing it to the memlock_add_range() and memblock_reserve_range() functions. Let's remove these variables and will pass a type directly. Signed-off-by: Alexander Kuleshov <kulesh

[PATCH] init: print warning message if /dev/console failed to open

Signed-off-by: Alexander Kuleshov --- init/do_mounts.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/init/do_mounts.c b/init/do_mounts.c index dea5de9..ca5da05 100644 --- a/init/do_mounts.c +++ b/init/do_mounts.c @@ -505,7 +505,9 @@ void __init change_floppy(char *fmt

[PATCH] init: print warning message if /dev/console failed to open

Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- init/do_mounts.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/init/do_mounts.c b/init/do_mounts.c index dea5de9..ca5da05 100644 --- a/init/do_mounts.c +++ b/init/do_mounts.c @@ -505,7 +505,9 @@ void

[tip:x86/asm] x86/entry/traps: Refactor preemption and interrupt flag handling

Commit-ID: d99e1bd175f4291ddb6e62b22bb5bdbe3976389a Gitweb: http://git.kernel.org/tip/d99e1bd175f4291ddb6e62b22bb5bdbe3976389a Author: Alexander Kuleshov AuthorDate: Mon, 25 Jan 2016 20:41:46 +0100 Committer: Ingo Molnar CommitDate: Mon, 1 Feb 2016 10:45:14 +0100 x86/entry/traps

[PATCH] x86_64/setup: handle builtin command line early

checks 'dis_ucode_ldr' option which may be passed through builtin command line. Signed-off-by: Alexander Kuleshov --- arch/x86/include/asm/setup.h | 2 +- arch/x86/kernel/head64.c | 5 + arch/x86/kernel/setup.c | 31 +++ 3 files changed, 25 insertions

[PATCH] x86_64/setup: handle builtin command line early

checks 'dis_ucode_ldr' option which may be passed through builtin command line. Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com> --- arch/x86/include/asm/setup.h | 2 +- arch/x86/kernel/head64.c | 5 + arch/x86/kernel/setup.c | 31 +++ 3

[tip:x86/asm] x86/entry/traps: Refactor preemption and interrupt flag handling

Commit-ID: d99e1bd175f4291ddb6e62b22bb5bdbe3976389a Gitweb: http://git.kernel.org/tip/d99e1bd175f4291ddb6e62b22bb5bdbe3976389a Author: Alexander Kuleshov <kuleshovm...@gmail.com> AuthorDate: Mon, 25 Jan 2016 20:41:46 +0100 Committer: Ingo Molnar <mi...@kernel.org> CommitDate

  1   2   3   4   5   6   7   8   >