Re: [PATCH v7 3/3] drm: Switch to %p4cc format modifier

2021-02-16 Thread Thomas Zimmermann
Hi Am 15.02.21 um 12:40 schrieb Sakari Ailus: Switch DRM drivers from drm_get_format_name() to %p4cc. This gets rid of a large number of temporary variables at the same time. Signed-off-by: Sakari Ailus --- drivers/gpu/drm/amd/amdgpu/dce_v10_0.c| 5 ++--

Re: [PATCH v14 2/4] phy: Add media type and speed serdes configuration interfaces

2021-02-16 Thread Steen Hegelund
Hi Andrew and Kishon, On Mon, 2021-02-15 at 15:07 +0100, Andrew Lunn wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you > know the content is safe > > On Mon, Feb 15, 2021 at 05:25:10PM +0530, Kishon Vijay Abraham I > wrote: > > Okay. Is it going to be some sort of manual

Re: [PATCH v4] tpm: fix reference counting for struct tpm_chip

2021-02-16 Thread Jarkko Sakkinen
On Tue, Feb 16, 2021 at 01:31:00AM +0100, Lino Sanfilippo wrote: > From: Lino Sanfilippo > > The following sequence of operations results in a refcount warning: > > 1. Open device /dev/tpmrm Add '.' to end. > 2. Remove module tpm_tis_spi Add '.' to end. > 3. Write a TPM command to the file

[ANNOUNCE] v5.10.16-rt30

2021-02-16 Thread Sebastian Andrzej Siewior
Dear RT folks! I'm pleased to announce the v5.10.16-rt30 patch set. Changes since v5.10.16-rt29: - Due to recent softirq rework it was not possible to compile a kernel with RT && !SMP. Reported by Jonathan Schwender, patch by Christian Eggers. - Update the block-mq patches to the

Re: [PATCH 1/2] mmc: core: Drop superfluous validations in mmc_hw|sw_reset()

2021-02-16 Thread Adrian Hunter
On 12/02/21 3:15 pm, Ulf Hansson wrote: > The mmc_hw|sw_reset() APIs are designed to be called solely from upper > layers, which means drivers that operates on top of the struct mmc_card, > like the mmc block device driver and an SDIO functional driver. > > Additionally, as long as the struct

Re: [PATCH v5 3/3] x86/sgx: Remove redundant if conditions in sgx_encl_create

2021-02-16 Thread Jarkko Sakkinen
On Tue, Feb 16, 2021 at 11:31:33AM +0800, Tianjia Zhang wrote: > In this scenario, there is no case where va_page is NULL, and > the error has been checked. The if-condition statement here is > redundant, so remove the condition detection. > > Signed-off-by: Tianjia Zhang > Acked-by: Jarkko

Re: [PATCH v5 1/3] dt-bindings: reset: microchip sparx5 reset driver bindings

2021-02-16 Thread Steen Hegelund
Hi Alex, On Mon, 2021-02-15 at 18:32 +0100, Alexandre Belloni wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you > know the content is safe > > On 10/02/2021 10:19:50+0100, Steen Hegelund wrote: > > Document the Sparx5 reset device driver bindings > > > > The driver uses

Re: [PATCH -next] fs: libfs: fix kernel-doc for mnt_userns

2021-02-16 Thread Christoph Hellwig
On Mon, Feb 15, 2021 at 08:29:27PM -0800, Randy Dunlap wrote: > Fix kernel-doc warning in libfs.c. > > ../fs/libfs.c:498: warning: Function parameter or member 'mnt_userns' not > described in 'simple_setattr' Shouldn't the subject say simple_setattr instead of mnt_userns? Otherwise looks good:

Re: [PATCH 6/7] dma-iommu: implement ->alloc_noncontiguous

2021-02-16 Thread Christoph Hellwig
On Tue, Feb 16, 2021 at 05:14:55PM +0900, Tomasz Figa wrote: > When working on the videobuf2 integration with Sergey I noticed that > we always pass 0 as DMA attrs here, which removes the ability for > drivers to use DMA_ATTR_ALLOC_SINGLE_PAGES. > > It's quite important from a system stability

Re: [PATCH] nvme-tcp: Check if request has started before processing it

2021-02-16 Thread Hannes Reinecke
On 2/15/21 10:23 PM, Sagi Grimberg wrote: blk_mq_tag_to_rq() will always return a request if the command_id is in the valid range. Check if the request has been started. If we blindly process the request we might double complete a request which can be fatal. How did you get to this one? did

[PATCH v6 1/2] dt-bindings: counter: add event-counter binding

2021-02-16 Thread Oleksij Rempel
Add binding for the event counter node Signed-off-by: Oleksij Rempel Reviewed-by: Linus Walleij --- .../bindings/counter/interrupt-counter.yaml | 62 +++ 1 file changed, 62 insertions(+) create mode 100644 Documentation/devicetree/bindings/counter/interrupt-counter.yaml

Re: [PATCH 6/7] dma-iommu: implement ->alloc_noncontiguous

2021-02-16 Thread Tomasz Figa
Hi Christoph On Tue, Feb 2, 2021 at 6:51 PM Christoph Hellwig wrote: > > Implement support for allocating a non-contiguous DMA region. > > Signed-off-by: Christoph Hellwig > --- > drivers/iommu/dma-iommu.c | 35 +++ > 1 file changed, 35 insertions(+) > > diff

[PATCH v6 2/2] counter: add IRQ or GPIO based event counter

2021-02-16 Thread Oleksij Rempel
Add simple IRQ or GPIO base event counter. This device is used to measure rotation speed of some agricultural devices, so no high frequency on the counter pin is expected. The maximal measurement frequency depends on the CPU and system load. On the idle iMX6S I was able to measure up to 20kHz

[PATCH] mutex: nuke mutex_trylock_recursive

2021-02-16 Thread Christian König
The last user went away in the 5.11 cycle. Signed-off-by: Christian König --- include/linux/mutex.h | 25 - kernel/locking/mutex.c | 10 -- scripts/checkpatch.pl | 6 -- 3 files changed, 41 deletions(-) diff --git a/include/linux/mutex.h

Re: [PATCH v6 2/3] perf tools: Add lexical definition of event name

2021-02-16 Thread Namhyung Kim
On Mon, Feb 15, 2021 at 9:34 PM Jiri Olsa wrote: > > On Fri, Feb 12, 2021 at 06:03:17PM +0900, Shunsuke Nakamura wrote: > > Add the lexical definition of event name so that the numbers are > > recognizable. > > > > A64FX defines an event name that starts with a number. > > - 0inst_commit > > -

Re: [External] Re: [PATCH v15 4/8] mm: hugetlb: alloc the vmemmap pages associated with each HugeTLB page

2021-02-16 Thread David Hildenbrand
If current node has no memory and other nodes have enough memory. We can fail to allocate vmemmap pages. But actually it is suitable to allocate vmemmap pages from other nodes. Right? Falling back to a different node would be very suboptimal because then you would have vmemmap back by remote

Re: [PATCH v5 1/1] mm: refactor initialization of struct page for holes in memory layout

2021-02-16 Thread Michal Hocko
On Mon 15-02-21 23:24:40, Mike Rapoport wrote: > On Mon, Feb 15, 2021 at 10:00:31AM +0100, Michal Hocko wrote: > > On Sun 14-02-21 20:00:16, Mike Rapoport wrote: > > > On Fri, Feb 12, 2021 at 02:18:20PM +0100, Michal Hocko wrote: > > > > > We can correctly set the zone links for the reserved

Sira X - The best of Mandrake and Ubuntu, readied for Fair Pay netcurrency.

2021-02-16 Thread Ywe Cærlyn
A further iteration: Sira X - The best of Mandrake and Ubuntu, readied for Fair Pay netcurrency. With Islam as background philosophy of LA J. Serenity, Ywe Cærlyn. https://www.youtube.com/channel/UCqt17eaSO66UV4xvIYJvD4g

Re: [PATCH 0/2] block: avoid to drop & re-add partitions if partitions aren't changed

2021-02-16 Thread Christoph Hellwig
On Mon, Feb 15, 2021 at 12:03:41PM +0800, Ming Lei wrote: > Hello, I think this is a fundamentally bad idea. We should not keep the parsed partition state around forever just to work around some buggy user space software.

Re: [PATCH 00/21] [Set 2] Rid W=1 warnings from Clock

2021-02-16 Thread Lee Jones
On Mon, 15 Feb 2021, Jakub Kicinski wrote: > On Mon, 15 Feb 2021 08:49:52 + Lee Jones wrote: > > > Jakub can explain how he added these checks. > > > > Yes, please share. > > https://github.com/kuba-moo/nipa Thanks for this. Oh, I see. So you conduct tests locally, then post them up in

Re: [External] Re: [PATCH v15 4/8] mm: hugetlb: alloc the vmemmap pages associated with each HugeTLB page

2021-02-16 Thread Michal Hocko
On Tue 16-02-21 09:13:09, David Hildenbrand wrote: > On 15.02.21 20:02, Michal Hocko wrote: > > Would it be feasible to reused parts of the freed page in > > the worst case? > > As already discussed, this is only possible when the huge page does not > reside on ZONE_MOVABLE/CMA. Right. But

[RFC PATCH] z3fold: prevent reclaim/free race for headless pages

2021-02-16 Thread Thomas Hebb
commit ca0246bb97c2 ("z3fold: fix possible reclaim races") introduced the PAGE_CLAIMED flag "to avoid racing on a z3fold 'headless' page release." By atomically testing and setting the bit in each of z3fold_free() and z3fold_reclaim_page(), a double-free was avoided. However, commit 746d179b0e66

Re: [PATCH] proc_sysctl: clamp sizes using table->maxlen

2021-02-16 Thread Christoph Hellwig
On Mon, Feb 15, 2021 at 09:53:05AM -0500, Alex Xu (Hello71) wrote: > Since maxlen is already exposed, we can allocate approximately the right > amount directly, fixing up those drivers which set a bogus maxlen. These > drivers were located based on those which had copy_x_user replaced in >

Re: [External] Re: [PATCH v15 4/8] mm: hugetlb: alloc the vmemmap pages associated with each HugeTLB page

2021-02-16 Thread Michal Hocko
On Tue 16-02-21 12:34:41, Muchun Song wrote: > On Tue, Feb 16, 2021 at 3:39 AM Michal Hocko wrote: [...] > > > Using GFP_KERNEL will also use the current task cpuset to allocate > > > memory. Do we have an interface to ignore current task cpuset?If not, > > > WQ may be the only option and it also

Re: [PATCH v5 1/2] pinctrl: use to octal permissions for debugfs files

2021-02-16 Thread Geert Uytterhoeven
On Fri, Feb 12, 2021 at 11:30 PM Drew Fustini wrote: > Switch over pinctrl debugfs files to use octal permissions as they are > preferred over symbolic permissions. Refer to commit f90774e1fd27 > ("checkpatch: look for symbolic permissions and suggest octal instead"). > > Note: S_IFREG flag is

Re: [PATCH 34/33] netfs: Use in_interrupt() not in_softirq()

2021-02-16 Thread Christoph Hellwig
On Mon, Feb 15, 2021 at 10:46:23PM +, David Howells wrote: > The in_softirq() in netfs_rreq_terminated() works fine for the cache being > on a normal disk, as the completion handlers may get called in softirq > context, but for an NVMe drive, the completion handler may get called in > IRQ

Re: [PATCH] mfd: syscon: Don't free allocated name for regmap_config

2021-02-16 Thread Marc Zyngier
Hi Tomi, On Tue, 16 Feb 2021 06:34:50 +, Tomi Valkeinen wrote: > > Hi Marc, > > On 03/09/2020 19:02, Marc Zyngier wrote: > > The name allocated for the regmap_config structure is freed > > pretty early, right after the registration of the MMIO region. > > > > Unfortunately, that doesn't

[PATCH] coccinelle: misc: add swap script

2021-02-16 Thread Denis Efremov
Check for opencoded swap() implementation. Signed-off-by: Denis Efremov --- scripts/coccinelle/misc/swap.cocci | 77 ++ 1 file changed, 77 insertions(+) create mode 100644 scripts/coccinelle/misc/swap.cocci diff --git a/scripts/coccinelle/misc/swap.cocci

Re: [PATCH v4 0/8] Make fw_devlink=on more forgiving

2021-02-16 Thread Geert Uytterhoeven
Hi Saravana, On Mon, Feb 15, 2021 at 10:27 PM Saravana Kannan wrote: > On Mon, Feb 15, 2021 at 4:38 AM Geert Uytterhoeven > wrote: > > On Fri, Feb 12, 2021 at 4:00 AM Saravana Kannan > > wrote: > > > On Thu, Feb 11, 2021 at 5:00 AM Geert Uytterhoeven > > > wrote: > > > > - I2C on

Re: [External] Re: [PATCH v15 4/8] mm: hugetlb: alloc the vmemmap pages associated with each HugeTLB page

2021-02-16 Thread David Hildenbrand
On 15.02.21 20:02, Michal Hocko wrote: On Tue 16-02-21 01:48:29, Muchun Song wrote: On Tue, Feb 16, 2021 at 12:28 AM Michal Hocko wrote: On Mon 15-02-21 23:36:49, Muchun Song wrote: [...] There shouldn't be any real reason why the memory allocation for vmemmaps, or handling vmemmap in

[PATCH v6 0/2] add support for GPIO or IRQ based evemt counter

2021-02-16 Thread Oleksij Rempel
changes v6: - rename it to interrupt-counter - driver fixes - device tree fixes changes v5: - rename it to event counter, since it support different event sources - make it work with gpio-only or irq-only configuration - update yaml binding changes v4: - use IRQ_NOAUTOEN to not enable IRQ by

Re: [PATCH] mtd: rawnand: qcom: update last code word register

2021-02-16 Thread Miquel Raynal
Hello, Md Sadre Alam wrote on Tue, 16 Feb 2021 00:46:42 +0530: > From QPIC version 2.0 onwards new register got added to > read last codeword. This change will add the READ_LOCATION_LAST_CW_n > register. > > For first three code word READ_LOCATION_n register will be > use.For last code word

Re: [PATCH 2/2] mmc: core: Drop reference counting of the bus_ops

2021-02-16 Thread Adrian Hunter
On 12/02/21 3:16 pm, Ulf Hansson wrote: > When the mmc_rescan work is enabled for execution (host->rescan_disable), > it's the only instance per mmc host that is allowed to set/clear the > host->bus_ops pointer. > > Besides the mmc_rescan work, there are a couple of scenarios when the >

[PATCH mips-next] vmlinux.lds.h: catch more UBSAN symbols into .data

2021-02-16 Thread Alexander Lobakin
LKP triggered lots of LD orphan warnings [0]: mipsel-linux-ld: warning: orphan section `.data.$Lubsan_data299' from `init/do_mounts_rd.o' being placed in section `.data.$Lubsan_data299' mipsel-linux-ld: warning: orphan section `.data.$Lubsan_data183' from `init/do_mounts_rd.o' being placed in

Re: [PATCH 1/2] v4l2-ctrl: Add decoder conceal color control

2021-02-16 Thread Stanimir Varbanov
On 2/15/21 1:57 PM, Hans Verkuil wrote: > On 15/02/2021 12:32, Stanimir Varbanov wrote: >> >> >> On 2/9/21 1:05 PM, Hans Verkuil wrote: >>> On 09/02/2021 10:45, Stanimir Varbanov wrote: Add decoder v4l2 control to set conceal color. Signed-off-by: Stanimir Varbanov ---

Re: [RFC PATCH] mm, oom: introduce vm.sacrifice_hugepage_on_oom

2021-02-16 Thread Michal Hocko
On Tue 16-02-21 03:07:13, Eiichi Tsukata wrote: > Hugepages can be preallocated to avoid unpredictable allocation latency. > If we run into 4k page shortage, the kernel can trigger OOM even though > there were free hugepages. When OOM is triggered by user address page > fault handler, we can use

[PATCH v6 2/6] dt: bindings: add mt7621-clk device tree binding documentation

2021-02-16 Thread Sergio Paracuellos
Adds device tree binding documentation for clocks in the MT7621 SOC. Signed-off-by: Sergio Paracuellos --- .../bindings/clock/mediatek,mt7621-clk.yaml | 66 +++ 1 file changed, 66 insertions(+) create mode 100644

[PATCH v6 1/6] dt-bindings: clock: add dt binding header for mt7621 clocks

2021-02-16 Thread Sergio Paracuellos
Adds dt binding header for 'mediatek,mt7621-clk' clocks. Acked-by: Rob Herring Signed-off-by: Sergio Paracuellos --- include/dt-bindings/clock/mt7621-clk.h | 41 ++ 1 file changed, 41 insertions(+) create mode 100644 include/dt-bindings/clock/mt7621-clk.h diff --git

[PATCH v6 3/6] clk: ralink: add clock driver for mt7621 SoC

2021-02-16 Thread Sergio Paracuellos
The documentation for this SOC only talks about two registers regarding to the clocks: * SYSC_REG_CPLL_CLKCFG0 - provides some information about boostrapped refclock. PLL and dividers used for CPU and some sort of BUS. * SYSC_REG_CPLL_CLKCFG1 - a banch of gates to enable/disable clocks for all or

[PATCH v6 5/6] staging: mt7621-dts: use valid vendor 'mediatek' instead of invalid 'mtk'

2021-02-16 Thread Sergio Paracuellos
Vendor listed for mediatek in kernel vendor file 'vendor-prefixes.yaml' contains 'mediatek' as a valid vendor string. Some nodes in the device tree are using an invalid vendor string vfor 'mtk' instead. Fix all of them in dts file. Update also ralink mt7621 related code to properly match new

[PATCH v6 6/6] MAINTAINERS: add MT7621 CLOCK maintainer

2021-02-16 Thread Sergio Paracuellos
Adding myself as maintainer for mt7621 clock driver. Signed-off-by: Sergio Paracuellos --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 809a68af5efd..be5ada6b4309 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11288,6 +11288,12 @@ L:

[PATCH v6 0/6] MIPS: ralink: add CPU clock detection and clock driver for MT7621

2021-02-16 Thread Sergio Paracuellos
This patchset ports CPU clock detection for MT7621 from OpenWrt and adds a complete clock plan for the mt7621 SOC. The documentation for this SOC only talks about two registers regarding to the clocks: * SYSC_REG_CPLL_CLKCFG0 - provides some information about boostrapped refclock. PLL and

[PATCH v6 4/6] staging: mt7621-dts: make use of new 'mt7621-clk'

2021-02-16 Thread Sergio Paracuellos
Clocks for SoC mt7621 have been properly integrated so there is no need to declare fixed clocks at all in the device tree. Remove all of them, add new device tree nodes for mt7621-clk and update the rest of the nodes to use them. Acked-by: Greg Kroah-Hartman Signed-off-by: Sergio Paracuellos

Re: [PATCH for 5.10] vdpa_sim: fix param validation in vdpasim_get_config()

2021-02-16 Thread Stefano Garzarella
On Mon, Feb 15, 2021 at 04:23:54PM +0100, Greg KH wrote: On Mon, Feb 15, 2021 at 04:03:21PM +0100, Stefano Garzarella wrote: On Mon, Feb 15, 2021 at 03:32:19PM +0100, Greg KH wrote: > On Thu, Feb 11, 2021 at 05:25:19PM +0100, Stefano Garzarella wrote: > > Commit

NACK: [PATCH][next] tracing/tools: fix spelling mistake "functionph" -> "graph"

2021-02-16 Thread Colin Ian King
On 16/02/2021 14:01, Colin King wrote: > From: Colin Ian King > > There is a spelling mistake in the -g help option, I believe > it should be "graph". Fix it. > > Signed-off-by: Colin Ian King > --- > tools/tracing/latency/latency-collector.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH 1/2] driver core: component: remove dentry pointer in "struct master"

2021-02-16 Thread Rafael J. Wysocki
On Tue, Feb 16, 2021 at 3:24 PM Greg Kroah-Hartman wrote: > > There is no need to keep around a pointer to a dentry when all it is > used for is to remove the debugfs file when tearing things down. As the > name is simple, have debugfs look up the dentry when removing things, > keeping the logic

Re: [PATCH v2 06/25] arm64: arch_timer: implement support for interrupt-names

2021-02-16 Thread Hector Martin
On 16/02/2021 03.23, Tony Lindgren wrote: * Hector Martin [210215 12:18]: This allows the devicetree to correctly represent the available set of timers, which varies from device to device, without the need for fake dummy interrupts for unavailable slots. I like the idea of using

[PATCH v5 bpf-next 0/6] xsk: build skb by page (aka generic zerocopy xmit)

2021-02-16 Thread Alexander Lobakin
This series introduces XSK generic zerocopy xmit by adding XSK umem pages as skb frags instead of copying data to linear space. The only requirement for this for drivers is to be able to xmit skbs with skb_headlen(skb) == 0, i.e. all data including hard headers starts from frag 0. To indicate

[PATCH v5 bpf-next 2/6] netdevice: check for net_device::priv_flags bitfield overflow

2021-02-16 Thread Alexander Lobakin
We almost ran out of unsigned int bitwidth. Define priv flags and check for potential overflow in the fashion of netdev_features_t. Defined this way, priv_flags can be easily expanded later with just changing its typedef. Signed-off-by: Alexander Lobakin Reported-by: kernel test robot #

[PATCH v5 bpf-next 1/6] netdev_priv_flags: add missing IFF_PHONY_HEADROOM self-definition

2021-02-16 Thread Alexander Lobakin
This is harmless for now, but comes fatal for the subsequent patch. Fixes: 871b642adebe3 ("netdev: introduce ndo_set_rx_headroom") Signed-off-by: Alexander Lobakin --- include/linux/netdevice.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/netdevice.h

Re: [PATCH v1 1/4] PCI: Introduce pcim_alloc_irq_vectors()

2021-02-16 Thread Krzysztof Wilczyński
Hi Dejin and Andy, [...] > > > Question: wouldn't you need to call pci_free_irq_vectors() somewhere, > > > possibly to pcim_release() callback? Although, I am not sure where the > > > right place would be. > > > > > > I am asking, as the documentation (see [4]) suggests that one would have > >

Re: [PATCH 1/2] media: camss: use v4l2_get_link_freq() to calculate the relevant clocks

2021-02-16 Thread Jacopo Mondi
Hi Andrey, On Mon, Feb 15, 2021 at 11:58:27PM +0300, Andrey Konovalov wrote: > Hi Jacopo, > > Thank you for the detailed review! > > On 15.02.2021 15:00, Jacopo Mondi wrote: > > Hi Andrey, > > nice to see progress in this direction > > > > On Mon, Feb 15, 2021 at 12:34:03AM +0300, Andrey

[PATCH 0/2] add proximity rate option for vcnl3020

2021-02-16 Thread Ivan Mikhaylov
Add the control of proximity rate from runtime and DTS for vcnl3020 proximity sensor. Ivan Mikhaylov (2): iio: proximity: vcnl3020: add proximity rate dt-bindings: vcnl3020: add proximity rate in hz .../iio/proximity/vishay,vcnl3020.yaml| 6 + drivers/iio/proximity/vcnl3020.c

[PATCH 1/2] iio: proximity: vcnl3020: add proximity rate

2021-02-16 Thread Ivan Mikhaylov
Add the proximity rate optional option and handling of it for vishay vcnl3020. Signed-off-by: Ivan Mikhaylov --- drivers/iio/proximity/vcnl3020.c | 123 ++- 1 file changed, 120 insertions(+), 3 deletions(-) diff --git a/drivers/iio/proximity/vcnl3020.c

[PATCH 2/2] dt-bindings: vcnl3020: add proximity rate in hz

2021-02-16 Thread Ivan Mikhaylov
Describe the possible proximity values in herzes for vcnl3020. Signed-off-by: Ivan Mikhaylov --- .../devicetree/bindings/iio/proximity/vishay,vcnl3020.yaml | 6 ++ 1 file changed, 6 insertions(+) diff --git a/Documentation/devicetree/bindings/iio/proximity/vishay,vcnl3020.yaml

Re: [PATCH 3/4] ibmvfc: treat H_CLOSED as success during sub-CRQ registration

2021-02-16 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center

Re: [RFT][PATCH v1] cpufreq: ACPI: Set cpuinfo.max_freq directly if max boost is known

2021-02-16 Thread Giovanni Gherdovich
On Mon, 2021-02-15 at 20:24 +0100, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Commit 3c55e94c0ade ("cpufreq: ACPI: Extend frequency tables to cover > boost frequencies") attempted to address a performance issue involving > acpi-cpufreq, the schedutil governor and scale-invariance on

Re: [PATCH v4 4/9] cxl/mem: Add basic IOCTL interface

2021-02-16 Thread Jonathan Cameron
On Mon, 15 Feb 2021 17:45:33 -0800 Ben Widawsky wrote: > Add a straightforward IOCTL that provides a mechanism for userspace to > query the supported memory device commands. CXL commands as they appear > to userspace are described as part of the UAPI kerneldoc. The command > list returned via

Re: [PATCH] Add CHARGE_NOW support to cw2015_battery.c

2021-02-16 Thread Tobias Schramm
Hi Martin, thanks for the patch. Looks good and tests fine. eh, scratch that. Didn't have my cup of morning coffee yet. The patch is fine as far as the code change goes. However it is missing a "Signed-off-by" line, certifying your authorship. git can generate that automagically for you

Re: usb: dwc3: gadget: Change runtime pm function for DWC3 runtime suspend

2021-02-16 Thread Alan Stern
On Tue, Feb 16, 2021 at 10:30:52AM +0900, Jung Daehwan wrote: > Hello, Alan > > On Mon, Feb 15, 2021 at 12:41:45PM -0500, Alan Stern wrote: > > On Mon, Feb 15, 2021 at 11:38:58AM +0900, Daehwan Jung wrote: > > > It seems pm_runtime_put calls runtime_idle callback not runtime_suspend > > >

[PATCH v9 2/4] v4l: ioctl: Use %p4cc printk modifier to print FourCC codes

2021-02-16 Thread Sakari Ailus
Now that we can print FourCC codes directly using printk, make use of the feature in V4L2 core. Signed-off-by: Sakari Ailus Acked-by: Mauro Carvalho Chehab Reviewed-by: Petr Mladek Reviewed-by: Andy Shevchenko --- drivers/media/v4l2-core/v4l2-ioctl.c | 85 +++- 1 file

Re: [PATCH] virtio_console: remove pointless check for debugfs_create_dir()

2021-02-16 Thread Amit Shah
On Tue, 2021-02-16 at 16:04 +0100, Greg Kroah-Hartman wrote: > It is impossible for debugfs_create_dir() to return NULL, so checking > for it gives people a false sense that they actually are doing something > if an error occurs. As there is no need to ever change kernel logic if > debugfs is

debugfs: was: Re: [PATCH v4] printk: Userspace format enumeration support

2021-02-16 Thread Petr Mladek
On Fri 2021-02-12 15:30:16, Chris Down wrote: > We have a number of systems industry-wide that have a subset of their > functionality that works as follows: > > 1. Receive a message from local kmsg, serial console, or netconsole; > 2. Apply a set of rules to classify the message; > 3. Do

Re: [PATCH] mtd: spi-nor: gigadevice: Add support for gd25lb256e

2021-02-16 Thread Tudor.Ambarus
Hi, On 2/16/21 5:10 PM, mda...@codeaurora.org wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > On 2021-02-09 16:41, Md Sadre Alam wrote: >> Add support for gd25lb256e. This device tested on IPQ5018 >> platform with dd from/to the flash

[PATCH v2] USB: serial: cp210x: add IDs for nLight Air Adapter, Acuity Brands, Inc.

2021-02-16 Thread Karan Singhal
IDs of nLight Air Adapter, Acuity Brands, Inc.: vid: 10c4 pid: 88d8 Signed-off-by: Karan Singhal --- drivers/usb/serial/cp210x.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/serial/cp210x.c b/drivers/usb/serial/cp210x.c index 7bec1e730b20..b82bb130710d 100644 ---

[PATCH v1 00/12] Driver of Intel(R) Gaussian & Neural Accelerator

2021-02-16 Thread Maciej Kwapulinski
Dear kernel maintainers, This submission is a kernel driver to support Intel(R) Gaussian & Neural Accelerator (Intel(R) GNA). Intel(R) GNA is a PCI-based neural co-processor available on multiple Intel platforms. AI developers and users can offload continuous inference workloads to an Intel(R)

[PATCH v1 01/12] gna: add driver module

2021-02-16 Thread Maciej Kwapulinski
From: Tomasz Jankowski Add a new PCI driver for Intel(R) Gaussian & Neural Accelerator with basic support like module loading and unloading. The full function of the driver will be added by further changes. Signed-off-by: Tomasz Jankowski Tested-by: Savo Novakovic Co-developed-by: Jianxun

[PATCH v1 12/12] gna: add a char device

2021-02-16 Thread Maciej Kwapulinski
From: Tomasz Jankowski The new char device is the node for applications in user space to interact with the driver. Signed-off-by: Tomasz Jankowski Tested-by: Savo Novakovic Co-developed-by: Jianxun Zhang Signed-off-by: Jianxun Zhang Co-developed-by: Maciej Kwapulinski Signed-off-by: Maciej

Re: [PATCH v4] tpm: fix reference counting for struct tpm_chip

2021-02-16 Thread Jarkko Sakkinen
On Tue, Feb 16, 2021 at 06:04:42PM +0200, Jarkko Sakkinen wrote: > On Tue, Feb 16, 2021 at 08:53:42AM -0400, Jason Gunthorpe wrote: > > On Tue, Feb 16, 2021 at 01:31:00AM +0100, Lino Sanfilippo wrote: > > > > > > +static int tpm_add_tpm2_char_device(struct tpm_chip *chip) > > BTW, this naming

[PATCH v1 02/12] gna: add component of hardware operation

2021-02-16 Thread Maciej Kwapulinski
From: Tomasz Jankowski Add definitions and utilities to interact with the hardware device. Signed-off-by: Tomasz Jankowski Tested-by: Savo Novakovic Co-developed-by: Jianxun Zhang Signed-off-by: Jianxun Zhang Co-developed-by: Maciej Kwapulinski Signed-off-by: Maciej Kwapulinski ---

Re: [Freedreno] [v2] drm/msm/disp/dpu1: turn off vblank irqs aggressively in dpu driver

2021-02-16 Thread Rob Clark
On Tue, Feb 16, 2021 at 7:21 AM wrote: > > On 2021-02-12 23:19, Rob Clark wrote: > > On Thu, Feb 11, 2021 at 7:31 AM wrote: > >> > >> On 2021-02-11 01:56, Rob Clark wrote: > >> > On Wed, Feb 10, 2021 at 3:41 AM wrote: > >> >> > >> >> On 2021-02-01 00:46, Rob Clark wrote: > >> >> > On Fri, Dec

WARNING in iov_iter_revert (2)

2021-02-16 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:dcc0b490 Merge tag 'powerpc-5.11-8' of git://git.kernel.or.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=162f2d02d0 kernel config: https://syzkaller.appspot.com/x/.config?x=1106b4b91e8dfab8

[PATCH v2] leds: apu: extend support for PC Engines APU1 with newer firmware

2021-02-16 Thread Andreas Eberlein
The DMI_PRODUCT_NAME entry on current firmware of PC Engines APU1 changed from "APU" to "apu1" This modification adds the missing DMI data and thereby the LED support for the PC Engines APU1 with firmware versions >= 4.6.8. Signed-off-by: Andreas Eberlein --- Changes in v2: - Add missing

Re: [RFC PATCH] mm, oom: introduce vm.sacrifice_hugepage_on_oom

2021-02-16 Thread Chris Down
Hi Eiichi, I agree with Michal's points, and I think there are also some other design questions which don't quite make sense to me. Perhaps you can clear them up? :-) Eiichi Tsukata writes: diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 4bdb58ab14cb..e2d57200fd00 100644 --- a/mm/hugetlb.c

Re: [PATCH] arch: s390: kvm: Fix oustanding to outstanding in the file kvm-s390.c

2021-02-16 Thread Bhaskar Chowdhury
On 10:08 Mon 15 Feb 2021, Janosch Frank wrote: On 2/13/21 4:32 PM, Bhaskar Chowdhury wrote: s/oustanding/outstanding/ Hey Bhaskar, while I do encourage anyone to send in changes I'm not a big fan of comment fixes if they are only a couple of characters and when the meaning is still intact

Re: riscv+KASAN does not boot

2021-02-16 Thread Dmitry Vyukov
On Tue, Feb 16, 2021 at 12:25 PM Dmitry Vyukov wrote: > > On Tue, Feb 16, 2021 at 12:17 PM Dmitry Vyukov wrote: > > > > On Fri, Jan 29, 2021 at 9:11 AM Dmitry Vyukov wrote: > > > > I was fixing KASAN support for my sv48 patchset so I took a look at your > > > > issue: I built a kernel on top of

Re: [PATCH v2 6/8] cxl/mem: Enable commands via CEL

2021-02-16 Thread Bartosz Golaszewski
On Thu, Feb 11, 2021 at 1:12 PM Jonathan Cameron wrote: > [snip!] > > > > @@ -869,6 +891,14 @@ static struct cxl_mem *cxl_mem_create(struct pci_dev > > *pdev, u32 reg_lo, > > mutex_init(>mbox_mutex); > > cxlm->pdev = pdev; > > cxlm->regs = regs + offset; > > +

[PATCH][next] tracing/tools: fix spelling mistake "functionph" -> "graph"

2021-02-16 Thread Colin King
From: Colin Ian King There is a spelling mistake in the -g help option, I believe it should be "graph". Fix it. Signed-off-by: Colin Ian King --- tools/tracing/latency/latency-collector.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v2 1/4] PCI: Introduce pcim_alloc_irq_vectors()

2021-02-16 Thread Dejin Zheng
Introduce pcim_alloc_irq_vectors(), a device-managed version of pci_alloc_irq_vectors(). Introducing this function can simplify the error handling path in many drivers. Suggested-by: Andy Shevchenko Signed-off-by: Dejin Zheng --- v1 -> v2: - Use pci_free_irq_vectors() to replace some

[PATCH v2 0/4] Introduce pcim_alloc_irq_vectors()

2021-02-16 Thread Dejin Zheng
Introduce pcim_alloc_irq_vectors(), a device-managed version of pci_alloc_irq_vectors(), In some i2c drivers, If pcim_enable_device() has been called before, then pci_alloc_irq_vectors() is actually a device-managed function. It is used as a device-managed function, So replace it with

[PATCH v2 2/4] Documentation: devres: Add pcim_alloc_irq_vectors()

2021-02-16 Thread Dejin Zheng
Add pcim_alloc_irq_vectors(), a device-managed version of pci_alloc_irq_vectors(). introducing this function can simplify the error handling path in many drivers. Signed-off-by: Dejin Zheng --- v1 -> v2: - Modify some commit messages. Documentation/driver-api/driver-model/devres.rst |

[PATCH] watchdog: wdat_wdg: fix typo

2021-02-16 Thread Flavio Suligoi
Fix the following typo: "recommeded" --> "recommended" "firmare"--> "firmware" Signed-off-by: Flavio Suligoi --- drivers/watchdog/wdat_wdt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/watchdog/wdat_wdt.c b/drivers/watchdog/wdat_wdt.c index

Re: [PATCH] fs: export kern_path_locked

2021-02-16 Thread Denis Kirjanov
On 2/14/21, Al Viro wrote: > On Fri, Jan 29, 2021 at 01:18:55PM +, Christoph Hellwig wrote: >> On Fri, Jan 29, 2021 at 04:11:05PM +0300, Denis Kirjanov wrote: >> > Do you mean just: >> >> We'll still need to lock the parent inode. > > Not just "lock", we wouldd need to have the lock _held_

Re: [PATCH v2 02/25] dt-bindings: arm: apple: Add bindings for Apple ARM platforms

2021-02-16 Thread Hector Martin
On 16/02/2021 02.48, Krzysztof Kozlowski wrote: On Mon, Feb 15, 2021 at 09:16:50PM +0900, Hector Martin wrote: +description: | + Apple ARM ("Apple Silicon") platforms should contain compatible strings + in the following format: + + - apple,j274 (board/device ID) + - apple,m1 (SoC name) + -

Re: [PATCH v3 1/3] platform/x86: dell-privacy: Add support for Dell hardware privacy

2021-02-16 Thread Pierre-Louis Bossart
+static const struct acpi_device_id privacy_acpi_device_ids[] = { +    {"PNP0C09", 0}, +    { }, +}; +MODULE_DEVICE_TABLE(acpi, privacy_acpi_device_ids); + +static struct platform_driver dell_privacy_platform_drv = { +    .driver = { +    .name = PRIVACY_PLATFORM_NAME, +   

Re: [PATCH 4/4] ibmvfc: store return code of H_FREE_SUB_CRQ during cleanup

2021-02-16 Thread Brian King
On 2/11/21 12:57 PM, Tyrel Datwyler wrote: > diff --git a/drivers/scsi/ibmvscsi/ibmvfc.c b/drivers/scsi/ibmvscsi/ibmvfc.c > index ba6fcf9cbc57..23b803ac4a13 100644 > --- a/drivers/scsi/ibmvscsi/ibmvfc.c > +++ b/drivers/scsi/ibmvscsi/ibmvfc.c > @@ -5670,7 +5670,7 @@ static int

Re: [PATCH] Add CHARGE_NOW support to cw2015_battery.c

2021-02-16 Thread Tobias Schramm
Hi Martin, thanks for the patch. Looks good and tests fine. CHARGE_NOW is expected by some user software (such as waybar) instead of 'CAPACITY', in order to correctly calculate remaining battery life. --- drivers/power/supply/cw2015_battery.c | 6 ++ 1 file changed, 6 insertions(+)

Re: [PATCH 14/33] fscache, cachefiles: Add alternate API to use kiocb for read/write to cache

2021-02-16 Thread David Howells
Christoph Hellwig wrote: > > Filesystems wanting to use the new API must #define FSCACHE_USE_NEW_IO_API > > before #including the header > > What exactly does this ifdef buys us? It seems like the old and new > APIs don't even conflict. I was asked to add this. The APIs look like they don't

[PATCH] drivers: habanalabs: remove unused dentry pointer for debugfs files

2021-02-16 Thread Greg Kroah-Hartman
The dentry for the created debugfs file was being saved, but never used anywhere. As the pointer isn't needed for anything, and the debugfs files are being properly removed by removing the parent directory, remove the saved pointer as well, saving a tiny bit of memory and logic. Cc: Oded Gabbay

[PATCH net-next v3 3/3] net: phy: mscc: coma mode disabled for VSC8514

2021-02-16 Thread Bjarni Jonasson
The 'coma mode' (configurable through sw or hw) provides an optional feature that may be used to control when the PHYs become active. The typical usage is to synchronize the link-up time across all PHY instances. This patch releases coma mode if not done by hardware, otherwise the phys will not

Re: [PATCH v4 5/9] cxl/mem: Add a "RAW" send command

2021-02-16 Thread Jonathan Cameron
On Mon, 15 Feb 2021 17:45:34 -0800 Ben Widawsky wrote: > The CXL memory device send interface will have a number of supported > commands. The raw command is not such a command. Raw commands allow > userspace to send a specified opcode to the underlying hardware and > bypass all driver checks on

Re: [PATCH] staging: greybus: minor code style fix

2021-02-16 Thread Alex Elder
On 2/12/21 4:50 PM, Manikantan Ravichandran wrote: checkpatch warning fix for string split across lines Signed-off-by: Manikantan Ravichandran I think what you're doing here *looks* reasonable. But the GB_AUDIO_MANAGER_MODULE_NAME_LEN_SSCANF symbol is a (string) numeric value that is

Re: [PATCH] Staging: ks7010: ks_hostif: fixed parentheses coding style issue

2021-02-16 Thread Greg KH
On Tue, Feb 16, 2021 at 08:40:58PM +0530, Selvakumar Elangovan wrote: > Fixed a coding style issue. > > Signed-off-by: Selvakumar Elangovan > --- > drivers/staging/ks7010/ks_hostif.h | 24 > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git

[v2] drm/i915: Use function pointer typedef long_pulse_detect_func in intel_get_hpd_pins

2021-02-16 Thread Chen Lin
From: Chen Lin Use function pointer typedef long_pulse_detect_func in intel_get_hpd_pins. Signed-off-by: Chen Lin --- drivers/gpu/drm/i915/i915_irq.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c index

Re: [PATCH] ftrace: Do not reference symbols in sections without size

2021-02-16 Thread Greg Kroah-Hartman
On Tue, Feb 16, 2021 at 09:45:39AM -0600, Josh Poimboeuf wrote: > On Tue, Feb 16, 2021 at 09:51:21AM -0500, Steven Rostedt wrote: > > On Tue, 16 Feb 2021 12:04:06 +0100 > > Greg Kroah-Hartman wrote: > > > > > Thanks for this. > > > > > > Should I also queue these up for 4.9 and 4.14 which do

Re: [reiser4 SFRN 5.1.3] kernel [5.10.x] read not supported for file /test-exec \(pid: 10094 comm: debootstrap\)

2021-02-16 Thread Jose R Rodriguez
On Mon, 2021-02-08 at 17:03 +0100, Edward Shishkin wrote: > On 02/08/2021 01:54 PM, Metztli Information Technology wrote: > > On Wed, Dec 23, 2020 at 3:40 PM Edward Shishkin < > > edward.shish...@gmail.com> wrote: > > > > > > On 12/23/2020 05:01 PM, Metztli Information Technology wrote: > > > >

Re: [PATCH 2/4] perf report: Load PE files from debug cache only

2021-02-16 Thread Nicholas Fraser
Oops, you are correct; it is doing these checks twice. Not sure how I missed that. I'll send another patch to fix it. Nick On 2021-02-15 9:36 a.m., Jiri Olsa wrote: > On Wed, Feb 10, 2021 at 02:17:38PM -0500, Nicholas Fraser wrote: >> dso__load_bfd_symbols() attempts to load a DSO at its

[PATCH v1 11/12] gna: add ioctl handler

2021-02-16 Thread Maciej Kwapulinski
From: Tomasz Jankowski Add ioctl handler into GNA driver. The ioctl interface provides the ability to do the following: - Map and unmap memory buffers for GNA computation requests. - Retrieve capabilities of the underlying GNA IP. - Submit GNA computation requests. - Request notification of

[PATCH v1 10/12] gna: add interrupt handler

2021-02-16 Thread Maciej Kwapulinski
From: Tomasz Jankowski An interrupt is generated by the hardware when a scoring job is done. The interrupt handler wakes up the work queue to resume the processing on the current request. Signed-off-by: Tomasz Jankowski Tested-by: Savo Novakovic Co-developed-by: Jianxun Zhang Signed-off-by:

Re: [PATCH 10/10] drm/qxl: add lock asserts to qxl_bo_kmap_locked + qxl_bo_kunmap_locked

2021-02-16 Thread Thomas Zimmermann
Hi Am 16.02.21 um 12:37 schrieb Gerd Hoffmann: Try avoid re-introducing locking bugs. Signed-off-by: Gerd Hoffmann Acked-by: Thomas Zimmermann IMHO this should eventually be done in the rsp TTM functions. But so far, I'd expect this to break some drivers. Best regards Thomas ---

  1   2   3   4   5   6   7   8   9   10   >