Re: [PATCH net-next] ath5k: fix 'mode' kernel-doc warning in ath5k_hw_pcu_init()

2020-09-10 Thread Kalle Valo
Wang Hai wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/ath5k/pcu.c:955: warning: Excess function parameter > 'mode' description in 'ath5k_hw_pcu_init' > > This parameter is not in use. Remove it. > > Reported-by: Hulk Robot > Signed-off-by: Wang Hai

Re: [PATCH 27/29] ath9k: ar5008_initvals: Move ar5416Bank{0,1,2,3,7} to where they are used

2020-09-10 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/ath9k/ar5008_initvals.h:627:18: warning: > ‘ar5416Bank7’ defined but not used [-Wunused-const-variable=] > 627 | static const u32 ar5416Bank7[][2] = { > | ^~~ >

Re: [PATCH] vfs: add fchmodat2 syscall

2020-09-10 Thread Rich Felker
On Thu, Sep 10, 2020 at 06:16:15PM +0200, Greg KH wrote: > On Thu, Sep 10, 2020 at 10:23:37AM -0400, Rich Felker wrote: > > POSIX defines fchmodat as having a 4th argument, flags, that can be > > AT_SYMLINK_NOFOLLOW. Support for changing the access mode of symbolic > > links is optional

Re: [PATCH v2 1/7] perf/amd/uncore: Set all slices and threads to restore perf stat -a behaviour

2020-09-10 Thread Sasha Levin
Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: 2f217d58a8a0 ("perf/x86/amd/uncore: Set the thread mask for F17h L3 PMCs"). The bot has tested the following trees: v5.8.7, v5.4.63, v4.19.143, v4.14.196. v5.8.7: Build OK!

Re: [PATCH] PCI/ASPM: Enable ASPM for links under VMD domain

2020-09-10 Thread Derrick, Jonathan
Hi Bjorn On Wed, 2020-09-09 at 20:55 -0500, Bjorn Helgaas wrote: > [+cc Saheed] > > On Fri, Aug 21, 2020 at 08:32:20PM +0800, Kai-Heng Feng wrote: > > New Intel laptops with VMD cannot reach deeper power saving state, > > renders very short battery time. > > > > As BIOS may not be able to

Re: [PATCH RFC 01/10] mm: add Kernel Electric-Fence infrastructure

2020-09-10 Thread Alexander Potapenko
On Thu, Sep 10, 2020 at 5:43 PM Dmitry Vyukov wrote: > > + /* Calculate address for this allocation. */ > > + if (right) > > + meta->addr += PAGE_SIZE - size; > > + meta->addr = ALIGN_DOWN(meta->addr, cache->align); > > I would move this ALIGN_DOWN under the

[PATCH v2 1/2] dt-bindings: iio: adc: exynos-adc: require second interrupt with touch screen

2020-09-10 Thread Krzysztof Kozlowski
The ADC in S3C/S5P/Exynos SoCs can be used also for handling touch screen. In such case the second interrupt is required. This second interrupt can be anyway provided, even without touch screens. This fixes dtbs_check warnings like: arch/arm/boot/dts/s5pv210-aquila.dt.yaml: adc@e170:

Re: [PATCH 25/29] ath9k: ar9001_initvals: Remove unused array 'ar5416Bank6_9100'

2020-09-10 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/ath9k/ar9001_initvals.h:462:18: warning: > ‘ar5416Bank6_9100’ defined but not used [-Wunused-const-variable=] > > Cc: QCA ath9k Development > Cc: Kalle Valo > Cc: "David S. Miller" > Cc: Jakub

Re: [PATCH 26/29] ath9k: ar5008_initvals: Remove unused table entirely

2020-09-10 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/ath9k/ar5008_initvals.h:553:18: warning: > ‘ar5416Bank6’ defined but not used [-Wunused-const-variable=] > > Cc: QCA ath9k Development > Cc: Kalle Valo > Cc: "David S. Miller" > Cc: Jakub

[PATCH v2 2/2] dt-bindings: iio: adc: exynos-adc: do not require syscon on S5Pv210

2020-09-10 Thread Krzysztof Kozlowski
The ADC in S5Pv210 does not have ADC phy registers in separate block for which syscon would be needed. Remove this requirement to fix dtbs_check warnings like: arch/arm/boot/dts/s5pv210-fascinate4g.dt.yaml: adc@e170: 'samsung,syscon-phandle' is a required property Signed-off-by:

Re: [PATCH v3 01/19] dlb2: add skeleton for DLB 2.0 driver

2020-09-10 Thread Greg KH
On Thu, Sep 10, 2020 at 04:24:31PM +, Eads, Gage wrote: > > > > -Original Message- > > From: Greg KH > > Sent: Monday, September 7, 2020 8:02 AM > > To: Eads, Gage > > Cc: linux-kernel@vger.kernel.org; a...@arndb.de; Karlsson, Magnus > > ; Topel, Bjorn > > Subject: Re: [PATCH v3

Re: [PATCH 1/4] kbuild: remove cc-option test of -fno-strict-overflow

2020-09-10 Thread Nathan Chancellor
On Thu, Sep 10, 2020 at 10:51:17PM +0900, Masahiro Yamada wrote: > The minimal compiler versions, GCC 4.9 and Clang 10 support this flag. > > Here is the godbolt: > https://godbolt.org/z/odq8h9 > > Signed-off-by: Masahiro Yamada Appeared in clang 3.0.0 in commit

Re: [PATCH 24/29] ath9k: ar9002_initvals: Remove unused array 'ar9280PciePhy_clkreq_off_L1_9280'

2020-09-10 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/ath9k/ar9002_initvals.h:900:18: warning: > ‘ar9280PciePhy_clkreq_off_L1_9280’ defined but not used > [-Wunused-const-variable=] > > Cc: QCA ath9k Development > Cc: Kalle Valo > Cc: "David S.

Re: [PATCH 23/29] ath6kl: wmi: Remove unused variable 'rate'

2020-09-10 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/ath6kl/wmi.c: In function > ‘ath6kl_wmi_bitrate_reply_rx’: > drivers/net/wireless/ath/ath6kl/wmi.c:1204:6: warning: variable ‘rate’ set > but not used [-Wunused-but-set-variable] > > Cc: Kalle

Re: [PATCH RFC 01/10] mm: add Kernel Electric-Fence infrastructure

2020-09-10 Thread Marco Elver
On Thu, 10 Sep 2020 at 16:58, Dmitry Vyukov wrote: > > On Mon, Sep 7, 2020 at 3:41 PM Marco Elver wrote: > > +config KFENCE_NUM_OBJECTS > > + int "Number of guarded objects available" > > + default 255 > > + range 1 65535 > > + help > > + The number of guarded

Re: [PATCH v2 15/28] init: lto: ensure initcall ordering

2020-09-10 Thread Sami Tolvanen
On Thu, Sep 10, 2020 at 10:25:40AM +0100, David Woodhouse wrote: > On Thu, 2020-09-03 at 13:30 -0700, Sami Tolvanen wrote: > > With LTO, the compiler doesn't necessarily obey the link order for > > initcalls, and initcall variables need globally unique names to avoid > > collisions at link time. >

Re: [PATCH 3/3] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset

2020-09-10 Thread Greg KH
On Thu, Sep 10, 2020 at 11:13:51AM +0200, Christoph Hellwig wrote: > On Thu, Sep 10, 2020 at 09:53:51AM +0200, Greg KH wrote: > > > /* > > >* Please refer to usb_alloc_dev() to see why we set > > > - * dma_mask and dma_pfn_offset. > > > + * dma_mask and

Re: [PATCH RFC 01/10] mm: add Kernel Electric-Fence infrastructure

2020-09-10 Thread Marco Elver
On Thu, 10 Sep 2020 at 17:48, Dmitry Vyukov wrote: > > On Thu, Sep 10, 2020 at 5:06 PM Marco Elver wrote: > > > On Mon, Sep 7, 2020 at 3:41 PM Marco Elver wrote: > > > > +config KFENCE_NUM_OBJECTS > > > > + int "Number of guarded objects available" > > > > + default 255 > > > > +

Re: [PATCH v11 11/11] PCI: brcmstb: Add bcm7211, bcm7216, bcm7445, bcm7278 to match list

2020-09-10 Thread Rob Herring
On Mon, 24 Aug 2020 15:30:24 -0400, Jim Quinlan wrote: > Now that the support is in place with previous commits, we add several > chips that use the BrcmSTB driver. > > Signed-off-by: Jim Quinlan > Acked-by: Florian Fainelli > --- > drivers/pci/controller/pcie-brcmstb.c | 4 > 1 file

Re: [PATCH v11 10/11] PCI: brcmstb: Set bus max burst size by chip type

2020-09-10 Thread Rob Herring
On Mon, 24 Aug 2020 15:30:23 -0400, Jim Quinlan wrote: > From: Jim Quinlan > > The proper value of the parameter SCB_MAX_BURST_SIZE varies per chip. The > 2711 family requires 128B whereas other devices can employ 512. The > assignment is complicated by the fact that the values for this

Re: [PATCH] vfs: add fchmodat2 syscall

2020-09-10 Thread Christoph Hellwig
On Thu, Sep 10, 2020 at 10:23:37AM -0400, Rich Felker wrote: > userspace emulation done in libc implementations. No change is made to > the underlying chmod_common(), so it's still possible to attempt > changes via procfs, if desired. And that is the goddamn problem. We need to fix that _first_.

[PATCH 1/1] of: properties of reserved-memory nodes

2020-09-10 Thread Heinrich Schuchardt
The reusable and the no-map property are mutually exclusive. Clarify this in the documentation. Signed-off-by: Heinrich Schuchardt --- .../devicetree/bindings/reserved-memory/reserved-memory.txt| 3 +++ 1 file changed, 3 insertions(+) diff --git

Re: [PATCH v11 09/11] PCI: brcmstb: Accommodate MSI for older chips

2020-09-10 Thread Rob Herring
On Mon, 24 Aug 2020 15:30:22 -0400, Jim Quinlan wrote: > From: Jim Quinlan > > Older BrcmSTB chips do not have a separate register for MSI interrupts; the > MSIs are in a register that also contains unrelated interrupts. In > addition, the interrupts lie in bits [31..24] for these legacy chips.

Re: [PATCH v3 1/2] idr: Document calling context for IDA APIs mustn't use locks

2020-09-10 Thread Greg KH
On Wed, Sep 09, 2020 at 10:52:45PM -0700, Stephen Boyd wrote: > The documentation for these functions indicates that callers don't need > to hold a lock while calling them, but that documentation is only in one > place under "IDA Usage". Let's state the same information on each IDA > function so

Re: [PATCH v2 9/9] arm64: defconfig: Enable Samsung S3FWRN5 NFC driver

2020-09-10 Thread Krzysztof Kozlowski
On Sun, Sep 06, 2020 at 05:36:54PM +0200, Krzysztof Kozlowski wrote: > Enable the Samsung S3FWRN5 NFC driver present in Exynos5433-based > TM2/TM2E boards. > > Signed-off-by: Krzysztof Kozlowski > --- > arch/arm64/configs/defconfig | 3 +++ Applied. Best regards, Krzysztof

[PATCH] mm/slab.h: Fix duplicate include file kmemleak.h

2020-09-10 Thread Pintu Kumar
As I was browsing through this code, I found that linux/kmemleak.h was declared twice. When I run ./scripts/checkincludes.pl it reported the same: mm/slab.h: linux/kmemleak.h is included more than once. Thus removing the last included header. Signed-off-by: Pintu Kumar --- mm/slab.h | 1 - 1

Re: [PATCH] PCI: Don't use Printk in raw_spinlocks

2020-09-10 Thread Scott Branden
On 2020-09-10 7:21 a.m., Rob Herring wrote: > On Wed, Sep 9, 2020 at 8:07 PM Bjorn Helgaas wrote: >> [+cc Mark, Florian, Rob, Scott] >> >> On Sat, Aug 01, 2020 at 09:25:49AM +0800, Xingxing Su wrote: >>> Do not use printk in raw_spinlocks, >>> it will cause BUG: Invalid wait context. >>> >>>

[PATCH 07/15] habanalabs/gaudi: allow user to get MAC addresses in INFO IOCTL

2020-09-10 Thread Oded Gabbay
From: Omer Shpigelman The user needs this information when working in a distributed environment with master/slave configuration. All the slaves get their MAC addresses from the driver and send them to the master. Signed-off-by: Omer Shpigelman Reviewed-by: Oded Gabbay Signed-off-by: Oded

[PATCH 08/15] habanalabs/gaudi: add a new IOCTL for NIC control operations

2020-09-10 Thread Oded Gabbay
From: Omer Shpigelman Add Queue Pair (QP) opcodes to the NIC ioctl. A QP represents a connection between two Gaudi ports. Each port currently supports 1024 QPs where QP 0 is reserved for the driver for Ethernet. User-space process needs to create a QP in order to communicate with other Gaudis.

[PATCH 00/15] Adding GAUDI NIC code to habanalabs driver

2020-09-10 Thread Oded Gabbay
This patch-set adds support for initializing and using the GAUDI NIC ports, functioning as scale-out interconnect when doing distributed Deep Learning training. The training can be performed over tens of thousands of GAUDIs and it is done using the RDMA-over-converged-Ethernet (RoCE) v2 protocol.

Re: [PATCH v11 06/11] PCI: brcmstb: Add control of rescal reset

2020-09-10 Thread Rob Herring
On Mon, Aug 24, 2020 at 03:30:19PM -0400, Jim Quinlan wrote: > From: Jim Quinlan > > Some STB chips have a special purpose reset controller named RESCAL (reset > calibration). The PCIe HW can now control RESCAL to start and stop its > operation. On probe(), the RESCAL is deasserted and the

Re: [PATCH] EDAC: sb_edac: simplify switch statement

2020-09-10 Thread Borislav Petkov
On Tue, Sep 08, 2020 at 10:08:28PM +, Luck, Tony wrote: > [Boris: Pushed to edac-misc and edac-for-next branches in RAS tree] Yap, looks good. Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH] memcg: introduce per-memcg reclaim interface

2020-09-10 Thread Shakeel Butt
On Wed, Sep 9, 2020 at 11:37 PM SeongJae Park wrote: > > On 2020-09-09T14:57:52-07:00 Shakeel Butt wrote: > > > Introduce an memcg interface to trigger memory reclaim on a memory cgroup. > > > > Use cases: > > -- > > > > 1) Per-memcg uswapd: > > > > Usually applications consists of

[PATCH REBASE] spi: sprd: Simplify with dev_err_probe()

2020-09-10 Thread Krzysztof Kozlowski
Common pattern of handling deferred probe can be simplified with dev_err_probe(). Less code and the error value gets printed. Signed-off-by: Krzysztof Kozlowski Acked-by: Chunyan Zhang --- Rebased on broonie/spi.git, for-5.10 --- drivers/spi/spi-sprd-adi.c | 5 + drivers/spi/spi-sprd.c

Re: [PATCH 00/15] Adding GAUDI NIC code to habanalabs driver

2020-09-10 Thread Greg KH
On Thu, Sep 10, 2020 at 06:59:41PM +0300, Oded Gabbay wrote: > On Thu, Sep 10, 2020 at 6:54 PM Greg KH wrote: > > > > On Thu, Sep 10, 2020 at 06:03:13PM +0300, Oded Gabbay wrote: > > > This patch-set adds support for initializing and using the GAUDI NIC > > > ports, > > > functioning as

Re: [PATCH v2 1/2] scsi: ufs: Abort tasks before clear them from doorbell

2020-09-10 Thread James Bottomley
On Thu, 2020-09-10 at 16:18 +0800, Stanley Chu wrote: [...] > > + if (!err) { > > +cleanup: > > Yeah, considering Bean Huo's patch "scsi: ufs: No need to send Abort > Task if the task in DB was cleared", "cleanup" label shall be added > back here. > > So your resolution looks good to me. > >

Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Andi Kleen
On Tue, Sep 08, 2020 at 01:42:24PM +0900, Namhyung Kim wrote: > When we profile cgroup events with perf stat, it's very annoying to > specify events and cgroups on the command line as it requires the > mapping between events and cgroups. (Note that perf record can use > cgroup sampling but it's

[PATCH] mtd: mtdconcat: map: remove redundant assignment to variable 'size'

2020-09-10 Thread Colin King
From: Colin Ian King Variable 'size' is being assigned the value zero that will never be read. The assignment is redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King --- drivers/mtd/mtdconcat.c | 1 - 1 file changed, 1 deletion(-) diff --git

[PATCH net-next 3/3] i40e: Fix a kernel-doc warning in i40e_ptp.c

2020-09-10 Thread Wang Hai
Fixes the following W=1 kernel build warning(s): drivers/net/ethernet/intel/i40e/i40e_ptp.c:270: warning: Excess function parameter 'vsi' description in 'i40e_ptp_rx_hang' Reported-by: Hulk Robot Signed-off-by: Wang Hai --- drivers/net/ethernet/intel/i40e/i40e_ptp.c | 1 - 1 file changed, 1

[PATCH] Drivers: hv: vmbus: Copy packets sent by Hyper-V out of the ring buffer

2020-09-10 Thread Andrea Parri (Microsoft)
From: Andres Beltran Pointers to ring-buffer packets sent by Hyper-V are used within the guest VM. Hyper-V can send packets with erroneous values or modify packet fields after they are processed by the guest. To defend against these scenarios, return a copy of the incoming VMBus packet after

Re: [PATCH v2 3/7] arch/x86/amd/ibs: Fix re-arming IBS Fetch

2020-09-10 Thread Kim Phillips
On 9/10/20 3:50 AM, pet...@infradead.org wrote: > On Thu, Sep 10, 2020 at 10:32:23AM +0200, pet...@infradead.org wrote: >>> @@ -363,7 +363,14 @@ perf_ibs_event_update(struct perf_ibs *perf_ibs, >>> struct perf_event *event, >>> static inline void perf_ibs_enable_event(struct perf_ibs *perf_ibs,

Re: [PATCH v11 04/11] PCI: brcmstb: Add suspend and resume pm_ops

2020-09-10 Thread Rob Herring
On Mon, Aug 24, 2020 at 03:30:17PM -0400, Jim Quinlan wrote: > From: Jim Quinlan > > Broadcom Set-top (BrcmSTB) boards typically support S2, S3, and S5 suspend > and resume. Now the PCIe driver may do so as well. > > Signed-off-by: Jim Quinlan > Acked-by: Florian Fainelli > --- >

Re: [PATCH] [v2] blkcg: add plugging support for punt bio

2020-09-10 Thread Jens Axboe
Applied, thanks. -- Jens Axboe

[PATCH V5] ASoC: Intel: boards: Use FS as nau8825 sysclk in nau88125_* machine

2020-09-10 Thread Radoslaw Biernacki
Since 256xFS clocks cannot be generated by SKL, the NAU8825 is configured to re-generate its system clock from the BCLK using the FLL. The link is configured to use a 48kHz frame rate, and 24 bits in 25-bit slot. The SSP configuration is extracted from NHLT settings and not dynamically changed.

[PATCH 3/4] dt-bindings: fsi: Aspeed master: Add bus-frequency property

2020-09-10 Thread Eddie James
Document the bus-frequency property. Signed-off-by: Eddie James --- Documentation/devicetree/bindings/fsi/fsi-master-aspeed.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/fsi/fsi-master-aspeed.txt

Re: [PATCH RFC 01/10] mm: add Kernel Electric-Fence infrastructure

2020-09-10 Thread Dmitry Vyukov
On Mon, Sep 7, 2020 at 3:41 PM Marco Elver wrote: > + meta->addr = metadata_to_pageaddr(meta); > + /* Unprotect if we're reusing this page. */ > + if (meta->state == KFENCE_OBJECT_FREED) > + kfence_unprotect(meta->addr); > + > + /* Calculate address for this

[PATCH net-next 1/3] e1000e: Fix some kernel-doc warnings in ich8lan.c

2020-09-10 Thread Wang Hai
Fixes the following W=1 kernel build warning(s): drivers/net/ethernet/intel/e1000e/ich8lan.c:2277: warning: Excess function parameter 'enable' description in 'e1000_configure_k1_ich8lan' drivers/net/ethernet/intel/e1000e/ich8lan.c:754: warning: Excess function parameter 'addr' description in

Re: remove the last set_fs() in common code, and remove it for x86 and powerpc v3

2020-09-10 Thread Segher Boessenkool
On Thu, Sep 10, 2020 at 09:26:28AM +, David Laight wrote: > From: Christophe Leroy > > Sent: 10 September 2020 09:14 > > > > Le 10/09/2020 à 10:04, David Laight a écrit : > > > From: Linus Torvalds > > >> Sent: 09 September 2020 22:34 > > >> On Wed, Sep 9, 2020 at 11:42 AM Segher Boessenkool

[PATCH net-next 0/3] Fix some kernel-doc warnings for e1000/e1000e

2020-09-10 Thread Wang Hai
Wang Hai (3): e1000e: Fix some kernel-doc warnings in ich8lan.c e1000e: Fix some kernel-doc warnings in netdev.c e1000: Fix a bunch of kerneldoc parameter issues in e1000_hw.c drivers/net/ethernet/intel/e1000/e1000_hw.c | 7 +-- drivers/net/ethernet/intel/e1000e/ich8lan.c | 4 ++--

[PATCH net-next 3/3] e1000: Fix a bunch of kerneldoc parameter issues in e1000_hw.c

2020-09-10 Thread Wang Hai
Fix W=1 compile warnings (invalid kerneldoc): drivers/net/ethernet/intel/e1000/e1000_hw.c:1907: warning: Excess function parameter 'mii_reg' description in 'e1000_config_mac_to_phy' drivers/net/ethernet/intel/e1000/e1000_hw.c:2930: warning: Excess function parameter 'data' description in

[PATCH net-next 2/3] e1000e: Fix some kernel-doc warnings in netdev.c

2020-09-10 Thread Wang Hai
Fixes the following W=1 kernel build warning(s): drivers/net/ethernet/intel/e1000e/netdev.c:2525: warning: Excess function parameter 'adapter' description in 'e1000_update_itr' drivers/net/ethernet/intel/e1000e/netdev.c:609: warning: Excess function parameter 'csum' description in

Re: [PATCH V4] ASoC: Intel: boards: Use FS as nau8825 sysclk in nau88125_* machine

2020-09-10 Thread Radosław Biernacki
Sorry, V4 got one stray change for SSP0 ops. Need to send V5. czw., 10 wrz 2020 o 17:40 Radoslaw Biernacki napisał(a): > > Since 256xFS clocks cannot be generated by SKL, the NAU8825 is > configured to re-generate its system clock from the BCLK using the > FLL. The link is configured to use a

Re: [PATCH RFC 01/10] mm: add Kernel Electric-Fence infrastructure

2020-09-10 Thread Dmitry Vyukov
On Thu, Sep 10, 2020 at 5:06 PM Marco Elver wrote: > > On Mon, Sep 7, 2020 at 3:41 PM Marco Elver wrote: > > > +config KFENCE_NUM_OBJECTS > > > + int "Number of guarded objects available" > > > + default 255 > > > + range 1 65535 > > > + help > > > + The number of

Re: [RFC PATCH v2 1/3] mm/gup: fix gup_fast with dynamic page table folding

2020-09-10 Thread Jason Gunthorpe
On Thu, Sep 10, 2020 at 03:28:03PM +0200, Gerald Schaefer wrote: > On Thu, 10 Sep 2020 10:02:33 -0300 > Jason Gunthorpe wrote: > > > On Thu, Sep 10, 2020 at 11:39:25AM +0200, Alexander Gordeev wrote: > > > > > As Gerald mentioned, it is very difficult to explain in a clear way. > > > Hopefully,

Re: [PATCH] mm: don't rely on system state to detect hot-plug operations

2020-09-10 Thread Michal Hocko
On Thu 10-09-20 14:49:28, David Hildenbrand wrote: > On 10.09.20 14:47, Michal Hocko wrote: > > On Thu 10-09-20 14:03:48, Oscar Salvador wrote: > >> On Thu, Sep 10, 2020 at 01:35:32PM +0200, Laurent Dufour wrote: > >> > >>> That points has been raised by David, quoting him here: > >>> >

[PATCH 2/2] ARM: s3c24xx: fix Wunused-variable warning on !MMU

2020-09-10 Thread Krzysztof Kozlowski
If S3C24xx machine code is build without CONFIG_MMU, the iotable() macros do nothing so annotate structures to get rid of warnings: arch/arm/mach-s3c24xx/common.c:140:24: warning: ‘s3c_iodesc’ defined but not used [-Wunused-variable] arch/arm/mach-s3c24xx/s3c2410.c:49:24: warning:

[PATCH net] net: ipa: fix u32_replace_bits by u32p_xxx version

2020-09-10 Thread Vadym Kochan
Looks like u32p_replace_bits() should be used instead of u32_replace_bits() which does not modifies the value but returns the modified version. Fixes: 2b9feef2b6c2 ("soc: qcom: ipa: filter and routing tables") Signed-off-by: Vadym Kochan Reviewed-by: Alex Elder --- Found it while grepping of

Re: [PATCH 05/28] media/v4l2: remove V4L2-FLAG-MEMORY-NON-CONSISTENT

2020-09-10 Thread Sergey Senozhatsky
On (20/09/10 23:48), Sergey Senozhatsky wrote: > > I've a kernel patch (I think I got all the pieces). This is not a > format submission yet, because I don't have v4l2-compliance patch > yet, so didn't really test it. > I guess,,, the v4l-utils patch should look something like this: From:

[PATCH 4/4] fsi: Aspeed master: Set bus frequency from devicetree

2020-09-10 Thread Eddie James
Set the FSI bus frequency based on the value encoded in the devicetree. The default value is 166MHz, which is the highest frequency some FSI slaves can accept. Signed-off-by: Eddie James Reviewed-by: Joel Stanley --- drivers/fsi/fsi-master-aspeed.c | 5 + 1 file changed, 5 insertions(+)

Re: [PATCH] vfs: add fchmodat2 syscall

2020-09-10 Thread Al Viro
On Thu, Sep 10, 2020 at 10:23:37AM -0400, Rich Felker wrote: > It was determined (see glibc issue #14578 and commit a492b1e5ef) that, > on some filesystems, performing chmod on the link itself produces a > change in the inode's access mode, but returns an EOPNOTSUPP error. Which filesystem types

[PATCH][next] binder: remove redundant assignment to pointer n

2020-09-10 Thread Colin King
From: Colin Ian King The pointer n is being initialized with a value that is never read and it is being updated later with a new value. The initialization is redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King --- drivers/android/binder_alloc.c | 2

[PATCH] spi: qup: remove redundant assignment to variable ret

2020-09-10 Thread Colin King
From: Colin Ian King The variable ret is being initialized with a value that is never read and it is being updated later with a new value. The initialization is redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King --- drivers/spi/spi-qup.c | 2 +- 1

Re: [PATCH V5 2/4] arm64: dts: qcom: sc7180: Add necessary pinctrl and interrupt config for BT UART

2020-09-10 Thread Matthias Kaehlcke
El Thu, Sep 10, 2020 at 06:23:56PM +0530 satya priya ha dit: > Add a suitable sleep configuration for uart3 to support Bluetooth wakeup. > > If QUP function is selected in sleep state, UART RTS/RFR is pulled high > during suspend and BT SoC not able to send wakeup bytes. So, configure > GPIO

[PATCH 3/4] spi: bcm-qspi: Fix probe regression on iProc platforms

2020-09-10 Thread Ray Jui
iProc chips have QSPI controller that does not have the MSPI_REV offset. Reading from that offset will cause a bus error. Fix it by having MSPI_REV query disabled in the generic compatible string. Fixes: 3a01f04d74ef ("spi: bcm-qspi: Handle lack of MSPI_REV offset") Link:

Re: [PATCH] checkpatch: accept longer commit description lines

2020-09-10 Thread Joe Perches
On Thu, 2020-09-10 at 23:32 +0900, Tetsuo Handa wrote: > Given that commit bdc48fa11e46f867 ("checkpatch/coding-style: deprecate > 80-column warning") was introduced because nowadays most of us are using > screens that can print more than 100 columns, most of us will be also > using mail clients

[PATCH 1/4] dt-bindings: spi: Add compatible string for brcmstb SoCs

2020-09-10 Thread Ray Jui
Add compatible string for brcmstb 7445 SoCs. Signed-off-by: Ray Jui --- Documentation/devicetree/bindings/spi/brcm,spi-bcm-qspi.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/spi/brcm,spi-bcm-qspi.txt

RE: remove the last set_fs() in common code, and remove it for x86 and powerpc v3

2020-09-10 Thread David Laight
> -Original Message- > From: Segher Boessenkool > Sent: 10 September 2020 16:21 > To: David Laight > Cc: 'Christophe Leroy' ; 'Linus Torvalds' > foundation.org>; linux-arch ; Kees Cook > ; the > arch/x86 maintainers ; Nick Desaulniers > ; Linux Kernel > Mailing List ; Alexey

RE: [PATCH 1/1] RAS: Add CPU Correctable Error Collector to isolate an erroneous CPU core

2020-09-10 Thread Shiju Jose
Hello Boris, >-Original Message- >From: Borislav Petkov [mailto:b...@alien8.de] >Sent: 09 September 2020 13:02 >To: Shiju Jose >Cc: linux-e...@vger.kernel.org; linux-a...@vger.kernel.org; linux- >ker...@vger.kernel.org; tony.l...@intel.com; r...@rjwysocki.net; >james.mo...@arm.com;

Re: [PATCH] usb: host: ehci-platform: Add workaround for brcm,xgs-iproc-ehci

2020-09-10 Thread Alan Stern
On Thu, Sep 10, 2020 at 02:51:53PM +1200, Chris Packham wrote: > The ehci controller found in some Broadcom switches with integrated SoCs > has an issue which causes a soft lockup with large transfers like you > see when running ext4 on USB3 flash drive. > > Port the fix from the Broadcom XLDK to

[PATCH 15/15] habanalabs/gaudi: add NIC init/fini calls from common code

2020-09-10 Thread Oded Gabbay
From: Omer Shpigelman Finally, enable the NIC engines. Initialize the NIC ports mask variable with full mask so all ports will be initialized. Call the NIC init/fini from the common code. Signed-off-by: Omer Shpigelman Reviewed-by: Oded Gabbay Signed-off-by: Oded Gabbay ---

[net-next v8 0/6] net: marvell: prestera: Add Switchdev driver for Prestera family ASIC device 98DX3255 (AC3x)

2020-09-10 Thread Vadym Kochan
Marvell Prestera 98DX3255 integrates up to 24 ports of 1GbE with 8 ports of 10GbE uplinks or 2 ports of 40Gbps stacking for a largely wireless SMB deployment. Prestera Switchdev is a firmware based driver that operates via PCI bus. The current implementation supports only boards designed for the

Re: [PATCH 5.8 000/186] 5.8.8-rc1 review

2020-09-10 Thread Guenter Roeck
On 9/9/20 11:36 PM, Geert Uytterhoeven wrote: > Hi Günter, > > On Wed, Sep 9, 2020 at 8:24 PM Guenter Roeck wrote: >> On 9/9/20 11:01 AM, Greg Kroah-Hartman wrote: >>> On Wed, Sep 09, 2020 at 09:47:05AM -0700, Guenter Roeck wrote: On Tue, Sep 08, 2020 at 05:22:22PM +0200, Greg Kroah-Hartman

Re: [PATCH v2] platform/chrome: Don't populate lightbar device if it isn't there

2020-09-10 Thread Guenter Roeck
On Thu, Sep 10, 2020 at 7:32 AM Enric Balletbo i Serra wrote: > > Hi, > > On 10/9/20 16:18, Guenter Roeck wrote: > > On Wed, Sep 9, 2020 at 3:42 PM Stephen Boyd wrote: > >> > >> The cros ec lightbar driver has a check in probe to fail early if the ec > >> device isn't the cros_ec one or if it

[PATCH] vfs: add fchmodat2 syscall

2020-09-10 Thread Rich Felker
POSIX defines fchmodat as having a 4th argument, flags, that can be AT_SYMLINK_NOFOLLOW. Support for changing the access mode of symbolic links is optional (EOPNOTSUPP allowed if not supported), but this flag is important even on systems where symlinks do not have access modes, since it's the only

Re: [PATCH -next] media: marvell-ccic: Fix -Wunused-function warnings

2020-09-10 Thread Lubomir Rintel
On Thu, Sep 10, 2020 at 05:18:15PM +0800, Yuehaibing wrote: > On 2020/9/10 16:22, Lubomir Rintel wrote: > > On Thu, Sep 10, 2020 at 04:09:33PM +0800, YueHaibing wrote: > >> If CONFIG_PM is n, gcc warns: > >> > >> drivers/media/platform/marvell-ccic/mmp-driver.c:347:12: warning: > >>

Re: [PATCH] perf: Fix race in perf_mmap_close function

2020-09-10 Thread Namhyung Kim
On Thu, Sep 10, 2020 at 7:42 PM Jiri Olsa wrote: > > There's a possible race in perf_mmap_close when checking ring buffer's > mmap_count refcount value. The problem is that the mmap_count check is > not atomic because we call atomic_dec and atomic_read separately. > > perf_mmap_close: > ... >

[PATCH] soc: fsl: dpio: remove set but not used 'addr_cena'

2020-09-10 Thread Jason Yan
This addresses the following gcc warning with "make W=1": drivers/soc/fsl/dpio/qbman-portal.c: In function ‘qbman_swp_enqueue_multiple_direct’: drivers/soc/fsl/dpio/qbman-portal.c:650:11: warning: variable ‘addr_cena’ set but not used [-Wunused-but-set-variable] 650 | uint64_t addr_cena;

Re: [PATCH] sh: fix syscall tracing

2020-09-10 Thread Rich Felker
On Thu, Sep 10, 2020 at 06:02:05AM -0500, Rob Landley wrote: > On 9/10/20 4:55 AM, John Paul Adrian Glaubitz wrote: > > Hi Rich! > > > > On 9/7/20 7:44 PM, Rich Felker wrote: > >>> Can we still get this merged as a hotfix for 5.9? > >> > >> Yes, fixes for regressions in the same release cycle are

Re: [PATCH V5 3/4] arm64: dts: qcom: sc7180-trogdor: Add pinctrl and interrupt config for BT UART

2020-09-10 Thread Doug Anderson
Hi, On Thu, Sep 10, 2020 at 5:55 AM satya priya wrote: > > Add a suitable sleep configuration for uart3 to support Bluetooth wakeup. > > If QUP function is selected in sleep state, UART RTS/RFR is pulled high > during suspend and BT SoC not able to send wakeup bytes. So, configure > GPIO mode in

[PATCH 11/12] bdi: invert BDI_CAP_NO_ACCT_WB

2020-09-10 Thread Christoph Hellwig
Replace BDI_CAP_NO_ACCT_WB with a positive BDI_CAP_WRITEBACK_ACCT to make the checks more obvious. Also remove the pointless bdi_cap_account_writeback wrapper that just obsfucates the check. Signed-off-by: Christoph Hellwig --- fs/fuse/inode.c | 3 ++- include/linux/backing-dev.h

[PATCH 03/12] drbd: remove RB_CONGESTED_REMOTE

2020-09-10 Thread Christoph Hellwig
This case isn't ever used. Signed-off-by: Christoph Hellwig Reviewed-by: Johannes Thumshirn --- drivers/block/drbd/drbd_req.c | 4 include/linux/drbd.h | 1 - 2 files changed, 5 deletions(-) diff --git a/drivers/block/drbd/drbd_req.c b/drivers/block/drbd/drbd_req.c index

Re: [PATCH 05/28] media/v4l2: remove V4L2-FLAG-MEMORY-NON-CONSISTENT

2020-09-10 Thread Sergey Senozhatsky
On (20/09/10 12:23), Hans Verkuil wrote: > On 10/09/2020 12:14, Sergey Senozhatsky wrote: > > On (20/09/10 11:57), Hans Verkuil wrote: > >> > >> Perhaps, but this patch is meant to revert *all* changes relating to > >> V4L2_FLAG_MEMORY_NON_CONSISTENT. We don't want to have unused fields > >> in

Re: [PATCH net-next + leds v2 6/7] net: phy: marvell: add support for LEDs controlled by Marvell PHYs

2020-09-10 Thread Andrew Lunn
> Moreover I propose (and am willing to do) this: > Rewrite phy_led_trigger so that it registers one trigger, `phydev`. > The identifier of the PHY which should be source of the trigger can be > set via a separate sysfs file, `device_name`, like in netdev trigger. > The linked speed on

Re: [PATCH] mm: don't rely on system state to detect hot-plug operations

2020-09-10 Thread Michal Hocko
On Thu 10-09-20 15:51:07, Michal Hocko wrote: > On Thu 10-09-20 15:39:00, Oscar Salvador wrote: > > On Thu, Sep 10, 2020 at 02:48:47PM +0200, Michal Hocko wrote: [...] > > > Forgot to ask one more thing. Who is going to online that memory when > > > userspace is not running yet? > > > > Depends,

Re: [PATCH V5 2/4] arm64: dts: qcom: sc7180: Add necessary pinctrl and interrupt config for BT UART

2020-09-10 Thread Doug Anderson
Hi, On Thu, Sep 10, 2020 at 5:55 AM satya priya wrote: > > Add a suitable sleep configuration for uart3 to support Bluetooth wakeup. > > If QUP function is selected in sleep state, UART RTS/RFR is pulled high > during suspend and BT SoC not able to send wakeup bytes. So, configure > GPIO mode in

Re: [RFC PATCH 00/16] 1GB THP support on x86_64

2020-09-10 Thread Zi Yan
On 10 Sep 2020, at 10:34, David Hildenbrand wrote: >>> As long as we stay in safe zone boundaries you get a benefit in most >>> scenarios. As soon as we would have a (temporary) workload that would >>> require more unmovable allocations we would fallback to polluting some >>> pageblocks only. >>

Re: [PATCH] pcmcia: db1xxx_ss: remove unneeded semicolon

2020-09-10 Thread Gustavo A. R. Silva
On Thu, Sep 10, 2020 at 10:05:24PM +0800, Jason Yan wrote: > Eliminate the following coccicheck warning: > > drivers/pcmcia/db1xxx_ss.c:455:2-3: Unneeded semicolon > > Reported-by: Hulk Robot > Signed-off-by: Jason Yan Acked-by: Gustavo A. R. Silva Thanks -- Gustavo > --- >

[PATCH v3 06/11] hv: hyperv.h: Introduce some hvpfn helper functions

2020-09-10 Thread Boqun Feng
When a guest communicate with the hypervisor, it must use HV_HYP_PAGE to calculate PFN, so introduce a few hvpfn helper functions as the counterpart of the page helper functions. This is the preparation for supporting guest whose PAGE_SIZE is not 4k. Signed-off-by: Boqun Feng ---

Re: [PATCH] video: fbdev: fsl-diu-fb: remove unneeded variable 'res'

2020-09-10 Thread Gustavo A. R. Silva
On Thu, Sep 10, 2020 at 10:05:58PM +0800, Jason Yan wrote: > Eliminate the following coccicheck warning: > > drivers/video/fbdev/fsl-diu-fb.c:1428:5-8: Unneeded variable: "res". > Return "0" on line 1450 > > Reported-by: Hulk Robot > Signed-off-by: Jason Yan Acked-by: Gustavo A. R. Silva

Re: [PATCH] taskstats: fix CGROUPSTATS_CMD_GET for cgroup v2

2020-09-10 Thread Tejun Heo
On Thu, Sep 10, 2020 at 01:52:07PM +0800, Chengming Zhou wrote: > We found cgroupstats_build would return -EINVAL when using netlink > CGROUPSTATS_CMD_GET interface to get stats on cgroup v2. Fix it by > supporting cgroup v2 kernfs directory in cgroupstats_build, and export > cgroup2_fs_type like

Re: [PATCH v2] platform/chrome: Don't populate lightbar device if it isn't there

2020-09-10 Thread Enric Balletbo i Serra
Hi, On 10/9/20 16:18, Guenter Roeck wrote: > On Wed, Sep 9, 2020 at 3:42 PM Stephen Boyd wrote: >> >> The cros ec lightbar driver has a check in probe to fail early if the ec >> device isn't the cros_ec one or if it can't read the lightbar version >> from the EC. Let's move this check to the

Re: [PATCH v2 -next] drm/amdkfd: Fix -Wunused-const-variable warning

2020-09-10 Thread Felix Kuehling
Am 2020-09-10 um 3:50 a.m. schrieb YueHaibing: > If KFD_SUPPORT_IOMMU_V2 is not set, gcc warns: > > drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_device.c:121:37: warning: > ‘raven_device_info’ defined but not used [-Wunused-const-variable=] > static const struct kfd_device_info raven_device_info = {

Re: [PATCH] mm: don't rely on system state to detect hot-plug operations

2020-09-10 Thread David Hildenbrand
>> Also, under QEMU, just do a reboot with hotplugged memory and you're in >> the very same situation. > > OK, I didn't know that. I thought the memory would be presented as a > normal memory after reboot. Thanks for the clarification. That's one of the cases where QEMU differs to actual

Re: [PATCH v2] platform/chrome: Don't populate lightbar device if it isn't there

2020-09-10 Thread Guenter Roeck
On Wed, Sep 9, 2020 at 3:42 PM Stephen Boyd wrote: > > The cros ec lightbar driver has a check in probe to fail early if the ec > device isn't the cros_ec one or if it can't read the lightbar version > from the EC. Let's move this check to the place where the lightbar > device is registered. This

Re: [PATCH] scripts/setlocalversion: make git describe output more reliable

2020-09-10 Thread Guenter Roeck
On 9/10/20 4:26 AM, Rasmus Villemoes wrote: > When building for an embedded target using Yocto, we're sometimes > observing that the version string that gets built into vmlinux (and > thus what uname -a reports) differs from the path under /lib/modules/ > where modules get installed in the rootfs,

[PATCH -next 0/3] rtlwifi: rtl8723ae: fix comparison pointer to bool warning

2020-09-10 Thread Zheng Bin
Zheng Bin (3): rtlwifi: rtl8723ae: fix comparison pointer to bool warning in rf.c rtlwifi: rtl8723ae: fix comparison pointer to bool warning in trx.c rtlwifi: rtl8723ae: fix comparison pointer to bool warning in phy.c drivers/net/wireless/realtek/rtlwifi/rtl8723ae/phy.c | 8

[PATCH -next 2/3] rtlwifi: rtl8723ae: fix comparison pointer to bool warning in trx.c

2020-09-10 Thread Zheng Bin
Fixes coccicheck warning: drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c:592:5-9: WARNING: Comparison to bool drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c:633:5-9: WARNING: Comparison to bool Signed-off-by: Zheng Bin --- drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c | 4

[PATCH -next 1/3] rtlwifi: rtl8723ae: fix comparison pointer to bool warning in rf.c

2020-09-10 Thread Zheng Bin
Fixes coccicheck warning: drivers/net/wireless/realtek/rtlwifi/rtl8723ae/rf.c:52:5-22: WARNING: Comparison to bool drivers/net/wireless/realtek/rtlwifi/rtl8723ae/rf.c:482:6-14: WARNING: Comparison to bool Signed-off-by: Zheng Bin --- drivers/net/wireless/realtek/rtlwifi/rtl8723ae/rf.c | 4

[PATCH 1/4] kbuild: remove cc-option test of -fno-strict-overflow

2020-09-10 Thread Masahiro Yamada
The minimal compiler versions, GCC 4.9 and Clang 10 support this flag. Here is the godbolt: https://godbolt.org/z/odq8h9 Signed-off-by: Masahiro Yamada --- Makefile | 2 +- arch/arm64/kernel/vdso32/Makefile | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)

[PATCH rdma-next 00/10] Prepare drivers to move QP allocation to ib_core

2020-09-10 Thread Leon Romanovsky
From: Leon Romanovsky Hi, This series mainly changes mlx4, mlx5, and mthca drivers to future change of QP allocation scheme. The rdmavt driver will be sent separately. Thanks Leon Romanovsky (10): RDMA/mlx5: Embed GSI QP into general mlx5_ib QP RDMA/mlx5: Reuse existing fields in parent

<    5   6   7   8   9   10   11   12   13   14   >