Re: PROBLEM: 4.15.0-rc3 APIC causes lockups on Core 2 Duo laptop

2017-12-22 Thread Alexandru Chirvasitu
I was just now trying to track down my other issue, whereby somewhere along the tree kexec stops working properly. In the process of doing that I realized I had initially made one change to the original 4.9 config beyond oldconfig: I'd turned off WX debugging. I've now compiled a bunch of

RE: PROBLEM: 4.15.0-rc3 APIC causes lockups on Core 2 Duo laptop

2017-12-22 Thread Dexuan Cui
> From: Alexandru Chirvasitu [mailto:achirva...@gmail.com] > Sent: Friday, December 22, 2017 14:29 > > The output of that precise command run just now on a freshly-compiled > copy of that commit is attached. > > On Fri, Dec 22, 2017 at 09:31:28PM +, Dexuan Cui wrote: > > > From: Alexandru

RE: PROBLEM: 4.15.0-rc3 APIC causes lockups on Core 2 Duo laptop

2017-12-22 Thread Dexuan Cui
> From: Alexandru Chirvasitu [mailto:achirva...@gmail.com] > Sent: Friday, December 22, 2017 06:21 > > In the absence of logs, the best I can do at the moment is attach a > picture of the screen I am presented with on the apic=debug boot > attempt. > Alex The panic happens in

[PATCH] Staging: vt6656: fix unnecessary parantheses

2017-12-22 Thread Sumit Pundir
This patch fixes a coding style issue as noted by checkpatch.pl related to unnecessary parentheses. This patch fixes the following checkpatch.pl warning: WARNING: Unnecessary parentheses around 'priv->basic_rates'. Signed-off-by: Sumit Pundir ---

[PATCH 2/2] drivers: android: Fix logtags in methods

2017-12-22 Thread Harsh Shandilya
From: Harsh Shandilya Several methods in the driver were hardcoding the function name in their logging calls which is a checkpatch violation. Utilise the __func__ macro to avoid needing to add the function name as is to the string. Test: logtags in dmesg remain unaffected before

[PATCH 1/2] drivers: android: Use octal permissions

2017-12-22 Thread Harsh Shandilya
From: Harsh Shandilya checkpatch warns against the use of symbolic permissions, this patch migrates all symbolic permissions in the binder driver to octal permissions. Test: debugfs nodes created by binder have the same unix permissions prior to and after this patch was applied.

[PATCH 7/8] staging: greybus: audio_codec.h: Prefer kernel type 'u32' over 'uint32_t'

2017-12-22 Thread Kamal Heib
Fix the following errors found by checkpatch.pl: CHECK: Prefer kernel type 'u32' over 'uint32_t' Cc: Vaibhav Agarwal Signed-off-by: Kamal Heib --- drivers/staging/greybus/audio_codec.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-)

[PATCH 8/8] staging: greybus: audio_codec.h Fix alignment should match open parenthesis

2017-12-22 Thread Kamal Heib
Fix "Alignment should match open parenthesis" checkpatch.pl errors. Cc: Vaibhav Agarwal Signed-off-by: Kamal Heib --- drivers/staging/greybus/audio_codec.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 3/8] staging: greybus: audio_codec.c: Cleanup blank lines

2017-12-22 Thread Kamal Heib
Remove the blank lines. Cc: Vaibhav Agarwal Signed-off-by: Kamal Heib --- drivers/staging/greybus/audio_codec.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c index

[PATCH 6/8] staging: greybus: audio_codec.c: Prefer kernel type 'u32' over 'uint32_t'

2017-12-22 Thread Kamal Heib
Fix the following errors found by checkpatch.pl: CHECK: Prefer kernel type 'u32' over 'uint32_t' Cc: Vaibhav Agarwal Signed-off-by: Kamal Heib --- drivers/staging/greybus/audio_codec.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH 4/8] staging: greybus: audio_codec.c: Space required around ':'

2017-12-22 Thread Kamal Heib
Space is required when using the question mark operator around ':' Cc: Vaibhav Agarwal Signed-off-by: Kamal Heib --- drivers/staging/greybus/audio_codec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 2/8] staging: greybus: audio_codec.c: Fix alignment should match open parenthesis

2017-12-22 Thread Kamal Heib
Cleanup "Alignment should match open parenthesis" checkpatch.pl errors. Cc: Vaibhav Agarwal Signed-off-by: Kamal Heib --- drivers/staging/greybus/audio_codec.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

[PATCH 5/8] staging: greybus: audio_codec.c: Logical continuations should be on the previous line

2017-12-22 Thread Kamal Heib
Fix the following error found by checkpatch.pl: CHECK: Logical continuations should be on the previous line Cc: Vaibhav Agarwal Signed-off-by: Kamal Heib --- drivers/staging/greybus/audio_codec.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 1/8] staging: greybus: authentication.c: Fix alignment should match open parenthesis

2017-12-22 Thread Kamal Heib
Fix the following error found by checkpatch.pl: CHECK: Alignment should match open parenthesis +static int cap_ioctl(struct gb_cap *cap, unsigned int cmd, +void __user *buf) Cc: Viresh Kumar Signed-off-by: Kamal Heib ---

[PATCH 0/8] staging: greybus: Fix few issues reported by

2017-12-22 Thread Kamal Heib
This patch set include multiple fixes for the issues reported by checkpatch.pl Kamal Heib (8): staging: greybus: authentication.c: Fix alignment should match open parenthesis staging: greybus: audio_codec.c: Fix alignment should match open parenthesis staging: greybus:

[PATCH] Changed misspelled variable name

2017-12-22 Thread Julian Geus
The misspelled variable name 'splitted' was changed into 'split'. Signed-off-by: Julian Geus Signed-off-by: Kilian Bender --- drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

Re: PROBLEM: 4.15.0-rc3 APIC causes lockups on Core 2 Duo laptop

2017-12-22 Thread Dou Liyang
Hi Alexandru, At 12/21/2017 10:23 AM, Alexandru Chirvasitu wrote: This might be more helpful. I ran another bisect with the following final log: --- git bisect start # bad: [d6ffc6ac83b1f9f12652d89b9cb5bcbfbea7796c] x86/vector: Respect affinity mask in irq descriptor git bisect bad

[PATCH] staging: most: Fix identifiers to function parameters

2017-12-22 Thread venkataravi . e
From: Ravi Eluri fixed "function definition argument should have an identifier name", with appropriate identifier names. Pointed out by checkpatch. Signed-off-by: Ravi Eluri Signed-off-by: Suniel Mahesh --- Note: -

[PATCH 4.14 065/159] x86/virt: Add enum for hypervisors to replace x86_hyper

2017-12-22 Thread Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know. -- From: Juergen Gross commit 03b2a320b19f1424e9ac9c21696be9c60b6d0d93 upstream. The x86_hyper pointer is only used for checking whether a virtual device is supporting the hypervisor

[PATCH 4.14 064/159] x86/virt, x86/platform: Merge struct x86_hyper into struct x86_platform and struct x86_init

2017-12-22 Thread Greg Kroah-Hartman
4.14-stable review patch. If anyone has any objections, please let me know. -- From: Juergen Gross commit f72e38e8ec8869ac0ba5a75d7d2f897d98a1454e upstream. Instead of x86_hyper being either NULL on bare metal or a pointer to a struct hypervisor_x86 in case

[PATCH] staging: pi433: fix CamelCase for maxMinus variables

2017-12-22 Thread Valentin Vidic
Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433.txt | 10 +-

[PATCH] staging: most: Avoid trailing semicolon for macros

2017-12-22 Thread venkataravi . e
From: Ravi Eluri Fixes checkpatch warning: macros should not use a trailing semicolon. Signed-off-by: Ravi Eluri --- Note: - Patch was compile tested and built(ARCH=arm) on linux-next (latest). - No build issues reported. ---