Re: [PATCH] scripts/gdb: fix lx-version for gdb 7.3-

2018-10-16 Thread Jan Kiszka
On 17.10.18 04:36, Du Changbin wrote: For gdb version less than 7.3, lx-version only one character. ^^^ prints? (gdb) lx-version L(gdb) This can be fixed by casting 'linux_banner' as (char *). (gdb) lx-version Linux version 4.19.0-rc1+ (changbin@acer

Re: [PATCH] iio: dpot-dac: mark expected switch fall-through

2018-10-16 Thread Peter Rosin
On 2018-10-16 13:01, Gustavo A. R. Silva wrote: > On 10/14/18 7:01 PM, Jonathan Cameron wrote: >> Done the first of the above... > > Thank you, Jonathan. Indeed. I meant to respond earlier, but then forgot... Thank you! Cheers, Peter

Re: [RFC PATCH for 4.21 03/16] sched: Implement push_task_to_cpu (v2)

2018-10-16 Thread Srikar Dronamraju
Hi Mathieu, > +int push_task_to_cpu(struct task_struct *p, unsigned int dest_cpu) > +{ In your use case, is the task going to be current? If yes, we should simply be using migrate_task_to. > + struct rq_flags rf; > + struct rq *rq; > + int ret = 0; > + > + rq = task_rq_lock(p, &r

[PATCH v6 07/20] tpm: move tpm_getcap to tpm1-cmd.c

2018-10-16 Thread Tomas Winkler
1. Move tpm_getcap to tpm1-cmd. Rename the function to tpm1_getcap. 2. Remove unused tpm_getcap_header with unused constant as this functionality is already implemented using tpm_buf construct. Fixes warning: drivers/char/tpm/tpm-interface.c:452:38: warning: ‘tpm_getcap_header’ defined but not us

[PATCH v6 20/20] tpm: use u32 instead of int for PCR index

2018-10-16 Thread Tomas Winkler
The TPM specs defines PCR index as a positive number, and there is no reason to use a signed number. It is also a possible security issue as currently no functions check for a negative index, which may become a large number when converted to u32. Adjust the API to use u32 instead of int in all PCR

[PATCH v6 18/20] tpm1: reimplement SAVESTATE using tpm_buf

2018-10-16 Thread Tomas Winkler
In tpm1_pm_suspend() function reimplement, TPM_ORD_SAVESTATE comamnd using tpm_buf. Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko Sakkinen --- V3: New in the series. V4-V6: Resend drivers/char/tpm/tpm1-cmd.c | 22 -- 1 file changed, 12 insert

[PATCH v6 10/20] tpm: factor out tpm 1.x pm suspend flow into tpm1-cmd.c

2018-10-16 Thread Tomas Winkler
Factor out TPM 1.x suspend flow from tpm-interface.c into a new function tpm1_pm_suspend() in tpm1-cmd.c Signed-off-by: Tomas Winkler --- V2-V3: Rebase V4: 1. Add kdoc for tpm1_pm_suspend(). 2. Fix commit message. V5: Adjust the kdoc to the new standard. V6: Rebase drivers/char/tpm/tpm-inte

[PATCH v6 11/20] tpm: factor out tpm_startup function

2018-10-16 Thread Tomas Winkler
TPM manual startup is used only from within TPM 1.x or TPM 2.x code, hence remove tpm_startup() function from tpm-interface.c and add two static functions implementations tpm1_startup() and tpm2_startup() into to tpm1-cmd.c and tpm2-cmd.c respectively. Signed-off-by: Tomas Winkler Tested-by: Jark

[PATCH] tracing: Export trace_dump_stack to modules

2018-10-16 Thread Nikolay Borisov
There is no reason for this function to be unexprted and it's a useful debugging aid. Signed-off-by: Nikolay Borisov --- kernel/trace/trace.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index bf6f1d70484d..15c7a7d01505 100644 --- a/kernel/trace

[PATCH v6 19/20] tpm1: reimplement tpm1_continue_selftest() using tpm_buf

2018-10-16 Thread Tomas Winkler
Reimplement tpm1_continue_selftest() using tpm_buf structure. This is the last command using the old tpm_cmd_t structure and now the structure can be removed. Cc: Nayna Jain Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko Sakkinen --- V3: New in the series. V4: Res

[PATCH v6 17/20] tmp1: rename tpm1_pcr_read_dev to tpm1_pcr_read()

2018-10-16 Thread Tomas Winkler
Rename tpm1_pcr_read_dev to tpm1_pcr_read() to match the counterpart tpm2_pcr_read(). Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen --- V4: New in the series V5-V6: Resend. drivers/char/tpm/tpm-interface.c | 2 +- drivers/char/tpm/tpm-sysfs.c | 2 +- drivers/char/tpm/tpm.h

[PATCH v6 16/20] tpm1: implement tpm1_pcr_read_dev() using tpm_buf structure

2018-10-16 Thread Tomas Winkler
Implement tpm1_pcr_read_dev() using tpm_buf and remove now unneeded structures from tpm.h Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko Sakkinen --- V3: New in the series. V4-V6: Resend. drivers/char/tpm/tpm.h | 18 ++ drivers/char/tpm/tpm1-

[PATCH v6 13/20] tpm: tpm-interface.c drop unused macros

2018-10-16 Thread Tomas Winkler
The commit: "tpm_tis: further simplify calculation of ordinal duration" left unused macros, we can drop them now. Fixes compilation warnings: tpm-interface.c:37:0: warning: macro "TPM_PROTECTED_COMMAND" is not used [-Wunused-macros] tpm-interface.c:36:0: warning: macro "TSC_MAX_ORDINAL" is not us

[PATCH v6 15/20] tpm: tpm1: rewrite tpm1_get_random() using tpm_buf structure

2018-10-16 Thread Tomas Winkler
1. Use tpm_buf in tpm1_get_random() 2. Fix comment in tpm_get_random() so it is clear that the function is expected to return number of random bytes. Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko Sakkinen --- V3: New in the series. V4: Resend. V5: A small adjustme

[PATCH v6 08/20] tpm: factor out tpm1_get_random into tpm1-cmd.c

2018-10-16 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 Reviewed-by: Jarkko Sakkine --- V2-V3: Rebase V4-V8: Resend drivers/char/tpm/tpm-interface.c | 58 +---

[PATCH v6 09/20] tpm: move tpm 1.x selftest code from tpm-interface.c tpm1-cmd.c

2018-10-16 Thread Tomas Winkler
Move the tpm1 selftest code functions to tpm1-cmd.c and adjust callers to use the new function names. 1. tpm_pcr_read_dev() to tpm1_pcr_read_dev(). 2. tpm_continue_selftest() to tpm1_continue_selftest(). 3. tpm_do_selftest() to tpm1_do_selftest() Signed-off-by: Tomas Winkler Tested-by: Jarkko

[PATCH v6 14/20] tpm: tpm-space.c remove unneeded semicolon

2018-10-16 Thread Tomas Winkler
Remove unneeded semicolon in tpm2_map_response_header() Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen --- V3: New in the series. V4-V6: Resend. drivers/char/tpm/tpm2-space.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/tpm/tpm2-space.c b/drivers/

[PATCH v6 02/20] tpm: sort objects in the Makefile

2018-10-16 Thread Tomas Winkler
Make the tpm Makefile a bit more in order by putting objects in one column. Signed-off-by: Tomas Winkler Tested-by: Jarkko Sakkinen Reviewed-by: Jarkko Sakkinen --- V2: 1. back to tpm-y notation 2. Partially sort files alphanumerically. V3: Rebase V4: Remove redunant sentence in commit mes

[PATCH v6 01/20] tpm2: add new tpm2 commands according to TCG 1.36

2018-10-16 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, the switch statement function is called tpm2_ordinal_duration_index(). 3. Fix

[PATCH v6 00/21] tpm: separate tpm 1.x and tpm 2.x commands

2018-10-16 Thread Tomas Winkler
This patch series provides initial separation of tpm 1.x and tpm 2.x commands, in foresight that the tpm 1.x chips will eventually phase out and can be compiled out for modern systems. A new file is added tpm1-cmd.c that contains tpm 1.x specific commands. In addition, tpm 1.x commands are now imp

[PATCH v6 03/20] tpm: factor out tpm 1.x duration calculation to tpm1-cmd.c

2018-10-16 Thread Tomas Winkler
Factor out TPM 1.x 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 Tested-by: Jarkko Sakkinen --- V2-V3: Rebase V4: 1. Remove the licence statement it's already covered by SPDX. 2. Add kdoc t

[PATCH v6 12/20] tpm: add tpm_auto_startup() into tpm-interface.c

2018-10-16 Thread Tomas Winkler
Add wrapper tpm_auto_startup() to tpm-interface.c instead of open coded decision between TPM 1.x and TPM 2.x in tpm-chip.c Signed-off-by: Tomas Winkler Tested-by: Jarkko Sakkinen --- V3: New in the series. V4: Fix the commit message. V5-V6: Resend. drivers/char/tpm/tpm-chip.c | 11 +++

[PATCH v6 05/20] tpm: factor out tpm_get_timeouts()

2018-10-16 Thread Tomas Winkler
Factor out tpm_get_timeouts() into tpm2_get_timeouts() and tpm1_get_timeouts() V2-V3: Rebase V4: Fix the commit message. V5: Resend. V6: Remove tpm1_auto_startup() declaration. Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko Sakkinen --- drivers/char/tpm/tpm-interf

[PATCH v6 04/20] tpm: add tpm_calc_ordinal_duration() wrapper

2018-10-16 Thread Tomas Winkler
Add convenient wrapper for ordinal duration computation to remove boiler plate if else statement over TPM2. if (chip->flags & TPM_CHIP_FLAG_TPM2) tpm2_calc_ordinal_duration(chip, ordinal); else tpm1_calc_ordinal_duration(chip, ordinal); V2-V3: Rebase. V4: Add Kdoc. V5: Adjust the

[PATCH v6 06/20] tpm: move tpm1_pcr_extend to tpm1-cmd.c

2018-10-16 Thread Tomas Winkler
Move tpm1_pcr_extend to tpm1-cmd.c and remove unused pcrextend_header structure and EXTEND_PCR_RESULT_SIZE and EXTEND_PCR_RESULT_BODY_SIZE defines. Fixes warning: drivers/char/tpm/tpm-interface.c:609:38: warning: ‘pcrextend_header’ defined but not used [-Wunused-const-variable=] static const str

Re: [PATCH] extract-vmlinux: Check for uncompressed image as fallback

2018-10-16 Thread Masahiro Yamada
Hi Helge, On Tue, Oct 16, 2018 at 3:06 PM Helge Deller wrote: > > As on x86-64 and other architectures, the boot kernel on parisc (vmlinuz > and bzImage) contains a full compressed copy of the final kernel > executable (vmlinux.bin.gz), which one should be able to extract with > the extract-vml

[GIT PULL] parisc fix for kernel v4.19

2018-10-16 Thread Helge Deller
Hi Greg, would you please include the trivial patch below for 4.19? My older cross-compiler didn't complained, so it went unnoticed for me until now and the faulty code was added during the 4.19 merge window. It can be pulled from my git tree as well: git://git.kernel.org/pub/scm/linux/kernel/

Re: [PATCH v6 1/2] spi: Add MXIC controller driver

2018-10-16 Thread Boris Brezillon
On Wed, 17 Oct 2018 10:08:11 +0800 masonccy...@mxic.com.tw wrote: > From: Mason Yang > > Add a driver for Macronix SPI controller IP. > > Signed-off-by: Mason Yang Reviewed-by: Boris Brezillon > --- > drivers/spi/Kconfig| 6 + > drivers/spi/Makefile | 1 + > drivers/spi/spi-mxic.

Re: [PATCH v6 2/2] dt-binding: spi: Document Macronix controller bindings

2018-10-16 Thread Boris Brezillon
+Rob and the DT ML Hi Mason, Remember to Cc the DT mailing list and maintainers when you add/update a binding. On Wed, 17 Oct 2018 10:08:12 +0800 masonccy...@mxic.com.tw wrote: > From: Mason Yang > > Document the bindings used by the Macronix controller. > > Signed-off-by: Mason Yang > ---

[RFC v4 PATCH 5/5] mm/can_skip_merge(): make it more aggressive to attempt cluster alloc/free

2018-10-16 Thread Aaron Lu
After system runs a long time, it's easy for a zone to have no suitable high order page available and that will stop cluster alloc and free in current implementation due to compact_considered > 0. To make it favour order0 alloc/free, relax the condition to only disallow cluster alloc/free when pro

[RFC v4 PATCH 1/5] mm/page_alloc: use helper functions to add/remove a page to/from buddy

2018-10-16 Thread Aaron Lu
There are multiple places that add/remove a page into/from buddy, introduce helper functions for them. This also makes it easier to add code when a page is added/removed to/from buddy. No functionality change. Acked-by: Vlastimil Babka Signed-off-by: Aaron Lu --- mm/page_alloc.c | 65

[RFC v4 PATCH 4/5] mm/free_pcppages_bulk: reduce overhead of cluster operation on free path

2018-10-16 Thread Aaron Lu
After "no_merge for order 0", the biggest overhead in free path for order 0 pages is now add_to_cluster(). As pages are freed one by one, it caused frequent operation of add_to_cluster(). Ideally, if only one migratetype pcp list has pages to free and count=pcp->batch in free_pcppages_bulk(), we c

[RFC v4 PATCH 3/5] mm/rmqueue_bulk: alloc without touching individual page structure

2018-10-16 Thread Aaron Lu
Profile on Intel Skylake server shows the most time consuming part under zone->lock on allocation path is accessing those to-be-returned page's "struct page" on the free_list inside zone->lock. One explanation is, different CPUs are releasing pages to the head of free_list and those page's 'struct

[RFC v4 PATCH 2/5] mm/__free_one_page: skip merge for order-0 page unless compaction failed

2018-10-16 Thread Aaron Lu
Running will-it-scale/page_fault1 process mode workload on a 2 sockets Intel Skylake server showed severe lock contention of zone->lock, as high as about 80%(42% on allocation path and 35% on free path) CPU cycles are burnt spinning. With perf, the most time consuming part inside that lock on free

[RFC v4 PATCH 0/5] Eliminate zone->lock contention for will-it-scale/page_fault1 and parallel free

2018-10-16 Thread Aaron Lu
This series is meant to improve zone->lock scalability for order 0 pages. With will-it-scale/page_fault1 workload, on a 2 sockets Intel Skylake server with 112 CPUs, CPU spend 80% of its time spinning on zone->lock. Perf profile shows the most time consuming part under zone->lock is the cache miss

Re: [PATCH] dt-bindings: ufs: Fix the compatible string definition

2018-10-16 Thread Vivek Gautam
Hi Doug, On 10/16/2018 10:29 PM, Doug Anderson wrote: Hi, On Mon, Oct 15, 2018 at 10:51 PM Vivek Gautam wrote: P.S.: While you are at it, can you please move 'ufs-qcom.txt' to Documentation/devicetree/bindings/phy/qcom,ufs-phy.txt. The current name and file location is misleading. I'd rathe

[PATCH] xfs: Fix error code in 'xfs_ioc_getbmap()'

2018-10-16 Thread Christophe JAILLET
In this function, once 'buf' has been allocated, we unconditionally return 0. However, 'error' is set to some error codes in several error handling paths. Before commit 232b51948b99 ("xfs: simplify the xfs_getbmap interface") this was not an issue because all error paths were returning directly, bu

Re: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread Dan Carpenter
On Wed, Oct 17, 2018 at 07:07:05AM +0200, Greg KH wrote: > On Wed, Oct 17, 2018 at 03:14:04AM +, k...@linuxonhyperv.com wrote: > > From: Dexuan Cui > > > > In kvp_send_key(), we do need call process_ib_ipinfo() if > > message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out > >

Re:Business proposition for you

2018-10-16 Thread Melvin Greg
Hello, Business proposition for you. I have a client from Syrian who will like to invest with your company. My client is willing to invest $4 Million. Can I have your company website to show to my client your company so that they will check and decide if they will invest there funds with you

Re: [PATCH] crypto: inside-secure: safexcel - fix memory allocation

2018-10-16 Thread Herbert Xu
On Tue, Oct 16, 2018 at 09:44:02PM +0200, Gustavo A. R. Silva wrote: > Hi all, > > On 10/9/18 12:20 AM, Kees Cook wrote: > > On Mon, Oct 8, 2018 at 12:17 PM, Gustavo A. R. Silva > > wrote: > >> The original intention is to allocate space for EIP197_DEFAULT_RING_SIZE > >> *pointers* to struct, so

Re: linux-next: Signed-off-by missing for commits in the parisc-hd tree

2018-10-16 Thread Helge Deller
On 17.10.2018 08:15, Stephen Rothwell wrote: > Commits > 17714e83ebff ("parisc: Release spinlocks using ordered store") > c389eafd8ac2 ("parisc: Reorder TLB flush timing calculation") > are missing a Signed-off-by from their committer. Fixed. Thanks, Helge signature.asc Description: OpenPGP

linux-next: Signed-off-by missing for commits in the parisc-hd tree

2018-10-16 Thread Stephen Rothwell
Hi all, Commits 17714e83ebff ("parisc: Release spinlocks using ordered store") c389eafd8ac2 ("parisc: Reorder TLB flush timing calculation") are missing a Signed-off-by from their committer. -- Cheers, Stephen Rothwell pgp0PRi7HHTC0.pgp Description: OpenPGP digital signature

Re: [PATCH v4 5/7] power: supply: add AC power supply driver for AXP813

2018-10-16 Thread Chen-Yu Tsai
On Sat, Oct 13, 2018 at 4:09 PM Oskari Lemmela wrote: > > AXP813 and AXP803 PMICs can control input current and > minimum voltage. > > Both of these values are configurable. > > Signed-off-by: Oskari Lemmela > Reviewed-by: Quentin Schulz > --- > drivers/power/supply/axp20x_ac_power.c | 92 +

[PATCH 2/2] clk: imx7d: remove CLK_IS_CRITICAL flag for arm_a7_root_clk

2018-10-16 Thread Anson Huang
i.MX7D uses virtual cpu clock of "arm" clock to be child clock of "arm_a7_root_clk" and it is with CLK_IS_CRITICAL flag set, so no need to add CLK_IS_CRITICAL flag for keeping "arm_a7_root_clk" use count correct, latest clock tree is as below in clk_summary: pll_arm_main 1

[PATCH 1/2] clk: imx: cpu clock should be always critical

2018-10-16 Thread Anson Huang
Add CLK_IS_CRITICAL flag for cpu clock type to make cpu clock use count correct, as cpu clock should be always critical. Signed-off-by: Anson Huang --- drivers/clk/imx/clk-cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/imx/clk-cpu.c b/drivers/clk/imx/clk-cp

linux-next: Tree for Oct 17

2018-10-16 Thread Stephen Rothwell
Hi all, Changes since 20181016: The net-next tree gained a conflict against the net tree. The netfilter-next tree lost its build failure. The char-misc tree gained a conflict against the drm tree. Non-merge commits (relative to Linus' tree): 10826 9851 files changed, 513320 inser

Re: [PATCH v5 20/21] tpm1: reimplement tpm1_continue_selftest() using tpm_buf

2018-10-16 Thread Nayna Jain
On 09/29/2018 04:00 AM, Tomas Winkler wrote: Reimplement tpm1_continue_selftest() using tpm_buf structure. This is the last command using the old tpm_cmd_t structure and now the structure can be removed. Cc: Nayna Jain Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen Tested-by: Ja

Re: [[PATCH] 8/9] DMA-UART-Driver-for-AST2500

2018-10-16 Thread Vinod
On 17-10-18, 09:41, sudheer.v wrote: Please add the change log describing the driver and its features > Signed-off-by: sudheer.v > --- > drivers/tty/serial/8250/8250_aspeed_uart_dma.c | 1594 > > 1 file changed, 1594 insertions(+) > create mode 100644 drivers/tty/se

Re: [PATCH v3 9/9] clk: actions: Add Actions Semi S900 SoC Reset Management Unit support

2018-10-16 Thread Stephen Boyd
Quoting Manivannan Sadhasivam (2018-08-10 02:51:13) > Add Reset Management Unit (RMU) support for Actions Semi S900 SoC. > > Signed-off-by: Manivannan Sadhasivam > --- Applied to clk-next

Re: [PATCH v3 3/9] dt-bindings: reset: Add binding constants for Actions Semi S700 RMU

2018-10-16 Thread Stephen Boyd
Quoting Manivannan Sadhasivam (2018-08-10 02:51:07) > Add device tree binding constants for Actions Semi S700 SoC Reset > Management Unit (RMU). > > Signed-off-by: Manivannan Sadhasivam > Reviewed-by: Rob Herring > --- Applied to clk-next

Re: [PATCH v3 2/9] dt-bindings: clock: Add reset controller bindings for Actions Semi Owl SoCs

2018-10-16 Thread Stephen Boyd
Quoting Manivannan Sadhasivam (2018-08-10 02:51:06) > Add Reset Controller bindings to clock bindings for Actions Semi Owl > SoCs, S700 and S900. > > Signed-off-by: Manivannan Sadhasivam > Reviewed-by: Rob Herring > --- Applied to clk-next

Re: [PATCH v3 4/9] dt-bindings: reset: Add binding constants for Actions Semi S900 RMU

2018-10-16 Thread Stephen Boyd
Quoting Manivannan Sadhasivam (2018-08-10 02:51:08) > Add device tree binding constants for Actions Semi S900 SoC Reset > Management Unit (RMU). > > Signed-off-by: Manivannan Sadhasivam > Reviewed-by: Rob Herring > --- Applied to clk-next

Re: [PATCH v3 7/9] clk: actions: Add Actions Semi Owl SoCs Reset Management Unit support

2018-10-16 Thread Stephen Boyd
Quoting Manivannan Sadhasivam (2018-08-10 02:51:11) > Add Reset Management Unit (RMU) support for Actions Semi Owl SoCs. > > Signed-off-by: Manivannan Sadhasivam > --- Applied to clk-next

Re: [PATCH v3 8/9] clk: actions: Add Actions Semi S700 SoC Reset Management Unit support

2018-10-16 Thread Stephen Boyd
Quoting Manivannan Sadhasivam (2018-08-10 02:51:12) > Add Reset Management Unit (RMU) support for Actions Semi S700 SoC. > > Signed-off-by: Manivannan Sadhasivam > --- Applied to clk-next

RE: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread KY Srinivasan
> -Original Message- > From: Greg KH > Sent: Tuesday, October 16, 2018 10:07 PM > To: KY Srinivasan > Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; Stephen > Hemminger ; Michael Kelley (EOSG) > ; vkuznets ; >

Re: [PATCH v3 1/9] clk: actions: Cache regmap info in private clock descriptor

2018-10-16 Thread Stephen Boyd
Quoting Manivannan Sadhasivam (2018-08-10 02:51:05) > In order to support the reset controller, regmap info needs to > be cached in the private clock descriptor, owl_clk_desc. Hence, > save that and also make the clock descriptor struct non const. > > Signed-off-by: Manivannan Sadhasivam > --- A

Re: [PATCH 2/4] Renames variable to fix shadow warning.

2018-10-16 Thread Ingo Molnar
* Leonardo Brás wrote: > Renames the char variable to avoid shadowing a variable previously > declared on this function. > > Signed-off-by: Leonardo Brás > --- > arch/x86/entry/vdso/vdso2c.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/entry/vdso/vds

RE: [PATCH 1/5] Drivers: hv: vmbus: Get rid of unnecessary state in hv_context

2018-10-16 Thread KY Srinivasan
> -Original Message- > From: Greg KH > Sent: Tuesday, October 16, 2018 10:05 PM > To: KY Srinivasan > Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; Stephen > Hemminger ; Michael Kelley (EOSG) > ; vkuznets > S

RE: [PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread KY Srinivasan
> -Original Message- > From: Greg KH > Sent: Tuesday, October 16, 2018 10:04 PM > To: KY Srinivasan > Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; Stephen > Hemminger ; Michael Kelley (EOSG) > ; vkuznets ; >

[v5 3/4] mpt3sas:Fix driver modifying persistent data.

2018-10-16 Thread Suganath Prabu
* If EEDPTagMode field in manufacturing page11 is set, unset it. This is needed to fix a hardware bug in SAS3/SAS2 cards, So, skipping EEDPTagMode changes in Manufacturing page11 for SAS35 controllers. * Fix driver modifying NVRAM/persistent data in Manufacturing page11 along with current copy. Dr

Fate of ihex2fw tool

2018-10-16 Thread Andrey Smirnov
Everyone: Since commit 5620a0d1aacd554ebebcff373e31107bb1ef7769 ("firmware: delete in-kernel firmware"): https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.19-rc7&id=5620a0d1aacd554ebebcff373e31107bb1ef7769 firmware/ihex2fw.c is gone from the source tree and with it

Re: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread Gustavo A. R. Silva
On 10/17/18 7:07 AM, Greg KH wrote: > On Wed, Oct 17, 2018 at 03:14:04AM +, k...@linuxonhyperv.com wrote: >> From: Dexuan Cui >> >> In kvp_send_key(), we do need call process_ib_ipinfo() if >> message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out >> the userland hv_kvp_daem

Re: [PATCH 5/5] Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:06AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > The patch fixes: > > hv_kvp_daemon.c: In function 'kvp_set_ip_info': > hv_kvp_daemon.c:1305:2: note: 'snprintf' output between 41 and 4136 bytes > into a destination of size 4096 > > Signed-off-by: De

Re: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:04AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > In kvp_send_key(), we do need call process_ib_ipinfo() if > message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out > the userland hv_kvp_daemon needs the info of operation, adapter_id an

Re: [PATCH 1/5] Drivers: hv: vmbus: Get rid of unnecessary state in hv_context

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:02AM +, k...@linuxonhyperv.com wrote: > From: "K. Y. Srinivasan" > > Currently we are replicating state in struct hv_context that is unnecessary - > this state can be retrieved from the hypervisor. Furthermore, this is a > per-cpu > state that is being maintaine

Re: [PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:05AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > I didn't find a real issue. Let's just make it consistent with the > next "case REG_U64:" where %llu is used. > > Signed-off-by: Dexuan Cui > Cc: K. Y. Srinivasan > Cc: Haiyang Zhang > Cc: Stephen H

Re: [PATCH v2 6/6] arm64: platform: Add enable Exynos Multi-Core Timer driver

2018-10-16 Thread Chanwoo Choi
Hi Marek, On 2018년 10월 15일 21:31, Marek Szyprowski wrote: > On Exynos SoCs enabling MCT driver is required even if ARM Architected > Timer driver is used to for managing timer hardware and clock source > events. > > Signed-off-by: Marek Szyprowski > --- > arch/arm64/Kconfig.platforms | 1 + > 1

Re: [PATCH] Kbuild: Hide Clang's -Wempty-body behind W=1

2018-10-16 Thread Nathan Chancellor
Hi Masahiro, On Wed, Oct 17, 2018 at 01:48:46PM +0900, Masahiro Yamada wrote: > Hi Nathan, > > > On Tue, Oct 16, 2018 at 11:15 AM Nathan Chancellor > wrote: > > > > There are only a few instances of this warning in an arm64 allyesconfig > > build but none of them appear useful. I believe the in

Re: [PATCH v2 5/6] arm64: dts: exynos: Move arch-timer node to right place

2018-10-16 Thread Chanwoo Choi
Hi Marek, On 2018년 10월 15일 21:31, Marek Szyprowski wrote: > Move ARM architected timer device-tree node to the beginning of 'soc' > node, to group it together with other ARM CPU core devices (like PMU). > > Signed-off-by: Marek Szyprowski > --- > arch/arm64/boot/dts/exynos/exynos5433.dtsi | 23

Re: [PATCH v2 3/6] clocksource: exynos_mct: Add arch_timer cooperation mode for ARM64

2018-10-16 Thread Chanwoo Choi
Hi Marek, I tested it about kernel booting and CPU hotplug through sysfs path on ARM64 Exynos5433-based TM2 board. It is well working. Reviewed-by: Chanwoo Choi Tested-by: Chanwoo Choi But, I have a minor comment for code clean-up. On exynos4_mct_starting_cpu() function, the initialization of

Re: [PATCH] Kbuild: Hide Clang's -Wempty-body behind W=1

2018-10-16 Thread Masahiro Yamada
Hi Nathan, On Tue, Oct 16, 2018 at 11:15 AM Nathan Chancellor wrote: > > There are only a few instances of this warning in an arm64 allyesconfig > build but none of them appear useful. I believe the intention of the > warning is to avoid situations like this: > > if (condition); > statem

Re: [RFC][PATCHv2 1/4] panic: avoid deadlocks in re-entrant console drivers

2018-10-16 Thread Sergey Senozhatsky
On (10/16/18 14:04), Sergey Senozhatsky wrote: > > Fix this by setting oops_in_progress before console_flush_on_panic(), > so re-entrant console drivers will trylock the port->lock instead of > spinning on it forever. > Just a small note: Regardless of what's going to happen to the series, t

Re: [Lkcamp] [PATCH 0/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patches. On 10/16/18 9:07 PM, Leonardo Brás wrote: > This patchset add -Wshadow=local on KBUILD_HOSTCFLAGS and fixes > all code that show this warning. > > The third patch was already submitted, but was not merged yet. > I like to think it's part of this patchset, bu

Re: [RFC][PATCHv2 2/4] printk: move printk_safe macros to printk header

2018-10-16 Thread Sergey Senozhatsky
On (10/16/18 14:54), Peter Zijlstra wrote: > On Tue, Oct 16, 2018 at 09:27:34PM +0900, Sergey Senozhatsky wrote: > > per-CPU printk_safe _semi-magic_ makes some things simple to handle. > > We can't just remove per-CPU buffers and add a wake_up_process() at > > the bottom of vprintk_emit(). Because

Re: [Lkcamp] [PATCH 1/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch. On 10/16/18 9:08 PM, Leonardo Brás wrote: > Adds -Wshadow=local on KBUILD_HOSTCFLAGS to show shadow warnings > on tools built for HOST. > > Signed-off-by: Leonardo Brás > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a

Re: [Lkcamp] [PATCH 4/4] Changes macro usage to avoid shadowing a variable.

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch, just some small comments below. On 10/16/18 9:09 PM, Leonardo Brás wrote: > Changes the usage of DEF_FIELD_ADDR in this function to create a > reference and operate over it using an aux variable. > It also changes the loop logic used to find duplicates, to avoid

[PATCH] perf: Add Sparc jitdump support.

2018-10-16 Thread David Miller
Signed-off-by: David S. Miller --- tools/perf/arch/sparc/Makefile | 2 ++ tools/perf/util/genelf.h | 6 ++ 2 files changed, 8 insertions(+) diff --git a/tools/perf/arch/sparc/Makefile b/tools/perf/arch/sparc/Makefile index 7fbca175099e..275dea7ff59a 100644 --- a/tools/perf/arch/sparc

[[PATCH] 2/9] Defconfig-changes-for-DMA-UART-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- arch/arm/configs/aspeed_g5_defconfig | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/arm/configs/aspeed_g5_defconfig b/arch/arm/configs/aspeed_g5_defconfig index b7f8fa1..25813b5 100644 --- a/arch/arm/configs/aspeed_g5_defconfig +++ b/a

[[PATCH] 8/9] DMA-UART-Driver-for-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- drivers/tty/serial/8250/8250_aspeed_uart_dma.c | 1594 1 file changed, 1594 insertions(+) create mode 100644 drivers/tty/serial/8250/8250_aspeed_uart_dma.c diff --git a/drivers/tty/serial/8250/8250_aspeed_uart_dma.c b/drivers/tty/serial/825

[[PATCH] 4/9] Documentation-DTbindings-DMA-controller-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- .../devicetree/bindings/dma/ast-uart-sdma.txt | 23 ++ 1 file changed, 23 insertions(+) create mode 100644 Documentation/devicetree/bindings/dma/ast-uart-sdma.txt diff --git a/Documentation/devicetree/bindings/dma/ast-uart-sdma.txt b/Docume

[[PATCH] 0/9] *** DMA support for UART in ASPEED's AST2500 ***

2018-10-16 Thread sudheer.v
DMA controller driver and UART dma client driver for aspeed's AST2500 sudheer.v (9): DT-changes-for-DMA-UART-of-AST2500 Defconfig-changes-for-DMA-UART-of-AST2500 configuration-for-DMA-of-AST2500 Documentation-DTbindings-DMA-controller-of-AST2500 DMA-driver-for-AST2500 configuration-fo

[[PATCH] 1/9] DT-changes-for-DMA-UART-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- arch/arm/boot/dts/aspeed-ast2500-evb.dts | 20 arch/arm/boot/dts/aspeed-g5.dtsi | 85 2 files changed, 105 insertions(+) diff --git a/arch/arm/boot/dts/aspeed-ast2500-evb.dts b/arch/arm/boot/dts/aspeed-ast2500-evb.dt

Re: [PATCH 4.14 000/109] 4.14.77-stable review

2018-10-16 Thread Dan Rue
On Tue, Oct 16, 2018 at 07:04:28PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.77 release. > There are 109 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

[PATCH] perf: Add Sparc annotate support.

2018-10-16 Thread David Miller
Signed-off-by: David S. Miller --- tools/perf/arch/sparc/annotate/instructions.c | 169 ++ tools/perf/util/annotate.c| 8 + 2 files changed, 177 insertions(+) create mode 100644 tools/perf/arch/sparc/annotate/instructions.c diff --git a/tools/perf/arch/sp

RE: [PATCH v3 1/2] mtd: spi-nor: add support to non-uniform SFDP SPI NOR flash memories

2018-10-16 Thread Yogesh Narayan Gaur
Hi Tudor, > -Original Message- > From: Yogesh Narayan Gaur > Sent: Wednesday, October 17, 2018 7:38 AM > To: 'Cyrille Pitchen' ; Tudor Ambarus > ; marek.va...@gmail.com; > dw...@infradead.org; computersforpe...@gmail.com; > boris.brezil...@bootlin.com; rich...@nod.at > Cc: linux-kernel@vge

Re: [PATCH v4 3/4] perf/smmuv3: Add MSI irq support

2018-10-16 Thread kbuild test robot
Hi Shameer, Thank you for the patch! Yet something to improve: [auto build test ERROR on linux-sof-driver/master] [also build test ERROR on v4.19-rc8 next-20181016] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com

[PATCH] selftests/ftrace: Strip escape sequences for log file

2018-10-16 Thread Masami Hiramatsu
Strip escape sequences from the stream to the ftracetest summary log file. Note that all test-case results are dumped raw in each file. Signed-off-by: Masami Hiramatsu --- tools/testing/selftests/ftrace/ftracetest | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/

Re: [PATCH 0/4] Add clock support for Hi3670 SoC

2018-10-16 Thread Manivannan Sadhasivam
On Thu, Sep 20, 2018 at 11:00:59PM -0700, Manivannan Sadhasivam wrote: > This patchset adds clock support for Hi3670 SoC from HiSilicon utilizing > the HiSi common clock code. While adding clock support, let's remove the > fixed clock for UART and source SoC clock on HiKey970 board. > > This patch

Re: [PATCH 11/19] PCI: keystone: Cleanup PHY handling

2018-10-16 Thread Kishon Vijay Abraham I
Lorenzo, On Tuesday 16 October 2018 10:36 PM, Lorenzo Pieralisi wrote: > On Mon, Oct 15, 2018 at 06:37:13PM +0530, Kishon Vijay Abraham I wrote: >> Cleanup PHY handling by using devm_phy_optional_get to get PHYs if >> the PHYs are optional, creating a device link between the PHY device >> and the

[PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread kys
From: Dexuan Cui In kvp_send_key(), we do need call process_ib_ipinfo() if message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out the userland hv_kvp_daemon needs the info of operation, adapter_id and addr_family. With the incorrect fc62c3b1977d, the host can't get the VM's IP via

[PATCH 1/5] Drivers: hv: vmbus: Get rid of unnecessary state in hv_context

2018-10-16 Thread kys
From: "K. Y. Srinivasan" Currently we are replicating state in struct hv_context that is unnecessary - this state can be retrieved from the hypervisor. Furthermore, this is a per-cpu state that is being maintained as a global state in struct hv_context. Get rid of this state in struct hv_context.

[PATCH 2/5] hv_utils: update name in struct hv_driver util_drv

2018-10-16 Thread kys
From: Haiyang Zhang The correct module name is hv_utils. This patch corrects the name in struct hv_driver util_drv. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hv/hv_util.c b/

[PATCH 5/5] Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1

2018-10-16 Thread kys
From: Dexuan Cui The patch fixes: hv_kvp_daemon.c: In function 'kvp_set_ip_info': hv_kvp_daemon.c:1305:2: note: 'snprintf' output between 41 and 4136 bytes into a destination of size 4096 Signed-off-by: Dexuan Cui Cc: K. Y. Srinivasan Cc: Haiyang Zhang Cc: Stephen Hemminger Cc: Signed-off-

[PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread kys
From: Dexuan Cui I didn't find a real issue. Let's just make it consistent with the next "case REG_U64:" where %llu is used. Signed-off-by: Dexuan Cui Cc: K. Y. Srinivasan Cc: Haiyang Zhang Cc: Stephen Hemminger Cc: Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_kvp.c | 2 +- 1 file ch

[PATCH 0/5] Drivers: hv: Miscellaneous fixes

2018-10-16 Thread kys
From: "K. Y. Srinivasan" Miscellaneous fixes. Dexuan Cui (3): Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up Drivers: hv: kvp: Use %u to print U32 Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1 Haiyang Zhang (1): hv_utils: update name in st

Re: [Lkcamp] [PATCH 3/4] kbuild: Removes unnecessary shadowed local variable and optimize testing.

2018-10-16 Thread Helen Koike
Hi Leonardo, On 10/16/18 9:09 PM, Leonardo Brás wrote: > Removes an unnecessary shadowed local variable (start). > Optimize test of isdigit: > - If isalpha returns true, isdigit will return false, so no need to test. This patch does two different things, it should be in two separated

Re: [PATCH v3 6/9] arm64: dts: actions: Add Reset Controller support for S900 SoC

2018-10-16 Thread Manivannan Sadhasivam
On Tue, Oct 16, 2018 at 02:42:19PM -0700, Stephen Boyd wrote: > Quoting Manivannan Sadhasivam (2018-08-10 02:51:10) > > Add reset controller property and bindings header for the > > Actions Semi S900 SoC DTS. > > > > Signed-off-by: Manivannan Sadhasivam > > --- > > Are these DT patches necessary

Re: [Lkcamp] [PATCH 2/4] Renames variable to fix shadow warning.

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch, just some small comments below. Please, check previous log messages with git log arch/x86/entry/vdso/vdso2c.h, you will see that most patches had the prefix "x86/vdso:" in the commit message. On 10/16/18 9:08 PM, Leonardo Brás wrote: > Renames the char variable

Re: [PATCH v9 5/5] lib/dlock-list: Scale dlock_lists_empty()

2018-10-16 Thread Davidlohr Bueso
On Thu, 04 Oct 2018, Waiman Long wrote: On 10/04/2018 03:16 AM, Jan Kara wrote: On Wed 12-09-18 15:28:52, Waiman Long wrote: From: Davidlohr Bueso Instead of the current O(N) implementation, at the cost of adding an atomic counter, we can convert the call to an atomic_read(). The counter onl

Re: [Ksummit-discuss] [PATCH v3 1/3] code-of-conduct: Fix the ambiguity about collecting email addresses

2018-10-16 Thread James Bottomley
On Tue, 2018-10-16 at 19:10 -0700, Frank Rowand wrote: > On 10/16/18 07:58, James Bottomley wrote: > > The current code of conduct has an ambiguity in the it considers > > publishing > > private information such as email addresses unacceptable > > behaviour. Since > > the Linux kernel collects and

  1   2   3   4   5   6   7   8   9   10   >