Re: [PATCH] usb: dwc3: st: fix incorrect kernel-doc comment syntax in file

2021-03-29 Thread Randy Dunlap
On 3/29/21 6:20 AM, Aditya Srivastava wrote: > The opening comment mark '/**' is used for highlighting the beginning of > kernel-doc comments. > The header for drivers/usb/dwc3/dwc3-st.c follows this syntax, but the > content inside does not comply with kernel-doc. > > This line was probably not

Re: Candidate Linux ABI for Intel AMX and hypothetical new related features

2021-03-29 Thread Andy Lutomirski
> On Mar 29, 2021, at 8:47 AM, Len Brown wrote: > > On Sat, Mar 27, 2021 at 5:58 AM Greg KH wrote: >>> On Fri, Mar 26, 2021 at 11:39:18PM -0400, Len Brown wrote: >>> Hi Andy, >>> Say a mainline links with a math library that uses AMX without the >>> knowledge of the mainline. > > sorry for

Re: [PATCH v2 1/2] fuse: Add support for FUSE_SETXATTR_V2

2021-03-29 Thread Vivek Goyal
On Mon, Mar 29, 2021 at 03:50:37PM +0100, Luis Henriques wrote: > On Thu, Mar 25, 2021 at 11:18:22AM -0400, Vivek Goyal wrote: > > Fuse client needs to send additional information to file server when > > it calls SETXATTR(system.posix_acl_access). Right now there is no extra > > space in

[PATCH v4 9/9] media: uapi: Move MPEG-2 stateless controls out of staging

2021-03-29 Thread Ezequiel Garcia
Until now, the MPEG-2 V4L2 API was not exported as a public API, and only defined in a private media header (media/mpeg2-ctrls.h). After reviewing the MPEG-2 specification in detail, and reworking the controls so they match the MPEG-2 semantics properly, we can consider it ready. Signed-off-by:

Re: [PATCH][next] mm/vmalloc: Fix read of pointer area after it has been free'd

2021-03-29 Thread Uladzislau Rezki
> > > } > > > > > > if (!pages) { > > > - free_vm_area(area); > > > warn_alloc(gfp_mask, NULL, > > > "vmalloc size %lu allocation failure: " > > > "page array size %lu allocation f

Re: [PATCH v4 14/22] x86/fpu/xstate: Expand the xstate buffer on the first use of dynamic user state

2021-03-29 Thread Bae, Chang Seok
On Mar 26, 2021, at 09:34, Jann Horn wrote: > On Sun, Feb 21, 2021 at 7:56 PM Chang S. Bae wrote: >> >> + if (handle_xfirstuse_event(>thread.fpu)) >> + return; > > What happens if handle_xfirstuse_event() fails because vmalloc() > failed in alloc_xstate_buffer()? I think

Re: Candidate Linux ABI for Intel AMX and hypothetical new related features

2021-03-29 Thread Andy Lutomirski
> On Mar 29, 2021, at 9:39 AM, Len Brown wrote: > >  >> >> In particular, the library may use instructions that main() doesn't know >> exist. > > And so I'll ask my question another way. > > How is it okay to change the value of XCR0 during the run time of a program? > > I submit that

[PATCH v4 7/9] media: controls: Log MPEG-2 stateless control in .std_log

2021-03-29 Thread Ezequiel Garcia
Simply print the type of the control. Signed-off-by: Ezequiel Garcia --- drivers/media/v4l2-core/v4l2-ctrls.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c index 28657f2f679b..063a373c8965 100644 ---

[PATCH v4 6/9] media: uapi: mpeg2: Remove V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS

2021-03-29 Thread Ezequiel Garcia
The Hantro and Cedrus drivers work in frame-mode, meaning they expect all the slices in a picture (either frame or field structure) to be passed in each OUTPUT buffer. These two are the only V4L2 MPEG-2 stateless decoders currently supported. Given the VA-API drivers also work per-frame,

[PATCH v4 8/9] media: uapi: Move the MPEG-2 stateless control type out of staging

2021-03-29 Thread Ezequiel Garcia
Move the MPEG-2 stateless control types out of staging, and re-number it to avoid any confusion. Signed-off-by: Ezequiel Garcia --- include/media/mpeg2-ctrls.h| 4 include/uapi/linux/videodev2.h | 4 2 files changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH v4 4/9] media: uapi: mpeg2: Move reference buffer fields

2021-03-29 Thread Ezequiel Garcia
The forward and backwards references are specified per-picture and not per-slice. Move it to V4L2_CID_MPEG_VIDEO_MPEG2_PICTURE. Signed-off-by: Ezequiel Garcia --- .../media/v4l/ext-ctrls-codec.rst | 28 +-- .../media/hantro/hantro_g1_mpeg2_dec.c| 6 ++--

[PATCH] irqchip/sifive-plic: Mark two global variables __ro_after_init

2021-03-29 Thread Jisheng Zhang
From: Jisheng Zhang All of these two are never modified after init, so they can be __ro_after_init. Signed-off-by: Jisheng Zhang --- drivers/irqchip/irq-sifive-plic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-sifive-plic.c

[PATCH v4 5/9] media: hantro/cedrus: Remove unneeded slice size and slice offset

2021-03-29 Thread Ezequiel Garcia
The MPEG2_SLICE_PARAMS control is designed to refer to a single slice. However, the Hantro and Cedrus drivers operate in per-frame mode, and so does the current Ffmpeg and GStreamer implementations that are tested with these two drivers. In other words, the drivers are expecting all the slices in

[PATCH v4 3/9] media: uapi: mpeg2: Split sequence and picture parameters

2021-03-29 Thread Ezequiel Garcia
Typically, bitstreams are composed of a sequence header, followed by a number of picture header and picture coding extension headers. Each picture can be composed by a number of slices. Let's split the MPEG-2 uAPI to follow these semantics more closely, allowing more usage flexibility. Having

[PATCH v4 2/9] media: uapi: mpeg2: Cleanup flags

2021-03-29 Thread Ezequiel Garcia
Our current MPEG-2 uAPI uses 1-byte fields for MPEG-2 boolean syntax elements. Clean these by adding a 'flags' field and flag macro for each boolean syntax element. A follow-up change will refactor this uAPI so we don't need to add padding fields just yet. Signed-off-by: Ezequiel Garcia

[PATCH v4 1/9] media: uapi: mpeg2: Rework quantization matrices semantics

2021-03-29 Thread Ezequiel Garcia
As stated in the MPEG-2 specification, section 6.3.7 "Quant matrix extension": Each quantisation matrix has a default set of values. When a sequence_header_code is decoded all matrices shall be reset to their default values. User defined matrices may be downloaded and this can occur in a

[PATCH v4 0/9] MPEG-2 stateless API cleanup and destaging

2021-03-29 Thread Ezequiel Garcia
Hi everyone, Over the last few weeks Daniel Almeida and Nicolas Dufresne have been working on GStreamer v4l2codecs support for stateless MPEG-2 decoding. This allowed us to re-review the MPEG-2 specification and re-discuss the API with some additional insight. The API now looks quite cleaner,

Re: [PATCH 1/1] block: fix trivial typos in comments

2021-03-29 Thread Tom Saeger
On Fri, Mar 26, 2021 at 09:46:36AM -0600, Jens Axboe wrote: > On 3/26/21 9:45 AM, Tom Saeger wrote: > > On Fri, Mar 26, 2021 at 09:41:49AM -0600, Jens Axboe wrote: > >> On 3/25/21 9:04 PM, Tom Saeger wrote: > >>> > >>> s/Additonal/Additional/ > >>> s/assocaited/associated/ > >>>

[PATCH] selftests/bpf: add LDFLAGS when building test_verifier

2021-03-29 Thread Jisheng Zhang
From: Jisheng Zhang This is useful for cross compile process to point linker to the correct libelf, libcap, libz path. Signed-off-by: Jisheng Zhang --- tools/testing/selftests/bpf/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/bpf/Makefile

Re: [PATCH] drm/amdgpu: fix an underflow on non-4KB-page systems

2021-03-29 Thread Christian König
Am 29.03.21 um 20:08 schrieb Xi Ruoyao: On 2021-03-29 20:04 +0200, Christian König wrote: Am 29.03.21 um 19:53 schrieb Xℹ Ruoyao: If the initial value of `num_entires` (calculated at line 1654) is not an integral multiple of `AMDGPU_GPU_PAGES_IN_CPU_PAGE`, in line 1681 a value greater than the

[PATCH] samples/kprobes: Add riscv support

2021-03-29 Thread Jisheng Zhang
From: Jisheng Zhang Add riscv specific info dump in both handler_pre() and handler_post(). Signed-off-by: Jisheng Zhang --- samples/kprobes/kprobe_example.c | 8 1 file changed, 8 insertions(+) diff --git a/samples/kprobes/kprobe_example.c b/samples/kprobes/kprobe_example.c index

Re: [PATCH] drm/amdgpu: fix an underflow on non-4KB-page systems

2021-03-29 Thread Xi Ruoyao
On 2021-03-29 20:04 +0200, Christian König wrote: > Am 29.03.21 um 19:53 schrieb Xℹ Ruoyao: > > If the initial value of `num_entires` (calculated at line 1654) is not > > an integral multiple of `AMDGPU_GPU_PAGES_IN_CPU_PAGE`, in line 1681 a > > value greater than the initial value will be

Re: [PATCH v3 0/4] add device drivers for Siemens Industrial PCs

2021-03-29 Thread Henning Schild
Guys, sorry for the delay. This one did in fact not change too much after all. The biggest points that are still kind of open are the naming of the LEDs. If what is proposed here is acceptable it is not open from my side. The other big point was "using a generic gpio" driver as a basis. My

Re: Linux 5.12-rc5

2021-03-29 Thread Linus Torvalds
On Sun, Mar 28, 2021 at 7:07 PM Guenter Roeck wrote: > > This is not really a new problem. I enabled devicetree unit tests > in the openrisc kernel and was rewarded with a crash. > https://lore.kernel.org/lkml/20210327224116.69309-1-li...@roeck-us.net/ > has all the glorious details. Hmm. I'm

Re: [PATCH] drm/amdgpu: fix an underflow on non-4KB-page systems

2021-03-29 Thread Christian König
Am 29.03.21 um 19:53 schrieb Xℹ Ruoyao: If the initial value of `num_entires` (calculated at line 1654) is not an integral multiple of `AMDGPU_GPU_PAGES_IN_CPU_PAGE`, in line 1681 a value greater than the initial value will be assigned to it. That causes `start > last + 1` after line 1708.

Re: [PATCH v6 4/5] drm/bridge: anx7625: add HDCP support

2021-03-29 Thread Sean Paul
On Mon, Mar 29, 2021 at 6:27 AM Xin Ji wrote: > > On Thu, Mar 25, 2021 at 02:19:23PM -0400, Sean Paul wrote: > > On Fri, Mar 19, 2021 at 2:35 AM Xin Ji wrote: > > > > > > Add HDCP feature, enable HDCP function through chip internal key > > > and downstream's capability. > > > > > >

Re: [PATCH v6 4/7] pwm: pca9685: Support staggered output ON times

2021-03-29 Thread Uwe Kleine-König
On Mon, Mar 29, 2021 at 07:16:38PM +0200, Clemens Gruber wrote: > On Mon, Mar 29, 2021 at 07:03:57PM +0200, Uwe Kleine-König wrote: > > On Mon, Mar 29, 2021 at 02:57:04PM +0200, Clemens Gruber wrote: > > > The PCA9685 supports staggered LED output ON times to minimize current > > > surges and

Re: [PATCH 5.11 000/252] 5.11.11-rc2 review

2021-03-29 Thread Naresh Kamboju
On Mon, 29 Mar 2021 at 15:45, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 5.11.11 release. > There are 252 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > >

Re: mmotm 2021-03-28-16-37 uploaded (cifs)

2021-03-29 Thread Randy Dunlap
On 3/28/21 4:37 PM, a...@linux-foundation.org wrote: > The mm-of-the-moment snapshot 2021-03-28-16-37 has been uploaded to > >https://www.ozlabs.org/~akpm/mmotm/ > > mmotm-readme.txt says > > README for mm-of-the-moment: > > https://www.ozlabs.org/~akpm/mmotm/ > > This is a snapshot of my

[PATCH] drm/amdgpu: fix an underflow on non-4KB-page systems

2021-03-29 Thread Xℹ Ruoyao
If the initial value of `num_entires` (calculated at line 1654) is not an integral multiple of `AMDGPU_GPU_PAGES_IN_CPU_PAGE`, in line 1681 a value greater than the initial value will be assigned to it. That causes `start > last + 1` after line 1708. Then in the next iteration an underflow

Re: [PATCH] lan743x: remove redundant intializations of pointers adapter and phydev

2021-03-29 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Sun, 28 Mar 2021 22:46:47 +0100 you wrote: > From: Colin Ian King > > The pointers adapter and phydev are being initialized with values that > are never read and are being updated later with a new value. The >

Re: [syzbot] WARNING: still has locks held in io_sq_thread

2021-03-29 Thread syzbot
syzbot has bisected this issue to: commit c8cc7e853192d520ab6a5957f5081034103587ae Author: Peter Zijlstra Date: Tue Feb 9 08:30:03 2021 + lockdep: Noinstr annotate warn_bogus_irq_restore() bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=152deb3ad0 start commit:

[PATCH v3 3/4] watchdog: simatic-ipc-wdt: add new driver for Siemens Industrial PCs

2021-03-29 Thread Henning Schild
This driver adds initial support for several devices from Siemens. It is based on a platform driver introduced in an earlier commit. Signed-off-by: Henning Schild Reviewed-by: Guenter Roeck --- drivers/watchdog/Kconfig | 11 ++ drivers/watchdog/Makefile | 1 +

[PATCH v3 2/4] leds: simatic-ipc-leds: add new driver for Siemens Industial PCs

2021-03-29 Thread Henning Schild
This driver adds initial support for several devices from Siemens. It is based on a platform driver introduced in an earlier commit. Signed-off-by: Henning Schild --- drivers/leds/Kconfig | 3 + drivers/leds/Makefile | 3 + drivers/leds/simple/Kconfig

Re: [PATCH v5 00/27] Memory Folios

2021-03-29 Thread Matthew Wilcox
On Mon, Mar 29, 2021 at 05:58:32PM +0100, Matthew Wilcox wrote: > In broad strokes, I think that having a Power Of Two Allocator > with Descriptor (POTAD) is a useful foundational allocator to have. > The specific allocator that we call the buddy allocator is very clever for > the 1990s, but

Re: [PATCH 1/2] iommu/arm-smmu-qcom: Skip the TTBR1 quirk for db820c.

2021-03-29 Thread Eric Anholt
On Mon, Mar 29, 2021 at 7:47 AM Will Deacon wrote: > > On Fri, Mar 26, 2021 at 04:13:02PM -0700, Eric Anholt wrote: > > db820c wants to use the qcom smmu path to get HUPCF set (which keeps > > the GPU from wedging and then sometimes wedging the kernel after a > > page fault), but it doesn't have

[PATCH v3 0/4] add device drivers for Siemens Industrial PCs

2021-03-29 Thread Henning Schild
changes since v2: - remove "simatic-ipc" prefix from LED names - fix style issues found in v2, mainly LED driver - fix OEM specific dmi code, and remove magic numbers - more "simatic_ipc" name prefixing - improved pmc quirk code using callbacks changes since v1: - fixed lots of style issues

[PATCH] arm: Define arch_is_kernel_initmem_freed() for lockdep

2021-03-29 Thread Jan Kardell
This fixes the warning: WARNING: CPU: 0 PID: 1994 at kernel/locking/lockdep.c:1119 alloc_netdev_mqs+0xb4/0x3b0 This warning is because the check in static_obj() assumes that all memory within [_stext, _end] belongs to static objects. The init section is also part of this range, and freeing it

[PATCH 0/2] drm/ingenic: IPU plane fixes

2021-03-29 Thread Paul Cercueil
Hi, A set of two fixes for the IPU plane of the ingenic-drm driver. Patch [1/2] changes the type of the IPU plane from PRIMARY to OVERLAY, since there can only be one PRIMARY plane per CRTC. Patch [2/2] enforces that a full modeset is only performed when the "sharpness" property is modified.

[PATCH 1/2] drm/ingenic: Switch IPU plane to type OVERLAY

2021-03-29 Thread Paul Cercueil
It should have been an OVERLAY from the beginning. The documentation stipulates that there should be an unique PRIMARY plane per CRTC. Fixes: fc1acf317b01 ("drm/ingenic: Add support for the IPU") Cc: # 5.8+ Signed-off-by: Paul Cercueil --- drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 11

[PATCH 2/2] drm/ingenic: Don't request full modeset if property is not modified

2021-03-29 Thread Paul Cercueil
Avoid requesting a full modeset if the sharpness property is not modified, because then we don't actually need it. Fixes: fc1acf317b01 ("drm/ingenic: Add support for the IPU") Cc: # 5.8+ Signed-off-by: Paul Cercueil --- drivers/gpu/drm/ingenic/ingenic-ipu.c | 4 +++- 1 file changed, 3

[PATCH v3 1/4] platform/x86: simatic-ipc: add main driver for Siemens devices

2021-03-29 Thread Henning Schild
This mainly implements detection of these devices and will allow secondary drivers to work on such machines. The identification is DMI-based with a vendor specific way to tell them apart in a reliable way. Drivers for LEDs and Watchdogs will follow to make use of that platform detection.

[PATCH v3 4/4] platform/x86: pmc_atom: improve critclk_systems matching for Siemens PCs

2021-03-29 Thread Henning Schild
Siemens industrial PCs unfortunately can not always be properly identified the way we used to. An earlier commit introduced code that allows proper identification without looking at DMI strings that could differ based on product branding. Switch over to that proper way and revert commits that used

Re: [PATCH v6 6/7] pwm: pca9685: Restrict period change for prescaler users

2021-03-29 Thread Uwe Kleine-König
Hello Clemens, On Mon, Mar 29, 2021 at 07:33:56PM +0200, Clemens Gruber wrote: > On Mon, Mar 29, 2021 at 07:15:59PM +0200, Uwe Kleine-König wrote: > > On Mon, Mar 29, 2021 at 02:57:06PM +0200, Clemens Gruber wrote: > > > @@ -330,14 +345,22 @@ static int pca9685_pwm_apply(struct pwm_chip *chip, >

Re: [PATCH 5.10 000/219] 5.10.27-rc2 review

2021-03-29 Thread Florian Fainelli
On 3/29/21 3:14 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.10.27 release. > There are 219 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses

Re: [PATCH] hwmon: Add driver for fsp-3y PSUs and PDUs

2021-03-29 Thread Guenter Roeck
On 3/29/21 7:38 AM, Václav Kubernát wrote: > After some testing, I have found out there is a timing issue with this > device. After setting page, the device doesn't immediately react and > gives values from the previous page for some time. This is why there > needs to be a delay between

Re: [PATCH 2/2] gpiolib: Allow drivers to return EOPNOTSUPP from config

2021-03-29 Thread Jakub Kicinski
On Mon, 29 Mar 2021 18:25:46 +0300 Andy Shevchenko wrote: > On Mon, Mar 29, 2021 at 08:08:52AM -0700, Joe Perches wrote: > > On Mon, 2021-03-29 at 14:59 +0300, Andy Shevchenko wrote: > > > On Mon, Mar 29, 2021 at 2:43 PM Matti Vaittinen > > > wrote: > > > > > > > > The checkpacth instructs

Re: I915 CI-run with kfence enabled, issues found

2021-03-29 Thread Marco Elver
On Mon, 29 Mar 2021 at 19:32, Dave Hansen wrote: > > On 3/29/21 9:40 AM, Marco Elver wrote: > > It looks like the code path from flush_tlb_one_kernel() to > > invalidate_user_asid()'s this_cpu_ptr() has several feature checks, so > > probably some feature difference between systems where it

Re: Mutual debugging of 2 processes can stuck in unkillable stopped state

2021-03-29 Thread Igor Zhbanov
Hi Oleg, On 29.03.2021 20:38, Oleg Nesterov wrote: Hi Igor, So. As expected, they sleep in EVENT_EXIT _after_ you have already sent SIGKILL. Here is the processes stack before sending any signals to them: izh@suse2:~> sudo cat /proc/1751/stack [<0>] ptrace_stop+0x14e/0x260 [<0>]

Re: [PATCH v4 14/22] x86/fpu/xstate: Expand the xstate buffer on the first use of dynamic user state

2021-03-29 Thread Andy Lutomirski
> On Mar 29, 2021, at 9:06 AM, Len Brown wrote: > > On Mon, Mar 29, 2021 at 11:43 AM Len Brown wrote: >> >> On Mon, Mar 29, 2021 at 9:33 AM Thomas Gleixner wrote: >> I found the author of this passage, and he agreed to revise it to say this was targeted primarily at VMMs. >>>

Re: [PATCH v6 2/7] pwm: pca9685: Support hardware readout

2021-03-29 Thread Uwe Kleine-König
On Mon, Mar 29, 2021 at 07:11:53PM +0200, Clemens Gruber wrote: > On Mon, Mar 29, 2021 at 06:54:29PM +0200, Uwe Kleine-König wrote: > > On Mon, Mar 29, 2021 at 02:57:02PM +0200, Clemens Gruber wrote: > > > [...] > > > + /* Calculate (chip-wide) period from prescale value */ > > > +

Re: [PATCH][next] mm/vmalloc: Fix read of pointer area after it has been free'd

2021-03-29 Thread Uladzislau Rezki
On Mon, Mar 29, 2021 at 06:14:34PM +0100, Matthew Wilcox wrote: > On Mon, Mar 29, 2021 at 06:07:30PM +0100, Colin King wrote: > > From: Colin Ian King > > > > Currently the memory pointed to by area is being freed by the > > free_vm_area call and then area->nr_pages is referencing the > > free'd

Re: Mutual debugging of 2 processes can stuck in unkillable stopped state

2021-03-29 Thread Oleg Nesterov
Hi Igor, So. As expected, they sleep in EVENT_EXIT _after_ you have already sent SIGKILL. Oh. I can only repeat that PTRACE_EVENT_EXIT must die ;) Or at least we should finally define its semantics. Igor, thanks for your report, but (I think) this has nothing to do with mutual debugging. I'll

Re: [PATCH] ACPI: processor: Fix CPU0 wakeup in acpi_idle_play_dead()

2021-03-29 Thread Rafael J. Wysocki
On Wed, Mar 24, 2021 at 4:37 PM Rafael J. Wysocki wrote: > > On Wed, Mar 24, 2021 at 4:23 PM Vitaly Kuznetsov wrote: > > > > Commit 496121c02127 ("ACPI: processor: idle: Allow probing on platforms > > with one ACPI C-state") broke CPU0 hotplug on certain systems, e.g. > > I'm observing the

Re: [PATCH 5.4 000/111] 5.4.109-rc1 review

2021-03-29 Thread Florian Fainelli
On 3/29/21 12:57 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.4.109 release. > There are 111 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses

Re: [syzbot] KASAN: null-ptr-deref Read in filp_close (2)

2021-03-29 Thread Christian Brauner
On Mon, Mar 29, 2021 at 11:21:34AM +0200, Christian Brauner wrote: > On Sat, Mar 27, 2021 at 11:33:37PM +, Al Viro wrote: > > On Fri, Mar 26, 2021 at 02:50:11PM +0100, Christian Brauner wrote: > > > @@ -632,6 +632,7 @@ EXPORT_SYMBOL(close_fd); /* for ksys_close() */ > > > static inline void

Re: [PATCH v6 6/7] pwm: pca9685: Restrict period change for prescaler users

2021-03-29 Thread Clemens Gruber
On Mon, Mar 29, 2021 at 07:15:59PM +0200, Uwe Kleine-König wrote: > On Mon, Mar 29, 2021 at 02:57:06PM +0200, Clemens Gruber wrote: > > @@ -330,14 +345,22 @@ static int pca9685_pwm_apply(struct pwm_chip *chip, > > struct pwm_device *pwm, > > > > if (!state->enabled || duty < 1) { > >

Re: [PATCH] drm/amd/display: dual cursors are seen if scaling is enabled

2021-03-29 Thread Harry Wentland
On 2021-03-29 3:54 a.m., Louis Li wrote: [Why] This issue is found when scaling is not equal to one from src to dest. When issue happens, there are offsets in both axis x and y between two cursors. Users cannot control APP under such a condition. What's the use case? I don't think we support

Re: I915 CI-run with kfence enabled, issues found

2021-03-29 Thread Dave Hansen
On 3/29/21 9:40 AM, Marco Elver wrote: > It looks like the code path from flush_tlb_one_kernel() to > invalidate_user_asid()'s this_cpu_ptr() has several feature checks, so > probably some feature difference between systems where it triggers and > it doesn't. > > As far as I'm aware, there is no

Re: [PATCH v4 2/2] gpio: Add Realtek Otto GPIO support

2021-03-29 Thread Sander Vanheule
Hi Andy, Thank you for clarifying your remarks. I'll support for building as a module, and have implemented the gpio_irq_chip->init_hw() callback. On Mon, 2021-03-29 at 13:26 +0300, Andy Shevchenko wrote: > On Fri, Mar 26, 2021 at 11:11 PM Sander Vanheule < > san...@svanheule.net> wrote: > > On

Re: [PATCH rfc 2/4] percpu: split __pcpu_balance_workfn()

2021-03-29 Thread Dennis Zhou
On Wed, Mar 24, 2021 at 12:06:24PM -0700, Roman Gushchin wrote: > __pcpu_balance_workfn() became fairly big and hard to follow, but in > fact it consists of two fully independent parts, responsible for > the destruction of excessive free chunks and population of necessarily > amount of free pages.

Re: [PATCH] arm64: dts: mt8173: fix wrong power-domain phandle of pmic

2021-03-29 Thread Matthias Brugger
On 18/03/2021 07:18, Chunfeng Yun wrote: > Due to power domain controller is added, the power domain's > phanle is also changed from 'scpsys' to 'spm', but forget to > modify pmic node's > > Fixes: 8b6562644df9 ("arm64: dts: mediatek: Add mt8173 power domain > controller") > Signed-off-by:

Re: [PATCH rfc 1/4] percpu: implement partial chunk depopulation

2021-03-29 Thread Dennis Zhou
On Wed, Mar 24, 2021 at 12:06:23PM -0700, Roman Gushchin wrote: > This patch implements partial depopulation of percpu chunks. > > As now, a chunk can be depopulated only as a part of the final > destruction, when there are no more outstanding allocations. However > to minimize a memory waste, it

Re: [PATCH -next] ASoC: amd: acp-da7219-max98357a: Fix -Wunused-variable warning

2021-03-29 Thread Mukunda,Vijendar
On 3/29/21 8:20 PM, YueHaibing wrote: While ACPI is not set, make W=1 warns: sound/soc/amd/acp-da7219-max98357a.c:684:28: warning: ‘cz_rt5682_card’ defined but not used [-Wunused-variable] static struct snd_soc_card cz_rt5682_card = { ^~

Re: [PATCH] amd: display: dc: struct dc_state is declared twice

2021-03-29 Thread Alex Deucher
On Sat, Mar 27, 2021 at 3:28 AM Wan Jiabing wrote: > > struct dc_state has been declared at 273rd line. > Remove the duplicate. > Delete duplicate blank lines. Can you split these into separate patches? Alex > > Signed-off-by: Wan Jiabing > --- > drivers/gpu/drm/amd/display/dc/dc.h | 10

Re: [PATCH v6 4/7] pwm: pca9685: Support staggered output ON times

2021-03-29 Thread Clemens Gruber
On Mon, Mar 29, 2021 at 07:03:57PM +0200, Uwe Kleine-König wrote: > On Mon, Mar 29, 2021 at 02:57:04PM +0200, Clemens Gruber wrote: > > The PCA9685 supports staggered LED output ON times to minimize current > > surges and reduce EMI. > > When this new option is enabled, the ON times of each

Re: [PATCH v3 2/6] PCI: brcmstb: Add control of EP voltage regulators

2021-03-29 Thread Mark Brown
On Mon, Mar 29, 2021 at 12:39:50PM -0400, Jim Quinlan wrote: > On Mon, Mar 29, 2021 at 12:25 PM Mark Brown > > Here you are figuring out a device local supply name... > > > + /* > > > + * Get the regulators that the EP devianswerces require. We cannot > > > use > > > + *

Re: [PATCH v6 6/7] pwm: pca9685: Restrict period change for prescaler users

2021-03-29 Thread Uwe Kleine-König
On Mon, Mar 29, 2021 at 02:57:06PM +0200, Clemens Gruber wrote: > @@ -330,14 +345,22 @@ static int pca9685_pwm_apply(struct pwm_chip *chip, > struct pwm_device *pwm, > > if (!state->enabled || duty < 1) { > pca9685_pwm_set_duty(pca, pwm->hwpwm, 0); > +

Re: [PATCH][next] mm/vmalloc: Fix read of pointer area after it has been free'd

2021-03-29 Thread Matthew Wilcox
On Mon, Mar 29, 2021 at 06:07:30PM +0100, Colin King wrote: > From: Colin Ian King > > Currently the memory pointed to by area is being freed by the > free_vm_area call and then area->nr_pages is referencing the > free'd object. Fix this swapping the order of the warn_alloc > message and the

Re: [PATCH] KVM: X86: Properly account for guest CPU time when considering context tracking

2021-03-29 Thread Sean Christopherson
+Thomas On Mon, Mar 29, 2021, Wanpeng Li wrote: > From: Wanpeng Li > > The bugzilla https://bugzilla.kernel.org/show_bug.cgi?id=209831 > reported that the guest time remains 0 when running a while true > loop in the guest. > > The commit 87fa7f3e98a131 ("x86/kvm: Move context tracking where

Re: [PATCH v2 1/1] x86/tdx: Handle MWAIT, MONITOR and WBINVD

2021-03-29 Thread Dave Hansen
On 3/27/21 3:54 PM, Kuppuswamy Sathyanarayanan wrote: > + /* > + * Per Guest-Host-Communication Interface (GHCI) for Intel Trust > + * Domain Extensions (Intel TDX) specification, sec 2.4, > + * some instructions that unconditionally cause #VE (such as WBINVD, > + *

Re: [PATCH v4 2/2] power: supply: mt6360_charger: add MT6360 charger support

2021-03-29 Thread Matthias Brugger
On 18/01/2021 13:41, Gene Chen wrote: > From: Gene Chen > > Add basic support for the battery charger for MT6360 PMIC > > Signed-off-by: Gene Chen Sebastian, can you have a look on that patch please? Regards, Matthias > --- > drivers/power/supply/Kconfig | 10 + >

Re: [PATCH V4 XRT Alveo 03/20] fpga: xrt: xclbin file helper functions

2021-03-29 Thread Tom Rix
On 3/23/21 10:29 PM, Lizhi Hou wrote: > Alveo FPGA firmware and partial reconfigure file are in xclbin format. This > code enumerates and extracts sections from xclbin files. xclbin.h is cross > platform and used across all platforms and OS. ok > > Signed-off-by: Sonal Santan > Signed-off-by:

Re: [v7,2/3] arm64: dts: mt8183: Configure CPU cooling

2021-03-29 Thread Matthias Brugger
On 16/03/2021 08:01, Michael Kao wrote: > From: Matthias Kaehlcke > > Add two passive trip points at 68°C and 80°C for the CPU temperature. > > Signed-off-by: Matthias Kaehlcke > Signed-off-by: Michael Kao > Tested-by: Hsin-Yi Wang Applied to v5.12-next/dts64 Thanks. Matthias > --- >

Re: [PATCH v6 2/7] pwm: pca9685: Support hardware readout

2021-03-29 Thread Clemens Gruber
On Mon, Mar 29, 2021 at 06:54:29PM +0200, Uwe Kleine-König wrote: > On Mon, Mar 29, 2021 at 02:57:02PM +0200, Clemens Gruber wrote: > > Implements .get_state to read-out the current hardware state. > > > > The hardware readout may return slightly different values than those > > that were set in

Re: Mutual debugging of 2 processes can stuck in unkillable stopped state

2021-03-29 Thread Igor Zhbanov
Hi Oleg! I've tried both 5.3.18 and 5.10.0. The behavior is the same. The important thing is to run "exec strace -p ..." on the second terminal to create the loop A->B->A. So the last line from the first strace we see is: ptrace(PTRACE_SEIZE, 1990, NULL,

Re: [v7,1/3] arm64: dts: mt8183: add thermal zone node

2021-03-29 Thread Matthias Brugger
On 22/03/2021 12:20, Hsin-Yi Wang wrote: > On Tue, Mar 16, 2021 at 3:02 PM Michael Kao wrote: >> >> From: "michael.kao" >> >> Add thermal zone node to Mediatek MT8183 dts file. >> >> Evaluate the thermal zone every 500ms while not cooling >> and every 100ms when passive cooling is performed.

[PATCH][next] mm/vmalloc: Fix read of pointer area after it has been free'd

2021-03-29 Thread Colin King
From: Colin Ian King Currently the memory pointed to by area is being freed by the free_vm_area call and then area->nr_pages is referencing the free'd object. Fix this swapping the order of the warn_alloc message and the free. Addresses-Coverity: ("Read from pointer after free") Fixes:

Re: [PATCH] lib: fix incorrect kernel-doc comment syntax in file

2021-03-29 Thread Randy Dunlap
On 3/29/21 5:10 AM, Aditya Srivastava wrote: > The opening comment mark '/**' is used for highlighting the beginning of > kernel-doc comments. > There are certain files in lib and lib/zstd, which follow this syntax, > but the content inside does not comply with kernel-doc. > Such lines were

Re: [PATCH] crypto: chelsio: fix incorrect kernel-doc comment syntax in file

2021-03-29 Thread Randy Dunlap
On 3/29/21 3:45 AM, Aditya Srivastava wrote: > The opening comment mark '/**' is used for highlighting the beginning of > kernel-doc comments. > The header for drivers/crypto/chelsio/chcr_core.c follows this syntax, but > the content inside does not comply with kernel-doc. > > This line was

RE: [PATCH next 2/2] misc: Add Renesas Synchronization Management Unit (SMU) support

2021-03-29 Thread Min Li
> > Where is patch 1/2 of this series? > > Also, please fix up the errors that the testing bot found, and properly > version > your patch submission so I know which one is the "latest" one to look at. > Hi Greg The first patch is mfd so I was not sure if I should send that to you guys in

Re: [PATCH v6 4/7] pwm: pca9685: Support staggered output ON times

2021-03-29 Thread Uwe Kleine-König
On Mon, Mar 29, 2021 at 02:57:04PM +0200, Clemens Gruber wrote: > The PCA9685 supports staggered LED output ON times to minimize current > surges and reduce EMI. > When this new option is enabled, the ON times of each channel are > delayed by channel number x counter range / 16, which avoids

Re: [PATCH] usb: dwc3: imx8mp: fix incorrect kernel-doc comment syntax

2021-03-29 Thread Randy Dunlap
On 3/29/21 7:26 AM, Aditya Srivastava wrote: > The opening comment mark '/**' is used for highlighting the beginning of > kernel-doc comments. > The header for drivers/usb/dwc3/dwc3-imx8mp.c follows this syntax, but the > content inside does not comply with kernel-doc. > > This line was probably

[PATCH next 1/2] mfd: Add Renesas Synchronization Management Unit (SMU) support

2021-03-29 Thread min.li.xe
From: Min Li Add support for ClockMatrix(TM) and 82P33xxx families of timing and synchronization devices. The access interface can be either SPI or I2C. Currently, it will create 2 types of MFD devices, which are to be used by the corresponding rsmu character device driver and the PTP hardware

Re: [PATCH v5 2/2] PCI: brcmstb: Use reset/rearm instead of deassert/assert

2021-03-29 Thread Florian Fainelli
On 3/29/21 9:58 AM, Lorenzo Pieralisi wrote: > On Mon, Mar 29, 2021 at 09:50:13AM -0700, Florian Fainelli wrote: >> On 3/29/21 9:10 AM, Lorenzo Pieralisi wrote: >>> On Fri, Mar 12, 2021 at 03:45:55PM -0500, Jim Quinlan wrote: The Broadcom STB PCIe RC uses a reset control "rescal" for certain

Re: [PATCH v5 00/27] Memory Folios

2021-03-29 Thread Matthew Wilcox
I'm going to respond to some points in detail below, but there are a couple of overarching themes that I want to bring out up here. Grand Vision I haven't outlined my long-term plan. Partly because it is a _very_ long way off, and partly because I think what I'm doing stands on its

Re: [PATCH v5 2/2] PCI: brcmstb: Use reset/rearm instead of deassert/assert

2021-03-29 Thread Lorenzo Pieralisi
On Mon, Mar 29, 2021 at 09:50:13AM -0700, Florian Fainelli wrote: > On 3/29/21 9:10 AM, Lorenzo Pieralisi wrote: > > On Fri, Mar 12, 2021 at 03:45:55PM -0500, Jim Quinlan wrote: > >> The Broadcom STB PCIe RC uses a reset control "rescal" for certain chips. > >> The "rescal" implements a "pulse

Re: [PATCH] PCI: Try to find two continuous regions for child resource

2021-03-29 Thread kernel test robot
Hi Kai-Heng, Thank you for the patch! Yet something to improve: [auto build test ERROR on pci/next] [also build test ERROR on powerpc/next sparc/master linus/master sparc-next/master v5.12-rc5] [cannot apply to next-20210329] [If your patch is applied to the wrong git tree, kindly drop us

Re: [PATCH v5 04/19] perf: aux: Add CoreSight PMU buffer formats

2021-03-29 Thread Mathieu Poirier
Hi Peter, On Tue, Mar 23, 2021 at 12:06:32PM +, Suzuki K Poulose wrote: > CoreSight PMU supports aux-buffer for the ETM tracing. The trace > generated by the ETM (associated with individual CPUs, like Intel PT) > is captured by a separate IP (CoreSight TMC-ETR/ETF until now). > > The TMC-ETR

Re: [PATCH v6 2/7] pwm: pca9685: Support hardware readout

2021-03-29 Thread Uwe Kleine-König
On Mon, Mar 29, 2021 at 02:57:02PM +0200, Clemens Gruber wrote: > Implements .get_state to read-out the current hardware state. > > The hardware readout may return slightly different values than those > that were set in apply due to the limited range of possible prescale and > counter register

[PATCH v3] lib: fix kconfig dependency on ARCH_WANT_FRAME_POINTERS

2021-03-29 Thread Julian Braha
When LATENCYTOP, LOCKDEP, or FAULT_INJECTION_STACKTRACE_FILTER is enabled and ARCH_WANT_FRAME_POINTERS is disabled, Kbuild gives a warning such as: WARNING: unmet direct dependencies detected for FRAME_POINTER Depends on [n]: DEBUG_KERNEL [=y] && (M68K || UML || SUPERH) ||

[PATCH v3 4/4] clk: ti: add am33xx spread spectrum clock support

2021-03-29 Thread Dario Binacchi
The patch enables spread spectrum clocking (SSC) for MPU and LCD PLLs. As reported by the TI spruh73x RM, SSC is only supported for the DISP/LCD and MPU PLLs on am33xx device. SSC is not supported for DDR, PER, and CORE PLLs. Calculating the required values and setting the registers accordingly

Re: [PATCH v5 2/2] PCI: brcmstb: Use reset/rearm instead of deassert/assert

2021-03-29 Thread Florian Fainelli
On 3/29/21 9:10 AM, Lorenzo Pieralisi wrote: > On Fri, Mar 12, 2021 at 03:45:55PM -0500, Jim Quinlan wrote: >> The Broadcom STB PCIe RC uses a reset control "rescal" for certain chips. >> The "rescal" implements a "pulse reset" so using assert/deassert is wrong >> for this device. Instead, we use

Re: Mutual debugging of 2 processes can stuck in unkillable stopped state

2021-03-29 Thread Oleg Nesterov
On 03/29, Igor Zhbanov wrote: > > Mutual debugging of 2 processes can stuck in unkillable stopped state can't reproduce and can't understand... > Hi! > > When one process, let's say "A", is tracing the another process "B", and the > process "B" is trying to attach to the process "A", then both

[PATCH v1] ovl: Fix leaked dentry

2021-03-29 Thread Mickaël Salaün
From: Mickaël Salaün Since commit 6815f479ca90 ("ovl: use only uppermetacopy state in ovl_lookup()"), overlayfs doesn't put temporary dentry when there is a metacopy error, which leads to dentry leaks when shutting down the related superblock: overlayfs: refusing to follow metacopy origin for

Re: Candidate Linux ABI for Intel AMX and hypothetical new related features

2021-03-29 Thread Florian Weimer
* Len Brown via Libc-alpha: >> In particular, the library may use instructions that main() doesn't know >> exist. > > And so I'll ask my question another way. > > How is it okay to change the value of XCR0 during the run time of a > program? > > I submit that it is not, and that is a deal-killer

[PATCH v3 2/4] dt-bindings: ti: dpll: add spread spectrum support

2021-03-29 Thread Dario Binacchi
DT bindings for enabling and adjusting spread spectrum clocking have been added. Signed-off-by: Dario Binacchi --- Changes in v3: - Add '-hz' suffix to "ti,ssc-modfreq" binding. .../devicetree/bindings/clock/ti/dpll.txt | 20 +++ 1 file changed, 20 insertions(+) diff

[PATCH v3 3/4] ARM: dts: am33xx-clocks: add spread spectrum support

2021-03-29 Thread Dario Binacchi
Registers for adjusting the spread spectrum clocking (SSC) have been added. As reported by the TI spruh73x RM, SSC is supported only for LCD and MPU PLLs. Signed-off-by: Dario Binacchi Acked-by: Tony Lindgren --- Changes in v3: - Add Tony Lindgren acked tag. Changes in v2: - Remove SSC

Re: [PATCH] PCI: Allow drivers to claim exclusive access to config regions

2021-03-29 Thread Dan Williams
On Fri, Mar 26, 2021 at 9:12 AM Bjorn Helgaas wrote: > > [+cc Christoph] > > On Wed, Mar 24, 2021 at 06:23:54PM -0700, Dan Williams wrote: > > The PCIE Data Object Exchange (DOE) mailbox is a protocol run over > > configuration cycles. It assumes one initiator at a time is > > reading/writing the

Re: [RFC PATCH v1 1/1] arm64: Implement stack trace termination record

2021-03-29 Thread Madhavan T. Venkataraman
On 3/29/21 6:27 AM, Mark Rutland wrote: > Hi Madhavan, > > Overall this looks pretty good; I have a few comments below. > > On Wed, Mar 24, 2021 at 01:46:07PM -0500, madve...@linux.microsoft.com wrote: >> From: "Madhavan T. Venkataraman" >> >> The unwinder needs to be able to reliably tell

<    3   4   5   6   7   8   9   10   11   12   >