Re: [PATCH] rcu-tasks: Fix compilation warning with !CONFIG_TASKS_RCU and CONFIG_TINY_RCU

2020-09-17 Thread Laurent Pinchart
Hi Paul, On Tue, Aug 25, 2020 at 09:16:29AM -0700, Paul E. McKenney wrote: > On Tue, Aug 25, 2020 at 06:22:49PM +0300, Laurent Pinchart wrote: > > On Tue, Aug 25, 2020 at 08:02:22AM -0700, Paul E. McKenney wrote: > > > On Sun, Aug 23, 2020 at 06:04:05AM +0300, Laurent Pinchart wrote: > > > >

Re: [PATCH] arm64: dts: ti: k3-j721e: Rename mux header and update macro names

2020-09-17 Thread Nishanth Menon
On 19:16-20200917, Roger Quadros wrote: > We intend to use one header file for SERDES MUX for all > TI SoCs so rename the header file. > > The exsting macros are too generic. Prefix them with SoC name. > > Signed-off-by: Roger Quadros Could you add Fixes and take care of

linux-next: Fixes tags need some work in the amdgpu tree

2020-09-17 Thread Stephen Rothwell
Hi all, In commit 2561adedd576 ("drm/amd/display: dc/clk_mgr: make function static") Fixes tag Fixes: 3ecb3b794e2 "drm/amd/display: dc/clk_mgr: add support for SI parts (v2)" has these problem(s): - SHA1 should be at least 12 digits long Can be fixed by setting core.abbrev to 12

Re: [PATCH 1/4] mm: Trial do_wp_page() simplification

2020-09-17 Thread Linus Torvalds
On Thu, Sep 17, 2020 at 3:09 PM Jason Gunthorpe wrote: > > My advice for this -rc fix is to go with a single bit in the mm_struct > set on any call to pin_user_pages* Ack, except make sure it's a byte rather than a bitfield that could have races. Or even just a separate atomic_t. Keep it simple

Re: [PATCH 1/2] vfio/pci: Remove redundant declaration of vfio_pci_driver

2020-09-17 Thread Alex Williamson
On Thu, 17 Sep 2020 11:31:27 +0800 Zenghui Yu wrote: > It was added by commit 137e5531351d ("vfio/pci: Add sriov_configure > support") and actually unnecessary. Remove it. Looks correct, but I might clarify as: s/unnecessary/duplicates a forward declaration earlier in the file/ I can change

[PATCH v2 2/6] Bluetooth: Break add adv into two mgmt commands

2020-09-17 Thread Daniel Winkler
This patch adds support for the new advertising add interface, with the first command setting advertising parameters and the second to set advertising data. The set parameters command allows the caller to leave some fields "unset", with a params bitfield defining which params were purposefully

[PATCH v2 4/6] Bluetooth: Emit tx power chosen on ext adv params completion

2020-09-17 Thread Daniel Winkler
Our hci call to set extended advertising parameters returns the actual tx power selected by the controller. This patch signals a new TX_POWER_SELECTED mgmt event to alert the caller of the actual tx power that is being used. This is important because the power selected will not necessarily match

[PATCH v2 5/6] Bluetooth: Query LE tx power on startup

2020-09-17 Thread Daniel Winkler
Queries tx power via HCI_LE_Read_Transmit_Power command when the hci device is initialized, and stores resulting min/max LE power in hdev struct. If command isn't available (< BT5 support), min/max values both default to HCI_TX_POWER_INVALID. This patch is manually verified by ensuring BT5

[PATCH v2 1/6] Bluetooth: Add helper to set adv data

2020-09-17 Thread Daniel Winkler
We wish to handle advertising data separately from advertising parameters in our new MGMT requests. This change adds a helper that allows the advertising data and scan response to be updated for an existing advertising instance. Reviewed-by: Sonny Sasaka Signed-off-by: Daniel Winkler ---

[PATCH v2 6/6] Bluetooth: Add MGMT command for controller capabilities

2020-09-17 Thread Daniel Winkler
For advertising, we wish to know the LE tx power capabilities of the controller in userspace, so this patch adds a new MGMT command to query controller capabilities. The data returned is in TLV format, so it can be easily used to convey any data determined to be useful in the future, but for now

[PATCH v2 0/6] Bluetooth: Add new MGMT interface for advertising add

2020-09-17 Thread Daniel Winkler
Hi Maintainers, This patch series defines the new two-call MGMT interface for adding new advertising instances. Similarly to the hci advertising commands, a mgmt call to set parameters is expected to be first, followed by a mgmt call to set advertising data/scan response. The members of the

[PATCH v2 3/6] Bluetooth: Use intervals and tx power from mgmt cmds

2020-09-17 Thread Daniel Winkler
This patch takes the min/max intervals and tx power optionally provided in mgmt interface, stores them in the advertisement struct, and uses them when configuring the hci requests. While tx power is not used if extended advertising is unavailable, software rotation will use the min and max

Re: [PATCH 06/14] Fix CFLAGS for UBSAN_BOUNDS on Clang

2020-09-17 Thread Kees Cook
On Thu, Sep 17, 2020 at 11:35:40AM +, George Popescu wrote: > On Thu, Sep 17, 2020 at 08:37:07AM +0200, Marco Elver wrote: > > So, it seems that local-bounds can still catch some rare OOB accesses, > > where KASAN fails to catch it because the access might skip over the > > redzone. > > > >

[PATCH v1 1/3] bus: mhi: core: Remove warnings for missing MODULE_LICENSE()

2020-09-17 Thread Bhaumik Bhatt
When building MHI as a module, missing MODULE_LICENSE() warnings are seen. Avoid them by adding the license and description information for the files where the warnings are seen. Signed-off-by: Bhaumik Bhatt --- drivers/bus/mhi/core/boot.c | 3 +++ drivers/bus/mhi/core/main.c | 3 +++

[PATCH v1 2/3] bus: mhi: core: Introduce debugfs entries for MHI

2020-09-17 Thread Bhaumik Bhatt
Introduce debugfs entries to show state, register, channel, device, and event rings information. Allow the host to dump registers, issue device wake, and change the MHI timeout to help in debug. Signed-off-by: Bhaumik Bhatt --- drivers/bus/mhi/Kconfig | 8 +

[PATCH v1 3/3] bus: mhi: core: Introduce sysfs entries for MHI

2020-09-17 Thread Bhaumik Bhatt
Introduce sysfs entries to enable userspace clients the ability to read the serial number and the OEM PK Hash values obtained from BHI. OEMs need to read these device-specific hardware information values through userspace for factory testing purposes and cannot be exposed via degbufs as it may

[PATCH v2] net: use exponential backoff in netdev_wait_allrefs

2020-09-17 Thread Francesco Ruggeri
The combination of aca_free_rcu, introduced in commit 2384d02520ff ("net/ipv6: Add anycast addresses to a global hashtable"), and fib6_info_destroy_rcu, introduced in commit 9b0a8da8c4c6 ("net/ipv6: respect rcu grace period before freeing fib6_info"), can result in an extra rcu grace period being

[PATCH v1 0/3] Debugfs and Sysfs entries for MHI

2020-09-17 Thread Bhaumik Bhatt
Introduce debugfs and sysfs entries for MHI. Fixes to allow building MHI as a module without warnings/errors. This set of patches was tested on arm64 and x86_64 architectures. Bhaumik Bhatt (3): bus: mhi: core: Remove warnings for missing MODULE_LICENSE() bus: mhi: core: Introduce debugfs

Re: [PATCH v2 9/9] iomap: Change calling convention for zeroing

2020-09-17 Thread Darrick J. Wong
On Thu, Sep 17, 2020 at 11:11:15PM +0100, Matthew Wilcox wrote: > On Thu, Sep 17, 2020 at 03:05:00PM -0700, Darrick J. Wong wrote: > > > -static loff_t > > > -iomap_zero_range_actor(struct inode *inode, loff_t pos, loff_t count, > > > - void *data, struct iomap *iomap, struct iomap

RE: [PATCH v14 5/5] remoteproc: Add initial zynqmp R5 remoteproc driver

2020-09-17 Thread Ben Levinsky
Hi Michael, Thanks for the comments, > -Original Message- > From: Michael Auchter > Sent: Thursday, September 17, 2020 3:11 PM > To: Ben Levinsky > Cc: punit1.agra...@toshiba.co.jp; Stefano Stabellini ; > Michal Simek ; devicet...@vger.kernel.org; > mathieu.poir...@linaro.org; Ed T.

Re: [PATCH] objtool: ignore unreachable trap after call to noreturn functions

2020-09-17 Thread Josh Poimboeuf
On Thu, Sep 17, 2020 at 11:49:04AM +0300, Ilie Halip wrote: > With CONFIG_UBSAN_TRAP enabled, the compiler may insert a trap instruction > after a call to a noreturn function. In this case, objtool warns that the > ud2 instruction is unreachable. > > objtool silences similar warnings (trap after

Re: [PATCH 06/14] Fix CFLAGS for UBSAN_BOUNDS on Clang

2020-09-17 Thread Kees Cook
On Tue, Sep 15, 2020 at 10:24:58AM +, George Popescu wrote: > This would mean losing the local-bounds coverage. I tried to test it without > local-bounds and with a locally defined array on the stack and it works fine > (the handler is called and the error reported). For me it feels like >

Re: [PATCH] add virtual PHY for PHY-less devices

2020-09-17 Thread Florian Fainelli
On 9/17/2020 2:40 PM, Sergej Bauer wrote: From: sba...@blackbox.su Here is a kernel related part of my work which was helps to develop brand new PHY device. It is migth be helpful for developers work with PHY-less lan743x (7431:0011 in my case). It's just a fake virtual PHY which

Re: [PATCH V7 1/4] perf/core: Add PERF_SAMPLE_DATA_PAGE_SIZE

2020-09-17 Thread Liang, Kan
On 9/17/2020 6:02 PM, Dave Hansen wrote: The problem is that the warning from the perf tool usually includes some hints regarding the cause of the warning or possible solution to workaround/fix the warning. What message should we deliver to the users? "Warning: Too many error page size.

Re: [PATCH] add virtual PHY for PHY-less devices

2020-09-17 Thread Andrew Lunn
On Fri, Sep 18, 2020 at 12:40:10AM +0300, Sergej Bauer wrote: > From: sba...@blackbox.su > > Here is a kernel related part of my work which was helps to develop brand > new PHY device. > > It is migth be helpful for developers work with PHY-less lan743x > (7431:0011 in my case). It's

RE: [PATCH] Introduce support for Systems Management Driver over WMI for Dell Systems

2020-09-17 Thread Limonciello, Mario
> > Those are very different flows to get to and change the same "types" of > data. By Dell's interface > > being Dell specific we can guarantee that the documented flow works how it > should. > > Documenting the flow could be part of the documentation for the > different passwd types. In the

Re: [PATCH v2 9/9] iomap: Change calling convention for zeroing

2020-09-17 Thread Matthew Wilcox
On Thu, Sep 17, 2020 at 03:05:00PM -0700, Darrick J. Wong wrote: > > -static loff_t > > -iomap_zero_range_actor(struct inode *inode, loff_t pos, loff_t count, > > - void *data, struct iomap *iomap, struct iomap *srcmap) > > +static loff_t iomap_zero_range_actor(struct inode *inode,

Re: [RFC][Patch v1 1/3] sched/isolation: API to get num of hosekeeping CPUs

2020-09-17 Thread Nitesh Narayan Lal
On 9/17/20 4:11 PM, Bjorn Helgaas wrote: > [+cc Ingo, Peter, Juri, Vincent (scheduler maintainers)] > > s/hosekeeping/housekeeping/ (in subject) > > On Wed, Sep 09, 2020 at 11:08:16AM -0400, Nitesh Narayan Lal wrote: >> Introduce a new API num_housekeeping_cpus(), that can be used to retrieve >>

Re: [PATCH 1/4] mm: Trial do_wp_page() simplification

2020-09-17 Thread Jason Gunthorpe
On Thu, Sep 17, 2020 at 05:40:59PM -0400, Peter Xu wrote: > On Thu, Sep 17, 2020 at 01:35:56PM -0700, Linus Torvalds wrote: > > For that to happen, we'd need to have the vma flag so that we wouldn't > > have any worry about non-pinners, but as you suggested, I think even > > just a mm-wide counter

Re: [PATCH] selftests/harness: Flush stdout before forking

2020-09-17 Thread Kees Cook
On Thu, Sep 17, 2020 at 02:15:19PM +1000, Michael Ellerman wrote: > The test harness forks() a child to run each test. Both the parent and > the child print to stdout using libc functions. That can lead to > duplicated (or more) output if the libc buffers are not flushed before > forking. > >

Re: [PATCH 2/2] vfio/pci: Remove bardirty from vfio_pci_device

2020-09-17 Thread Alex Williamson
On Thu, 17 Sep 2020 13:35:37 +0200 Cornelia Huck wrote: > On Thu, 17 Sep 2020 11:31:28 +0800 > Zenghui Yu wrote: > > > It isn't clear what purpose the @bardirty serves. It might be used to avoid > > the unnecessary vfio_bar_fixup() invoking on a user-space BAR read, which > > is not required

Re: [PATCH] selftests/harness: Flush stdout before forking

2020-09-17 Thread Kees Cook
On Thu, Sep 17, 2020 at 10:51:32AM -0600, Shuah Khan wrote: > On 9/17/20 6:58 AM, Shuah Khan wrote: > > On 9/16/20 10:53 PM, Max Filippov wrote: > > > On Wed, Sep 16, 2020 at 9:16 PM Michael Ellerman > > > wrote: > > > > > > > > The test harness forks() a child to run each test. Both the parent

Re: [PATCH v2 9/9] iomap: Change calling convention for zeroing

2020-09-17 Thread Darrick J. Wong
On Fri, Sep 11, 2020 at 12:47:07AM +0100, Matthew Wilcox (Oracle) wrote: > Pass the full length to iomap_zero() and dax_iomap_zero(), and have > them return how many bytes they actually handled. This is preparatory > work for handling THP, although it looks like DAX could actually take >

Re: [RFC PATCH 1/6] security/fbfam: Add a Kconfig to enable the fbfam feature

2020-09-17 Thread Kees Cook
On Thu, Sep 17, 2020 at 08:40:06PM +0200, John Wood wrote: > Hi, > > On Thu, Sep 10, 2020 at 04:18:08PM -0700, Kees Cook wrote: > > On Thu, Sep 10, 2020 at 01:21:02PM -0700, Kees Cook wrote: > > > From: John Wood > > > > > > Add a menu entry under "Security options" to enable the "Fork brute > >

Re: [PATCH v2 8/9] iomap: Convert iomap_write_end types

2020-09-17 Thread Darrick J. Wong
On Fri, Sep 11, 2020 at 12:47:06AM +0100, Matthew Wilcox (Oracle) wrote: > iomap_write_end cannot return an error, so switch it to return > size_t instead of int and remove the error checking from the callers. > Also convert the arguments to size_t from unsigned int, in case anyone > ever wants to

Re: [PATCH 01/13] mm: Add AOP_UPDATED_PAGE return value

2020-09-17 Thread Matthew Wilcox
On Thu, Sep 17, 2020 at 04:10:38PM +0100, Matthew Wilcox (Oracle) wrote: > +++ b/mm/filemap.c > @@ -2254,8 +2254,10 @@ ssize_t generic_file_buffered_read(struct kiocb *iocb, >* PG_error will be set again if readpage fails. >*/ > ClearPageError(page); >

Re: [PATCH v2 6/9] iomap: Convert read_count to read_bytes_pending

2020-09-17 Thread Darrick J. Wong
On Fri, Sep 11, 2020 at 12:47:04AM +0100, Matthew Wilcox (Oracle) wrote: > Instead of counting bio segments, count the number of bytes submitted. > This insulates us from the block layer's definition of what a 'same page' > is, which is not necessarily clear once THPs are involved. > >

Re: [PATCH V7 1/4] perf/core: Add PERF_SAMPLE_DATA_PAGE_SIZE

2020-09-17 Thread Dave Hansen
On 9/17/20 2:58 PM, Liang, Kan wrote: > The user space perf tool looks like a better place for this kind of > warning. The perf tool knows the total number of the samples. It also > knows the number of the page size 0 samples. We can set a threshold, > e.g., 90%. If 90% of the samples have the

Re: [PATCH v2 7/9] iomap: Convert write_count to write_bytes_pending

2020-09-17 Thread Darrick J. Wong
On Fri, Sep 11, 2020 at 12:47:05AM +0100, Matthew Wilcox (Oracle) wrote: > Instead of counting bio segments, count the number of bytes submitted. > This insulates us from the block layer's definition of what a 'same page' > is, which is not necessarily clear once THPs are involved. > >

Re: linux-next: build failure after merge of the rcu tree

2020-09-17 Thread Paul E. McKenney
On Thu, Sep 17, 2020 at 03:19:09PM +1000, Stephen Rothwell wrote: > Hi all, > > After merging the rcu tree, today's linux-next build (powerpc > ppc64_defconfig) failed like this: > > In file included from kernel/rcu/update.c:578: > kernel/rcu/tasks.h:601:20: error: static declaration of >

Re: [PATCH v2 5/9] iomap: Support arbitrarily many blocks per page

2020-09-17 Thread Darrick J. Wong
On Fri, Sep 11, 2020 at 12:47:03AM +0100, Matthew Wilcox (Oracle) wrote: > Size the uptodate array dynamically to support larger pages in the > page cache. With a 64kB page, we're only saving 8 bytes per page today, > but with a 2MB maximum page size, we'd have to allocate more than 4kB > per

Re: [PATCH V7 1/4] perf/core: Add PERF_SAMPLE_DATA_PAGE_SIZE

2020-09-17 Thread Liang, Kan
On 9/17/2020 5:24 PM, Dave Hansen wrote: On 9/17/20 2:16 PM, Liang, Kan wrote: One last concern as I look at this: I wish it was a bit more future-proof.  There are lots of weird things folks are trying to do with the page tables, like Address Space Isolation.  For instance, if you get a

Re: [PATCH] platform/x86: fix kconfig dependency warning for LG_LAPTOP

2020-09-17 Thread mark gross
Acked-by: mark gross --mark On Tue, Sep 15, 2020 at 12:09:23PM +0300, Necip Fazil Yildiran wrote: > When LG_LAPTOP is enabled and NEW_LEDS is disabled, it results in the > following Kbuild warning: > > WARNING: unmet direct dependencies detected for LEDS_CLASS > Depends on [n]: NEW_LEDS [=n]

Re: [PATCH 5/8] x86/mce: Avoid tail copy when machine check terminated a copy from user

2020-09-17 Thread Luck, Tony
On Thu, Sep 17, 2020 at 07:04:06PM +0200, Borislav Petkov wrote: > So actually, I'm thinking: > > .LMCE_during_user_access: > mov $-ENODEV, %eax > ASM_CLAC > ret > > I have no clue which error code we should put there but it should be an > error code which tells you not to

Re: [PATCH] platform/x86: asus-wmi: Add support for SW_TABLET_MODE on UX360

2020-09-17 Thread mark gross
On Wed, Sep 16, 2020 at 09:12:33PM +0200, Samuel Čavoj wrote: > The UX360CA has a WMI device id 0x00060062, which reports whether the > lid is flipped in tablet mode (1) or in normal laptop mode (0). > > This commit adds a quirk (quirk_asus_use_lid_flip_devid) for devices on > which this WMI

Re: [RFC][Patch v1 1/3] sched/isolation: API to get num of hosekeeping CPUs

2020-09-17 Thread Jacob Keller
On 9/17/2020 1:11 PM, Bjorn Helgaas wrote: > [+cc Ingo, Peter, Juri, Vincent (scheduler maintainers)] > > s/hosekeeping/housekeeping/ (in subject) > > On Wed, Sep 09, 2020 at 11:08:16AM -0400, Nitesh Narayan Lal wrote: >> Introduce a new API num_housekeeping_cpus(), that can be used to

[PATCH] add virtual PHY for PHY-less devices

2020-09-17 Thread Sergej Bauer
From: sba...@blackbox.su Here is a kernel related part of my work which was helps to develop brand new PHY device. It is migth be helpful for developers work with PHY-less lan743x (7431:0011 in my case). It's just a fake virtual PHY which can change speed of network card processing as a

Re: [PATCH] platform/x86: thinkpad_acpi: initialize tp_nvram_state variable

2020-09-17 Thread mark gross
Acked-by: mark gross --mark On Sun, Sep 13, 2020 at 12:02:03PM -0700, t...@redhat.com wrote: > From: Tom Rix > > clang static analysis flags this represenative problem > thinkpad_acpi.c:2523:7: warning: Branch condition evaluates > to a garbage value > if (!oldn->mute || >

[PATCH v3] nfs: remove incorrect fallthrough label

2020-09-17 Thread Nick Desaulniers
There is no case after the default from which to fallthrough to. Clang will error in this case (unhelpfully without context, see link below) and GCC will with -Wswitch-unreachable. The previous commit should have just replaced the comment with a break statement. If we consider implicit

Re: [PATCH net] ethtool: add and use message type for tunnel info reply

2020-09-17 Thread Andrew Lunn
> On the other hand, the enums are part of userspace API so I better take > a closer look to make sure we don't run into some trouble there. Hi Michal Yes, that is what i was thinking about. But i guess you can pass a tagged enum to a function expecting an int and the compiler will silently cast

Re: [PATCH v2] nfs: remove incorrect fallthrough label

2020-09-17 Thread Nick Desaulniers
On Wed, Sep 16, 2020 at 1:19 PM Joe Perches wrote: > > On Wed, 2020-09-16 at 13:02 -0700, Nick Desaulniers wrote: > > * (call of function with __attribute__(__noreturn__)) > > I guess panic counts. I count 11 of those. > > Are there any other uses of functions with __noreturn > in switch/case

Re: [PATCH 1/4] mm: Trial do_wp_page() simplification

2020-09-17 Thread Peter Xu
On Thu, Sep 17, 2020 at 01:35:56PM -0700, Linus Torvalds wrote: > For that to happen, we'd need to have the vma flag so that we wouldn't > have any worry about non-pinners, but as you suggested, I think even > just a mm-wide counter - or flag - to deal with the fast-bup case is > likely perfectly

Re: [PATCH v2 1/1] fpga: dfl: afu: harden port enable logic

2020-09-17 Thread Moritz Fischer
On Thu, Sep 17, 2020 at 01:28:22PM -0700, Tom Rix wrote: > > On 9/17/20 11:32 AM, Russ Weight wrote: > > Port enable is not complete until ACK = 0. Change > > __afu_port_enable() to guarantee that the enable process > > is complete by polling for ACK == 0. > > > > Signed-off-by: Russ Weight

Re: [PATCH 3/3] platform/x86: Intel PMT Crashlog capability driver

2020-09-17 Thread Alexander Duyck
On Thu, Sep 17, 2020 at 5:12 AM Hans de Goede wrote: > > Hi, > > On 9/15/20 12:35 AM, Alexander Duyck wrote: > > On Mon, Sep 14, 2020 at 11:07 AM Alexander Duyck > > wrote: > >> > >> On Mon, Sep 14, 2020 at 6:42 AM Hans de Goede wrote: > >>> > >>> Hi, > >>> > >>> On 9/11/20 9:45 PM, David E.

Re: [PATCH net] ethtool: add and use message type for tunnel info reply

2020-09-17 Thread Michal Kubecek
On Thu, Sep 17, 2020 at 03:41:51AM +0200, Andrew Lunn wrote: > On Thu, Sep 17, 2020 at 01:04:10AM +0200, Michal Kubecek wrote: > > Tunnel offload info code uses ETHTOOL_MSG_TUNNEL_INFO_GET message type (cmd > > field in genetlink header) for replies to tunnel info netlink request, i.e. > > the

[PATCH 3/3 v4] KVM: SVM: Don't flush cache if hardware enforces cache coherency across encryption domains

2020-09-17 Thread Krish Sadhukhan
In some hardware implementations, coherency between the encrypted and unencrypted mappings of the same physical page in a VM is enforced. In such a system, it is not required for software to flush the VM's page from all CPU caches in the system prior to changing the value of the C-bit for the

[PATCH 2/3 v4] x86: AMD: Don't flush cache if hardware enforces cache coherency across encryption domnains

2020-09-17 Thread Krish Sadhukhan
In some hardware implementations, coherency between the encrypted and unencrypted mappings of the same physical page is enforced. In such a system, it is not required for software to flush the page from all CPU caches in the system prior to changing the value of the C-bit for the page.

[PATCH 1/3 v4] x86: AMD: Add hardware-enforced cache coherency as a CPUID feature

2020-09-17 Thread Krish Sadhukhan
In some hardware implementations, coherency between the encrypted and unencrypted mappings of the same physical page is enforced. In such a system, it is not required for software to flush the page from all CPU caches in the system prior to changing the value of the C-bit for a page. This

Re: [RFC PATCH] Add bridge driver to connect sensors to CIO2 device via software nodes on ACPI platforms

2020-09-17 Thread Daniel Scally
On 17/09/2020 15:14, Andy Shevchenko wrote: > On Thu, Sep 17, 2020 at 4:53 PM Dan Scally wrote: >> Hi Andy, thanks for input (as always) > You're welcome! I'm really impressed by your activity in this area. Thanks - it's pretty fun so far >>> Ah, I think you misinterpreted the meaning of above.

Re: [PATCH V7 1/4] perf/core: Add PERF_SAMPLE_DATA_PAGE_SIZE

2020-09-17 Thread Liang, Kan
On 9/17/2020 3:00 PM, Dave Hansen wrote: On 9/17/20 6:52 AM, kan.li...@linux.intel.com wrote: + mm = current->mm; + if (!mm) { + /* +* For kernel threads and the like, use init_mm so that +* we can find kernel memory. +

[PATCH 0/3 v4] x86: AMD: Don't flush cache if hardware enforces cache coherency across encryption domains

2020-09-17 Thread Krish Sadhukhan
v3 -> v4: 1. Patch# 1 from v3 has been dropped. 2. The CPUID feature for hardware-enforced cache coherency has been renamed. [PATCH 1/3 v4] x86: AMD: Add hardware-enforced cache coherency as a [PATCH 2/3 v4] x86: AMD: Don't flush cache if hardware enforces cache [PATCH

Re: [PATCH] i40e: Fix use of uninitialized variable

2020-09-17 Thread Nguyen, Anthony L
On Wed, 2020-09-16 at 21:49 +0100, Alex Dewar wrote: > In i40e_clean_rx_irq_zc(), the variable failure is only set when a > condition is met, but then its value is used unconditionally. Fix > this. > > Addresses-Coverity: 1496986 ("Uninitialized value") > Fixes: 8cbf74149903 ("i40e, xsk: move

[PATCH] mmzone: clean code by removing unused macro parameter

2020-09-17 Thread mateusznosek0
From: Mateusz Nosek Previously 'for_next_zone_zonelist_nodemask' macro parameter 'zlist' was unused so this patch removes it. Signed-off-by: Mateusz Nosek --- include/linux/mmzone.h | 2 +- mm/page_alloc.c| 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH v4 1/5] x86/asm: move the raw asm in iosubmit_cmds512() to special_insns.h

2020-09-17 Thread Dave Jiang
The MOVDIR64B instruction can be used by other wrapper instructions. Move the core asm code to special_insns.h and have iosubmit_cmds512() call the core asm function. Signed-off-by: Dave Jiang Reviewed-by: Tony Luck --- arch/x86/include/asm/io.h| 17 +++--

Re: [BlueZ PATCH v2 1/6] Bluetooth: Update Adv monitor count upon removal

2020-09-17 Thread Luiz Augusto von Dentz
Hi Howard, On Thu, Sep 17, 2020 at 1:47 AM Howard Chung wrote: > > From: Miao-chen Chou > > This fixes the count of Adv monitor upon monitor removal. > > The following test was performed. > - Start two btmgmt consoles, issue a btmgmt advmon-remove command on one > console and observe a

[PATCH v4 0/5] Add shared workqueue support for idxd driver

2020-09-17 Thread Dave Jiang
v4: - Rebased against latest dmaengine/next tree - Split out enqcmd and pasid dependency. V3: - Rebased against latest dmaengine/next tree. - Updated API doc with new kernel version and dates. - Changed to allow driver to load without ENQCMD support. - Break out some patches that can be sent

[PATCH v4 2/5] x86/asm: add enqcmds() to support ENQCMDS instruction

2020-09-17 Thread Dave Jiang
Currently, the MOVDIR64B instruction is used to atomically submit 64-byte work descriptors to devices. Although it can encounter errors like faults, MOVDIR64B can not report back on errors from the device itself. This means that MOVDIR64B users need to separately interact with a device to see if a

[PATCH v4 5/5] dmaengine: idxd: add ABI documentation for shared wq

2020-09-17 Thread Dave Jiang
Add the sysfs attribute bits in ABI/stable for shared wq support. Signed-off-by: Jing Lin Signed-off-by: Dave Jiang Reviewed-by: Tony Luck Reviewed-by: Dan Williams --- Documentation/ABI/stable/sysfs-driver-dma-idxd | 14 ++ 1 file changed, 14 insertions(+) diff --git

[PATCH v4 4/5] dmaengine: idxd: clean up descriptors with fault error

2020-09-17 Thread Dave Jiang
Add code to "complete" a descriptor when the descriptor or its completion address hit a fault error when SVA mode is being used. This error can be triggered due to bad programming by the user. A lock is introduced in order to protect the descriptor completion lists since the fault handler will run

[PATCH tip/core/rcu 1/8] rcu-tasks: Prevent complaints of unused show_rcu_tasks_classic_gp_kthread()

2020-09-17 Thread paulmck
From: "Paul E. McKenney" Commit 8344496e8b49 ("rcu-tasks: Conditionally compile show_rcu_tasks_gp_kthreads()") introduced conditional compilation of several functions, but forgot one occurrence of show_rcu_tasks_classic_gp_kthread() that causes the compiler to warn of an unused static function.

[PATCH tip/core/rcu 3/8] rcu-tasks: Use more aggressive polling for RCU Tasks Trace

2020-09-17 Thread paulmck
From: "Paul E. McKenney" The RCU Tasks Trace grace periods are too slow, as in 40x slower than those of RCU Tasks. This is due to my having assumed a one-second grace period was OK, and thus not having optimized any further. This commit provides the first step in this optimization process,

[PATCH tip/core/rcu 6/8] rcu-tasks: Fix grace-period/unlock race in RCU Tasks Trace

2020-09-17 Thread paulmck
From: "Paul E. McKenney" The more intense grace-period processing resulting from the 50x RCU Tasks Trace grace-period speedups exposed the following race condition: o Task A running on CPU 0 executes rcu_read_lock_trace(), entering a read-side critical section. o When Task

[PATCH tip/core/rcu 4/8] rcu-tasks: Selectively enable more RCU Tasks Trace IPIs

2020-09-17 Thread paulmck
From: "Paul E. McKenney" Many workloads are quite sensitive to IPIs, and such workloads should build kernels with CONFIG_TASKS_TRACE_RCU_READ_MB=y to prevent RCU Tasks Trace from using them under normal conditions. However, other workloads are quite happy to permit more IPIs if doing so makes

[PATCH tip/core/rcu 5/8] rcu-tasks: Shorten per-grace-period sleep for RCU Tasks Trace

2020-09-17 Thread paulmck
From: "Paul E. McKenney" The various RCU tasks flavors currently wait 100 milliseconds between each grace period in order to prevent CPU-bound loops and to favor efficiency over latency. However, RCU Tasks Trace needs to have a grace-period latency of roughly 25 milliseconds, which is

Re: [PATCH v2 1/4] ARM: dts: imx6qdl-gw5xxx: correct interrupt flags

2020-09-17 Thread Tim Harvey
On Thu, Sep 17, 2020 at 11:54 AM Krzysztof Kozlowski wrote: > > GPIO_ACTIVE_x flags are not correct in the context of interrupt flags. > These are simple defines so they could be used in DTS but they will not > have the same meaning: > 1. GPIO_ACTIVE_HIGH = 0 = IRQ_TYPE_NONE > 2. GPIO_ACTIVE_LOW

[PATCH tip/core/rcu 2/8] rcu-tasks: Mark variables static

2020-09-17 Thread paulmck
From: "Paul E. McKenney" The n_heavy_reader_attempts, n_heavy_reader_updates, and n_heavy_reader_ofl_updates variables are not used outside of their translation unit, so this commit marks them static. Signed-off-by: Paul E. McKenney --- kernel/rcu/tasks.h | 6 +++--- 1 file changed, 3

[PATCH tip/core/rcu 7/8] rcu-tasks: Fix low-probability task_struct leak

2020-09-17 Thread paulmck
From: "Paul E. McKenney" When rcu_tasks_trace_postgp() function detects an RCU Tasks Trace CPU stall, it adds all tasks blocking the current grace period to a list, invoking get_task_struct() on each to prevent them from being freed while on the list. It then traverses that list, printing

[PATCH tip/core/rcu 8/8] rcu-tasks: Enclose task-list scan in rcu_read_lock()

2020-09-17 Thread paulmck
From: "Paul E. McKenney" The rcu_tasks_trace_postgp() function uses for_each_process_thread() to scan the task list without the benefit of RCU read-side protection, which can result in use-after-free errors on task_struct structures. This error was missed because the TRACE01 rcutorture scenario

Re: [PATCH RFC tip/core/rcu 0/4] Accelerate RCU Tasks Trace updates

2020-09-17 Thread Paul E. McKenney
Hello! This series accelerates RCU Tasks Trace updates, reducing the average grace-period latencies from about a second to about 20 milliseconds on my x86 laptop. These are benchmark numbers, based on a previously posted patch to rcuscale.c [1] running on my x86 laptop. Additional patches

[PATCH v2] mm/migrate: correct thp migration stats.

2020-09-17 Thread Zi Yan
From: Zi Yan PageTransHuge returns true for both thp and hugetlb, so thp stats was counting both thp and hugetlb migrations. Exclude hugetlb migration by setting is_thp variable right. Clean up thp handling code too when we are there. Fixes: 1a5bae25e3cf ("mm/vmstat: add events for THP

Re: [PATCH v7 6/7] KVM: x86: Introduce MSR filtering

2020-09-17 Thread Aaron Lewis
+4.126 KVM_X86_SET_MSR_FILTER + + +:Capability: KVM_X86_SET_MSR_FILTER +:Architectures: x86 +:Type: vm ioctl +:Parameters: struct kvm_msr_filter +:Returns: 0 on success, < 0 on error + +:: + + struct kvm_msr_filter_range { + #define KVM_MSR_FILTER_READ (1 << 0) +

Re: [PATCH -next 0/4] ubifs: Fix some kernel-doc warnings for ubifs

2020-09-17 Thread Richard Weinberger
On Mon, Sep 14, 2020 at 4:06 PM Wang Hai wrote: > > In the ubifs module, there are some kernel-doc warnings, > this series of patches can fix these warnings. > > Wang Hai (4): > ubifs: Fix 'hash' kernel-doc warning in auth.c > ubifs: Fix some kernel-doc warnings in gc.c > ubifs: Fix some

Re: [PATCH v2 3/4] i2c: designware: Allow SMBus block reads up to 255 bytes in length

2020-09-17 Thread Wolfram Sang
On Wed, Sep 16, 2020 at 10:22:55PM -0700, Sultan Alsawaf wrote: > From: Sultan Alsawaf > > According to the SMBus 3.0 protocol specification, block transfer limits > were increased from 32 bytes to 255 bytes. Remove the obsolete 32-byte > limitation. Sadly, it is not that easy. We are trying to

Re: [PATCH] mm/migrate: correct thp migration stats.

2020-09-17 Thread Zi Yan
On 17 Sep 2020, at 16:59, Daniel Jordan wrote: > On Thu, Sep 17, 2020 at 04:27:29PM -0400, Zi Yan wrote: >> From: Zi Yan >> >> PageTransHuge returns true for both thp and hugetlb, so thp stats was >> counting both thp and hugetlb migrations. Exclude hugetlb migration by >> setting is_thp

Re: [PATCH 11/13] ubifs: Tell the VFS that readpage was synchronous

2020-09-17 Thread Richard Weinberger
- Ursprüngliche Mail - > Von: "Matthew Wilcox" > An: "linux-fsdevel" > CC: "Matthew Wilcox" , "linux-mm" , > v9fs-develo...@lists.sourceforge.net, > "linux-kernel" , linux-...@lists.infradead.org, > "ceph-devel" > , linux-c...@vger.kernel.org, > ecryp...@vger.kernel.org, "linux-um" >

Re: [PATCH] mm/migrate: correct thp migration stats.

2020-09-17 Thread Daniel Jordan
On Thu, Sep 17, 2020 at 04:27:29PM -0400, Zi Yan wrote: > From: Zi Yan > > PageTransHuge returns true for both thp and hugetlb, so thp stats was > counting both thp and hugetlb migrations. Exclude hugetlb migration by > setting is_thp variable right. Yeah, shoot. > Fixes: 1a5bae25e3cf

Re: [PATCH 1/2] ubifs: xattr: Fix some potential memory leaks while iterating entries

2020-09-17 Thread Richard Weinberger
On Mon, Sep 14, 2020 at 5:25 AM Zhihao Cheng wrote: > 在 2020/9/14 3:08, Richard Weinberger 写道: > > On Mon, Jun 1, 2020 at 11:11 AM Zhihao Cheng > > wrote: > The kill_xattrs process is more intuitive without the pxent. However, > the release process for the memory (stores xent->name) is similar

Re: linux-next: Tree for Sep 17 (net/ipv4/devinet.c)

2020-09-17 Thread Randy Dunlap
On 9/17/20 3:23 AM, Stephen Rothwell wrote: > Hi all, > > Changes since 20200916: > Maybe some older versions of gcc just can't handle IS_ENABLED() inside an if (expression) very well. > gcc --version gcc (SUSE Linux) 7.5.0 This patch: commit 9efd6a3cecdde984d67e63d17fe6af53c7c50968 Author:

[GIT PULL] percpu fix for v5.9-rc6

2020-09-17 Thread Dennis Zhou
Hi Linus, This is a fix for the first chunk size calculation where the variable length array incorrectly used # of longs instead of bytes of longs. This came in as a code fix and not a bug report, so I don't think it was widely problematic. I believe it worked out due to it being memblock memory

Re: [PATCH] arm64: dts: ti: k3-j721e: Rename mux header and update macro names

2020-09-17 Thread Peter Rosin
Hi! A few questions. I don't really care, but if someone does then at least I highlighted few thing out. On 2020-09-17 18:16, Roger Quadros wrote: > We intend to use one header file for SERDES MUX for all > TI SoCs so rename the header file. > > The exsting macros are too generic. Prefix them

Re: [PATCH] media: rc: gpio-ir-recv: add QoS support for cpuidle system

2020-09-17 Thread Sean Young
Hi Joakim, On Thu, Sep 17, 2020 at 09:12:32AM +, Joakim Zhang wrote: > > > -Original Message- > > From: Sean Young > > Sent: 2020年9月15日 17:34 > > To: Joakim Zhang > > Cc: mche...@kernel.org; linux-me...@vger.kernel.org; > > linux-kernel@vger.kernel.org; dl-linux-imx > > Subject:

Re: [PATCH v4] perf metricgroup: Fix uncore metric expressions

2020-09-17 Thread Arnaldo Carvalho de Melo
Em Thu, Sep 17, 2020 at 01:18:07PM -0700, Ian Rogers escreveu: > A metric like DRAM_BW_Use has on SkylakeX events uncore_imc/cas_count_read/ > and uncore_imc/case_count_write/. These events open 6 events per socket > with pmu names of uncore_imc_[0-5]. The current metric setup code in >

Re: [RFC PATCH v7 11/23] sched/fair: core wide cfs task priority comparison

2020-09-17 Thread Vineeth Pillai
+ +bool cfs_prio_less(struct task_struct *a, struct task_struct *b) +{ +   struct sched_entity *se_a = >se, *se_b = >se; +   struct cfs_rq *cfs_rq_a, *cfs_rq_b; +   u64 vruntime_a, vruntime_b; + +#ifdef CONFIG_FAIR_GROUP_SCHED +   while (!is_same_tg(se_a, se_b)) { + 

Re: [PATCH 1/4] mm: Trial do_wp_page() simplification

2020-09-17 Thread Linus Torvalds
On Thu, Sep 17, 2020 at 1:06 PM Jason Gunthorpe wrote: > > Given that this is a user visible regression, it is nearly rc6, what > do you prefer for next steps? Since I had to deal with the page lock performance regression too, and I think we have a fairly simple way forward, I'd rather do an rc8

[PATCH] mm/migrate: correct thp migration stats.

2020-09-17 Thread Zi Yan
From: Zi Yan PageTransHuge returns true for both thp and hugetlb, so thp stats was counting both thp and hugetlb migrations. Exclude hugetlb migration by setting is_thp variable right. Fixes: 1a5bae25e3cf ("mm/vmstat: add events for THP migration without split") Signed-off-by: Zi Yan Cc:

Re: linux-next: Tree for Sep 17 (netdevice.h: net_has_fallback_tunnels when SYSCTL is not set)

2020-09-17 Thread Randy Dunlap
On 9/17/20 3:23 AM, Stephen Rothwell wrote: > Hi all, > > Changes since 20200916: > I am seeing build errors when CONFIG_SYSCTL is not set: ld: net/ipv4/ip_tunnel.o: in function `ip_tunnel_init_net': ip_tunnel.c:(.text+0x2ea0): undefined reference to `sysctl_fb_tunnels_only_for_init_net' ld:

Re: [PATCH v2 1/1] fpga: dfl: afu: harden port enable logic

2020-09-17 Thread Tom Rix
On 9/17/20 11:32 AM, Russ Weight wrote: > Port enable is not complete until ACK = 0. Change > __afu_port_enable() to guarantee that the enable process > is complete by polling for ACK == 0. > > Signed-off-by: Russ Weight > --- > drivers/fpga/dfl-afu-error.c | 2 +- >

[PATCH] KVM: SVM: Use a separate vmcb for the nested L2 guest

2020-09-17 Thread Cathy Avery
svm->vmcb will now point to either a separate vmcb L1 ( not nested ) or L2 vmcb ( nested ). Issues: 1) There is some wholesale copying of vmcb.save and vmcb.contol areas which will need to be refined. 2) There is a workaround in nested_svm_vmexit() where if (svm->vmcb01->control.asid ==

Re: [PATCH v6 7/7] perf diff: Support hot streams comparison

2020-09-17 Thread Arnaldo Carvalho de Melo
Em Fri, Sep 11, 2020 at 04:03:53PM +0800, Jin Yao escreveu: > This patch enables perf-diff with "--stream" option. > > "--stream": Enable hot streams comparison > > Now let's see examples. > > perf record -b ... Generate perf.data.old with branch data > perf record -b ... Generate

Re: [PATCH 1/4] mm: Trial do_wp_page() simplification

2020-09-17 Thread Jason Gunthorpe
On Thu, Sep 17, 2020 at 01:19:02PM -0700, John Hubbard wrote: > On 9/17/20 1:06 PM, Jason Gunthorpe wrote: > > On Thu, Sep 17, 2020 at 12:42:11PM -0700, Linus Torvalds wrote: > ... > > > Is there possibly somethign else we can filter on than just > > > GUP_PIN_COUNTING_BIAS? Because it could be as

<    7   8   9   10   11   12   13   14   15   16   >