Re: [PATCH 1/2] dt-bindings: power: Add ZynqMP power domain bindings

2018-03-06 Thread Geert Uytterhoeven
Hi Rob, Jolly, On Mon, Mar 5, 2018 at 11:39 PM, Rob Herring wrote: > On Tue, Feb 27, 2018 at 03:55:49PM -0800, Jolly Shah wrote: >> Add documentation to describe ZynqMP power domain bindings. >> >> Signed-off-by: Jolly Shah >> Signed-off-by: Rajan Vaja >> --- >> .../devicetree/bindings/power/z

Re: [PATCH] ipvs: use true and false for boolean values

2018-03-06 Thread Simon Horman
On Mon, Mar 05, 2018 at 03:35:57PM -0600, Gustavo A. R. Silva wrote: > Assign true or false to boolean variables instead of an integer value. > > This issue was detected with the help of Coccinelle. > > Signed-off-by: Gustavo A. R. Silva Signed-off-by: Simon Horman Pablo, could you take this

Re: [PATCH 1/3 RESEND] tpm: add longer timeouts for creation commands.

2018-03-06 Thread Jarkko Sakkinen
On Mon, Mar 05, 2018 at 06:04:56PM +, Winkler, Tomas wrote: > > > > On Mon, Mar 05, 2018 at 01:09:09PM +, Winkler, Tomas wrote: > > > > enum tpm_duration { > > > > TPM_DURATION_DEFAULT = 2000, > > > > TPM_DURATION_LONG = 30, > > > > }; > > > > > > > How is this aligned

Re: [PATCH 1/2] dt-bindings: power: Add ZynqMP power domain bindings

2018-03-06 Thread Marek Szyprowski
Hi All, On 2018-03-06 08:59, Geert Uytterhoeven wrote: Hi Rob, Jolly, On Mon, Mar 5, 2018 at 11:39 PM, Rob Herring wrote: On Tue, Feb 27, 2018 at 03:55:49PM -0800, Jolly Shah wrote: Add documentation to describe ZynqMP power domain bindings. Signed-off-by: Jolly Shah Signed-off-by: Rajan V

RE: [PATCH 1/3 RESEND] tpm: add longer timeouts for creation commands.

2018-03-06 Thread Winkler, Tomas
> > On Mon, Mar 05, 2018 at 01:09:09PM +, Winkler, Tomas wrote: > > Why you need cover letter? What are u missing in the patch description > > If you submit a *patch set* I *require* a cover letter, yes. It's good but it is not must, you are inventing your own rules. Thanks Tomas

[PATCH v2 2/3] ARM: dts: STi: Remove useless stdout-path for STi boards

2018-03-06 Thread patrice.chotard
From: Patrice Chotard As serial interface is already specified into bootargs, stdout-path can be removed. Signed-off-by: Patrice Chotard --- v2: _ none arch/arm/boot/dts/stih407-b2120.dts | 1 - arch/arm/boot/dts/stih410-b2120.dts | 1 - arch/arm/boot/dts/stih410-b2260.dts | 1 - arch/arm/bo

[PATCH v2 3/3] tty: st-asc: Update tty alias

2018-03-06 Thread patrice.chotard
From: Patrice Chotard Since dtc v1.4.6-9-gaadd0b65c987, aliases property name must include only lowercase and '-'. After having updated all STi boards serial aliases from "ttyASN" to "serialN", st-asc driver need to be updated accordingly as tty aliases id is retrieved using of_alias_get_id(np,

[PATCH v2 1/3] ARM: dts: STi: Fix aliases property name for STi boards

2018-03-06 Thread patrice.chotard
From: Patrice Chotard Update serial aliases from "ttyASN" to more common "serialN". Since dtc v1.4.6-9-gaadd0b65c987, aliases property name must be lowercase only. This allows to fix following dtc warnings: arch/arm/boot/dts/stih418-b2199.dtb: Warning (alias_paths): /aliases: aliases property

[PATCH v2 0/3] Fix STi aliases property name

2018-03-06 Thread patrice.chotard
From: Patrice Chotard Since dtc v1.4.6-9-gaadd0b65c987, when compiling dtb with W=1 option, the following warnings are triggered : arch/arm/boot/dts/stih418-b2199.dtb: Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-' arch/arm/boot/dts/stih407-b2120.

RE: [PATCH 1/3 RESEND] tpm: add longer timeouts for creation commands.

2018-03-06 Thread Winkler, Tomas
> > On Mon, Mar 05, 2018 at 06:04:56PM +, Winkler, Tomas wrote: > > > > > > On Mon, Mar 05, 2018 at 01:09:09PM +, Winkler, Tomas wrote: > > > > > enum tpm_duration { > > > > > TPM_DURATION_DEFAULT = 2000, > > > > > TPM_DURATION_LONG = 30, > > > > > }; > > > > > > > > > Ho

Re: Would you help to tell why async printk solution was not taken to upstream kernel ?

2018-03-06 Thread Sergey Senozhatsky
On (03/05/18 22:16), Steven Rostedt wrote: > > Yes. My point was that "CPU can print one full buffer max" is not > > guaranteed and not exactly true. There are ways for CPUs to break > > that O(logbuf) boundary. > > Yes, when printk or the consoles have a bug, it can be greater than > O(logbuf).

Re: [PATCH 2/2] trace_uprobe: Simplify probes_seq_show()

2018-03-06 Thread Ravi Bangoria
On 02/08/2018 09:13 AM, Ravi Bangoria wrote: > > On 02/08/2018 08:59 AM, Kees Cook wrote: >> On Tue, Feb 6, 2018 at 8:34 PM, Ravi Bangoria >> wrote: >>> Simplify probes_seq_show() function. We are using %lx to display >>> the offset and we don't prepend unnecessary 0s in the offset. >>> >>> Sign

Re: [RFC/RFT][PATCH 6/7] sched: idle: Predict idle duration before stopping the tick

2018-03-06 Thread Rafael J. Wysocki
On Tuesday, March 6, 2018 12:27:01 AM CET Rik van Riel wrote: > On Sun, 2018-03-04 at 23:28 +0100, Rafael J. Wysocki wrote: > > > > +++ linux-pm/kernel/sched/idle.c > > @@ -188,13 +188,14 @@ static void cpuidle_idle_call(void) > > } else { > > unsigned int duration_us; > > > > -

Re: "x86/boot/compressed/64: Prepare trampoline memory" breaks boot on Zotac CI-321

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 07:57:06PM +0100, Heiner Kallweit wrote: > Am 05.03.2018 um 09:19 schrieb Kirill A. Shutemov: > > On Sat, Mar 03, 2018 at 12:46:28PM +0100, Heiner Kallweit wrote: > >> I wanted to apply the fix mentioned in the link but found that the > >> statement was movq already. > >> T

Re: [RFC, PATCH 13/22] mm, rmap: Free encrypted pages once mapcount drops to zero

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:12:15AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > extern void prep_encrypt_page(struct page *page, gfp_t gfp, unsigned int > > order); > > +extern void free_encrypt_page(struct page *page, int keyid, unsigned int > > order); > >

Re: [PATCH v2] mtdchar: fix usage of mtd_ooblayout_ecc()

2018-03-06 Thread Boris Brezillon
On Tue, 6 Mar 2018 11:13:25 +0800 欧阳志忠 wrote: > Section was not properly computed. The value of OOB region definition is > always ECC section 0 information in the OOB area, but we want to get all > the ECC bytes information, so we should call > mtd_ooblayout_ecc(mtd, section++, &oobregion) until

Re: [PATCH 3/3] tpm_crb: use __le64 annotated variable for response buffer address

2018-03-06 Thread Jarkko Sakkinen
On Mon, Mar 05, 2018 at 03:03:20PM +0200, Jarkko Sakkinen wrote: > On Sun, Mar 04, 2018 at 02:12:05PM +0200, Tomas Winkler wrote: > > This suppresses sparse warning > > drivers/char/tpm/tpm_crb.c:558:18: warning: cast to restricted __le64 > > > > Signed-off-by: Tomas Winkler > > --- > > drivers/

Re: [RFC, PATCH 13/22] mm, rmap: Free encrypted pages once mapcount drops to zero

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:13:36AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > @@ -1292,6 +1308,12 @@ static void page_remove_anon_compound_rmap(struct > > page *page) > > __mod_node_page_state(page_pgdat(page), NR_ANON_MAPPED, -nr); > >

Re: [PATCH] kernel/memremap: Remove stale devres_free() call

2018-03-06 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

Re: [RFC, PATCH 16/22] x86/mm: Preserve KeyID on pte_modify() and pgprot_modify()

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:09:23AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > + * It includes full range of PFN bits regardless if they were claimed for > > KeyID > > + * or not: we want to preserve KeyID on pte_modify() and pgprot_modify(). > > */ > > -#de

RE: [PATCH 3/3] tpm_crb: use __le64 annotated variable for response buffer address

2018-03-06 Thread Winkler, Tomas
> On Mon, Mar 05, 2018 at 03:03:20PM +0200, Jarkko Sakkinen wrote: > > On Sun, Mar 04, 2018 at 02:12:05PM +0200, Tomas Winkler wrote: > > > This suppresses sparse warning > > > drivers/char/tpm/tpm_crb.c:558:18: warning: cast to restricted > > > __le64 > > > > > > Signed-off-by: Tomas Winkler > >

Re: [PATH 0/4] usbip: make vhci_hcd.* objects independent of vhci_hcd.0

2018-03-06 Thread Salvador Fandiño
On 03/06/2018 01:03 AM, Shuah Khan wrote: On 03/05/2018 02:00 AM, Salvador Fandiño wrote: On 02/21/2018 01:35 AM, Shuah Khan wrote: Hi Salvador, On 01/30/2018 01:36 AM, Salvador Fandino wrote: Let me start by explaining the problem that have motivated me to write this patches: I work on th

Re: [RFC, PATCH 18/22] x86/mm: Handle allocation of encrypted pages

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:03:55AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > -#define __alloc_zeroed_user_highpage(movableflags, vma, vaddr) \ > > - alloc_page_vma(GFP_HIGHUSER | __GFP_ZERO | movableflags, vma, vaddr) > > #define __HAVE_ARCH_ALLOC_ZEROED_U

Re: [PATCH 3/3] i2c: xlp9xx: Add support for SMBAlert

2018-03-06 Thread Jan Glauber
On Tue, Feb 27, 2018 at 01:26:20PM +, George Cherian wrote: > Add support for SMBus alert mechanism to i2c-xlp9xx driver. > The second interrupt is parsed to use for SMBus alert. > The first interrupt is the i2c controller main interrupt. > > Signed-off-by: Kamlakant Patel > Signed-off-by: Ge

Re: [RFC, PATCH 18/22] x86/mm: Handle allocation of encrypted pages

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:07:55AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > kmap_atomic_keyid() would map the page with the specified KeyID. > > For now it's dummy implementation that would be replaced later. > > I think you need to explain the tradeoffs he

Re: [PATCH 07/34] x86/entry/32: Restore segments before int registers

2018-03-06 Thread Joerg Roedel
On Mon, Mar 05, 2018 at 01:58:32PM -0800, Linus Torvalds wrote: > On Mon, Mar 5, 2018 at 1:35 PM, Joerg Roedel wrote: > > I could probably add some debug instrumentation to check for that in my > > future testing, as there is no NX protection in the user address-range > > for the kernel-cr3. > >

[PATCH resend] usb: quirks: add control message delay for 1b1c:1b20

2018-03-06 Thread Danilo Krummrich
Corsair Strafe RGB keyboard does not respond to usb control messages sometimes and hence generates timeouts. Commit de3af5bf259d ("usb: quirks: add delay init quirk for Corsair Strafe RGB keyboard") tried to fix those timeouts by adding USB_QUIRK_DELAY_INIT. Unfortunately, even with this quirk ti

Re: [RFC, PATCH 19/22] x86/mm: Implement free_encrypt_page()

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:00:00AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > +void free_encrypt_page(struct page *page, int keyid, unsigned int order) > > +{ > > + int i; > > + void *v; > > + > > + for (i = 0; i < (1 << order); i++) { > > + v

Re: [RFC/RFT][PATCH 4/7] cpuidle: menu: Split idle duration prediction from state selection

2018-03-06 Thread Peter Zijlstra
On Tue, Mar 06, 2018 at 10:15:10AM +0800, Li, Aubrey wrote: > On 2018/3/5 21:53, Peter Zijlstra wrote: > > On Mon, Mar 05, 2018 at 02:05:10PM +0100, Rafael J. Wysocki wrote: > >> On Mon, Mar 5, 2018 at 1:50 PM, Peter Zijlstra > >> wrote: > >>> On Mon, Mar 05, 2018 at 12:47:23PM +0100, Rafael J. W

Re: Simplifying our RCU models

2018-03-06 Thread Ingo Molnar
* Paul E. McKenney wrote: > > > But if we look at the bigger API picture: > > > > > > !PREEMPT_RCU PREEMPT_RCU=y > > > rcu_read_lock():atomicpreemptible > > > rcu_read_lock_sched(): atomicatomic > > > srcu_read_lock(): preemptible

Re: [PATCHv2 1/3] i2c: xlp9xx: Check for Bus state before every transfer

2018-03-06 Thread Jan Glauber
I don't know how valuable same-company reviewed-by's are in the end, but the patches look good to me with the small change in SMBalert, so you could add: Reviewed-by: Jan Glauber --Jan On Tue, Feb 27, 2018 at 01:26:18PM +, George Cherian wrote: > I2C bus enters the STOP condition after the

Re: [PATCH v4 15/38] drm/bridge: analogix_dp: Ensure edp is disabled when shutting down the panel

2018-03-06 Thread Enric Balletbo i Serra
Hi Marek, On 06/03/18 08:35, Marek Szyprowski wrote: > Hi All, > > This is the patch, which introduces the issue I've pointed here: > > https://lists.freedesktop.org/archives/dri-devel/2018-March/167794.html > > On 2018-03-05 23:23, Enric Balletbo i Serra wrote: >> From: Lin Huang >> >> When p

Re: [RFC, PATCH 19/22] x86/mm: Implement free_encrypt_page()

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:07:16AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > +void free_encrypt_page(struct page *page, int keyid, unsigned int order) > > +{ > > + int i; > > + void *v; > > + > > + for (i = 0; i < (1 << order); i++) { > > + v

Re: [PATCH] perf: correct ctx_event_type in ctx_resched()

2018-03-06 Thread Peter Zijlstra
On Mon, Mar 05, 2018 at 09:55:04PM -0800, Song Liu wrote: > In ctx_resched(), EVENT_FLEXIBLE should be sched_out when EVENT_PINNED is > added. However, ctx_resched() calculates ctx_event_type before checking > this condition. As a result, pinned events will NOT get higher priority > than flexible e

Re: [RFC, PATCH 21/22] x86/mm: Introduce page_keyid() and page_encrypted()

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 09:08:53AM -0800, Dave Hansen wrote: > On 03/05/2018 08:26 AM, Kirill A. Shutemov wrote: > > +static inline bool page_encrypted(struct page *page) > > +{ > > + /* All pages with non-zero KeyID are encrypted */ > > + return page_keyid(page) != 0; > > +} > > Is this true?

Re: [RFC, PATCH 00/22] Partial MKTME enabling

2018-03-06 Thread Kirill A. Shutemov
On Mon, Mar 05, 2018 at 11:05:50AM -0800, Matthew Wilcox wrote: > On Mon, Mar 05, 2018 at 10:30:50AM -0800, Christoph Hellwig wrote: > > On Mon, Mar 05, 2018 at 07:25:48PM +0300, Kirill A. Shutemov wrote: > > > Hi everybody, > > > > > > Here's updated version of my patchset that brings support of

Re: [PATCH 0/2] ESRT fixes for relocatable kexec'd kernel

2018-03-06 Thread AKASHI Takahiro
Tyler, Jeffrey, On Fri, Mar 02, 2018 at 08:27:11AM -0500, Tyler Baicar wrote: > On 3/2/2018 12:53 AM, AKASHI Takahiro wrote: > >Tyler, Jeffrey, > > > >[Note: This issue takes place in kexec, not kdump. So to be precise, > >it is not the same phenomenon as what I addressed in [1],[2]: > > [1] >

Re: Simplifying our RCU models

2018-03-06 Thread Ingo Molnar
* Ingo Molnar wrote: >I.e. the new RCU namespace would be something like: > > call_rcu => rcu_call_rcu typo: rcu_call(). > synchronize_rcu => rcu_wait_ typo: rcu_wait(). Here's the updated table: # RCU APIs: rcu_read_lock

[PATCH v4 0/6] Fix the overlapping registers issue for the MediTek audio system

2018-03-06 Thread Ryder Lee
Hi, This is the new round of the audsys. I add devm_of_platform_populate() in the clock driver to populate the child devices (AFE) so that we can handle the dependency for all functions. changes since v4: - remove "simple-mfd" from mediatek,audsys.txt. - switch to use devm_of_platform_popula

[PATCH v4 1/6] clk: mediatek: update missing clock data for MT7622 audsys

2018-03-06 Thread Ryder Lee
Add missing clock data 'CLK_AUDIO_AFE_CONN' for MT7622 audsys. Signed-off-by: Ryder Lee Reviewed-by: Rob Herring Reviewed-by: Matthias Brugger --- drivers/clk/mediatek/clk-mt7622-aud.c | 1 + include/dt-bindings/clock/mt7622-clk.h | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff

[PATCH v4 4/6] dt-bindings: clock: mediatek: update audsys documentation to adapt MFD device

2018-03-06 Thread Ryder Lee
The MediaTek audio hardware block that exposes functionalities that are handled by separate subsystems in the kernel. These functions are all mapped somewhere at 0x112x, and there are some control bits are mixed up with other functions within the same registers. This patch modifies example to

[PATCH v4 3/6] clk: mediatek: add audsys support for MT2701

2018-03-06 Thread Ryder Lee
Add clock driver support for MT2701 audsys. Signed-off-by: Ryder Lee --- drivers/clk/mediatek/Kconfig | 6 ++ drivers/clk/mediatek/Makefile | 1 + drivers/clk/mediatek/clk-mt2701-aud.c | 177 ++ 3 files changed, 184 insertions(+) create mode

[PATCH v4 2/6] clk: mediatek: add devm_of_platform_populate() for MT7622 audsys

2018-03-06 Thread Ryder Lee
Add devm_of_platform_populate() to populate devices which are children of the root node. Signed-off-by: Ryder Lee --- drivers/clk/mediatek/clk-mt7622-aud.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/clk/mediatek/clk-mt7622-aud.c b/drivers/clk/mediatek/cl

[PATCH v4 5/6] dt-bindings: clock: mediatek: add audsys support for MT2701

2018-03-06 Thread Ryder Lee
This patch adds a compatible string for MT2701. Signed-off-by: Ryder Lee --- Documentation/devicetree/bindings/arm/mediatek/mediatek,audsys.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,audsys.txt b/Documentation/devicetree/bindi

[PATCH v4 6/6] arm: dts: mediatek: modify audio related nodes for both MT2701 and MT7623

2018-03-06 Thread Ryder Lee
Modify audio related nodes to reflect the actual usage in binding documents. Signed-off-by: Ryder Lee --- arch/arm/boot/dts/mt2701.dtsi | 188 - arch/arm/boot/dts/mt7623.dtsi | 190 -- 2 files changed, 182 insertions

[RFC/RFT][PATCH v2 6/6] time: tick-sched: Avoid running the same code twice in a row

2018-03-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki To avoid running the same piece of code twice in a row, move the tick stopping part of __tick_nohz_next_event() into a new function called __tick_nohz_stop_tick() and invoke them both separately. Make __tick_nohz_idle_enter() avoid calling __tick_nohz_next_event() if it h

[PATCH] staging: fsl-mc/dpio: remove unused function

2018-03-06 Thread Anders Roxell
gcc warns that function 'qbman_pull_desc_set_token' is not used. drivers/staging/fsl-mc/bus/dpio/qbman-portal.c:525:13: warning: ‘qbman_pull_desc_set_token’ defined but not used [-Wunused-function] In the current code we remove that function. Fixes: 321eecb06bfb ("bus: fsl-mc: dpio: add QBMan p

[RFC/RFT][PATCH v2 4/6] cpuidle: Return nohz hint from cpuidle_select()

2018-03-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Add a new pointer argument to cpuidle_select() and to the ->select cpuidle governor callback to allow a boolean value indicating whether or not the tick should be stopped before entering the selected state to be returned from there. Make the ladder governor ignore that po

[RFC/RFT][PATCH v2 2/6] sched: idle: Do not stop the tick upfront in the idle loop

2018-03-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Push the decision whether or not to stop the tick somewhat deeper into the idle loop. Stopping the tick upfront leads to unpleasant outcomes in case the idle governor doesn't agree with the timekeeping code on the duration of the upcoming idle period. Specifically, if th

[PATCH 7/7] perf auxtrace: Make auxtrace_queues__add_buffer() do CPU filtering

2018-03-06 Thread Adrian Hunter
In preparation for supporting AUX area sampling buffers, auxtrace_queues__add_buffer() needs to be more generic. To that end, move CPU filtering into it. Signed-off-by: Adrian Hunter --- tools/perf/util/auxtrace.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff

[RFC/RFT][PATCH v2 3/6] sched: idle: Do not stop the tick before cpuidle_idle_call()

2018-03-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Make cpuidle_idle_call() decide whether or not to stop the tick. First, there are code paths in cpuidle_idle_call() that don't need the tick to be stopped. In particular, the cpuidle_enter_s2idle() path deals with the tick (and with the entire timekeeping for that matter

[RFC/RFT][PATCH v2 1/6] time: tick-sched: Reorganize idle tick management code

2018-03-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Prepare two pieces of code in tick_nohz_idle_enter() for being called separately from each other. First, make it possible to call the initial preparatory part of tick_nohz_idle_enter() without the tick-stopping part following it and introduce the tick_nohz_idle_prepare()

[PATCH 3/7] perf auxtrace: Add missing parameters from kernel-doc comments

2018-03-06 Thread Adrian Hunter
Add missing parameters from kernel-doc comments. Signed-off-by: Adrian Hunter --- tools/perf/util/auxtrace.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/perf/util/auxtrace.h b/tools/perf/util/auxtrace.h index 453c148d2158..e731f55da072 100644 --- a/tools/perf/util/auxtrace.h +++

[PATCH 6/7] perf auxtrace: Make auxtrace_queues__add_buffer() allocate struct buffer

2018-03-06 Thread Adrian Hunter
In preparation for supporting AUX area sampling buffers, auxtrace_queues__add_buffer() needs to be more generic. To that end, move memory allocation for struct buffer into it. Signed-off-by: Adrian Hunter --- tools/perf/util/auxtrace.c | 54 +- 1 file

[PATCH 5/7] perf auxtrace: Make auxtrace_queues__add_buffer() return buffer_ptr

2018-03-06 Thread Adrian Hunter
In preparation for supporting AUX area sampling buffers, auxtrace_queues__add_buffer() needs to be more generic. To that end, make it return buffer_ptr instead of the caller. Signed-off-by: Adrian Hunter --- tools/perf/util/auxtrace.c | 23 +++ 1 file changed, 15 insertions(+

[PATCH 0/7] perf tools: Prepare for AUX area sampling support

2018-03-06 Thread Adrian Hunter
Hi Here are a few patches to help clear the way for adding AUX area sampling support. There is one minor fix "Prevent decoding when --no-itrace" but otherwise there are no functional changes. Adrian Hunter (7): perf record: Combine some auxtrace initialization into a single function

[PATCH 4/7] perf auxtrace: Rename some buffer-queuing functions

2018-03-06 Thread Adrian Hunter
Rename some buffer-queuing functions in preparation for supporting AUX area sampling buffers. Signed-off-by: Adrian Hunter --- tools/perf/util/auxtrace.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tools/perf/util/auxtrace.c b/tools/perf/util/auxtra

[PATCH 2/7] perf auxtrace: Prevent decoding when --no-itrace

2018-03-06 Thread Adrian Hunter
Prevent auxtrace_queues__process_index() from queuing AUX area data for decoding when the --no-itrace option has been used. Signed-off-by: Adrian Hunter --- tools/perf/util/auxtrace.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/tools/perf/util/auxtrace.c b

[PATCH 1/7] perf record: Combine some auxtrace initialization into a single function

2018-03-06 Thread Adrian Hunter
In preparation for adding AUX area sampling support, combine some auxtrace initialization into a single function. Signed-off-by: Adrian Hunter --- tools/perf/builtin-record.c | 36 1 file changed, 24 insertions(+), 12 deletions(-) diff --git a/tools/perf/bui

[RFC/RFT][PATCH v2 0/6] sched/cpuidle: Idle loop rework

2018-03-06 Thread Rafael J. Wysocki
Hi All, Thanks a lot for the discussion so far! Here's a new version of the series addressing some comments from the discussion and (most importantly) replacing patches 4 and 5 with another (simpler) patch. The summary below still applies: On Sunday, March 4, 2018 11:21:30 PM CET Rafael J. Wyso

[RFC/RFT][PATCH v2 5/6] sched: idle: Select idle state before stopping the tick

2018-03-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki In order to address the issue with short idle duration predictions by the idle governor after the tick has been stopped, reorder the code in cpuidle_idle_call() so that the governor idle state selection runs before tick_nohz_idle_go_idle() and use the "nohz" hint returned

[PATCH v2 1/2] staging: most: Add a blank line.

2018-03-06 Thread Quytelda Kahja
Use a blank line after components_show() function declaration. Signed-off-by: Quytelda Kahja --- drivers/staging/most/core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/most/core.c b/drivers/staging/most/core.c index 0ab2de5ecf18..67e2d7f29967 100644 --- a/drivers/staging

Re: [PATCH 3/3] i2c: xlp9xx: Add support for SMBAlert

2018-03-06 Thread Phil Reid
On 6/03/2018 16:36, Jan Glauber wrote: On Tue, Feb 27, 2018 at 01:26:20PM +, George Cherian wrote: Add support for SMBus alert mechanism to i2c-xlp9xx driver. The second interrupt is parsed to use for SMBus alert. The first interrupt is the i2c controller main interrupt. Signed-off-by: Kaml

Re: [PATCH v2 08/11] kconfig: unittest: test defconfig when two choices interact

2018-03-06 Thread Masahiro Yamada
2018-03-02 19:41 GMT+09:00 Ulf Magnusson : > On Fri, Mar 2, 2018 at 5:31 AM, Masahiro Yamada > wrote: >> Commit fbe98bb9ed3d ("kconfig: Fix defconfig when one choice menu >> selects options that another choice menu depends on") fixed defconfig >> when two choices interact (i.e. calculating the vis

Re: [REPOST PATCH] arm/arm64: KVM: Add PSCI version selection API

2018-03-06 Thread Andrew Jones
On Mon, Mar 05, 2018 at 04:47:55PM +, Peter Maydell wrote: > On 2 March 2018 at 11:11, Marc Zyngier wrote: > > On Fri, 02 Mar 2018 10:44:48 +, > > Auger Eric wrote: > >> I understand the get/set is called as part of the migration process. > >> So my understanding is the benefit of this ser

Re: [PATCH 2/4] staging: most: Replace calls to BUG_ON() with WARN_ONCE() and return.

2018-03-06 Thread Quytelda Kahja
> Are you sure this will work? Well, my goal was just to replace the code that could crash the kernel and let somebody with a better understanding of this particular driver write the recovery code, if necessary. It seemed from context that the BUG_ON() calls were being used like assert() though, s

Re: [PATCH] vsprintf: Make "null" pointer dereference more robust

2018-03-06 Thread Petr Mladek
On Mon 2018-03-05 16:16:37, Rasmus Villemoes wrote: > On 2 March 2018 at 13:53, Petr Mladek wrote: > > %p has many modifiers where the pointer is dereferenced. An invalid > > pointer might cause kernel to crash silently. > > > > Note that printk() formats the string under logbuf_lock. Any recursiv

Re: [PATCH v3 10/25] ASoC: qcom: q6asm: Add support to memory map and unmap

2018-03-06 Thread Srinivas Kandagatla
Thanks for the review, On 01/03/18 21:28, Mark Brown wrote: On Tue, Feb 13, 2018 at 04:58:22PM +, srinivas.kandaga...@linaro.org wrote: + num_regions = is_contiguous ? 1 : periods; + buf_sz = is_contiguous ? (period_sz * periods) : period_sz; Please write normal if statements

Re: [PATCH 9/9] drm/xen-front: Implement communication with backend

2018-03-06 Thread Daniel Vetter
On Mon, Mar 05, 2018 at 11:30:35AM +0200, Oleksandr Andrushchenko wrote: > On 03/05/2018 11:25 AM, Daniel Vetter wrote: > > On Wed, Feb 21, 2018 at 10:03:42AM +0200, Oleksandr Andrushchenko wrote: > > > From: Oleksandr Andrushchenko > > > > > > Handle communication with the backend: > > > - sen

Re: [PATCH v3 07/25] ASoC: qcom: qdsp6: Add support to Q6ADM

2018-03-06 Thread Srinivas Kandagatla
Thanks for the review, On 01/03/18 21:24, Mark Brown wrote: On Tue, Feb 13, 2018 at 04:58:19PM +, srinivas.kandaga...@linaro.org wrote: +static struct copp *adm_find_copp(struct q6adm *adm, int port_idx, + int copp_idx) +{ + struct copp *c; + + s

Re: [PATCH v3 11/25] ASoC: qcom: q6asm: add support to audio stream apis

2018-03-06 Thread Srinivas Kandagatla
Thanks for the review comments. sorry for delay, for some reason these emails endup in my SPAM folder. On 01/03/18 21:33, Mark Brown wrote: On Tue, Feb 13, 2018 at 04:58:23PM +, srinivas.kandaga...@linaro.org wrote: uint32_t used; @@ -131,7 +191,7 @@ static int q6asm_apr_send_sess

[PATCH 5/5 V2] tmp: factor out tpm_get_timeouts

2018-03-06 Thread Tomas Winkler
Factor out tpm_get_timeouts into tpm2_get_timeouts and tpm1_get_timeouts. Signed-off-by: Tomas Winkler --- V2: Rebase. drivers/char/tpm/tpm-interface.c | 127 ++- drivers/char/tpm/tpm.h | 5 +- drivers/char/tpm/tpm1-cmd.c | 107 ++

[PATCH 4/5 V2] tpm2: add new tpm2 commands according to TCG 1.36

2018-03-06 Thread Tomas Winkler
1. TPM2_CC_LAST has moved from 182 to 193 2. Convert tpm2_ordinal_duration from an array into a switch statement, as there are not so many commands that require special duration relative to a number of commands. Signed-off-by: Tomas Winkler --- V2: Rebase. drivers/char/tpm/tpm.h | 4

[PATCH 2/5 V2] tpm: factor out tpm 1.2 duration calculation to tpm1-cmd.c

2018-03-06 Thread Tomas Winkler
Factor out tpm1.2 commands calculation into tpm1-cmd.c file. and change the prefix from tpm_ to tpm1_. No functional change is done here. Signed-off-by: Tomas Winkler --- V2: Rebase. drivers/char/tpm/Makefile| 1 + drivers/char/tpm/st33zp24/st33zp24.c | 2 +- drivers/char/tpm/tp

[PATCH 0/5 V2] tpm: timeouts revamp

2018-03-06 Thread Tomas Winkler
This series cleans up tpm timeouts setting and handling. First motivation was to fix failures coming from too short timeouts for commands that creates keys. Key generation may take significant time depending on the underlying hardware. Rather than increasing default timeout a new constant is added

[PATCH 3/5 V2] tpm2: add longer timeouts for creation commands.

2018-03-06 Thread Tomas Winkler
TPM2_CC_Create(0x153) and TPM2_CC_CreatePrimary (0x131) involve generation of crypto keys which can be a computationally intensive task. The timeout is set to 3min. Rather than increasing default timeout a new constant is added, to not stall for too long on regular commands failures. Signed-off-by

Re: [RFC/RFT][PATCH v2 4/6] cpuidle: Return nohz hint from cpuidle_select()

2018-03-06 Thread Rafael J. Wysocki
Bummer. :-( On Tue, Mar 6, 2018 at 10:05 AM, Rafael J. Wysocki wrote: > + if (drv->states[idx].flags & CPUIDLE_FLAG_POLLING) { > + *nohz_ret = false; > + } else if (drv->states[idx].target_residency < TICK_USEC) { > + /* > +* Do not stop th

[PATCH 1/5 V2] tpm: sort objects in the Makefile

2018-03-06 Thread Tomas Winkler
Make the tpm Makefile a bit more in order by putting objects in one column and group together tpm2 modules Signed-off-by: Tomas Winkler --- V2: 1. back to tpm-y notation 2. Partially sort files alphanumerically. drivers/char/tpm/Makefile | 15 --- 1 file changed, 12 insertions(+

Re: [PATCH v4.15.7 1/1] on Intel, VDSO should handle CLOCK_MONOTONIC_RAW and export 'tsc_calibration' pointer

2018-03-06 Thread Thomas Gleixner
Jason, On Mon, 5 Mar 2018, Jason Vas Dias wrote: thanks for providing this. A few formal nits first. Please read Documentation/process/submitting-patches.rst Patches need a concise subject line and the subject line wants a prefix, in this case 'x86/vdso'. Please don't put anything past the pat

[PATCH 4/4] tpm: move tpm1 selftest code from tpm-interface tpm1-cmd.c

2018-03-06 Thread Tomas Winkler
Move the tmp1 selftest code functions to tpm1-cmd.c tpm_pcr_read_dev to tpm1_pcr_read_dev tpm_continue_selftest to tpm1_continue_selftest tpm_do_selftest to tpm1_do_selftest Signed-off-by: Tomas Winkler --- drivers/char/tpm/st33zp24/st33zp24.c | 2 +- drivers/char/tpm/tpm-interface.c | 144

[PATCH v2 2/2] staging: most: Indent function parameter.

2018-03-06 Thread Quytelda Kahja
Indent the parameters for a function call that extends past 80 characters. Signed-off-by: Quytelda Kahja --- drivers/staging/most/core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/most/core.c b/drivers/staging/most/core.c index 67e2d7f29967..8d311970225

[PATCH 2/4] tmp: move tpm_getcap to tpm1-cmd.c

2018-03-06 Thread Tomas Winkler
1. Move tpm_getcap to tpm1-cmd. Rename the function to tpm1_getcap. 2. Remove unsed tpm_getcap_header with unused constant as the functionality is implemented using tpm_buf construct. Signed-off-by: Tomas Winkler --- drivers/char/tpm/tpm-interface.c | 47 +--- dri

[PATCH 3/4] tmp: factor out tpm1_get_random into tpm1-cmd.c

2018-03-06 Thread Tomas Winkler
Factor out get random implementation from tpm-interface.c into tpm1_get_random function in tpm1-cmd.c. No functional changes. Signed-off-by: Tomas Winkler --- drivers/char/tpm/tpm-interface.c | 56 drivers/char/tpm/tpm.h | 2 ++ drivers/char/tp

[PATCH 0/4] tpm: factor out tpm1 code into tpm1-cmd.c

2018-03-06 Thread Tomas Winkler
Move tpm 1.x implementation to tpm1-cmd.c similarly to tpm2-cmd.c This is work in progress of possible dropping compiling out one tpm 1.x or tpm 2.x in case not needed on a target platform. Tomas Winkler (4): tmp: move tpm1_pcr_extend to tpm1-cmd.c tmp: move tpm_getcap to tpm1-cmd.c tmp: fac

[PATCH 1/4] tmp: move tpm1_pcr_extend to tpm1-cmd.c

2018-03-06 Thread Tomas Winkler
Move tpm1_pcr_extend to tpm1-cmd.c and remove unused pcrextend_header structure. Signed-off-by: Tomas Winkler --- drivers/char/tpm/tpm-interface.c | 28 drivers/char/tpm/tpm.h | 2 ++ drivers/char/tpm/tpm1-cmd.c | 23 +++ 3 files c

Re: [PATCH 1/7] platform/x86: fujitsu-laptop: Define constants for FUNC operations

2018-03-06 Thread Andy Shevchenko
On Tue, Mar 6, 2018 at 1:16 AM, Darren Hart wrote: > On Sun, Mar 04, 2018 at 08:44:26PM +0100, Michał Kępień wrote: >> > On Wed, Feb 28, 2018 at 06:08:52PM +0200, Andy Shevchenko wrote: >> > > On Tue, Feb 27, 2018 at 11:15 PM, Micha?? K??pie?? >> > > wrote: >> > > > Various functions exposed by

[PATCH V2] tpm_crb: use __le64 annotated variable for response buffer address

2018-03-06 Thread Tomas Winkler
use __le64 annotated variable for response buffer address as this is read in little endian format form the register. This suppresses sparse warning drivers/char/tpm/tpm_crb.c:558:18: warning: cast to restricted __le64 Signed-off-by: Tomas Winkler --- V2: Add more info into the commit message d

Re: [PATCH v2 09/11] kconfig: unittest: test randconfig for choice in choice

2018-03-06 Thread Masahiro Yamada
2018-03-03 18:34 GMT+09:00 Ulf Magnusson : > On Fri, Mar 2, 2018 at 10:29 PM, Luis R. Rodriguez wrote: >> >> >> On Fri, Mar 2, 2018, 3:14 AM Ulf Magnusson wrote: >>> >>> On Fri, Mar 2, 2018 at 5:31 AM, Masahiro Yamada >>> > This is complicated usage, but it is still used in the real world; >>> >

[PATCH RESEND v5] perf/core: Fix installing arbitrary cgroup event into cpu

2018-03-06 Thread linxiulei
From: "leilei.lin" Do not install cgroup event into the CPU context and schedule it if the cgroup is not running on this CPU While there is no task of cgroup running specified CPU, current kernel still install cgroup event into CPU context that causes another cgroup event can't be installed into

Re: [PATCH v3 2/2] kconfig: rename silentoldconfig to syncconfig

2018-03-06 Thread Masahiro Yamada
2018-03-01 15:34 GMT+09:00 Masahiro Yamada : > As commit cedd55d49dee ("kconfig: Remove silentoldconfig from help > and docs; fix kconfig/conf's help") mentioned, 'silentoldconfig' is a > historical misnomer. That commit removed it from help and docs since > it is an internal interface. If so, it

[PATCH 0/2] crypto: introduce SM4 symmetric cipher algorithm

2018-03-06 Thread Gilad Ben-Yossef
Introduce the SM4 cipher algorithms (OSCCA GB/T 32907-2016) and related tests. SM4 (GBT.32907-2016) is a cryptographic standard issued by the Organization of State Commercial Administration of China (OSCCA) as an authorized cryptographic algorithms for the use within China. SMS4 was originally cr

[PATCH 1/2] crypto: introduce SM4 symmetric cipher algorithm

2018-03-06 Thread Gilad Ben-Yossef
Introduce the SM4 cipher algorithms (OSCCA GB/T 32907-2016). SM4 (GBT.32907-2016) is a cryptographic standard issued by the Organization of State Commercial Administration of China (OSCCA) as an authorized cryptographic algorithms for the use within China. SMS4 was originally created for use in p

Re: [PATCH 9/9] drm/xen-front: Implement communication with backend

2018-03-06 Thread Oleksandr Andrushchenko
On 03/06/2018 11:26 AM, Daniel Vetter wrote: On Mon, Mar 05, 2018 at 11:30:35AM +0200, Oleksandr Andrushchenko wrote: On 03/05/2018 11:25 AM, Daniel Vetter wrote: On Wed, Feb 21, 2018 at 10:03:42AM +0200, Oleksandr Andrushchenko wrote: From: Oleksandr Andrushchenko Handle communication with

[PATCH 2/2] crypto: introduce SM4 testmgr tests

2018-03-06 Thread Gilad Ben-Yossef
Add testmgr tests for the newly introduced SM4 ECB symmetric cipher. Signed-off-by: Gilad Ben-Yossef --- crypto/tcrypt.c | 3 ++ crypto/testmgr.c | 9 crypto/testmgr.h | 131 +++ 3 files changed, 143 insertions(+) diff --git a/crypto

Re: [PATCH 2/4] staging: most: Replace calls to BUG_ON() with WARN_ONCE() and return.

2018-03-06 Thread Dan Carpenter
On Tue, Mar 06, 2018 at 01:23:18AM -0800, Quytelda Kahja wrote: > > Are you sure this will work? > Well, my goal was just to replace the code that could crash the kernel > and let somebody with a better understanding of this particular driver > write the recovery code, if necessary. It seemed from

[PATCH] scsi: ufs: ufshcd_dump_regs to use memcpy_fromio

2018-03-06 Thread Tomas Winkler
ufshcd_dump_regs should use memcpy_fromio to read host registers instead of directly accessing using memcpy. The same function is utilized in ufs-qcom. Elminite compilation warning drivers/scsi/ufs/ufshcd.c:356:9: warning: incorrect type in argument 6 (different address spaces) drivers/scsi/ufs/u

[PATCH 6/9] x86/dumpstack: Add loglevel argument to show_opcodes()

2018-03-06 Thread Borislav Petkov
From: Borislav Petkov Will be used in the next patch. Signed-off-by: Borislav Petkov --- arch/x86/include/asm/stacktrace.h | 1 + arch/x86/kernel/dumpstack.c | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/arch/x86/include/asm/stacktrace.h b/arch/x86/include/a

Re: [PATCH] kconfig: remove redundant streamline_config.pl prerequisite

2018-03-06 Thread Masahiro Yamada
2018-03-02 0:23 GMT+09:00 Ulf Magnusson : > On Thu, Mar 1, 2018 at 3:39 PM, Masahiro Yamada > wrote: >> 2018-03-01 20:18 GMT+09:00 Ulf Magnusson : >>> The local{yes,mod}config targets currently have streamline_config.pl as >>> a prerequisite. This is redundant, because streamline_config.pl is a >>

[PATCH 2/9] x86/fault: Do not print IP in show_fault_oops()

2018-03-06 Thread Borislav Petkov
From: Borislav Petkov ... because __show_regs() already does that. Signed-off-by: Borislav Petkov --- arch/x86/mm/fault.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c index e6af2b464c3d..26865147a507 100644 --- a/arch/x86/mm/fault.c +++ b/arch/x8

Re: uprobes/perf: KASAN: use-after-free in uprobe_perf_close

2018-03-06 Thread Prashant Bhole
On 2/23/2018 2:40 AM, Oleg Nesterov wrote: On 02/22, Peter Zijlstra wrote: On Thu, Feb 22, 2018 at 06:04:27PM +0100, Peter Zijlstra wrote: On Thu, Feb 22, 2018 at 05:37:15PM +0100, Oleg Nesterov wrote: This all makes me think that we should change (fix) kernel/events/core.c... That's go

  1   2   3   4   5   6   7   8   9   >