[PATCH v10 01/14] mm/gup: don't pin migrated cma pages in movable zone

2021-02-11 Thread Pavel Tatashin
In order not to fragment CMA the pinned pages are migrated. However, they are migrated to ZONE_MOVABLE, which also should not have pinned pages. Remove __GFP_MOVABLE, so pages can be migrated to zones where pinning is allowed. Signed-off-by: Pavel Tatashin Reviewed-by: David Hildenbrand Reviewe

[PATCH v10 03/14] mm/gup: return an error on migration failure

2021-02-11 Thread Pavel Tatashin
When migration failure occurs, we still pin pages, which means that we may pin CMA movable pages which should never be the case. Instead return an error without pinning pages when migration failure happens. No need to retry migrating, because migrate_pages() already retries 10 times. Signed-off-

[PATCH v10 04/14] mm/gup: check for isolation errors

2021-02-11 Thread Pavel Tatashin
It is still possible that we pin movable CMA pages if there are isolation errors and cma_page_list stays empty when we check again. Check for isolation errors, and return success only when there are no isolation errors, and cma_page_list is empty after checking. Because isolation errors are trans

Re: [PATCH 00/49] Add Alder Lake support for perf

2021-02-11 Thread Liang, Kan
On 2/11/2021 6:40 AM, Jiri Olsa wrote: On Mon, Feb 08, 2021 at 07:24:57AM -0800, kan.li...@linux.intel.com wrote: SNIP Jin Yao (24): perf jevents: Support unit value "cpu_core" and "cpu_atom" perf util: Save pmu name to struct perf_pmu_alias perf pmu: Save detected hybrid pmus to a

[PATCH v10 02/14] mm/gup: check every subpage of a compound page during isolation

2021-02-11 Thread Pavel Tatashin
When pages are isolated in check_and_migrate_movable_pages() we skip compound number of pages at a time. However, as Jason noted, it is not necessary correct that pages[i] corresponds to the pages that we skipped. This is because it is possible that the addresses in this range had split_huge_pmd()/

[PATCH v10 00/14] prohibit pinning pages in ZONE_MOVABLE

2021-02-11 Thread Pavel Tatashin
Changelog - v10 - Fixed !CONFIG_MMU compiler issues by adding is_zero_pfn() stub. v9 - Renamed gpf_to_alloc_flags() to gfp_to_alloc_flags_cma(); thanks Lecopzer Chen for noticing. - Fixed warning reported scripts/checkpatch.pl: "Logical continuations should be on the previous line" v8

[PATCH v10 05/14] mm cma: rename PF_MEMALLOC_NOCMA to PF_MEMALLOC_PIN

2021-02-11 Thread Pavel Tatashin
PF_MEMALLOC_NOCMA is used ot guarantee that the allocator will not return pages that might belong to CMA region. This is currently used for long term gup to make sure that such pins are not going to be done on any CMA pages. When PF_MEMALLOC_NOCMA has been introduced we haven't realized that it is

[PATCH v10 11/14] mm/gup: change index type to long as it counts pages

2021-02-11 Thread Pavel Tatashin
In __get_user_pages_locked() i counts number of pages which should be long, as long is used in all other places to contain number of pages, and 32-bit becomes increasingly small for handling page count proportional values. Signed-off-by: Pavel Tatashin Acked-by: Michal Hocko --- mm/gup.c | 2 +-

Re: [PATCH 1/2] spi: spi-mem: add spi_mem_dtr_supports_op()

2021-02-11 Thread Mark Brown
On Thu, 4 Feb 2021 19:42:17 +0530, Pratyush Yadav wrote: > spi_mem_default_supports_op() rejects DTR ops by default to ensure that > the controller drivers that haven't been updated with DTR support > continue to reject them. It also makes sure that controllers that don't > support DTR mode at all

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

2021-02-11 Thread Stefano Garzarella
Commit 65b709586e222fa6ffd4166ac7fdb5d5dad113ee upstream. Before this patch, if 'offset + len' was equal to sizeof(struct virtio_net_config), the entire buffer wasn't filled, returning incorrect values to the caller. Since 'vdpasim->config' type is 'struct virtio_net_config', we can safely copy i

[PATCH v10 10/14] memory-hotplug.rst: add a note about ZONE_MOVABLE and page pinning

2021-02-11 Thread Pavel Tatashin
Document the special handling of page pinning when ZONE_MOVABLE present. Signed-off-by: Pavel Tatashin Suggested-by: David Hildenbrand Acked-by: Michal Hocko --- Documentation/admin-guide/mm/memory-hotplug.rst | 9 + 1 file changed, 9 insertions(+) diff --git a/Documentation/admin-gui

[PATCH v10 07/14] mm: honor PF_MEMALLOC_PIN for all movable pages

2021-02-11 Thread Pavel Tatashin
PF_MEMALLOC_PIN is only honored for CMA pages, extend this flag to work for any allocations from ZONE_MOVABLE by removing __GFP_MOVABLE from gfp_mask when this flag is passed in the current context. Add is_pinnable_page() to return true if page is in a pinnable page. A pinnable page is not in ZONE

[PATCH v10 09/14] mm/gup: migrate pinned pages out of movable zone

2021-02-11 Thread Pavel Tatashin
We should not pin pages in ZONE_MOVABLE. Currently, we do not pin only movable CMA pages. Generalize the function that migrates CMA pages to migrate all movable pages. Use is_pinnable_page() to check which pages need to be migrated Signed-off-by: Pavel Tatashin Reviewed-by: John Hubbard --- inc

[PATCH v10 06/14] mm: apply per-task gfp constraints in fast path

2021-02-11 Thread Pavel Tatashin
Function current_gfp_context() is called after fast path. However, soon we will add more constraints which will also limit zones based on context. Move this call into fast path, and apply the correct constraints for all allocations. Also update .reclaim_idx based on value returned by current_gfp_c

[PATCH v10 08/14] mm/gup: do not migrate zero page

2021-02-11 Thread Pavel Tatashin
On some platforms ZERO_PAGE(0) might end-up in a movable zone. Do not migrate zero page in gup during longterm pinning as migration of zero page is not allowed. For example, in x86 QEMU with 16G of memory and kernelcore=5G parameter, I see the following: Boot#1: zero_pfn 0x48a8d zero_pfn zone: Z

[PATCH] Add the latency-collector to tools

2021-02-11 Thread Viktor Rosendahl
Thanks Steve for the long and detailed answer. On Tue, 2021-01-26 at 14:26 -0500, Steven Rostedt wrote: > > > +#define verbose_sizechange() (verbosity >= 1) > > +#define verbose_lostevent() (verbosity >= 2) > > + > > +static const char *debug_tracefile; > > +static const char *debug_tracefile_df

[PATCH v10 12/14] mm/gup: longterm pin migration cleanup

2021-02-11 Thread Pavel Tatashin
When pages are longterm pinned, we must migrated them out of movable zone. The function that migrates them has a hidden loop with goto. The loop is to retry on isolation failures, and after successful migration. Make this code better by moving this loop to the caller. Signed-off-by: Pavel Tatashi

[PATCH v10 13/14] selftests/vm: gup_test: fix test flag

2021-02-11 Thread Pavel Tatashin
In gup_test both gup_flags and test_flags use the same flags field. This is broken. Farther, in the actual gup_test.c all the passed gup_flags are erased and unconditionally replaced with FOLL_WRITE. Which means that test_flags are ignored, and code like this always performs pin dump test: 155

[PATCH v10 14/14] selftests/vm: gup_test: test faulting in kernel, and verify pinnable pages

2021-02-11 Thread Pavel Tatashin
When pages are pinned they can be faulted in userland and migrated, and they can be faulted right in kernel without migration. In either case, the pinned pages must end-up being pinnable (not movable). Add a new test to gup_test, to help verify that the gup/pup (get_user_pages() / pin_user_pages(

Re: [PATCH ghak124 v3] audit: log nftables configuration change events

2021-02-11 Thread Paul Moore
On Thu, Feb 11, 2021 at 10:16 AM Phil Sutter wrote: > Hi, > > On Thu, Jun 04, 2020 at 09:20:49AM -0400, Richard Guy Briggs wrote: > > iptables, ip6tables, arptables and ebtables table registration, > > replacement and unregistration configuration events are logged for the > > native (legacy) iptab

Re: [PATCH 8/9] vfio/pci: use x86 naming instead of igd

2021-02-11 Thread Matthew Rosato
On 2/11/21 10:47 AM, Max Gurtovoy wrote: On 2/2/2021 7:10 PM, Jason Gunthorpe wrote: On Tue, Feb 02, 2021 at 05:06:59PM +0100, Cornelia Huck wrote: On the other side, we have the zdev support, which both requires s390 and applies to any pci device on s390. Is there a reason why CONFIG_VFIO_P

Re: [PATCH iproute2-next V4] devlink: add support for port params get/set

2021-02-11 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to iproute2/iproute2-next.git (refs/heads/main): On Tue, 9 Feb 2021 12:31:51 +0200 you wrote: > Add implementation for the port parameters > getting/setting. > Add bash completion for port param. > Add man description for port param. > > Signed-off-by: Oleksandr Ma

Re: [PATCH v6 2/3] v4l: ioctl: Use %p4cc printk modifier to print FourCC codes

2021-02-11 Thread Petr Mladek
On Mon 2021-02-08 22:09:02, Sakari Ailus wrote: > Now that we can print FourCC codes directly using printk, make use of the > feature in V4L2 core. > > Signed-off-by: Sakari Ailus > --- > drivers/media/v4l2-core/v4l2-ioctl.c | 85 +++- > 1 file changed, 21 insertions(+),

Re: [PATCH] platform/surface: Add platform profile driver

2021-02-11 Thread Hans de Goede
Hi, On 2/11/21 5:17 PM, Maximilian Luz wrote: > > > On 2/11/21 4:56 PM, Hans de Goede wrote: >> Hi, >> >> On 2/8/21 10:38 PM, Maximilian Luz wrote: >>> >>> >>> On 2/8/21 9:27 PM, Hans de Goede wrote: >> >> >> > +static int convert_ssam_to_profile(struct ssam_device *sdev, enum > ssam_t

Re: [PATCH] platform/surface: Add platform profile driver

2021-02-11 Thread Maximilian Luz
On 2/11/21 5:31 PM, Hans de Goede wrote: Hi, On 2/11/21 5:17 PM, Maximilian Luz wrote: On 2/11/21 4:56 PM, Hans de Goede wrote: Hi, On 2/8/21 10:38 PM, Maximilian Luz wrote: On 2/8/21 9:27 PM, Hans de Goede wrote: +static int convert_ssam_to_profile(struct ssam_device *sdev, enum

Re: [PATCH 13/24] perf daemon: Allow only one daemon over base directory

2021-02-11 Thread Jiri Olsa
On Thu, Feb 11, 2021 at 10:20:18AM -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Feb 08, 2021 at 09:08:57PM +0100, Jiri Olsa escreveu: > > Add 'lock' file under daemon base and flock it, so only one > > perf daemon can run on top of it. > > > > Each daemon tries to create and lock BASE/lock fil

Re: [PATCH 05/24] perf daemon: Add client socket support

2021-02-11 Thread Jiri Olsa
On Thu, Feb 11, 2021 at 09:52:05AM -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Feb 08, 2021 at 09:08:49PM +0100, Jiri Olsa escreveu: > > +__maybe_unused > > +static int send_cmd(struct daemon *daemon, union cmd *cmd) > > +{ > > + int ret = -1, fd; > > + char *line = NULL; > > + size_t le

Re: [PATCH] staging: wimax: Fix some coding style problems

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 09:45:53PM +0530, Hemansh Agnihotri wrote: > This fixes following warnings and errors as reported by checkpatch.pl: > 1) WARNING: Missing a blank line after declarations > 2) WARNING: Block comments use a trailing */ on a separate line > 3) ERROR: code inde

Re: [PATCH] platform/surface: Add platform profile driver

2021-02-11 Thread Hans de Goede
Hi, On 2/11/21 5:34 PM, Maximilian Luz wrote: > > > On 2/11/21 5:31 PM, Hans de Goede wrote: >> Hi, >> >> On 2/11/21 5:17 PM, Maximilian Luz wrote: >>> >>> >>> On 2/11/21 4:56 PM, Hans de Goede wrote: Hi, On 2/8/21 10:38 PM, Maximilian Luz wrote: > > > On 2/8/21 9:27 P

Re: [PATCH v1 7/7] ACPI: property: Allow counting a single value as an array of 1 element

2021-02-11 Thread Rafael J. Wysocki
On Thu, Feb 11, 2021 at 4:42 PM Andy Shevchenko wrote: > > On Wed, Feb 10, 2021 at 04:44:34PM +0100, Rafael J. Wysocki wrote: > > On Wed, Feb 10, 2021 at 4:42 PM Andy Shevchenko > > wrote: > > > On Wed, Feb 10, 2021 at 04:01:16PM +0100, Rafael J. Wysocki wrote: > > > > On Wed, Feb 10, 2021 at 3:4

Re: possible deadlock in start_this_handle (2)

2021-02-11 Thread Michal Hocko
On Thu 11-02-21 14:26:30, Matthew Wilcox wrote: > On Thu, Feb 11, 2021 at 03:20:41PM +0100, Michal Hocko wrote: > > On Thu 11-02-21 13:25:33, Matthew Wilcox wrote: > > > On Thu, Feb 11, 2021 at 02:07:03PM +0100, Michal Hocko wrote: > > > > On Thu 11-02-21 12:57:17, Matthew Wilcox wrote: > > > > > >

Re: [PATCH v2] Documentation/admin-guide: kernel-parameters: Update nohlt section

2021-02-11 Thread Jonathan Corbet
Florian Fainelli writes: > Update the documentation regarding "nohlt" and indicate that it is not > only for bugs, but can be useful to disable the architecture specific > sleep instructions. ARM, ARM64, SuperH and Microblaze all use > CONFIG_GENERIC_IDLE_POLL_SETUP which takes care of honoring t

Re: [PATCH] doc/admin-guide: fix spelling mistake: "perfomance" -> "performance"

2021-02-11 Thread Jonathan Corbet
Colin King writes: > From: Colin Ian King > > There is a spelling mistake in the perf-security documentation. Fix it. > > Signed-off-by: Colin Ian King > --- > Documentation/admin-guide/perf-security.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/adm

Re: [PATCH v2 5/8] cxl/mem: Add a "RAW" send command

2021-02-11 Thread Dan Williams
On Wed, Feb 10, 2021 at 7:27 AM wrote: > > > diff --git a/drivers/cxl/Kconfig b/drivers/cxl/Kconfig > > index c4ba3aa0a05d..08eaa8e52083 100644 > > --- a/drivers/cxl/Kconfig > > +++ b/drivers/cxl/Kconfig > > @@ -33,6 +33,24 @@ config CXL_MEM > > > > If unsure say 'm'. > > > > +config CXL

Re: [PATCH iproute2-next V4] devlink: add support for port params get/set

2021-02-11 Thread David Ahern
On 2/9/21 3:31 AM, Oleksandr Mazur wrote: > Add implementation for the port parameters > getting/setting. > Add bash completion for port param. > Add man description for port param. > > Signed-off-by: Oleksandr Mazur > --- applied to iproute2-next. In the future, please add example commands - g

Re: [PATCH 06/24] perf daemon: Add config file support

2021-02-11 Thread Jiri Olsa
On Thu, Feb 11, 2021 at 03:01:12PM +0900, Namhyung Kim wrote: > Hi Jiri, > > On Tue, Feb 9, 2021 at 5:09 AM Jiri Olsa wrote: > > +static int daemon__reconfig(struct daemon *daemon) > > +{ > > + struct daemon_session *session, *n; > > + > > + list_for_each_entry_safe(session, n, &daemo

Re: [PATCH v2] arm: dts: sun5i: Add GPU node

2021-02-11 Thread Maxime Ripard
Hi, On Thu, Feb 11, 2021 at 03:25:23PM +, Yassine Oudjana wrote: > sun5i has the same Mali 400 GPU as sun4i with the same interrupts, clocks > and resets. Add node for it in dts. > > Signed-off-by: Yassine Oudjana Unfortunately we already merged a similar patch for 5.12 Maxime signature.

Re: [v7 PATCH 05/12] mm: memcontrol: rename shrinker_map to shrinker_info

2021-02-11 Thread Kirill Tkhai
On 10.02.2021 02:33, Yang Shi wrote: > On Tue, Feb 9, 2021 at 12:50 PM Roman Gushchin wrote: >> >> On Tue, Feb 09, 2021 at 09:46:39AM -0800, Yang Shi wrote: >>> The following patch is going to add nr_deferred into shrinker_map, the >>> change will >>> make shrinker_map not only include map anymor

Re: [PATCH 1/1] s390/vfio-ap: fix circular lockdep when setting/clearing crypto masks

2021-02-11 Thread Halil Pasic
On Thu, 11 Feb 2021 09:21:26 -0500 Tony Krowiak wrote: > Yes, it makes sense. I guess I didn't look closely at your > suggestion when I said it was exactly what I implemented > after agreeing with Connie. I had a slight difference in > my implementation: > > static void vfio_ap_mdev_unset_kvm(st

Re: [PATCH iproute2-next V4] devlink: add support for port params get/set

2021-02-11 Thread Oleksandr Mazur
On 2/9/21 3:31 AM, Oleksandr Mazur wrote: > Add implementation for the port parameters > getting/setting. > Add bash completion for port param. > Add man description for port param. > > Signed-off-by: Oleksandr Mazur > --- > applied to iproute2-next. > In the future, please add example commands

Re: [PATCH v3 1/2] thermal: armada: ap806: use firmware SiP services for thermal operations

2021-02-11 Thread kernel test robot
Hi, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on clk/clk-next] [also build test WARNING on linus/master v5.11-rc7 next-20210211] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--bas

include/linux/compiler_types.h:315:38: error: call to '__compiletime_assert_498' declared with attribute error: BUILD_BUG_ON failed: offsetof(struct can_frame, len) != offsetof(struct canfd_frame, len

2021-02-11 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 291009f656e8eaebbdfd3a8d99f6b190a9ce9deb commit: c7b74967799b1af52b3045d69d4c26836b2d41de can: replace can_dlc as variable/element for payload length date: 3 months ago config: arm-randconfig-r015-20210210

Re: [PATCH v2 4/8] cxl/mem: Add basic IOCTL interface

2021-02-11 Thread Ben Widawsky
On 21-02-11 10:06:46, Jonathan Cameron wrote: > On Wed, 10 Feb 2021 20:40:52 -0800 > Dan Williams wrote: > > > On Wed, Feb 10, 2021 at 10:47 AM Jonathan Cameron > > wrote: > > [..] > > > > +#define CXL_CMDS > > > >\ > > > > + ___C(IN

Re: [PATCH] staging: wlan-ng: Fix comments typos

2021-02-11 Thread Mairo P. Rufus
> Hi, > > This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him > a patch that has triggered this response. He used to manually respond > to these common problems, but in order to save his sanity (he kept > writing the same thing over and over, yet to different people), I was > cr

Re: RFC: oftree based setup of composite board devices

2021-02-11 Thread Enrico Weigelt, metux IT consult
On 11.02.21 12:41, Andy Shevchenko wrote: Hi, On Thu, Feb 11, 2021 at 1:15 PM Enrico Weigelt, metux IT consult wrote: On 10.02.21 11:30, Andy Shevchenko wrote: Use cases are boards with non-oftree firmware (ACPI, etc) where certain platform devices can't be directly enumerated via firmware

Re: [RFC 1/9] mm, arm64: Update PR_SET/GET_TAGGED_ADDR_CTRL interface

2021-02-11 Thread Catalin Marinas
Hi Kirill, On Fri, Feb 05, 2021 at 06:16:21PM +0300, Kirill A. Shutemov wrote: > The interface for enabling tagged addresses is very inflexible. It > implies tag size and tag shift implemented by ARM TBI. > > Rework the interface to accommodate different shifts and tag sizes. > > PR_SET_TAGGED_A

Re: [PATCH] docs: kernel-hacking: Remove the word fuck,trying to be civil :)

2021-02-11 Thread Jonathan Corbet
Bhaskar Chowdhury writes: > s/fuck// > > > Signed-off-by: Bhaskar Chowdhury > --- > Documentation/kernel-hacking/locking.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/kernel-hacking/locking.rst > b/Documentation/kernel-hacking/locking.rst > index c3

Re: [RFC 1/9] mm, arm64: Update PR_SET/GET_TAGGED_ADDR_CTRL interface

2021-02-11 Thread Dave Hansen
Hi Catalin, I noticed there are some ELF bits for ARM's BTI feature: GNU_PROPERTY_AARCH64_FEATURE_1_BTI > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/include/uapi/linux/elf.h#n453 There's been talk of needing a similar set of bits on x86 for tagged pointers (

Re: [PATCH] docs: reporting-issues.rst: explain how to decode stack traces

2021-02-11 Thread Randy Dunlap
Hi Thorsten, Just a couple of small nits (or one that is repeated): On 2/9/21 9:48 PM, Thorsten Leemhuis wrote: > Replace placeholder text about decoding stack traces with a section that > properly describes what a typical user should do these days. To make > it works for them, add a paragraph in

Re: [PATCH 3/3] [v3] lib/vsprintf: debug_never_hash_pointers prints all addresses as unhashed

2021-02-11 Thread Timur Tabi
On 2/11/21 6:31 AM, Pavel Machek wrote: Can we make this something shorter? Clearly you don't want people placing this in their grub config, so they'll be most likely typing this a lot... debug_pointers or debug_ptrs would be better. dbg_unhash_ptrs? "debug_ptrs" is too vague IMHO, and I w

Re: [PATCH 06/24] perf daemon: Add config file support

2021-02-11 Thread Arnaldo Carvalho de Melo
Em Thu, Feb 11, 2021 at 05:45:38PM +0100, Jiri Olsa escreveu: > On Thu, Feb 11, 2021 at 03:01:12PM +0900, Namhyung Kim wrote: > > Hi Jiri, > > > > On Tue, Feb 9, 2021 at 5:09 AM Jiri Olsa wrote: > > > +static int daemon__reconfig(struct daemon *daemon) > > > +{ > > > + struct daemon_session

Re: [PATCH] docs: kernel-hacking: Remove the word fuck,trying to be civil :)

2021-02-11 Thread Randy Dunlap
On 2/11/21 9:04 AM, Jonathan Corbet wrote: > Bhaskar Chowdhury writes: > >> s/fuck// >> >> >> Signed-off-by: Bhaskar Chowdhury >> --- >> Documentation/kernel-hacking/locking.rst | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/Documentation/kernel-hacking/locking.rst

Re: [net-next] net: mvpp2: fix interrupt mask/unmask skip condition

2021-02-11 Thread Russell King - ARM Linux admin
On Thu, Feb 11, 2021 at 05:13:19PM +0200, stef...@marvell.com wrote: > From: Stefan Chulski > > The condition should be skipped if CPU ID equal to nthreads. > The patch doesn't fix any actual issue since > nthreads = min_t(unsigned int, num_present_cpus(), MVPP2_MAX_THREADS). > On all current Arm

[PATCH] staging: wimax: Fix some coding style problems

2021-02-11 Thread Hemansh Agnihotri
This fixes checkpatch errors :- "else should follow close brace '}'", "space required before the open parenthesis '('" and "spaces required around that '?' (ctx:VxW)" in drivers/staging/wimax/i2400m/rx.c file. Signed-off-by: Hemansh Agnihotri --- drivers/staging/wimax/i2400m/rx.c | 11 +-

[PATCH v3] Updated locking documentation for transaction_t

2021-02-11 Thread Alexander Lochmann
Some members of transaction_t are allowed to be read without any lock being held if accessed from the correct context. We used LockDoc's findings to determine those members. Each member of them is marked with a short comment: "no lock needed for jbd2 thread". Signed-off-by: Alexander Lochmann Sig

Re: [PATCH v6 1/3] lib/vsprintf: Add support for printing V4L2 and DRM fourccs

2021-02-11 Thread Petr Mladek
On Tue 2021-02-09 19:47:55, Sakari Ailus wrote: > Hi Andy, > > On Tue, Feb 09, 2021 at 11:58:40AM +0200, Andy Shevchenko wrote: > > On Tue, Feb 09, 2021 at 11:20:32AM +0200, Sakari Ailus wrote: > > > On Mon, Feb 08, 2021 at 10:43:30PM +0200, Andy Shevchenko wrote: > > > > On Mon, Feb 8, 2021 at 10

Re: [PATCH v1 0/5] Enable fw_devlink=on by default

2021-02-11 Thread Saravana Kannan
On Thu, Feb 11, 2021 at 7:03 AM Rafael J. Wysocki wrote: > > On Thu, Feb 11, 2021 at 1:02 AM Saravana Kannan wrote: > > > > On Thu, Jan 28, 2021 at 7:03 AM Jon Hunter wrote: > > > > > > > > > On 14/01/2021 16:56, Jon Hunter wrote: > > > > > > > > On 14/01/2021 16:47, Saravana Kannan wrote: > > >

Re: [PATCH] Fix jump parsing for C++ code.

2021-02-11 Thread Arnaldo Carvalho de Melo
Em Thu, Feb 11, 2021 at 01:37:55PM +0100, Martin Liška escreveu: > Considering the following testcase: > > int > foo(int a, int b) > { > for (unsigned i = 0; i < 10; i++) > a += b; > return a; > } > > int main() > { > foo (3, 4); > return 0; > } > > perf annotate displays: >

objtool segfault in 5.10 kernels with binutils-2.36.1

2021-02-11 Thread Ken Moffat
Hi, in 5.10 kernels up to and including 5.10.15 when trying to build the kernel for an x86_64 skylake using binutils-2.36.1, gcc-10.2 and glibic-2.33 I get a segfault in objtool if the orc unwinder is enabled. This has already been fixed in 5.11 by ''objtool: Fix seg fault with Clang non-section

Re: [PATCH] KVM: nVMX: Sync L2 guest CET states between L1/L2

2021-02-11 Thread Sean Christopherson
On Tue, Feb 09, 2021, Yang Weijiang wrote: > When L2 guest status has been changed by L1 QEMU/KVM, sync the change back > to L2 guest before the later's next vm-entry. On the other hand, if it's > changed due to L2 guest, sync it back so as to let L1 guest see the change. > > Signed-off-by: Yang W

Re: [PATCH] gen_compile_commands: prune some directories

2021-02-11 Thread Nathan Chancellor
On Fri, Feb 12, 2021 at 01:11:54AM +0900, Masahiro Yamada wrote: > If directories are passed to gen_compile_commands.py, os.walk() traverses > all the subdirectories to search for .cmd files, but we know some of them > are not worth traversing. > > Use the 'topdown' parameter of os.walk to prune t

[PATCH v2 1/3] coresight: etm-perf: Clarify comment on perf options

2021-02-11 Thread Mathieu Poirier
From: Leo Yan In theory, the options should be arbitrary values and are neutral for any ETM version; so far perf tool uses ETMv3.5/PTM ETMCR config bits except for register's bit definitions, also uses as options. This can introduce confusion, especially if we want to add a new option but the ne

[PATCH v2 0/3] coresight: Patches for v5.12 (part 2)

2021-02-11 Thread Mathieu Poirier
Good morning, My previous submission had the wrong baseline and as such was missing a patch. This set applies properly on [1]. Thanks for the patience, Mathieu [1]. 48139bad913d ACRN: update MAINTAINERS: mailing list is subscribers-only Leo Yan (2): coresight: etm-perf: Clarify comment on per

[PATCH v2 2/3] coresight: etm-perf: Support PID tracing for kernel at EL2

2021-02-11 Thread Mathieu Poirier
From: Suzuki K Poulose When the kernel is running at EL2, the PID is stored in CONTEXTIDR_EL2. So, tracing CONTEXTIDR_EL1 doesn't give us the pid of the process. Thus we should trace the VMID with VMIDOPT set to trace CONTEXTIDR_EL2 instead of CONTEXTIDR_EL1. Given that we have an existing confi

[PATCH v2 3/3] Documentation: coresight: Add PID tracing description

2021-02-11 Thread Mathieu Poirier
From: Leo Yan After support the PID tracing for the kernel in EL1 or EL2, the usage gets more complicated. This patch gives description for the PMU formats of contextID configs, this can help users to understand how to control the knobs for PID tracing when the kernel is in different ELs. Signe

Re: [PATCH 3/3] [v3] lib/vsprintf: debug_never_hash_pointers prints all addresses as unhashed

2021-02-11 Thread Matthew Wilcox
On Thu, Feb 11, 2021 at 11:08:12AM -0600, Timur Tabi wrote: > > > On 2/11/21 6:31 AM, Pavel Machek wrote: > > Can we make this something shorter? Clearly you don't want people > > placing this in their grub config, so they'll be most likely typing > > this a lot... > > > > debug_pointers or debu

Re: [PATCH 3/3] [v3] lib/vsprintf: debug_never_hash_pointers prints all addresses as unhashed

2021-02-11 Thread Petr Mladek
On Thu 2021-02-11 11:08:12, Timur Tabi wrote: > > > On 2/11/21 6:31 AM, Pavel Machek wrote: > > Can we make this something shorter? Clearly you don't want people > > placing this in their grub config, so they'll be most likely typing > > this a lot... > > > > debug_pointers or debug_ptrs would b

[PATCH] ASoC: wm_adsp: Remove unused control callback structure

2021-02-11 Thread Charles Keepax
This callback structure has never been used and it is not clear why it was added in the first place. Remove it to clear up the code a little. Signed-off-by: Charles Keepax --- sound/soc/codecs/wm_adsp.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/sound/soc/codecs/wm_adsp.c b/s

Re: [PATCH] staging: wlan-ng: Fix comments typos

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 05:55:18PM +0100, Mairo P. Rufus wrote: > > Hi, > > > > This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him > > a patch that has triggered this response. He used to manually respond > > to these common problems, but in order to save his sanity (he kept >

Re: [v7 PATCH 05/12] mm: memcontrol: rename shrinker_map to shrinker_info

2021-02-11 Thread Yang Shi
On Thu, Feb 11, 2021 at 8:47 AM Kirill Tkhai wrote: > > On 10.02.2021 02:33, Yang Shi wrote: > > On Tue, Feb 9, 2021 at 12:50 PM Roman Gushchin wrote: > >> > >> On Tue, Feb 09, 2021 at 09:46:39AM -0800, Yang Shi wrote: > >>> The following patch is going to add nr_deferred into shrinker_map, the

Re: [v7 PATCH 12/12] mm: vmscan: shrink deferred objects proportional to priority

2021-02-11 Thread Yang Shi
On Thu, Feb 11, 2021 at 5:10 AM Vlastimil Babka wrote: > > On 2/9/21 6:46 PM, Yang Shi wrote: > > The number of deferred objects might get windup to an absurd number, and it > > results in clamp of slab objects. It is undesirable for sustaining > > workingset. > > > > So shrink deferred objects

Re: [PATCH mm] arm64: kasan: fix MTE symbols exports

2021-02-11 Thread Andrey Konovalov
On Tue, Feb 9, 2021 at 7:45 PM Andrew Morton wrote: > > On Tue, 9 Feb 2021 17:02:56 + Catalin Marinas > wrote: > > > On Tue, Feb 09, 2021 at 04:32:30PM +0100, Andrey Konovalov wrote: > > > diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c > > > index a66c2806fc4d..788ef0c3a25e 1

Re: [PATCH v7 3/4] x509: Add support for parsing x509 certs with ECDSA keys

2021-02-11 Thread Stefan Berger
On 2/11/21 3:03 AM, kernel test robot wrote: Hi Stefan, Thank you for the patch! Yet something to improve: crypto/asymmetric_keys/public_key.c:97: undefined reference to `parse_OID' So the issue is that  only ASYMMETRIC_PUBLIC_KEY_SUBTYPE is selected in this config and the selection of OID

[PATCH v2] printk: avoid prb_first_valid_seq() where possible

2021-02-11 Thread John Ogness
() usage with prb_read_valid_*() functions, which provide a start sequence number to search from. Fixes: 896fbe20b4e2333fb55 ("printk: use the lockless ringbuffer") Reported-by: kernel test robot Reported-by: J. Avila Signed-off-by: John Ogness --- patch against next-20210211 v2: Abort and

Re: [PATCH v2 0/3] coresight: Patches for v5.12 (part 2)

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 10:20:35AM -0700, Mathieu Poirier wrote: > Good morning, > > My previous submission had the wrong baseline and as such was missing a patch. > This set applies properly on [1]. Much better, thanks, now queued up. greg k-h

Re: [PATCH 09/15] KVM: selftests: Move per-VM GPA into perf_test_args

2021-02-11 Thread Ben Gardon
On Thu, Feb 11, 2021 at 7:58 AM Sean Christopherson wrote: > > On Thu, Feb 11, 2021, Paolo Bonzini wrote: > > On 11/02/21 02:56, Sean Christopherson wrote: > > > > > + pta->gpa = (vm_get_max_gfn(vm) - guest_num_pages) * > > > > > pta->guest_page_size; > > > > > + pta->gpa &= ~(pta->ho

Re: [PATCH v3] driver core: auxiliary bus: Fix calling stage for auxiliary bus init

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 09:21:29AM -0700, Dave Jiang wrote: > > On 2/10/2021 11:54 PM, Greg KH wrote: > > On Wed, Feb 10, 2021 at 01:16:11PM -0700, Dave Jiang wrote: > > > When the auxiliary device code is built into the kernel, it can be > > > executed > > > before the auxiliary bus is registere

Re: [PATCH v13 05/15] iommu/smmuv3: Get prepared for nested stage support

2021-02-11 Thread Auger Eric
Hi Keqian, On 2/2/21 8:14 AM, Keqian Zhu wrote: > Hi Eric, > > On 2020/11/18 19:21, Eric Auger wrote: >> When nested stage translation is setup, both s1_cfg and >> s2_cfg are set. >> >> We introduce a new smmu domain abort field that will be set >> upon guest stage1 configuration passing. >> >> a

Re: [PATCH v13 06/15] iommu/smmuv3: Implement attach/detach_pasid_table

2021-02-11 Thread Auger Eric
Hi Keqian, On 2/2/21 9:03 AM, Keqian Zhu wrote: > Hi Eric, > > On 2020/11/18 19:21, Eric Auger wrote: >> On attach_pasid_table() we program STE S1 related info set >> by the guest into the actual physical STEs. At minimum >> we need to program the context descriptor GPA and compute >> whether the

Re: [PATCH] enetc: auto select PHYLIB and MDIO_DEVRES

2021-02-11 Thread Florian Fainelli
On 2/11/21 8:09 AM, Tong Zhang wrote: > FSL_ENETC_MDIO use symbols from PHYLIB and MDIO_DEVRES, however they are > not auto selected. > > ERROR: modpost: "__mdiobus_register" > [drivers/net/ethernet/freescale/enetc/fsl-enetc-mdio.ko] undefined! > ERROR: modpost: "mdiobus_unregister" > [drivers/n

Re: linux-next: manual merge of the rcu tree with the block tree

2021-02-11 Thread Paul E. McKenney
On Thu, Feb 11, 2021 at 04:48:52PM +1100, Stephen Rothwell wrote: > Hi all, > > Today's linux-next merge of the rcu tree got conflicts in: > > include/linux/rcupdate.h > kernel/rcu/tree.c > kernel/rcu/tree_plugin.h > > between commits: > > 3a7b5c87a0b2 ("rcu/nocb: Perform deferred wake

[RFC][PATCH v2 3/7] objtool,x86: Rewrite LEA decode

2021-02-11 Thread Peter Zijlstra
Current LEA decoding is a bunch of special cases, properly decode the instruction, with exception of full SIB and RIP-relative modes. Signed-off-by: Peter Zijlstra (Intel) --- tools/objtool/arch/x86/decode.c | 83 +++- 1 file changed, 24 insertions(+), 59 de

Re: [PATCH 8/9] vfio/pci: use x86 naming instead of igd

2021-02-11 Thread Cornelia Huck
On Thu, 11 Feb 2021 11:29:37 -0500 Matthew Rosato wrote: > On 2/11/21 10:47 AM, Max Gurtovoy wrote: > > > > On 2/2/2021 7:10 PM, Jason Gunthorpe wrote: > >> On Tue, Feb 02, 2021 at 05:06:59PM +0100, Cornelia Huck wrote: > >> > >>> On the other side, we have the zdev support, which both requi

RE: [RESEND PATCH v2] dt-bindings: pci: rcar-pci-ep: Document r8a7795

2021-02-11 Thread Prabhakar Mahadev Lad
Hi Hamamachi-san, Thank you for the patch. > -Original Message- > From: Yuya Hamamachi > Sent: 09 February 2021 07:49 > To: linux-...@vger.kernel.org; linux-renesas-...@vger.kernel.org; > devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org; marek.vasut+rene...@gmail.com; Yoshihi

[RFC][PATCH v2 0/7] objtool x86 decoder fixes

2021-02-11 Thread Peter Zijlstra
Hi, Since I had to look at instruction decoding for the stack swizzle thing, I ended up with a few more changes to the objtool x86 decoder. These patches are lightly tested (they build defconfig and allmodconfig using GCC10), but older versions have seen some robot exposure and other compilers.

[RFC][PATCH v2 4/7] objtool,x86: Rewrite LEAVE

2021-02-11 Thread Peter Zijlstra
Since we can now have multiple stack-ops per instruction, we don't need to special case LEAVE and can simply emit the composite operations. Signed-off-by: Peter Zijlstra (Intel) --- tools/objtool/arch/x86/decode.c | 14 +++--- tools/objtool/check.c| 24 ++

[RFC][PATCH v2 2/7] objtool,x86: Renumber CFI_reg

2021-02-11 Thread Peter Zijlstra
Make them match the instruction encoding numbering. Signed-off-by: Peter Zijlstra (Intel) --- tools/objtool/arch/x86/include/arch/cfi_regs.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) --- a/tools/objtool/arch/x86/include/arch/cfi_regs.h +++ b/tools/objtool/arch/x86/inc

[RFC][PATCH v2 6/7] objtool,x86: Support %riz encodings

2021-02-11 Thread Peter Zijlstra
When there's a SIB byte, the register otherwise denoted by r/m will then be denoted by SIB.base REX.b will now extend this. SIB.index == SP is magic and notes an index value zero. This means that there's a bunch of alternative (longer) encodings for the same thing. Eg. 'ModRM.mod != 3, ModRM.r/m =

[RFC][PATCH v2 7/7] objtool,x86: Rewrite ADD/SUB/AND

2021-02-11 Thread Peter Zijlstra
Support sign extending and imm8 forms. Signed-off-by: Peter Zijlstra (Intel) --- tools/objtool/arch/x86/decode.c | 70 +--- 1 file changed, 51 insertions(+), 19 deletions(-) --- a/tools/objtool/arch/x86/decode.c +++ b/tools/objtool/arch/x86/decode.c @@ -98,

[RFC][PATCH v2 5/7] objtool,x86: Simplify register decode

2021-02-11 Thread Peter Zijlstra
Since the CFI_reg number now matches the instruction encoding order do away with the op_to_cfi_reg[] and use direct assignment. Signed-off-by: Peter Zijlstra (Intel) --- tools/objtool/arch/x86/decode.c | 79 +++- 1 file changed, 39 insertions(+), 40 deletion

[RFC][PATCH v2 1/7] objtool: Allow UNWIND_HINT to suppress dodgy stack modifications

2021-02-11 Thread Peter Zijlstra
rewind_stack_do_exit() UNWIND_HINT_FUNC /* Prevent any naive code from trying to unwind to our caller. */ xorl%ebp, %ebp movqPER_CPU_VAR(cpu_current_top_of_stack), %rax leaq-PTREGS_SIZE(%rax), %rsp UNWIND_HINT_REGS calldo_exi

[PATCH v4] driver core: auxiliary bus: Fix calling stage for auxiliary bus init

2021-02-11 Thread Dave Jiang
When the auxiliary device code is built into the kernel, it can be executed before the auxiliary bus is registered. This causes bus->p to be not allocated and triggers a NULL pointer dereference when the auxiliary bus device gets added with bus_add_device(). Call the auxiliary_bus_init() under driv

adi-axi-adc.c:undefined reference to `devm_platform_ioremap_resource'

2021-02-11 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 291009f656e8eaebbdfd3a8d99f6b190a9ce9deb commit: ef04070692a21633ec6a60f80c19b6af44b3cf47 iio: adc: adi-axi-adc: add support for AXI ADC IP core date: 10 months ago config: s390-randconfig-r035-20210209 (a

Re: [PATCH v2] arm64: Fix warning in mte_get_random_tag()

2021-02-11 Thread Andrey Konovalov
On Thu, Feb 11, 2021 at 4:22 PM Vincenzo Frascino wrote: > > The simplification of mte_get_random_tag() caused the introduction of the > warning below: > > In file included from arch/arm64/include/asm/kasan.h:9, > from include/linux/kasan.h:16, > from mm/kasan/com

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

2021-02-11 Thread Ben Widawsky
On 21-02-11 12:02:15, Jonathan Cameron wrote: > On Tue, 9 Feb 2021 16:02:57 -0800 > Ben Widawsky wrote: > > > CXL devices identified by the memory-device class code must implement > > the Device Command Interface (described in 8.2.9 of the CXL 2.0 spec). > > While the driver already maintains a l

Re: [PATCH v5 2/2] ARM: kprobes: rewrite test-arm.c in UAL

2021-02-11 Thread Nathan Chancellor
On Wed, Feb 10, 2021 at 06:51:49PM -0800, Nick Desaulniers wrote: > Clang's integrated assembler only accepts UAL syntax, rewrite the > instructions that were changed by RVCTv2.1. The document "Assembly > language changes after RVCTv2.1" was very helpful. > > .syntax unified > > directive is a

Re: [PATCH v1 0/5] Enable fw_devlink=on by default

2021-02-11 Thread Rafael J. Wysocki
On Thu, Feb 11, 2021 at 6:15 PM Saravana Kannan wrote: > > On Thu, Feb 11, 2021 at 7:03 AM Rafael J. Wysocki wrote: > > > > On Thu, Feb 11, 2021 at 1:02 AM Saravana Kannan > > wrote: > > > > > > On Thu, Jan 28, 2021 at 7:03 AM Jon Hunter wrote: > > > > > > > > > > > > On 14/01/2021 16:56, Jon

[PATCH v1 7/7] dt-bindings: net: qcom-ipa: Document qcom,msm8998-ipa compatible

2021-02-11 Thread AngeloGioacchino Del Regno
MSM8998 support has been added: document the new compatible. Signed-off-by: AngeloGioacchino Del Regno --- Documentation/devicetree/bindings/net/qcom,ipa.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/net/qcom,ipa.yaml b/Documentation/devicetree/bind

[PATCH v1 4/7] net: ipa: gsi: Use right masks for GSI v1.0 channels hw param

2021-02-11 Thread AngeloGioacchino Del Regno
In GSI v1.0 the register GSI_HW_PARAM_2_OFFSET has different layout so the number of channels and events per EE are, of course, laid out in 8 bits each (0-7, 8-15 respectively). Signed-off-by: AngeloGioacchino Del Regno --- drivers/net/ipa/gsi.c | 16 +--- drivers/net/ipa/gsi_re

Re: [PATCH][next][V2] fs/jfs: fix potential integer overflow on shift of a int

2021-02-11 Thread Dave Kleikamp
On 2/11/21 7:01 AM, Colin King wrote: From: Colin Ian King The left shift of int 32 bit integer constant 1 is evaluated using 32 bit arithmetic and then assigned to a signed 64 bit integer. In the case where l2nb is 32 or more this can lead to an overflow. Avoid this by shifting the value 1LL

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