[PATCH] ANDROID: binder: fix binder work return error is wrongly consumed

2018-05-02 Thread gityuan
From: yuanhuihui transaction consume mismatched binder work. as follow: 1) Client send transaction when left binder work in the thread->todo list. 2) in kernel space, client consumed BC_TRANSACTION, and produced BINDER_WORK_RETURN_ERROR into thread->todo list as target proc is dead 3) Client s

[PATCH] ARM: EXYNOS: no need to select ARCH_HAS_BANDGAP any longer

2018-05-02 Thread Bartlomiej Zolnierkiewicz
This is not needed since commit fcbb1e02ee54 ("drivers: thermal: Remove ARCH_HAS_BANDGAP dependency for samsung") from 2014. Signed-off-by: Bartlomiej Zolnierkiewicz --- arch/arm/mach-exynos/Kconfig |1 - 1 file changed, 1 deletion(-) Index: b/arch/arm/mach-exynos/Kconfig ==

Re: [PATCH] swiotlb: Fix inversed DMA_ATTR_NO_WARN test

2018-05-02 Thread Michel Dänzer
On 2018-05-02 02:41 PM, Christoph Hellwig wrote: > On Wed, May 02, 2018 at 02:18:56PM +0200, Daniel Vetter wrote: >> Other dma-api backends like cma just shut up when __GFP_NOWARN is >> passed. And afaiui Christoph Hellwig has plans to nuke the DMA_ATTR >> stuff (or at least clean it up) - should w

Re: [PATCH v7 2/5] cpuset: Add cpuset.sched_load_balance to v2

2018-05-02 Thread Mike Galbraith
On Wed, 2018-05-02 at 16:02 +0200, Peter Zijlstra wrote: > On Wed, May 02, 2018 at 09:47:00AM -0400, Waiman Long wrote: > > > > I've read half of the next patch that adds the isolation thing. And > > > while that kludges around the whole root cgorup is magic thing, it > > > doesn't help if you mov

Re: [PATCH] Revert "dmaengine: pl330: add DMA_PAUSE feature"

2018-05-02 Thread Frank Mori Hess
On Wed, May 2, 2018 at 12:32 AM, Vinod Koul wrote: > On Sat, Apr 28, 2018 at 05:50:58PM -0400, Frank Mori Hess wrote: >> This reverts commit 88987d2c7534a0269f567fb101e6d71a08f0f01d. >> > > I am dropping the orignal patch for queue, so no need for revert patch. > I don't understand, 88987d2c7534a

[PATCH 2/2] tpm: self test failure should not cause suspend to fail

2018-05-02 Thread Jarkko Sakkinen
From: Chris Chiu commit 0803d7befa15cab5717d667a97a66214d2a4c083 upstream The Acer Acer Veriton X4110G has a TPM device detected as: tpm_tis 00:0b: 1.2 TPM (device-id 0xFE, rev-id 71) After the first S3 suspend, the following error appears during resume: tpm tpm0: A TPM error(38) occurred c

[PATCH 0/2] fixes for 4.9.96

2018-05-02 Thread Jarkko Sakkinen
Decided to add Enric's commit because it is also a bug fix instead of modifying Chris commit. Chris Chiu (1): tpm: self test failure should not cause suspend to fail Enric Balletbo i Serra (1): tpm: do not suspend/resume if power stays on drivers/char/tpm/tpm-interface.c | 7 +++ driver

Re: [PATCH 4/6] mm, arm64: untag user addresses in mm/gup.c

2018-05-02 Thread Andrey Konovalov
On Thu, Apr 26, 2018 at 7:47 PM, Catalin Marinas wrote: My approach with this was to add untagging to every gup.c function that is exposed for external use, but perhaps adding untagging only where it is actually required is a better approach. > On Wed, Apr 18, 2018 at 08:53:13PM +0200, Andrey Ko

[PATCH 1/2] tpm: do not suspend/resume if power stays on

2018-05-02 Thread Jarkko Sakkinen
From: Enric Balletbo i Serra commit b5d0ebc99bf5d0801a5ecbe958caa3d68b8eaee8 upstream The suspend/resume behavior of the TPM can be controlled by setting "powered-while-suspended" in the DTS. This is useful for the cases when hardware does not power-off the TPM. Signed-off-by: Sonny Rao Signed

Re: [PATCH] vhost: make msg padding explicit

2018-05-02 Thread Kevin Easton
On Wed, May 02, 2018 at 05:19:27PM +0300, Michael S. Tsirkin wrote: > On Wed, May 02, 2018 at 10:04:46AM -0400, David Miller wrote: > > From: "Michael S. Tsirkin" > > Date: Wed, 2 May 2018 16:36:37 +0300 > > > > > Ouch. True - and in particular the 32 bit ABI on 64 bit kernels doesn't > > > work

Re: Motorola Droid 4 progress, power consumption

2018-05-02 Thread Tony Lindgren
* Pavel Machek [180501 18:33]: > Hi! > > 4.17-rc1 is quite usable on droid 4; basically everything > works. OHCI is running all the time, which means we burn a lot of > power needlessly. To work around that we can use n_gsm and and then suspend USB device. That will need the modem wake gpio work

Re: [PATCH 1/1] vfio: ccw: fix error return in vfio_ccw_sch_event

2018-05-02 Thread Cornelia Huck
On Wed, 2 May 2018 09:25:59 +0200 Dong Jia Shi wrote: > If device has not been registered, or there is a pending work, s/a pending work/work pending/ > we should reschedule a sch_event call again. > > Signed-off-by: Dong Jia Shi Slightly off-topic question: Is the mail address with or witho

[tip:timers/core] clocksource: Remove kthread

2018-05-02 Thread tip-bot for Peter Zijlstra
Commit-ID: 7197e77abcb65a71d0b21d67beb24f153a96055e Gitweb: https://git.kernel.org/tip/7197e77abcb65a71d0b21d67beb24f153a96055e Author: Peter Zijlstra AuthorDate: Mon, 30 Apr 2018 12:00:15 +0200 Committer: Thomas Gleixner CommitDate: Wed, 2 May 2018 16:11:46 +0200 clocksource: Remove k

RE: [PATCH v2 1/2] usb: dwc3: gadget: Correct handling of scattergather lists

2018-05-02 Thread Anurag Kumar Vulisha
Hi All, Please let me know if the changes in this patch are okay . If the changes looks fine , can we proceed with this patch. Thanks, Anurag Kumar Vulisha >-Original Message- >From: v.anuragku...@gmail.com [mailto:v.anuragku...@gmail.com] >Sent: Tuesday, March 27, 2018 4:35 PM >To: F

[PATCH] drm/amd/display: clean up assignment of amdgpu_crtc

2018-05-02 Thread Colin King
From: Colin Ian King The declaration of pointer amdgpu_crtc has a redundant assignment to amdgpu_crtc. Clean this up by removing it. Detected by CoverityScan, CID#1460299 ("Evaluation order violation") Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-

Re: [PATCH v10 00/25] Speculative page faults

2018-05-02 Thread Laurent Dufour
On 02/05/2018 16:17, Punit Agrawal wrote: > Hi Laurent, > > One query below - > > Laurent Dufour writes: > > [...] > >> >> Ebizzy: >> --- >> The test is counting the number of records per second it can manage, the >> higher is the best. I run it like this 'ebizzy -mTRp'. To get consisten

Re: [PATCH 2/2] arm64/mm: add speculative page fault

2018-05-02 Thread Punit Agrawal
Hi Ganesh, I was looking at evaluating speculative page fault handling on arm64 and noticed your patch. Some comments below - Ganesh Mahendran writes: > This patch enables the speculative page fault on the arm64 > architecture. > > I completed spf porting in 4.9. From the test result, > we can

[tip:x86/urgent] x86/cpu: Restore CPUID_8000_0008_EBX reload

2018-05-02 Thread tip-bot for Thomas Gleixner
Commit-ID: c65732e4f72124ca5a3a0dd3bee0d3cee39c7170 Gitweb: https://git.kernel.org/tip/c65732e4f72124ca5a3a0dd3bee0d3cee39c7170 Author: Thomas Gleixner AuthorDate: Mon, 30 Apr 2018 21:47:46 +0200 Committer: Thomas Gleixner CommitDate: Wed, 2 May 2018 16:44:38 +0200 x86/cpu: Restore CPU

Re: [RFC PATCH] driver core: make deferring probe forever optional

2018-05-02 Thread Rob Herring
On Wed, May 2, 2018 at 6:40 AM, Robin Murphy wrote: > On 01/05/18 22:31, Rob Herring wrote: >> >> Deferred probe will currently wait forever on dependent devices to probe, >> but sometimes a driver will never exist. It's also not always critical for >> a driver to exist. Platforms can rely on defa

Re: [PATCH 2/3] kexec: call LSM hook for kexec_load syscall

2018-05-02 Thread Eric W. Biederman
Mimi Zohar writes: > Allow LSMs and IMA to differentiate between the kexec_load and > kexec_file_load syscalls by adding an "unnecessary" call to > security_kernel_read_file() in kexec_load. This would be similar to the > existing init_module syscall calling security_kernel_read_file(). Given t

Re: [PATCH v5 2/5] efi: Add embedded peripheral firmware support

2018-05-02 Thread Hans de Goede
Hi, On 05/01/2018 09:29 PM, Andy Lutomirski wrote: On Sun, Apr 29, 2018 at 2:36 AM Hans de Goede wrote: +The EFI embedded-fw code works by scanning all EFI_BOOT_SERVICES_CODE memory +segments for an eight byte sequence matching prefix, if the prefix is found it +then does a crc32 over leng

Re: [Xen-devel] [PATCH 1/4] xen/PVH: Replace GDT_ENTRY with explicit constant

2018-05-02 Thread Boris Ostrovsky
On 05/02/2018 04:00 AM, Jan Beulich wrote: On 30.04.18 at 18:23, wrote: >> Latest binutils release (2.29.1) will no longer allow proper computation >> of GDT entries on 32-bits, with warning: >> >> arch/x86/xen/xen-pvh.S: Assembler messages: >> arch/x86/xen/xen-pvh.S:150: Warning: shift count

Re: [PATCH] arm: dts: am437x-sk-evm: add wilink8 support

2018-05-02 Thread Tony Lindgren
* Eyal Reizer [180501 00:26]: > enable mmc3 used for wlan and uart1 used for bluetooth > configure the gpios used for wlan and bluetooth controls > add fixed voltage regulator used for wlan power control ... > / { > model = "TI AM437x SK EVM"; > @@ -158,6 +159,22 @@ >

[GIT PULL] SCSI fixes for 4.17-rc3

2018-05-02 Thread James Bottomley
Three small bug fixes: an illegally overlapping memcmp in target code, a potential infinite loop in isci under certain rare phy conditions and an ATA queue depth (performance) correction for storvsc. The patch is available here: git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git scsi-fix

Re: [Xen-devel] [PATCH 2/4] xen/PVH: Use proper CS selector in long mode

2018-05-02 Thread Boris Ostrovsky
On 05/02/2018 04:05 AM, Jan Beulich wrote: On 30.04.18 at 18:23, wrote: >> Signed-off-by: Boris Ostrovsky > Reviewed-by: Jan Beulich > > But to understand why things have been working nevertheless it would > have been nice if the commit message wasn't empty, but instead said > something lik

RE: [PATCH 01/10] crypto: aead - allow to allocate AEAD requests on the stack

2018-05-02 Thread David Laight
From: Antoine Tenart > Sent: 02 May 2018 10:57 > Adds the AEAD_REQUEST_ON_STACK primitive to allow allocating AEAD > requests on the stack, as it can already be done with various other > crypto algorithms within the kernel. > > Signed-off-by: Antoine Tenart > --- > include/crypto/aead.h | 5

Re: [PATCH v5 00/14] mmc: sdhci-omap: Add UHS/HS200 mode support

2018-05-02 Thread Tony Lindgren
* Ulf Hansson [180502 13:17]: > On 27 April 2018 at 13:47, Kishon Vijay Abraham I wrote: > > Add UHS/HS200 mode support in sdhci-omap. The programming sequence > > for voltage switching, tuning is followed from AM572x TRM > > http://www.ti.com/lit/ug/spruhz6j/spruhz6j.pdf > > (Similar to all AM57

Re: [PATCH v6 02/10] dt-bindings: introduce RPMH RSC bindings for Qualcomm SoCs

2018-05-02 Thread Lina Iyer
On Tue, May 01 2018 at 17:45 -0600, Doug Anderson wrote: Hi, On Thu, Apr 19, 2018 at 3:16 PM, Lina Iyer wrote: +TCS-OFFSET: 0xD00 + + apps_rsc: rsc@179e000 { This label is wrong in two ways: 1. I believe it's supposed to be the first reg listed, so 179c 2. There's only 3 0's at t

Re: [PATCH v2 0/5] kernel hacking: GCC optimization for debug experience (-Og)

2018-05-02 Thread Daniel Thompson
On Wed, May 02, 2018 at 09:44:55PM +0800, changbin...@intel.com wrote: > From: Changbin Du > > Hi all, > I know some kernel developers was searching for a method to dissable GCC > optimizations, probably they want to apply GCC '-O0' option. But since Linux > kernel replys on GCC optimization to r

Re: [RFC resend 2/2] modpost: sumversions: try to fix "same dir" logic

2018-05-02 Thread Masahiro Yamada
Hi. 2018-03-23 6:05 GMT+09:00 Rasmus Villemoes : > The current logic for deciding whether to parse a file listed in the > .o.cmd file matches neither "is in same dir" (implied by the comment > here), nor the "/* Sum all files in the same dir or subdirs. */" a > little higher up. For example, for a

Re: [Xen-devel] [PATCH 3/4] xen/PVH: Set up GS segment for stack canary

2018-05-02 Thread Boris Ostrovsky
On 05/02/2018 04:16 AM, Jan Beulich wrote: On 30.04.18 at 18:23, wrote: >> --- a/arch/x86/xen/xen-pvh.S >> +++ b/arch/x86/xen/xen-pvh.S >> @@ -54,6 +54,9 @@ >> * charge of setting up it's own stack, GDT and IDT. >> */ >> >> +#define PVH_GDT_ENTRY_CANARY4 >> +#define PVH_CANARY_SEL

Re: [PATCH v4 08/15] KVM: s390: interfaces to (de)configure guest's AP matrix

2018-05-02 Thread Pierre Morel
On 25/04/2018 18:21, Tony Krowiak wrote: On 04/23/2018 09:46 AM, Pierre Morel wrote: On 15/04/2018 23:22, Tony Krowiak wrote: Provides interfaces to assign AP adapters, usage domains and control domains to a KVM guest. ... +/** + * kvm_ap_matrix_create + * + * Create an AP matrix to hold a c

Re: [Xen-devel] [PATCH 2/4] xen/PVH: Use proper CS selector in long mode

2018-05-02 Thread Jan Beulich
>>> On 02.05.18 at 16:57, wrote: > On 05/02/2018 04:05 AM, Jan Beulich wrote: > On 30.04.18 at 18:23, wrote: >>> Signed-off-by: Boris Ostrovsky >> Reviewed-by: Jan Beulich >> >> But to understand why things have been working nevertheless it would >> have been nice if the commit message wasn

[PATCH] selftests: media_tests: fix spelling mistake: "iternations" -> "iterations"

2018-05-02 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in message text Signed-off-by: Colin Ian King --- tools/testing/selftests/media_tests/media_device_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/media_tests/media_device_test.c b/tools/te

Re: [Xen-devel] [PATCH 3/4] xen/PVH: Set up GS segment for stack canary

2018-05-02 Thread Jan Beulich
>>> On 02.05.18 at 17:00, wrote: > On 05/02/2018 04:16 AM, Jan Beulich wrote: > On 30.04.18 at 18:23, wrote: >>> --- a/arch/x86/xen/xen-pvh.S >>> +++ b/arch/x86/xen/xen-pvh.S >>> @@ -54,6 +54,9 @@ >>> * charge of setting up it's own stack, GDT and IDT. >>> */ >>> >>> +#define PVH_GDT_EN

Re: [PATCH 01/12] mtd: nand: davinci: store the core chipselect number in platform data

2018-05-02 Thread Sekhar Nori
On Wednesday 02 May 2018 04:15 PM, Bartosz Golaszewski wrote: > 2018-05-01 12:29 GMT+02:00 Sekhar Nori : >> On Tuesday 01 May 2018 03:23 PM, Sekhar Nori wrote: >>> On Tuesday 01 May 2018 02:55 PM, Sekhar Nori wrote: On Monday 30 April 2018 01:54 PM, Bartosz Golaszewski wrote: > From: Barto

Re: [Xen-devel] [PATCH 4/4] xen/PVH: Remove reserved entry in PVH GDT

2018-05-02 Thread Boris Ostrovsky
On 05/02/2018 04:26 AM, Jan Beulich wrote: On 01.05.18 at 14:34, wrote: >> On 05/01/2018 04:00 AM, Roger Pau Monné wrote: >>> On Mon, Apr 30, 2018 at 12:23:39PM -0400, Boris Ostrovsky wrote: And without it we can't use _BOOT_XX macros any longer so define new ones. >>> Not being that fam

Re: [PATCH] drm/amd/display: clean up assignment of amdgpu_crtc

2018-05-02 Thread Harry Wentland
On 2018-05-02 10:43 AM, Colin King wrote: > From: Colin Ian King > > The declaration of pointer amdgpu_crtc has a redundant assignment to > amdgpu_crtc. Clean this up by removing it. > > Detected by CoverityScan, CID#1460299 ("Evaluation order violation") > > Signed-off-by: Colin Ian King Loo

4.15.18 irq xx: nobody cared (try booting with the "irqpoll" option)

2018-05-02 Thread Udo van den Heuvel
Hello, Ever since booting into 4.15.18 I had two incidents of 'irq xx: nobody cared (try booting with the "irqpoll" option)'. Before that 4.9.60 ran for months without issue. What could be the cause? Kind regards, Udo 96.747946] snd_hda_codec_hdmi hdaudioC1D0: HDMI ATI/AMD: no speaker allocatio

Re: [Xen-devel] [PATCH 2/4] xen/PVH: Use proper CS selector in long mode

2018-05-02 Thread Boris Ostrovsky
On 05/02/2018 11:00 AM, Jan Beulich wrote: On 02.05.18 at 16:57, wrote: >> On 05/02/2018 04:05 AM, Jan Beulich wrote: >> On 30.04.18 at 18:23, wrote: Signed-off-by: Boris Ostrovsky >>> Reviewed-by: Jan Beulich >>> >>> But to understand why things have been working nevertheless it

Re: [Xen-devel] [PATCH 4/4] xen/PVH: Remove reserved entry in PVH GDT

2018-05-02 Thread Jan Beulich
>>> On 02.05.18 at 17:06, wrote: > On 05/02/2018 04:26 AM, Jan Beulich wrote: > On 01.05.18 at 14:34, wrote: >>> On 05/01/2018 04:00 AM, Roger Pau Monné wrote: On Mon, Apr 30, 2018 at 12:23:39PM -0400, Boris Ostrovsky wrote: > And without it we can't use _BOOT_XX macros any longer so

Re: [PATCH] net: ethernet: ti: cpsw: fix packet leaking in dual_mac mode

2018-05-02 Thread David Miller
From: Grygorii Strashko Date: Tue, 1 May 2018 12:41:22 -0500 > In dual_mac mode packets arrived on one port should not be forwarded by > switch hw to another port. Only Linux Host can forward packets between > ports. The below test case (reported in [1]) shows that packet arrived on > one port ca

Re: [Xen-devel] [PATCH 2/4] xen/PVH: Use proper CS selector in long mode

2018-05-02 Thread Jan Beulich
>>> On 02.05.18 at 17:08, wrote: > On 05/02/2018 11:00 AM, Jan Beulich wrote: > On 02.05.18 at 16:57, wrote: >>> On 05/02/2018 04:05 AM, Jan Beulich wrote: >>> On 30.04.18 at 18:23, wrote: > Signed-off-by: Boris Ostrovsky Reviewed-by: Jan Beulich But to understand wh

Re: [PATCH v9 03/27] clk: davinci: psc: allow for dev == NULL

2018-05-02 Thread Sekhar Nori
On Wednesday 02 May 2018 07:19 AM, David Lechner wrote: > On 05/01/2018 09:02 AM, Sekhar Nori wrote: >> On Friday 27 April 2018 05:47 AM, David Lechner wrote: >>> +static inline void *_devm_kzalloc(struct device *dev, size_t size, >>> gfp_t flags) >>> +{ >>> +    if (dev) >>> +    return devm_k

Re: [PATCH v6 0/8] ipc: Clamp *mni to the real IPCMNI limit & increase that limit

2018-05-02 Thread Eric W. Biederman
Waiman Long writes: > On 05/01/2018 10:18 PM, Eric W. Biederman wrote: >> >>> The sysctl parameters msgmni, shmmni and semmni have an inherent limit >>> of IPC_MNI (32k). However, users may not be aware of that because they >>> can write a value much higher than that without getting any error or

Re: [PATCH 1/2] dt-bindings: clock: Introduce QCOM RPMh clock bindings

2018-05-02 Thread Stephen Boyd
Quoting Taniya Das (2018-05-02 03:51:16) > Add RPMh clock device bindings for Qualcomm Technology Inc's SoCs. These > devices would be used for communicating resource state requests to control > the clocks managed by RPMh. > > Signed-off-by: Taniya Das > Reviewed-by: Rob Herring When you change

led: hw-trigger, global brightness and multi-colored leds

2018-05-02 Thread Uwe Kleine-König
Hello, on the Turris Omnia[1] the LEDs are controllable via an i2c device. Each LED can be either in "manual mode" or in a mode I'd call "native mode" which is the default. In this native mode the LED being on or off depends on an input line that is controlled by another hardware. So a LED signals

Re: [PATCH] net: stmmac: Avoid VLA usage

2018-05-02 Thread David Miller
From: Kees Cook Date: Tue, 1 May 2018 14:01:30 -0700 > In the quest to remove all stack VLAs from the kernel[1], this switches > the "status" stack buffer to use the existing small (8) upper bound on > how many queues can be checked for DMA, and adds a sanity-check just to > make sure it doesn't

Re: [RFC v3 4/5] virtio_ring: add event idx support in packed ring

2018-05-02 Thread Tiwei Bie
On Wed, May 02, 2018 at 04:51:01PM +0300, Michael S. Tsirkin wrote: > On Wed, May 02, 2018 at 03:28:19PM +0800, Tiwei Bie wrote: > > On Wed, May 02, 2018 at 10:51:06AM +0800, Jason Wang wrote: > > > On 2018年04月25日 13:15, Tiwei Bie wrote: > > > > This commit introduces the event idx support in packe

Re: [v6 0/2] clk: qcom: clk-rpmh: Add QCOM RPMh clock driver

2018-05-02 Thread Stephen Boyd
Quoting Taniya Das (2018-05-02 03:51:15) > [v6] >* Addressed comments from Rob >* Addressed comments from Stephen Please actually describe what you addressed. Nobody wants to look back at the old emails to figure out what comments you addressed.

Re: [PATCH] mm: provide a fallback for PAGE_KERNEL_RO for architectures

2018-05-02 Thread Luis R. Rodriguez
On Wed, May 02, 2018 at 12:08:57PM +0200, Geert Uytterhoeven wrote: > Hi Luis, > > On Sat, Apr 28, 2018 at 2:15 AM, Luis R. Rodriguez wrote: > > Some architectures do not define PAGE_KERNEL_RO, best we can do > > for them is to provide a fallback onto PAGE_KERNEL. Remove the > > hack from the fir

[GIT PULL 00/52] perf/core improvements and fixes

2018-05-02 Thread Arnaldo Carvalho de Melo
rg/pub/scm/linux/kernel/git/acme/linux into perf/urgent (2018-04-26 07:28:29 +0200) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-core-for-mingo-4.18-20180502 for you to fetch changes up to 107cad95ffd81afad295ed5c29d006e525f1f80

[PATCH 03/52] perf check-headers.sh: Simplify arguments passing

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Passing whole string instead of parsing them after. It simplifies things for the next patches, that adds another function call, which makes it hard to pass arguments in the correct shape. Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: David Ahern Cc: Namhyung Kim Cc: Pe

[PATCH 05/52] perf stat: Display time in precision based on std deviation

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Ingo suggested to display elapsed time for multirun workload (perf stat -e) with precision based on the precision of the standard deviation. In his own words: > This output is a slightly bit misleading: > Performance counter stats for 'make -j128' (10 runs): > 27.98

[PATCH 10/52] perf dso: Add dso__has_symbols() method

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo To replace longer code sequences in various places. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-tlk3klbkfyjrbfjvryyzn...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/pe

Re: [Xen-devel] [PATCH 3/4] xen/PVH: Set up GS segment for stack canary

2018-05-02 Thread Boris Ostrovsky
On 05/02/2018 11:01 AM, Jan Beulich wrote: On 02.05.18 at 17:00, wrote: >> On 05/02/2018 04:16 AM, Jan Beulich wrote: >> On 30.04.18 at 18:23, wrote: --- a/arch/x86/xen/xen-pvh.S +++ b/arch/x86/xen/xen-pvh.S @@ -54,6 +54,9 @@ * charge of setting up it's own stack, G

[PATCH 01/52] perf buildid-cache: Support --list option

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Ravi Bangoria 'perf buildid-cache' allows to add/remove files into cache but there is no option to list all cached files. Add --list option to list all _valid_ cached files. Ex, # perf buildid-cache --add /tmp/a.out # perf buildid-cache -l 8a86ef73e44067bca52cc3f6cd3e5446c783391c /

[PATCH 14/52] perf thread: Introduce thread__find_symbol()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Out of thread__find_addr_location(..., MAP__FUNCTION, ...), idea here is to continue removing references to MAP__{FUNCTION,VARIABLE} ahead of getting both types of symbols in the same rbtree, as various places do two lookups, looking first at MAP__FUNCTION, then at

[PATCH 17/52] perf thread: Make thread__find_symbol() return the symbol searched

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Instead of just returning it in al.sym, allowing for some simplification in its users, and to make it consistent with thread__find_map(). Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-4axi2sig

[PATCH 24/52] perf tests vmlinux-kallsyms: Use map__for_each_symbol() instead of open coded equivalent

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We had this much shorter map__for_each_symbol() helper for ages, use it and kill one more map_type use outside the code, in the tools. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-iswqjy1elgh

[PATCH 22/52] perf machine: Remove needless map_type from machine__load_vmlinux_path()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Since it uses machine__kernel_map() and this function always returns the MAP__FUNCTION map, it doesn't make sense to call it with MAP__VARIABLE. And also this is a step in the direction of nuking the MAP__{FUNCTION,VARIABLE} split. Cc: Adrian Hunter Cc: David Ahe

[PATCH 20/52] perf machine: Introduce machine__kernel_maps()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo That returns the a data structure contained the ordered list of kernel modules + the main kernel maps, one more step in removing the MAP__{FUNCTION,VARIABLE} split. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkm

[PATCH 26/52] perf ui stdio: Use map_groups__fprintf()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Instead of the variant that allows asking for just a specific map_type, because that map_type split will go away. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-eya0jvmu26qvro0nxxd49...@git.ker

[PATCH 31/52] perf map: Remove enum_type arg to map_groups__first()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Only the symbol core needs to use that, so provide a __ variant for that case, that will end up removed when we ditch the MAP__ split. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-x29k9e1ohas

[PATCH 33/52] perf thread: Remove addr_type arg from thread__find_cpumode_addr_location()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo All callers are for MAP__FUNCTION, so just ditch it and use thread__find_symbol(), that already ditched MAP__FUNCTION, i.e. internally uses it till we ditch it for good. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https:

[PATCH 38/52] perf symbols: Store the ELF symbol type in the symbol struct

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo There is code that needs to see if a resolved address is a function, so, since we're going to ditch the MAP__{FUNCTION,VARIABLE} split, store that info in the per symbol struct. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link

[PATCH 35/52] perf thread: Ditch __thread__find_symbol()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Simulate having all symbols in just one tree by searching the still existing two trees. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-uss70e8tvzzbzs326330t...@git.kernel.org Signed-off-by: Arn

[PATCH 46/52] perf symbols: Remove needless goto

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We can plain use the an else to the if block that is right after that goto, so simplify it. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-vnpc2rakf6vc98pcl5z1c...@git.kernel.org Signed-off-by:

[PATCH 43/52] perf map: Use map->prot in place of type==MAP__FUNCTION

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Equivalent, one step more in ditching enum map_type. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-mrjjc87a4tpf896j5u4sq...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/p

[PATCH 50/52] perf symbols: Move split_kallsyms to struct map_groups

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Since it mainly will populate symtabs of its maps (kernel modules). While looking at this I wonder if map_groups__split_kallsyms_for_kcore() shouldn't be all that we need, seems much simpler. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc:

[PATCH 41/52] perf symbols: No need to special case MAP__FUNCTION in fixup

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo In 39b12f781271 ("perf tools: Make it possible to read object code from vmlinux") we special case MAP__FUNCTION maps inconsistently, the first test tests the map type while the following tests added by this patch don't do that, be consistent and elliminate this spec

[PATCH 49/52] perf symbols: kallsyms__delta() needs the kmap, not the map

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo It was only using the map to obtain its kmap, so do the validation in its called, __dso__load_kallsyms() and pass the kmap, that will be used in the following patches in similar simplifications. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim C

[PATCH 51/52] perf tools: Fix spelling mistake: "builid" -> "buildid"

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin King Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Namhyung Kim Cc: Peter Zijlstra Cc: kernel-janit...@vger.kernel.org Link: http://lkml.kernel.org/r/20180427193158.17932-1-colin.k...@canonical.com Si

[PATCH 52/52] perf machine: Ditch find_kernel_function variants

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Since we do not have split symtabs anymore, no need to have explicit find_kernel_function variants, use the find_kernel_symbol ones. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-hiw2ryflju000

[PATCH 47/52] perf symbols: Split kernel symbol processing from dso__load_sym()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo More should be done to split this function, removing stuff map relocation steps from the actual symbol table loading. Arch specific stuff also should go elsewhere, to tools/arch/ and we should have it keyed by data from the perf_env either in the perf.data header o

[PATCH 48/52] perf symbols: Remove unused dso__load_all_kallsyms() 'map' parameter

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Only the 'dso' is needed, so ditch the struct used to pass (map, dso), passing just the used 'dso' pointer. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-17a4gkk1cs4up4smkviym...@git.kernel.or

[PATCH 45/52] perf symbols: Unify symbol maps

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Remove the split of symbol tables for data (MAP__VARIABLE) and for functions (MAP__FUNCTION), its unneeded and there were various places doing two lookups to find a symbol, so simplify this. We still will consider only the symbols that matched the filters in place,

[PATCH 42/52] perf symbols: Use symbol type instead of map->type

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo map->type is going away, we can derive it from map->prot, so use the same logic as in the kernel's arch/arm/kernel/module.c file: ELF32_ST_TYPE(sym->st_info) == STT_FUNC && !(sym->st_value & 1)) This was introduced in b2f8fb237e9c ("perf symbols: Fix annotation

[PATCH 44/52] perf symbols: Use map->prot in place of type==MAP__FUNCTION

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Its equivalent, one less use of enum map_type. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-6m18iv1ty7nh7kxlfmn89...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/ut

[PATCH 37/52] perf map: Remove map_type arg from map_groups__find()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo One more step in ditching the split. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-4pour7egur07tkrpbynaw...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/arch/arm/tes

[PATCH 40/52] perf sort: Use mmap->prot on "dcacheline" formatting

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo To match the kernel when setting the PERF_RECORD_MISC_MMAP_DATA bit in perf_event_attr.header.misc, that gets set when VM_EXEC is not set in the vm_flags. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.o

[PATCH 39/52] perf machine: Set PROT_EXEC for executable PERF_RECORD_MMAP records

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo The kernel doesn't fill the map 'prot' field for PERF_RECORD_MMAP records, and we will use that info to replace checking for MAP__VARIABLE, so store that when processing the PERF_RECORD_MISC_MMAP_DATA perf_event_attr.header.misc bit. Cc: Adrian Hunter Cc: David Ah

[PATCH 36/52] perf thread: Make thread__find_map() search all maps

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We still have the split internally, but users don't see it anymore, simplifying the growing number of cases where we end up searching in the MAP__VARIABLE maps. This further paves the way for ditching the split. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa C

[PATCH 30/52] perf symbols: Unexport symbol_type__is_a()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Now this is only used in the symbols.c file, where it will finally disappear when we remove the MAP_{FUNCTION,VARIABLE} split. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-a9t4d4hfrycczq9vpsk

[PATCH 34/52] perf machine: Use machine__find_kernel_function() instead of open coded version

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We have that equivalent, shorter helper, use it. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-1hcgu3k7vxdy4vknqf3kb...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/

[PATCH] ARM: multi_v7_defconfig: enable STM32 analog & timer drivers

2018-05-02 Thread Fabrice Gasnier
This enables drivers for STM32 timer, low power timer and analog hardware that can be used on STM32MP1 SoC: - Timer & LP Timer MFD core, PWM, trigger & encoder drivers - IIO ADC/DAC/DFSDM - vrefbuf regu driver (voltage reference buffer). Signed-off-by: Fabrice Gasnier --- arch/arm/configs/multi_

[PATCH 32/52] perf symbols: Remove map_type arg from dso__find_symbol()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo One more step to ditch MAP__{VARIABLE,FUNCTION} Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-919d1k13ts62pjipnpibv...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/u

Re: [PATCH 2/2] clk: qcom: clk-rpmh: Add QCOM RPMh clock driver

2018-05-02 Thread Stephen Boyd
Quoting Taniya Das (2018-05-02 03:51:17) > + > +static DEFINE_MUTEX(rpmh_clk_lock); > + > +#define __DEFINE_CLK_RPMH(_platform, _name, _name_active, _res_name, \ > + _res_en_offset, _res_on) \ > + static struct clk_rpmh _platform##_##_name_active

[PATCH 27/52] perf symbols: Shorten dso__(first|last)_symbol()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo All users want MAP__FUNCTION, and this split is going away. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-sm72zwt1f03ma5uw78l6z...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo ---

[PATCH 29/52] perf tools: Use kallsyms__is_function()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Replacing equivalent, the equivalent and longer variation: symbol__is_a(type, MAP__FUNCTION); Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-9t3dqogher54owfl9o2mi...@git.kernel.org Si

[PATCH 28/52] tools lib symbols: Introduce kallsyms__is_function()

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Out of symbol_type__is_a(type, MAP__FUNCTION), which is the only variant used so far, useful in a kallsyms library and one more step in ditching the MAP__FUNCTION/VARIABLE split. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Lin

Re: [Xen-devel] [PATCH 2/4] xen/PVH: Use proper CS selector in long mode

2018-05-02 Thread Andrew Cooper
On 02/05/18 16:09, Jan Beulich wrote: On 02.05.18 at 17:08, wrote: >> On 05/02/2018 11:00 AM, Jan Beulich wrote: >> On 02.05.18 at 16:57, wrote: On 05/02/2018 04:05 AM, Jan Beulich wrote: On 30.04.18 at 18:23, wrote: >> Signed-off-by: Boris Ostrovsky > Reviewed-by

[PATCH 25/52] perf map: Shorten map_groups__find() signature

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Removing the map_type, that is going away. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-18iiiw25r75xn7zlppjld...@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/arch/a

Re: [PATCH 3/6] arm64: untag user addresses in copy_from_user and others

2018-05-02 Thread Andrey Konovalov
On Thu, Apr 26, 2018 at 5:47 PM, Catalin Marinas wrote: > On Wed, Apr 18, 2018 at 08:53:12PM +0200, Andrey Konovalov wrote: >> @@ -238,12 +239,15 @@ static inline void uaccess_enable_not_uao(void) >> /* >> * Sanitise a uaccess pointer such that it becomes NULL if above the >> * current addr_l

[PATCH 23/52] perf tests vmlinux-kallsyms: Use machine__find_kernel_function(_by_name)

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We had this for ages, IIRC for 'perf probe' use initially, so use them instead of the variants that pass the map_type, that is going away. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-x1jpogs

[PATCH 19/52] perf Documentation: Support for asciidoctor

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Takashi Iwai The asciidoc package seems behind the recent big wave of python3 conversion, and we were advised to switch to asciidoctor instead. It's almost compatible but some extensions used for perf documentation don't work with it. Here is the patch to cover them, and add the proper su

[PATCH 21/52] perf machine: Shorten machine__load_kallsyms() signature

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo So far the only use is for MAP__FUNCTION, and since we're going to remove that split, remove the map_type argument in machine__load_kallsyms(). Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link: https://lkml.kernel.org/n/tip-5d

Re: [PATCH 2/2] clk: qcom: clk-rpmh: Add QCOM RPMh clock driver

2018-05-02 Thread Stephen Boyd
Quoting Taniya Das (2018-05-02 03:51:17) > + ret = devm_clk_hw_register(&pdev->dev, hw_clks[i]); > + if (ret) { > + dev_err(&pdev->dev, "failed to register %s\n", > + hw_clks[i]->init->name); > +

[PATCH 16/52] perf thread: Make thread__find_map() return the map

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo It was returning the searched map just on the addr_location passed, with the function itself returning void. Make it return the map so that we can make the code more compact. Cc: Adrian Hunter Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Cc: Wang Nan Link:

[PATCH 15/52] perf script: Use thread__find_symbol() instead of ad-hoc equivalent

2018-05-02 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo In dc323ce8e72d ("perf script: Enable printing of branch stack") it first tries to find the map for an address, then the symbol in the DSO backing that map, for that address, well, this is what thread__find_symbol() does, so just use it and make the code shorter. C

<    1   2   3   4   5   6   7   8   9   10   >