Re: [PATCH v5 1/7] PCI/ATS: Fix pci_prg_resp_pasid_required() dependency issues

2019-08-12 Thread Bjorn Helgaas
On Thu, Aug 01, 2019 at 05:05:58PM -0700, sathyanarayanan.kuppusw...@linux.intel.com wrote: > From: Kuppuswamy Sathyanarayanan > > Since pci_prg_resp_pasid_required() function has dependency on both > PASID and PRI, define it only if both CONFIG_PCI_PRI and > CONFIG_PCI_PASID config options are

Re: [PATCH v5 3/7] PCI/ATS: Initialize PASID in pci_ats_init()

2019-08-12 Thread Bjorn Helgaas
On Thu, Aug 01, 2019 at 05:06:00PM -0700, sathyanarayanan.kuppusw...@linux.intel.com wrote: > From: Kuppuswamy Sathyanarayanan > > Currently, PASID Capability checks are repeated across all PASID API's. > Instead, cache the capability check result in pci_pasid_init() and use > it in other PASID

Re: [PATCH v5 5/7] PCI/ATS: Add PASID support for PCIe VF devices

2019-08-12 Thread Bjorn Helgaas
On Thu, Aug 01, 2019 at 05:06:02PM -0700, sathyanarayanan.kuppusw...@linux.intel.com wrote: > From: Kuppuswamy Sathyanarayanan > > When IOMMU tries to enable PASID for VF device in > iommu_enable_dev_iotlb(), it always fails because PASID support for PCIe > VF device is currently broken in PCIE

Re: [RFC][Patch v12 1/2] mm: page_reporting: core infrastructure

2019-08-12 Thread David Hildenbrand
>> --- >> include/linux/mmzone.h | 11 ++ >> include/linux/page_reporting.h | 63 +++ >> mm/Kconfig | 6 + >> mm/Makefile| 1 + >> mm/page_alloc.c| 42 - >> mm/page_reporting.c| 332

[PATCH] checkpatch: Prefer __section over __attribute__((section(...)))

2019-08-12 Thread Joe Perches
Add another test for __attribute__((section("foo"))) uses that should be __section(foo) Suggested-by: Nick Desaulniers Signed-off-by: Joe Perches Tested-by: Nick Desaulniers # post PEBKAC --- scripts/checkpatch.pl | 12 1 file changed, 12 insertions(+) diff --git

[PATCH v2 20/22] watchdog: ziirave_wdt: Fix DOWNLOAD_START payload

2019-08-12 Thread Andrey Smirnov
Bootloader firmware expects the following traffic for DOWNLOAD_END: S Addr Wr [A] 0x10 [A] P using ziirave_firm_write_byte() will result in S Addr Wr [A] 0x10 [A] 0x01 [A] 0x01 [A] P which happens to work because firmware will ignore any extra bytes sent. Fix this by converting the code to use

[PATCH v2 01/22] watchdog: ziirave_wdt: Add missing newline

2019-08-12 Thread Andrey Smirnov
Add missing newline. Signed-off-by: Andrey Smirnov Reviewed-by: Guenter Roeck Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/watchdog/ziirave_wdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH v2 00/22] Ziirave_wdt driver fixes

2019-08-12 Thread Andrey Smirnov
Everyone, This series contains various fixes/improvements for ziirave_wdt driver. Hopefully each commit is self-explanatory. Feedback is welcome! Thanks, Andrey Smirnov Changes since [v1]: - Collected Reviewied-by from Guenter - Added two more error messages to "watchdog:

[PATCH v2 19/22] watchdog: ziirave_wdt: Drop status polling code

2019-08-12 Thread Andrey Smirnov
Bootloader firmware doesn't implement DOWNLOAD_START or DOWNLOAD_PACKET in a non-blocking way. It will stretch the clock of the first status byte read until the operation is complete. Polling for the status is not really necessary since it will always succed on the first try. Replace polling code

[PATCH v2 05/22] watchdog: ziirave_wdt: Log bootloader/firmware info during probe

2019-08-12 Thread Andrey Smirnov
Log bootloader/firmware info during probe. This information is available via sysfs already, but it's really helpful to have this in kernel log during startup as well. Signed-off-by: Andrey Smirnov Reviewed-by: Guenter Roeck Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc:

[PATCH v2 11/22] watchdog: ziirave_wdt: Make use of put_unaligned_le16

2019-08-12 Thread Andrey Smirnov
Instead of doing this explicitly use put_unaligned_le16() to place 16-bit address value into command payload. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org ---

[PATCH v2 09/22] watchdog: ziirave_wdt: Fix incorrect use of ARRAY_SIZE

2019-08-12 Thread Andrey Smirnov
Both memset() and ziirave_firm_write_block_data() expect length in bytes as an argument, not a number of elements in array. It just happens that in this particular case both values are equal. Modify the code to use sizeof() instead. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter

[PATCH v2 15/22] watchdog: ziirave_wdt: Fix misleading error message

2019-08-12 Thread Andrey Smirnov
Fix misleading error message in ziirave_wdt_init_duration(). Saying "unable to set ..." implies that an attempt at communication with watchdog device has taken palce and was not successful. In this case, however, all it indicates is that no reset pulse duration was specified either via kernel

[PATCH v2 21/22] watchdog: ziirave_wdt: Drop ziirave_firm_write_block_data()

2019-08-12 Thread Andrey Smirnov
There's only one user of ziirave_firm_write_block_data(), so we may as well inline it. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/watchdog/ziirave_wdt.c | 31

[PATCH v2 08/22] watchdog: ziirave_wdt: Skip zeros when calculating checksum

2019-08-12 Thread Andrey Smirnov
Zeros don't contribute anything to checksum value, so we can skip unused portion of the packet when calculating its checksum. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org ---

[PATCH v2 02/22] watchdog: ziirave_wdt: Be verbose about errors in probe()

2019-08-12 Thread Andrey Smirnov
The driver is quite silent in case of probe failure, which makes it more difficult to diagnose problem from the kernel log. Add logging to all of the silent error paths ziirave_wdt_probe() to improve that. Signed-off-by: Andrey Smirnov Reviewed-by: Guenter Roeck Cc: Chris Healy Cc: Guenter

[PATCH v2 16/22] watchdog: ziirave_wdt: Fix JUMP_TO_BOOTLOADER payload

2019-08-12 Thread Andrey Smirnov
Bootloader firmware expects the following traffic for JUMP_TO_BOOTLOADER: S Addr Wr [A] 0x0c [A] 0x01 [A] P using ziirave_firm_write_byte() will result in S Addr Wr [A] 0x0c [A] 0x01 [A] 0x01 [A] P which happens to work because firmware will ignore any extra bytes and expected magic value

[PATCH v2 14/22] watchdog: ziirave_wdt: Don't try to program readonly flash

2019-08-12 Thread Andrey Smirnov
Bootloader code will ignore any attempts to write data to any flash area outside of [ZIIRAVE_FIRM_FLASH_MEMORY_START; ZIIRAVE_FIRM_FLASH_MEMORY_END]. Firmware update code already have an appropriate check to skip those areas when validating updated firmware. Firmware programming code, OTOH, does

[PATCH v2 07/22] watchdog: ziirave_wdt: Check packet length only once

2019-08-12 Thread Andrey Smirnov
We don't need to check for packet length more than once, so drop the extra check in ziirave_firm_upload(). While at it move the check at the very start of __ziirave_firm_write_pkt(), as to not waste any time preparing a packet we'll never use. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc:

[PATCH v6 2/2] selftests: add tests for clone3()

2019-08-12 Thread Adrian Reber
This tests clone3() with and without set_tid to see if all desired PIDs are working as expected. The test tries to clone3() with a set_tid of -1, 1, pid_max, a PID which is already in use and an unused PID. The same tests are also running in PID namespace. In addition the clone3 test (without

[PATCH v2 06/22] watchdog: ziirave_wdt: Simplify ziirave_firm_write_pkt()

2019-08-12 Thread Andrey Smirnov
There no reason why ziirave_firm_write_pkt() has to take firmware data via 'struct ihex_binrec' and it can just take address, data pointer and data length as individual arguments. Make this change to allow us to drastically simplify handling page crossing case by removing all of the extra code

[PATCH v6 1/2] fork: extend clone3() to support setting a PID

2019-08-12 Thread Adrian Reber
The main motivation to add set_tid to clone3() is CRIU. To restore a process with the same PID/TID CRIU currently uses /proc/sys/kernel/ns_last_pid. It writes the desired (PID - 1) to ns_last_pid and then (quickly) does a clone(). This works most of the time, but it is racy. It is also slow as it

[PATCH v2 17/22] watchdog: ziirave_wdt: Fix DOWNLOAD_END payload

2019-08-12 Thread Andrey Smirnov
Bootloader firmware expects the following traffic for DOWNLOAD_END: S Addr Wr [A] 0x11 [A] P using ziirave_firm_write_byte() will result in S Addr Wr [A] 0x11 [A] 0x01 [A] 0x01 [A] P which happens to work because firmware will ignore any extra bytes sent. Fix this by converting the code to use

[PATCH v2 03/22] watchdog: ziirave_wdt: Be more verbose during firmware update

2019-08-12 Thread Andrey Smirnov
Add more error logging to ziirave_firm_upload() for diagnostics. Signed-off-by: Andrey Smirnov Reviewed-by: Guenter Roeck Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/watchdog/ziirave_wdt.c | 18

[PATCH v2 04/22] watchdog: ziirave_wdt: Don't bail out on unexpected timeout value

2019-08-12 Thread Andrey Smirnov
Reprogramming bootloader on watchdog MCU will result in reported default timeout value of "0". That in turn will be unnecessarily rejected by the driver as invalid device (-ENODEV). Simplify probe to read stored timeout value, set it to a sane default if it is bogus, and then program that value

[PATCH v2 22/22] watchdog: ziirave_wdt: Update checked I2C functionality mask

2019-08-12 Thread Andrey Smirnov
Update checked I2C functionality mask to reflect all of the SMBus primitives used by this driver. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/watchdog/ziirave_wdt.c | 5

[PATCH v2 12/22] watchdog: ziirave_wdt: Don't check if ihex record length is zero

2019-08-12 Thread Andrey Smirnov
Ihex_next_binrec() will return NULL if next record's 'len' is zero, so explicit checks for that in the driver are unnecessary. Drop them. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org ---

[PATCH v2 18/22] watchdog: ziirave_wdt: Fix RESET_PROCESSOR payload

2019-08-12 Thread Andrey Smirnov
Bootloader firmware expects the following traffic for RESET_PROCESSOR: S Addr Wr [A] 0x0b [A] 0x01 [A] P using ziirave_firm_write_byte() will result in S Addr Wr [A] 0x0b [A] 0x01 [A] 0x01 [A] P which happens to work because firmware will ignore any extra bytes and expected magic value matches

[PATCH v2 10/22] watchdog: ziirave_wdt: Zero out only what's necessary

2019-08-12 Thread Andrey Smirnov
Instead of zeroing out all of the packet and then overwriting a significant portion of those zeros via memcpy(), zero out only a portion of the packet that is known to not contain any data. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc:

[PATCH v2 13/22] watchdog: ziirave_wdt: Don't read out more than 'len' firmware bytes

2019-08-12 Thread Andrey Smirnov
We only compare first 'len' bytes of read firmware, so we don't need to read more that that. Signed-off-by: Andrey Smirnov Cc: Chris Healy Cc: Guenter Roeck Cc: Rick Ramstetter Cc: linux-watch...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/watchdog/ziirave_wdt.c | 6 --

Re: [PATCH 0/7] Add definition for the number of standard PCI BARs

2019-08-12 Thread Thomas Gleixner
On Mon, 12 Aug 2019, Bjorn Helgaas wrote: > On Sun, Aug 11, 2019 at 06:07:55PM +0300, Denis Efremov wrote: > > Code that iterates over all standard PCI BARs typically uses > > PCI_STD_RESOURCE_END, but this is error-prone because it requires > > "i <= PCI_STD_RESOURCE_END" rather than something

Re: 5.3-rc3-ish VM crash: RIP: 0010:tcp_trim_head+0x20/0xe0

2019-08-12 Thread Sander Eikelenboom
On 12/08/2019 19:56, Eric Dumazet wrote: > > > On 8/12/19 2:50 PM, Sander Eikelenboom wrote: >> L.S., >> >> While testing a somewhere-after-5.3-rc3 kernel (which included the latest >> net merge (33920f1ec5bf47c5c0a1d2113989bdd9dfb3fae9), >> one of my Xen VM's (which gets quite some network

Re: [PATCH v3 2/4] perf: Use CAP_SYS_ADMIN with perf_event_paranoid checks

2019-08-12 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 12, 2019 at 05:01:34PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Aug 07, 2019 at 10:44:15AM -0400, Igor Lubashev escreveu: > > +++ b/tools/perf/util/evsel.c > > @@ -279,7 +279,7 @@ struct evsel *perf_evsel__new_idx(struct > > perf_event_attr *attr, int idx) > > > static

Re: [PATCH v10 09/15] dt-bindings: memory: tegra30: Convert to Tegra124 YAML

2019-08-12 Thread Dmitry Osipenko
12.08.2019 22:54, Rob Herring пишет: > On Mon, Aug 12, 2019 at 1:53 PM Rob Herring wrote: >> >> On Sun, Aug 11, 2019 at 3:01 PM Dmitry Osipenko wrote: >>> >>> The Tegra30 binding will actually differ from the Tegra124 a tad, in >>> particular the EMEM configuration description. Hence rename the

Re: [PATCH v3 4/4] perf: Use CAP_SYS_ADMIN instead of euid==0 with ftrace

2019-08-12 Thread Arnaldo Carvalho de Melo
Em Wed, Aug 07, 2019 at 10:44:17AM -0400, Igor Lubashev escreveu: > Kernel requires CAP_SYS_ADMIN instead of euid==0 to mount debugfs for ftrace. > Make perf do the same. > > Signed-off-by: Igor Lubashev > --- > tools/perf/builtin-ftrace.c | 4 +++- > 1 file changed, 3 insertions(+), 1

Re: [PATCH v5 1/7] PCI/ATS: Fix pci_prg_resp_pasid_required() dependency issues

2019-08-12 Thread sathyanarayanan kuppuswamy
On 8/12/19 1:04 PM, Bjorn Helgaas wrote: On Thu, Aug 01, 2019 at 05:05:58PM -0700, sathyanarayanan.kuppusw...@linux.intel.com wrote: From: Kuppuswamy Sathyanarayanan Since pci_prg_resp_pasid_required() function has dependency on both PASID and PRI, define it only if both CONFIG_PCI_PRI and

Re: [PATCH v3 4/4] perf: Use CAP_SYS_ADMIN instead of euid==0 with ftrace

2019-08-12 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 12, 2019 at 05:22:51PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Aug 07, 2019 at 10:44:17AM -0400, Igor Lubashev escreveu: > > Kernel requires CAP_SYS_ADMIN instead of euid==0 to mount debugfs for > > ftrace. > > Make perf do the same. > > > > Signed-off-by: Igor Lubashev

Re: [PATCH v8 14/21] clk: tegra210: Add suspend and resume support

2019-08-12 Thread Dmitry Osipenko
12.08.2019 22:03, Sowjanya Komatineni пишет: > > On 8/12/19 11:19 AM, Dmitry Osipenko wrote: >> 12.08.2019 20:28, Sowjanya Komatineni пишет: >>> On 8/12/19 9:25 AM, Dmitry Osipenko wrote: 11.08.2019 22:15, Sowjanya Komatineni пишет: > On 8/11/19 10:39 AM, Dmitry Osipenko wrote: >>

Re: [PATCH v3 4/4] perf: Use CAP_SYS_ADMIN instead of euid==0 with ftrace

2019-08-12 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 12, 2019 at 05:27:06PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Mon, Aug 12, 2019 at 05:22:51PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Wed, Aug 07, 2019 at 10:44:17AM -0400, Igor Lubashev escreveu: > > > @@ -281,7 +283,7 @@ static int __cmd_ftrace(struct perf_ftrace

Re: [PATCH v2] arm64: dts: meson-g12b-khadas-vim3: add initial device-tree

2019-08-12 Thread Kevin Hilman
Neil Armstrong writes: > From: Christian Hewitt > > The Khadas VIM3 uses the Amlogic S922X or A311S SoC, both based on the > Amlogic G12B SoC family, on a board with the same form factor as the > VIM/VIM2 models. It ships in two variants; basic and > pro which differ in RAM and eMMC size: > > -

Re: [PATCH] checkpatch: Prefer __section over __attribute__((section(...)))

2019-08-12 Thread Nick Desaulniers
On Mon, Aug 12, 2019 at 1:07 PM Joe Perches wrote: > > Add another test for __attribute__((section("foo"))) uses > that should be __section(foo) > > Suggested-by: Nick Desaulniers > Signed-off-by: Joe Perches > Tested-by: Nick Desaulniers # post PEBKAC lol please don't commit that (PEBKAC)

Re: [PATCH v10 2/7] filemap: check compound_head(page)->mapping in pagecache_get_page()

2019-08-12 Thread Johannes Weiner
On Thu, Aug 01, 2019 at 11:42:39AM -0700, Song Liu wrote: > Similar to previous patch, pagecache_get_page() avoids race condition > with truncate by checking page->mapping == mapping. This does not work > for compound pages. This patch let it check compound_head(page)->mapping > instead. > >

Re: Build regressions/improvements in v5.3-rc4

2019-08-12 Thread Randy Dunlap
On 8/12/19 3:20 AM, Geert Uytterhoeven wrote: > Below is the list of build error/warning regressions/improvements in > v5.3-rc4[1] compared to v5.2[2]. > > Summarized: > - build errors: +5/-1 > - build warnings: +137/-136 > > JFYI, when comparing v5.3-rc4[1] to v5.3-rc3[3], the summaries

Re: [PATCH v5 6/7] s390: vfio-ap: add logging to vfio_ap driver

2019-08-12 Thread Tony Krowiak
On 8/12/19 6:35 AM, Cornelia Huck wrote: On Wed, 31 Jul 2019 18:41:16 -0400 Tony Krowiak wrote: Added two DBF log files for logging events and errors; one for the vfio_ap driver, and one for each matrix mediated device. While the s390dbf is useful (especially for accessing the information

Re: [PATCH v10 6/7] mm,thp: add read-only THP support for (non-shmem) FS

2019-08-12 Thread Johannes Weiner
On Thu, Aug 01, 2019 at 11:42:43AM -0700, Song Liu wrote: > This patch is (hopefully) the first step to enable THP for non-shmem > filesystems. > > This patch enables an application to put part of its text sections to THP > via madvise, for example: > > madvise((void *)0x60, 0x20,

Re: [PATCH] ext4: set error return correctly when ext4_htree_store_dirent fails

2019-08-12 Thread Theodore Y. Ts'o
On Mon, Aug 05, 2019 at 11:44:19PM +0100, Colin King wrote: > From: Colin Ian King > > Currently when the call to ext4_htree_store_dirent fails the error return > variable 'ret' is is not being set to the error code and variable count is > instead, hence the error code is not being returned.

Re: [PATCH v10 7/7] mm,thp: avoid writes to file with THP in pagecache

2019-08-12 Thread Johannes Weiner
On Thu, Aug 01, 2019 at 11:42:44AM -0700, Song Liu wrote: > In previous patch, an application could put part of its text section in > THP via madvise(). These THPs will be protected from writes when the > application is still running (TXTBSY). However, after the application > exits, the file is

Re: [PATCH v2 1/4] PCI: pciehp: Add pciehp_set_indicators() to jointly set LED indicators

2019-08-12 Thread Lukas Wunner
On Mon, Aug 12, 2019 at 11:49:23AM -0700, sathyanarayanan kuppuswamy wrote: > > On 8/11/19 12:59 PM, Denis Efremov wrote: > > > +if ((!PWR_LED(ctrl) || pwr == PWR_NONE) && > > > +(!ATTN_LED(ctrl) || attn == ATTN_NONE)) > > > +return; > > Also I think this condition needs to

Re: [PATCH 4/7] pwm: jz4740: Improve algorithm of clock calculation

2019-08-12 Thread Paul Cercueil
Le lun. 12 août 2019 à 8:15, Uwe =?iso-8859-1?q?Kleine-K=F6nig?= a écrit : Hello Paul, On Fri, Aug 09, 2019 at 07:14:45PM +0200, Paul Cercueil wrote: Le ven. 9 août 2019 à 19:05, Uwe =?iso-8859-1?q?Kleine-K=F6nig?= a écrit : > On Fri, Aug 09, 2019 at 02:30:28PM +0200, Paul Cercueil

Re: [PATCH v6 1/2] fork: extend clone3() to support setting a PID

2019-08-12 Thread Andrei Vagin
On Mon, Aug 12, 2019 at 1:10 PM Adrian Reber wrote: > > The main motivation to add set_tid to clone3() is CRIU. > > To restore a process with the same PID/TID CRIU currently uses > /proc/sys/kernel/ns_last_pid. It writes the desired (PID - 1) to > ns_last_pid and then (quickly) does a clone().

[PATCH V2] checkpatch: Prefer __section over __attribute__((section(...)))

2019-08-12 Thread Joe Perches
Add another test for __attribute__((section("foo"))) uses that should be __section(foo) Suggested-by: Nick Desaulniers Signed-off-by: Joe Perches Tested-by: Nick Desaulniers --- V2: Remove (however dull) humor Fix fix missing ) removal scripts/checkpatch.pl | 12 1 file

Re: [PATCH v3 4/4] tpm: add driver for cr50 on SPI

2019-08-12 Thread Stephen Boyd
Quoting Jarkko Sakkinen (2019-08-09 13:31:04) > On Tue, 2019-08-06 at 15:07 -0700, Stephen Boyd wrote: > > From: Andrey Pronin > > > > Add TPM2.0 PTP FIFO compatible SPI interface for chips with Cr50 > > firmware. The firmware running on the currently supported H1 > > Secure Microcontroller

Re: [RFC PATCH v2 12/19] mm/gup: Prep put_user_pages() to take an vaddr_pin struct

2019-08-12 Thread Ira Weiny
On Fri, Aug 09, 2019 at 05:30:00PM -0700, John Hubbard wrote: > On 8/9/19 3:58 PM, ira.we...@intel.com wrote: > > From: Ira Weiny > > > > Once callers start to use vaddr_pin the put_user_pages calls will need > > to have access to this data coming in. Prep put_user_pages() for this > > data. >

Re: [PATCH 0/1] ARM: dts: meson8b: persistent MAC address for Odroid-C1

2019-08-12 Thread Kevin Hilman
Martin Blumenstingl writes: > This series makes Odroid-C1 use the MAC address which is programmed into > the eFuse. > > build-time dependencies: > none > > runtime dependencies (without these a random MAC address is assigned, > just like before these patches), both are already part of -next: > -

[PATCH] s390: vfio-ap: remove unnecessary calls to disable queue interrupts

2019-08-12 Thread Tony Krowiak
When an AP queue is reset (zeroized), interrupts are disabled. The queue reset function currently tries to disable interrupts unnecessarily. This patch removes the unnecessary calls to disable interrupts after queue reset. Signed-off-by: Tony Krowiak --- drivers/s390/crypto/vfio_ap_ops.c | 13

Re: [PATCH 6/7] pwm: jz4740: Make PWM start with the active part

2019-08-12 Thread Paul Cercueil
Le lun. 12 août 2019 à 7:55, Uwe =?iso-8859-1?q?Kleine-K=F6nig?= a écrit : On Fri, Aug 09, 2019 at 07:33:24PM +0200, Paul Cercueil wrote: Le ven. 9 août 2019 à 19:10, Uwe =?iso-8859-1?q?Kleine-K=F6nig?= a écrit : > On Fri, Aug 09, 2019 at 02:30:30PM +0200, Paul Cercueil wrote: > >

Re: [PATCH 7/7] vfio_pci: Use PCI_STD_NUM_BARS in loops instead of PCI_STD_RESOURCE_END

2019-08-12 Thread Alex Williamson
On Mon, 12 Aug 2019 15:02:34 -0500 Bjorn Helgaas wrote: > On Sun, Aug 11, 2019 at 06:08:04PM +0300, Denis Efremov wrote: > > This patch refactors the loop condition scheme from > > 'i <= PCI_STD_RESOURCE_END' to 'i < PCI_STD_NUM_BARS'. > > > > Signed-off-by: Denis Efremov > > --- > >

Re: [PATCH 0/6] arm64: g12a: add support for DVFS

2019-08-12 Thread Kevin Hilman
Kevin Hilman writes: > Kevin Hilman writes: > >> Neil Armstrong writes: >> >>> The G12A & G12B SoCs has kernel controllable CPU clocks and PWMs for >>> voltage regulators. >>> >>> This patchsets moves the meson-g12a.dtsi to meson-g12-common.dtsi to >>> simplify >>> handling the G12A & G12B

Re: [PATCH] vhost: do not reference a file that does not exist

2019-08-12 Thread Enrico Granata
Fair enough, yeah. I think what I found confusing was that the file had a precise (directly actionable in a file browser, if you will) path. If it was just listed as a filename, or a project name, it might have been more obvious that one shouldn't expect to find it within the kernel tree and just

Re: [PATCH 14/16] rtc: remove w90x900/nuc900 driver

2019-08-12 Thread Alexandre Belloni
On 09/08/2019 22:27:42+0200, Arnd Bergmann wrote: > The ARM w90x900 platform is getting removed, so this driver is obsolete. > > Signed-off-by: Arnd Bergmann > --- > drivers/rtc/Kconfig | 7 - > drivers/rtc/Makefile | 1 - > drivers/rtc/rtc-nuc900.c | 271

Re: [BUG] n_gsm: possible recursive locking detected

2019-08-12 Thread Martin Hundebøll
On 25/07/2019 13.26, Greg Kroah-Hartman wrote: On Wed, Jul 17, 2019 at 11:40:02AM +0200, Martin Hundebøll wrote: Hi, The GSM0710 line discipline driver triggers a lockdep warning when disabling the ldisc while holding a multiplexed virtual tty open:

Re: [RFC PATCH v2 15/19] mm/gup: Introduce vaddr_pin_pages()

2019-08-12 Thread Ira Weiny
On Fri, Aug 09, 2019 at 05:09:54PM -0700, John Hubbard wrote: > On 8/9/19 3:58 PM, ira.we...@intel.com wrote: > > From: Ira Weiny > > > > The addition of FOLL_LONGTERM has taken on additional meaning for CMA > > pages. > > > > In addition subsystems such as RDMA require new information to be

[rcu:rcu/next 27/86] drivers/base/core.c:102:9: error: implicit declaration of function 'lock_is_held'; did you mean 'lockref_get'?

2019-08-12 Thread kbuild test robot
tree: https://kernel.googlesource.com/pub/scm/linux/kernel/git/paulmck/linux-rcu.git rcu/next head: eee850b8c265f38ab5feeb8fe6793b1b86eb77c7 commit: 4a3a5474b4c14fc6bc57b2d30cfbf20b54d54989 [27/86] driver/core: Convert to use built-in RCU list checking config: parisc-allnoconfig (attached

Re: [PATCH 0/7] Add definition for the number of standard PCI BARs

2019-08-12 Thread Denis Efremov
On 12.08.2019 12:06, Andrew Murray wrote: > > Hi Denis, Hi! > > You could also fix up a few cases where the number of BARs is hard coded in > loops, e.g. > > drivers/pci/controller/pci-hyperv.c - look for uses of probed_bar in loops > drivers/pci/pci.c - pci_release_selected_regions and >

RE: [PATCH 1/4] arm64: dts: ls1088a: Fix incorrect I2C clock divider

2019-08-12 Thread Leo Li
> -Original Message- > From: Shawn Guo > Sent: Monday, August 12, 2019 9:17 AM > To: Chuanhua Han > Cc: Leo Li ; robh...@kernel.org; > mark.rutl...@arm.com; linux-arm-ker...@lists.infradead.org; > devicet...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 1/4]

Re: [RFC PATCH v2 15/19] mm/gup: Introduce vaddr_pin_pages()

2019-08-12 Thread Ira Weiny
On Sun, Aug 11, 2019 at 04:07:23PM -0700, John Hubbard wrote: > On 8/9/19 3:58 PM, ira.we...@intel.com wrote: > > From: Ira Weiny > > > > The addition of FOLL_LONGTERM has taken on additional meaning for CMA > > pages. > > > > In addition subsystems such as RDMA require new information to be

RE: [PATCH] arm64: dts: ls1028a: fix gpio nodes

2019-08-12 Thread Leo Li
> -Original Message- > From: Shawn Guo > Sent: Monday, August 12, 2019 8:47 AM > To: Hui Song ; Leo Li > Cc: Rob Herring ; Mark Rutland > ; Linus Walleij ; Bartosz > Golaszewski ; linux-arm- > ker...@lists.infradead.org; devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org;

Re: [PATCH] rtc: bd70528: fix driver dependencies

2019-08-12 Thread Alexandre Belloni
On 12/08/2019 09:36:11+0300, Matti Vaittinen wrote: > With CONFIG_BD70528_WATCHDOG=m, a built-in rtc driver cannot call > into the low-level functions that are part of the watchdog module: > > drivers/rtc/rtc-bd70528.o: In function `bd70528_set_time': > rtc-bd70528.c:(.text+0x22c): undefined

[PATCH 3/2] padata: initialize usable masks to reflect offlined CPU

2019-08-12 Thread Daniel Jordan
__padata_remove_cpu clears the offlined CPU from the usable masks after padata_alloc_pd has initialized pd->cpu, which means pd->cpu could be initialized to this CPU, causing padata to wait indefinitely for the next job in padata_get_next. Make the usable masks reflect the offline CPU when

Re: [PATCH RFC tip/core/rcu 14/14] rcu/nohz: Make multi_cpu_stop() enable tick on all online CPUs

2019-08-12 Thread Frederic Weisbecker
On Fri, Aug 02, 2019 at 08:15:01AM -0700, Paul E. McKenney wrote: > The multi_cpu_stop() function relies on the scheduler to gain control from > whatever is running on the various online CPUs, including any nohz_full > CPUs running long loops in kernel-mode code. Lack of the scheduler-clock >

Re: [PATCH v2] net: phy: at803x: stop switching phy delay config needlessly

2019-08-12 Thread David Miller
From: André Draszik Date: Fri, 9 Aug 2019 12:20:25 +0100 > This driver does a funny dance disabling and re-enabling > RX and/or TX delays. In any of the RGMII-ID modes, it first > disables the delays, just to re-enable them again right > away. This looks like a needless exercise. > > Just

Re: [PATCH v6 1/2] fork: extend clone3() to support setting a PID

2019-08-12 Thread Adrian Reber
On Mon, Aug 12, 2019 at 01:43:53PM -0700, Andrei Vagin wrote: > On Mon, Aug 12, 2019 at 1:10 PM Adrian Reber wrote: > > > > The main motivation to add set_tid to clone3() is CRIU. > > > > To restore a process with the same PID/TID CRIU currently uses > > /proc/sys/kernel/ns_last_pid. It writes

Re: [PATCH] Add optional chip erase functionality to AT25 EEPROM driver.

2019-08-12 Thread Joe Burmeister
On 12/08/2019 16:51, David Laight wrote: From: Joe Burmeister Sent: 09 August 2019 13:54 Many, though not all, AT25s have an instruction for chip erase. If there is one in the datasheet, it can be added to device tree. Erase can then be done in userspace via the sysfs API with a new "erase"

Re: [PATCH] rtc: mxc: use spin_lock_irqsave instead of spin_lock_irq in IRQ context

2019-08-12 Thread Alexandre Belloni
On 07/08/2019 16:23:10+0800, Fuqian Huang wrote: > As spin_unlock_irq will enable interrupts. > mxc_rtc_irq_enable is called from interrupt handler mxc_rtc_interrupt. > Interrupts are enabled in interrupt handler. > Use spin_lock_irqsave/spin_unlock_irqrestore instead of spin_(un)lock_irq > in IRQ

Re: [PATCH v12 5/6] khugepaged: enable collapse pmd for pte-mapped THP

2019-08-12 Thread Song Liu
> On Aug 12, 2019, at 7:40 AM, Kirill A. Shutemov wrote: > > On Mon, Aug 12, 2019 at 03:22:58PM +0200, Oleg Nesterov wrote: >> On 08/12, Kirill A. Shutemov wrote: >>> >>> On Fri, Aug 09, 2019 at 06:01:18PM +, Song Liu wrote: + if (pte_none(*pte) || !pte_present(*pte))

Re: [PATCH net-next v2 04/12] net: stmmac: Add Split Header support and enable it in XGMAC cores

2019-08-12 Thread David Miller
From: Jose Abreu Date: Mon, 12 Aug 2019 11:44:03 +0200 > - Add performance info (David) Ummm... Whilst cpu utilization is interesting, I might be mainly interested in how this effects "networking" performance. I find it very surprising that it isn't obvious that this is what I wanted.

Re: [PATCH 2/7] s390/pci: Replace PCI_BAR_COUNT with PCI_STD_NUM_BARS

2019-08-12 Thread Denis Efremov
Hi! > > There's a loop just below here that should be rewritten in the typical > style: > > i = PCI_IOV_RESOURCES; > for (; i < PCI_SRIOV_NUM_BARS + PCI_IOV_RESOURCES; i++) { > len = pci_resource_len(pdev, i); > > Again, not strictly related to this patch, but

Re: [PATCH v3 0/3] mm: kmemleak: Use a memory pool for kmemleak object allocations

2019-08-12 Thread Andrew Morton
On Mon, 12 Aug 2019 17:06:39 +0100 Catalin Marinas wrote: > Following the discussions on v2 of this patch(set) [1], this series > takes slightly different approach: > > - it implements its own simple memory pool that does not rely on the > slab allocator > > - drops the early log buffer

Re: [PATCH v2 1/4] PCI: pciehp: Add pciehp_set_indicators() to jointly set LED indicators

2019-08-12 Thread sathyanarayanan kuppuswamy
On 8/12/19 1:40 PM, Lukas Wunner wrote: On Mon, Aug 12, 2019 at 11:49:23AM -0700, sathyanarayanan kuppuswamy wrote: On 8/11/19 12:59 PM, Denis Efremov wrote: +if ((!PWR_LED(ctrl) || pwr == PWR_NONE) && +(!ATTN_LED(ctrl) || attn == ATTN_NONE)) +return; Also I think this

Re: [PATCH] mm: vmscan: do not share cgroup iteration between reclaimers

2019-08-12 Thread Roman Gushchin
On Mon, Aug 12, 2019 at 03:23:16PM -0400, Johannes Weiner wrote: > One of our services observed a high rate of cgroup OOM kills in the > presence of large amounts of clean cache. Debugging showed that the > culprit is the shared cgroup iteration in page reclaim. > > Under high allocation

Re: Build regressions/improvements in v5.3-rc4

2019-08-12 Thread Geert Uytterhoeven
Hi Randy, On Mon, Aug 12, 2019 at 10:34 PM Randy Dunlap wrote: > On 8/12/19 3:20 AM, Geert Uytterhoeven wrote: > > Below is the list of build error/warning regressions/improvements in > > v5.3-rc4[1] compared to v5.2[2]. > > > > Summarized: > > - build errors: +5/-1 > > - build warnings:

[PATCHv4] tty: n_gsm: add ioctl to map serial device to mux'ed tty

2019-08-12 Thread Martin Hundebøll
Guessing the first tty for a gsm0710 multiplexed serial device is not currently possible, which makes it racy to use with multiple modems. Add a way to map the physical serial tty to its related mux devices using an ioctl. Signed-off-by: Martin Hundebøll --- Changes since v3: * use __u32

Re: [PATCH v2 4/4] PCI: pciehp: Replace pciehp_green_led_{on,off,blink}()

2019-08-12 Thread Denis Efremov
> You must have a reason, but why didn't you completely remove > pciehp_green_led_on(), etc, and change the callers to use > pciehp_set_indicators() instead? Well, I don't have the exact reason here. I thought that it would be nice to preserve an existing interface and to hide some

Re: [RFC PATCH v2 16/19] RDMA/uverbs: Add back pointer to system file object

2019-08-12 Thread Ira Weiny
On Mon, Aug 12, 2019 at 02:56:15PM -0300, Jason Gunthorpe wrote: > On Mon, Aug 12, 2019 at 10:28:27AM -0700, Ira Weiny wrote: > > On Mon, Aug 12, 2019 at 10:00:40AM -0300, Jason Gunthorpe wrote: > > > On Fri, Aug 09, 2019 at 03:58:30PM -0700, ira.we...@intel.com wrote: > > > > From: Ira Weiny > >

Re: [PATCH v2 bpf-next] mm: mmap: increase sockets maximum memory size pgoff for 32bits

2019-08-12 Thread Andrew Morton
On Mon, 12 Aug 2019 15:43:26 +0300 Ivan Khoronzhuk wrote: > The AF_XDP sockets umem mapping interface uses XDP_UMEM_PGOFF_FILL_RING > and XDP_UMEM_PGOFF_COMPLETION_RING offsets. The offsets seems like are > established already and are part of configuration interface. > > But for 32-bit

Re: [RFC PATCH v2 15/19] mm/gup: Introduce vaddr_pin_pages()

2019-08-12 Thread John Hubbard
On 8/12/19 2:00 PM, Ira Weiny wrote: On Fri, Aug 09, 2019 at 05:09:54PM -0700, John Hubbard wrote: On 8/9/19 3:58 PM, ira.we...@intel.com wrote: From: Ira Weiny ... At one point I wanted to (and had in my tree) a new flag but I went away from it. Prior to the discussion on mlock last week

Re: [PATCH v3 0/8] PM / ACPI: sleep: Additional changes related to suspend-to-idle

2019-08-12 Thread Rafael J. Wysocki
On Mon, Aug 12, 2019 at 3:55 PM Bhardwaj, Rajneesh wrote: > > Hi Rafael > > On 02-Aug-19 4:03 PM, Rafael J. Wysocki wrote: > > Hi All, > > > >>> On top of the "Simplify the suspend-to-idle control flow" patch series > >>> posted previously: > >>> > >>>

[PATCH v5 2/6] mm: Move set/get_pcppage_migratetype to mmzone.h

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck In order to support page reporting it will be necessary to store and retrieve the migratetype of a page. To enable that I am moving the set and get operations for pcppage_migratetype into the mm/internal.h header so that they can be used outside of the page_alloc.c file.

[PATCH v5 0/6] mm / virtio: Provide support for unused page reporting

2019-08-12 Thread Alexander Duyck
This series provides an asynchronous means of reporting to a hypervisor that a guest page is no longer in use and can have the data associated with it dropped. To do this I have implemented functionality that allows for what I am referring to as unused page reporting The functionality for this is

[PATCH v5 1/6] mm: Adjust shuffle code to allow for future coalescing

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck This patch is meant to move the head/tail adding logic out of the shuffle code and into the __free_one_page function since ultimately that is where it is really needed anyway. By doing this we should be able to reduce the overhead and can consolidate all of the list

[PATCH v5 3/6] mm: Use zone and order instead of free area in free_list manipulators

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck In order to enable the use of the zone from the list manipulator functions I will need access to the zone pointer. As it turns out most of the accessors were always just being directly passed >free_area[order] anyway so it would make sense to just fold that into the

[PATCH v5 5/6] virtio-balloon: Pull page poisoning config out of free page hinting

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck Currently the page poisoning setting wasn't being enabled unless free page hinting was enabled. However we will need the page poisoning tracking logic as well for unused page reporting. As such pull it out and make it a separate bit of config in the probe function. In

[PATCH v5 6/6] virtio-balloon: Add support for providing unused page reports to host

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck Add support for the page reporting feature provided by virtio-balloon. Reporting differs from the regular balloon functionality in that is is much less durable than a standard memory balloon. Instead of creating a list of pages that cannot be accessed the pages are only

[PATCH v5 4/6] mm: Introduce Reported pages

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck In order to pave the way for free page reporting in virtualized environments we will need a way to get pages out of the free lists and identify those pages after they have been returned. To accomplish this, this patch adds the concept of a Reported Buddy, which is

[PATCH v5 QEMU 1/3] virtio-ballon: Implement support for page poison tracking feature

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck We need to make certain to advertise support for page poison tracking if we want to actually get data on if the guest will be poisoning pages. So if free page hinting is active we should add page poisoning support and let the guest disable it if it isn't using it. Page

[PATCH v5 QEMU 2/3] virtio-balloon: Add bit to notify guest of unused page reporting

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck Add a bit for the page reporting feature provided by virtio-balloon. This patch should be replaced once the feature is added to the Linux kernel and the bit is backported into this exported kernel header. Signed-off-by: Alexander Duyck ---

[PATCH v5 QEMU 3/3] virtio-balloon: Provide a interface for unused page reporting

2019-08-12 Thread Alexander Duyck
From: Alexander Duyck Add support for what I am referring to as "unused page reporting". Basically the idea is to function very similar to how the balloon works in that we basically end up madvising the page as not being used. However we don't really need to bother with any deflate type logic

Re: [PATCH v2 8/8] ACPI / property: Add two new Thunderbolt property GUIDs to the list

2019-08-12 Thread Rafael J. Wysocki
On Mon, Aug 12, 2019 at 2:39 PM Mika Westerberg wrote: > > Ice Lake Thunderbolt controller includes two new device property > compatible properties that we need to be able to extract in the driver > so add them to the growing array of GUIDs. > > Signed-off-by: Mika Westerberg Acked-by: Rafael

Re: [PATCH] hugetlbfs: fix hugetlb page migration/fault race causing SIGBUS

2019-08-12 Thread Andrew Morton
On Mon, 12 Aug 2019 11:33:26 -0400 Sasha Levin wrote: > >I thought that absence of the Cc is the indication :P. Anyway, I really > >do not understand why should we bother, really. I have tried to explain > >that stable maintainers should follow Cc: stable because we bother to > >consider that

Re: [PATCH v5 2/7] PCI/ATS: Initialize PRI in pci_ats_init()

2019-08-12 Thread sathyanarayanan kuppuswamy
Hi, Thanks for the review. On 8/12/19 1:04 PM, Bjorn Helgaas wrote: On Thu, Aug 01, 2019 at 05:05:59PM -0700, sathyanarayanan.kuppusw...@linux.intel.com wrote: From: Kuppuswamy Sathyanarayanan Currently, PRI Capability checks are repeated across all PRI API's. Instead, cache the capability

  1   2   3   4   5   6   7   8   9   10   >