[PATCH] net: systemport: Fix memleak in bcm_sysport_probe

2020-08-23 Thread Dinghao Liu
When devm_kcalloc() fails, dev should be freed just like what we've done in the subsequent error paths. Fixes: 7b78be48a8eb6 ("net: systemport: Dynamically allocate number of TX rings") Signed-off-by: Dinghao Liu --- drivers/net/ethernet/broadcom/bcmsysport.c | 6 -- 1 file changed, 4

Re: [RESEND PATCH v3 7/8] mtd: spi-nor: Convert cadence-quadspi to use spi-mem framework

2020-08-23 Thread Jan Kiszka
On 01.06.20 09:04, Vignesh Raghavendra wrote: > From: Ramuthevar Vadivel Murugan > > Move cadence-quadspi driver to use spi-mem framework. This is required > to make the driver support for SPI NAND flashes in future. > > Driver is feature compliant with existing SPI NOR version. > >

[PATCH] iio: dac: ad5593r: Dynamically set AD5593R channel modes

2020-08-23 Thread William Sung
To use ad5593r more flexibly, we use the module parameter to setting the channel modes dynamically whenever the module probe up. Users can pass the channel modes to the module parameter for allocating the functionality of channels as desired. For example: * Use in the kernel command line: Users

Re: [PATCH 07/22] dt-bindings: serial: fsl-imx-uart: imx-pwm: Add i.MX 8M compatibles

2020-08-23 Thread Sascha Hauer
The subject contains a "imx-pwm", presumably from the last patch. Sascha On Sun, Aug 23, 2020 at 06:15:35PM +0200, Krzysztof Kozlowski wrote: > DTSes with new i.MX 8M SoCs introduce their own compatibles so add them > to fix dtbs_check warnings like: > >

[PATCH] net: hns: Fix memleak in hns_nic_dev_probe

2020-08-23 Thread Dinghao Liu
hns_nic_dev_probe allocates ndev, but not free it on two error handling paths, which may lead to memleak. Fixes: 63434888aaf1b ("net: hns: net: hns: enet adds support of acpi") Signed-off-by: Dinghao Liu --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 9 ++--- 1 file changed, 6

Re: [RFC PATCH 1/5] printk: implement pr_cont_t

2020-08-23 Thread Sergey Senozhatsky
On (20/08/20 01:32), John Ogness wrote: > +#define CONT_LINE_MAX LOG_LINE_MAX > +#define CONT_BUF_COUNT 10 > +static char cont_buf[CONT_BUF_COUNT][CONT_LINE_MAX]; > +static DECLARE_BITMAP(cont_buf_map, CONT_BUF_COUNT); > + > +static int get_cont_buf(void) > +{ > + int bit; > + > + do { > +

Re: [PATCH 02/22] dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property

2020-08-23 Thread Sascha Hauer
On Sun, Aug 23, 2020 at 06:15:30PM +0200, Krzysztof Kozlowski wrote: > The GPIO controller node can have gpio-ranges property. This fixes > dtbs_check warnings like: > > arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: gpio@3020: > 'gpio-ranges' does not match any of the regexes:

[PATCH] driver/pci: reduce the single block time in pci_read_config

2020-08-23 Thread Jiang Biao
From: Jiang Biao pci_read_config() could block several ms in kernel space, mainly caused by the while loop to call pci_user_read_config_dword(). Singel pci_user_read_config_dword() loop could consume 130us+, |pci_user_read_config_dword() { |

Re: [PATCH v1 09/10] powerpc/pseries/iommu: Make use of DDW even if it does not map the partition

2020-08-23 Thread Alexey Kardashevskiy
On 18/08/2020 09:40, Leonardo Bras wrote: > As of today, if the biggest DDW that can be created can't map the whole > partition, it's creation is skipped and the default DMA window > "ibm,dma-window" is used instead. > > DDW is 16x bigger than the default DMA window, 16x only under very

Re: [PATCH v2] KVM: LAPIC: Narrow down the kick target vCPU

2020-08-23 Thread Wanpeng Li
On Mon, 24 Aug 2020 at 09:03, Wanpeng Li wrote: > > From: Wanpeng Li > > The kick after setting KVM_REQ_PENDING_TIMER is used to handle the timer > fires on a different pCPU which vCPU is running on, this kick is expensive > since memory barrier, rcu, preemption disable/enable operations. We

Re: [patch RFC 23/38] x86/xen: Rework MSI teardown

2020-08-23 Thread Jürgen Groß
On 21.08.20 02:24, Thomas Gleixner wrote: X86 cannot store the irq domain pointer in struct device without breaking XEN because the irq domain pointer takes precedence over arch_*_msi_irqs() fallbacks. XENs MSI teardown relies on default_teardown_msi_irqs() which invokes

Re: [PATCH v1 08/10] powerpc/pseries/iommu: Add ddw_property_create() and refactor enable_ddw()

2020-08-23 Thread Alexey Kardashevskiy
On 18/08/2020 09:40, Leonardo Bras wrote: > Code used to create a ddw property that was previously scattered in > enable_ddw() is now gathered in ddw_property_create(), which deals with > allocation and filling the property, letting it ready for > of_property_add(), which now occurs in

Re: [patch RFC 24/38] x86/xen: Consolidate XEN-MSI init

2020-08-23 Thread Jürgen Groß
On 21.08.20 02:24, Thomas Gleixner wrote: X86 cannot store the irq domain pointer in struct device without breaking XEN because the irq domain pointer takes precedence over arch_*_msi_irqs() fallbacks. To achieve this XEN MSI interrupt management needs to be wrapped into an irq domain. Move

linux-next: Tree for Aug 24

2020-08-23 Thread Stephen Rothwell
Hi all, Changes since 20200821: The hwmon-staging tree gained a build failure so I used the version from next-20200821. The bpf-next tree still had its build failure for which I disabled BPF_PRELOAD. It also gained a conflict against Linus' tree. Non-merge commits (relative to Linus' tree):

Re: [patch RFC 22/38] x86/xen: Make xen_msi_init() static and rename it to xen_hvm_msi_init()

2020-08-23 Thread Jürgen Groß
On 21.08.20 02:24, Thomas Gleixner wrote: The only user is in the same file and the name is too generic because this function is only ever used for HVM domains. Signed-off-by: Thomas Gleixner Cc: Konrad Rzeszutek Wilk Cc:linux-...@vger.kernel.org Cc:xen-de...@lists.xenproject.org Cc: Juergen

Re: [GIT PULL] irqchip fixes for 5.9, take #1

2020-08-23 Thread Lokesh Vutla
Hi, On 18/08/20 8:44 pm, Marc Zyngier wrote: > Hi Thomas, > > As -rc1 is out, here's the first batches of fixes for 5.9. The most > important one is a fix for a typo that broke modular irqchips using > the brand new set of macros. Oh hum... > > The rest of it is what I was hinting at when I

Re: [PATCH V2] arm64/hotplug: Improve memory offline event notifier

2020-08-23 Thread Anshuman Khandual
On 08/24/2020 09:34 AM, Anshuman Khandual wrote: > This brings about three different changes to the sole memory event notifier > for arm64 platform and improves it's robustness while also enhancing debug > capabilities during potential memory offlining error conditions. > > This moves the

Re: [PATCH] tools/power turbostat: call pread64 in kernel directly

2020-08-23 Thread Liwei Song
On 8/24/20 04:54, Alexander Monakov wrote: > Hi, > > I am not the original submitter, but I have answers and a proper patch :) > > On Fri, 21 Aug 2020, Len Brown wrote: > >> Re: offset size >> >> The offsets on this file are the MSR offsets. >> What MSR are you trying to access at offset

[PATCH V2] arm64/hotplug: Improve memory offline event notifier

2020-08-23 Thread Anshuman Khandual
This brings about three different changes to the sole memory event notifier for arm64 platform and improves it's robustness while also enhancing debug capabilities during potential memory offlining error conditions. This moves the memory notifier registration bit earlier in the boot process from

Re: [PATCH v1 06/10] powerpc/pseries/iommu: Add ddw_list_add() helper

2020-08-23 Thread Alexey Kardashevskiy
On 18/08/2020 09:40, Leonardo Bras wrote: > There are two functions adding DDW to the direct_window_list in a > similar way, so create a ddw_list_add() to avoid duplicity and > simplify those functions. > > Also, on enable_ddw(), add list_del() on out_free_window to allow > removing the window

Re: [PATCH v2] h8300: Replace by

2020-08-23 Thread Yoshinori Sato
On Wed, 19 Aug 2020 21:29:25 +0900, Geert Uytterhoeven wrote: > > The H8/300 platform code is not a clock provider, and just needs to call > of_clk_init(). > > Hence it can include instead of . > > Signed-off-by: Geert Uytterhoeven > Reviewed-by: Stephen Boyd > --- > v2: > - Add

Re: [PATCH v1 07/10] powerpc/pseries/iommu: Allow DDW windows starting at 0x00

2020-08-23 Thread Alexey Kardashevskiy
On 18/08/2020 09:40, Leonardo Bras wrote: > enable_ddw() currently returns the address of the DMA window, which is > considered invalid if has the value 0x00. > > Also, it only considers valid an address returned from find_existing_ddw > if it's not 0x00. > > Changing this behavior makes

[GIT PULL] fallthrough pseudo-keyword macro conversions for 5.9-rc3

2020-08-23 Thread Gustavo A. R. Silva
Hi Linus, Not sure what the problem was with my pull-request for -rc2. So, I'm giving this a second try because I think it is worth it. I have build-tested this patch on 10 different architectures: x86_64, i386, arm64, powerpc, s390, sparc64, sh, m68k, powerpc64 and alpha (allyesconfig/

Re: [PATCH v2] h8300: dts: Fix /chosen:stdout-path

2020-08-23 Thread Yoshinori Sato
On Wed, 19 Aug 2020 21:28:38 +0900, Geert Uytterhoeven wrote: > > arch/h8300/boot/dts/h8s_sim.dts:11.3-25: Warning > (chosen_node_stdout_path): /chosen:stdout-path: property is not a string > arch/h8300/boot/dts/h8300h_sim.dts:11.3-25: Warning > (chosen_node_stdout_path):

[PATCH v10 2/2] Add PWM fan controller driver for LGM SoC

2020-08-23 Thread Rahul Tanwar
Intel Lightning Mountain(LGM) SoC contains a PWM fan controller. This PWM controller does not have any other consumer, it is a dedicated PWM controller for fan attached to the system. Add driver for this PWM fan controller. Signed-off-by: Rahul Tanwar --- drivers/pwm/Kconfig | 11 ++

[PATCH v10 0/2] pwm: intel: Add PWM driver for a new SoC

2020-08-23 Thread Rahul Tanwar
Patch 1 adds dt binding document in YAML format. Patch 2 add PWM fan controller driver for LGM SoC. v10: - Removed unused of_device.h and added platform_device.h & mod_devicetable.h v9: - Address code quality related review concerns (Andy Shevchenko) - Use devm_add_action_or_reset() instead of

[PATCH v10 1/2] Add DT bindings YAML schema for PWM fan controller of LGM SoC

2020-08-23 Thread Rahul Tanwar
Intel's LGM(Lightning Mountain) SoC contains a PWM fan controller which is only used to control the fan attached to the system. This PWM controller does not have any other consumer other than fan. Add DT bindings documentation for this PWM fan controller. Signed-off-by: Rahul Tanwar ---

Re: [PATCH] media: staging/intel-ipu3: css: Correctly reset some memory

2020-08-23 Thread Bingbu Cao
Thanks for the patch. On 8/22/20 9:11 PM, Christophe JAILLET wrote: > The intent here is to reset the whole 'scaler_coeffs_luma' array, not just > the first element. > > Fixes:e0a5b744 ("media: staging/intel-ipu3: css: Compute and > program ccs") > Signed-off-by: Christophe JAILLET

[PATCH RESEND 02/10] crypto: hisilicon/zip - unify naming style for functions and macros

2020-08-23 Thread Yang Shen
1.Add prefix 'HZIP' for some macros 2.Add prefix 'hisi_zip' for some functions Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 35 + 1 file changed, 18 insertions(+), 17 deletions(-) diff --git

[PATCH RESEND 06/10] crypto: hisilicon/zip - add print for error branch

2020-08-23 Thread Yang Shen
Add print for some error branches. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 68 +++ drivers/crypto/hisilicon/zip/zip_main.c | 8 ++-- 2 files changed, 56 insertions(+), 20 deletions(-) diff --git

[PATCH RESEND 08/10] crypto: hisilicon/zip - move some private macros from 'zip.h' to 'zip_crypto.c'

2020-08-23 Thread Yang Shen
Some macros which are defined in 'zip.h' are related to the struct 'hisi_zip_sqe' and are only used in 'zip_crypto.c'. So move them from 'zip.h' to 'zip_crypto.c'. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip.h| 14 --

[PATCH RESEND 03/10] crypto: hisilicon/zip - modify debugfs interface parameters

2020-08-23 Thread Yang Shen
From: Shukun Tan Update debugfs interface parameters Signed-off-by: Shukun Tan Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 55 ++--- 1 file changed, 24 insertions(+), 31 deletions(-) diff --git

[PATCH RESEND 04/10] crypto: hisilicon/zip - replace 'sprintf' with 'scnprintf'

2020-08-23 Thread Yang Shen
Replace 'sprintf' with 'scnprintf' to avoid overrun. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/crypto/hisilicon/zip/zip_main.c

[PATCH RESEND 07/10] crypto: hisilicon/zip - fix static check warning

2020-08-23 Thread Yang Shen
Fix some code for PClint warning: Warning - Suspicious Cast Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/crypto/hisilicon/zip/zip_crypto.c

[PATCH RESEND 10/10] crypto: hisilicon/zip - fix some coding styles

2020-08-23 Thread Yang Shen
1.Unified alignment styles 2.Remove unnecessary goto branch 3.Remove address printf Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 13 ++--- drivers/crypto/hisilicon/zip/zip_main.c | 16 2 files changed, 10

[PATCH RESEND 05/10] crypto: hisilicon/zip - use a enum parameter instead of some macros

2020-08-23 Thread Yang Shen
Macros 'QPC_COMP', 'QPC_DECOMP' and 'HZIP_CTX_Q_NUM' are relative and incremental. So, use an enum instead. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 23 +-- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git

[PATCH RESEND 09/10] crypto: hisilicon/zip - supplement some comments

2020-08-23 Thread Yang Shen
Supplement some comments. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/hisilicon/zip/zip_main.c b/drivers/crypto/hisilicon/zip/zip_main.c index

[PATCH RESEND 00/10] crypto: hisilicon/zip - misc clean up

2020-08-23 Thread Yang Shen
This patchset make some clean up: patch 1:remove useless parameters patch 4:replace 'sprintf' with 'scnprintf' patch 7:fix static check warning and the rest patch fix some coding style Resend this patch series because it depends on https://patchwork.kernel.org/cover/11715927/ (crypto:

[PATCH RESEND 01/10] crypto: hisilicon/zip - remove some useless parameters

2020-08-23 Thread Yang Shen
1.Remove the macro 'HZIP_VF_NUM'. 2.Remove 'list' of the struct 'hisi_zip' Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip.h | 1 - drivers/crypto/hisilicon/zip/zip_main.c | 1 - 2 files changed, 2 deletions(-) diff --git

[PATCH] [v2] watchdog: Use put_device on error

2020-08-23 Thread Dinghao Liu
We should use put_device() instead of freeing device directly after device_initialize(). Fixes: cb36e29bb0e4b ("watchdog: initialize device before misc_register") Signed-off-by: Dinghao Liu --- Changelog: v2: - Use put_device() instead of just removing kfree. Move the memleak part to a

[PATCH] ceph: add column 'mds' to show caps in more user friendly

2020-08-23 Thread Yanhu Cao
In multi-mds, the 'caps' debugfs file will have duplicate ino, add the 'mds' column to indicate which mds session the cap belongs to. Signed-off-by: Yanhu Cao --- fs/ceph/debugfs.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/fs/ceph/debugfs.c b/fs/ceph/debugfs.c

RE: [PATCH -next] amdgpu: fix Documentation builds for pm/ file movement

2020-08-23 Thread Quan, Evan
[AMD Official Use Only - Internal Distribution Only] Thanks for fixing this. The patch is reviewed-by: Evan Quan BR Evan -Original Message- From: Randy Dunlap Sent: Monday, August 24, 2020 6:36 AM To: dri-devel ; LKML ; amd-...@lists.freedesktop.org; Deucher, Alexander Cc: Quan,

[PATCH] misc: rtsx: do not setting OC_POWER_DOWN reg in rtsx_pci_init_ocp()

2020-08-23 Thread ricky_wu
From: Ricky Wu this power saving action in rtsx_pci_init_ocp() cause INTEL-NUC6 platform missing card reader Signed-off-by: Ricky Wu --- drivers/misc/cardreader/rtsx_pcr.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/misc/cardreader/rtsx_pcr.c

[PATCH v5] kernel/trace: Add DISALLOW_TRACE_PRINTK make option

2020-08-23 Thread Nicolas Boichat
trace_printk is meant as a debugging tool, and should not be compiled into production code without specific debug Kconfig options enabled, or source code changes, as indicated by the warning that shows up on boot if any trace_printk is called: ** NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE

[PATCH] mtd: maps: physmap: Retain mtd-name property from dts

2020-08-23 Thread Chris Packham
In physmap_flash_of_init() the maps[].name can be populated based on the optional 'linux,mtd-name' property in the dts. Make sure this is retained when filling in the rest of the map[] data. Signed-off-by: Chris Packham --- drivers/mtd/maps/physmap-core.c | 3 ++- 1 file changed, 2

[PATCH] staging: greybus: fix warnings detected by sparse

2020-08-23 Thread Coiby Xu
This patch fix the following warnings from sparse, $ make C=2 drivers/staging/greybus/ drivers/staging/greybus/audio_codec.c:691:36: warning: incorrect type in initializer (different base types) drivers/staging/greybus/audio_codec.c:691:36:expected unsigned long long [usertype] formats

[PATCH] MAINTAINERS: Add maintenance information for MStar Interrupt Controller

2020-08-23 Thread Mark-PK Tsai
Add entry for MStar Interrupt Controller. Signed-off-by: Mark-PK Tsai --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index deaafb617361..8ab08fccd915 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11754,6 +11754,13 @@ Q:

Re: [RESEND PATCH 4/5] iommu: intel: Drop kerneldoc marker from regular comment

2020-08-23 Thread Lu Baolu
Hi, On 8/20/20 1:53 AM, Krzysztof Kozlowski wrote: Fix W=1 compile warnings (invalid kerneldoc): drivers/iommu/intel/dmar.c:389: warning: Function parameter or member 'header' not described in 'dmar_parse_one_drhd' Signed-off-by: Krzysztof Kozlowski Adjust the commit title to

Re: [mm] c566586818: BUG:kernel_hang_in_early-boot_stage,last_printk:Probing_EDD(edd=off_to_disable)...ok

2020-08-23 Thread Rong Chen
On 8/21/20 9:01 AM, Qian Cai wrote: On Tue, Aug 18, 2020 at 08:23:51AM +0800, kernel test robot wrote: Greeting, FYI, we noticed the following commit (built with gcc-9): commit: c5665868183fec689dbab9fb8505188b2c4f0757 ("mm: kmemleak: use the memory pool for early allocations") I might

Re: [PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Anton Altaparmakov
Hi Andrew, Can you please merge this patch? Thanks a lot in advance! Rustam, thank you for the updated patch! Best regards, Anton > On 24 Aug 2020, at 03:28, Rustam Kovhaev wrote: > > number of bytes allocated for mft record should be equal to the mft > record size stored in ntfs

Re: [PATCH v5 2/2] Add Intel LGM soc DMA support.

2020-08-23 Thread Reddy, MallikarjunaX
Hi Peter, Thanks for the review comments. Please see my comments inline.. On 8/18/2020 6:16 PM, Peter Ujfalusi wrote: Hi, On 14/08/2020 8.26, Amireddy Mallikarjuna reddy wrote: Add DMA controller driver for Lightning Mountain(LGM) family of SoCs. The main function of the DMA controller is

Re: [PATCH] usb: hcd: fix the error check

2020-08-23 Thread Peter Chen
On 20-08-20 20:20:38, Tang Bin wrote: > In the function usb_add_hcd(), usb_phy_roothub_alloc() > can return NULL in some cases, so IS_ERR() doesn't meet > the requirements. Thus fix it. > Signed-off-by: Zhang Shengju > Signed-off-by: Tang Bin > --- > drivers/usb/core/hcd.c | 5 +++-- > 1 file

[PATCH] watchdog: Fix memleak in watchdog_cdev_register

2020-08-23 Thread Dinghao Liu
When watchdog_kworker is NULL, we should free wd_data before the function returns to prevent memleak. Fixes: 664a39236e718 ("watchdog: Introduce hardware maximum heartbeat in watchdog core") Signed-off-by: Dinghao Liu --- drivers/watchdog/watchdog_dev.c | 4 +++- 1 file changed, 3

Re: [PATCH v2 1/2] irqchip: irq-mst: Add MStar interrupt controller support

2020-08-23 Thread Mark-PK Tsai
From: Daniel Palmer >Hi Mark-PK, > >On Thu, 20 Aug 2020 at 00:38, Mark-PK Tsai wrote: >> >> Add MStar interrupt controller support using hierarchy irq >> domain. >> >> Signed-off-by: Mark-PK Tsai > >I've integrated this version into my MStar/SigmaStar tree and tested >on an MStar MSC313E

[PATCH] regulator: mp886x: use "mps,switch-frequency-hz"

2020-08-23 Thread Jisheng Zhang
As Rob suggested, use the "mps,switch-frequency-hz" instead of the "mps,switch-frequency" for switch frequency. Fortunately, the switch frequency support isn't released, so we can modify it now without any concern. Signed-off-by: Jisheng Zhang ---

Re: [PATCH v2 4/9] iommu/ioasid: Add reference couting functions

2020-08-23 Thread Lu Baolu
Hi Jacob, On 8/22/20 12:35 PM, Jacob Pan wrote: There can be multiple users of an IOASID, each user could have hardware contexts associated with the IOASID. In order to align lifecycles, reference counting is introduced in this patch. It is expected that when an IOASID is being freed, each user

{standard input}:5973: Error: operand out of range (512 is not between -512 and 511)

2020-08-23 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d012a7190fc1fd72ed48911e77ca97ba4521bccd commit: de8f5e4f2dc1f032b46afda0a78cab5456974f89 lockdep: Introduce wait-type checks date: 5 months ago config: arc-randconfig-r004-20200824 (attached as .config)

Re: [PATCH v2 3/9] iommu/ioasid: Introduce ioasid_set APIs

2020-08-23 Thread Lu Baolu
Hi Jacob, On 8/22/20 12:35 PM, Jacob Pan wrote: ioasid_set was introduced as an arbitrary token that are shared by a group of IOASIDs. For example, if IOASID #1 and #2 are allocated via the same ioasid_set*, they are viewed as to belong to the same set. For guest SVA usages, system-wide IOASID

[PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Rustam Kovhaev
number of bytes allocated for mft record should be equal to the mft record size stored in ntfs superblock as reported by syzbot, userspace might trigger out-of-bounds read by dereferencing ctx->attr in ntfs_attr_find() Reported-and-tested-by: syzbot+aed06913f36eff9b5...@syzkaller.appspotmail.com

Re: [PATCH v8 2/3] drm: bridge: Add support for Cadence MHDP DPI/DP bridge

2020-08-23 Thread Laurent Pinchart
Hi Tomi, On Fri, Aug 14, 2020 at 12:29:35PM +0300, Tomi Valkeinen wrote: > On 11/08/2020 05:36, Laurent Pinchart wrote: > > >> +static int cdns_mhdp_mailbox_write(struct cdns_mhdp_device *mhdp, u8 val) > >> +{ > >> + int ret, full; > >> + > >> + WARN_ON(!mutex_is_locked(>mbox_mutex)); > >> + >

Re: [PATCH v8 2/3] drm: bridge: Add support for Cadence MHDP DPI/DP bridge

2020-08-23 Thread Laurent Pinchart
Hi Tomi, On Fri, Aug 14, 2020 at 11:22:09AM +0300, Tomi Valkeinen wrote: > On 11/08/2020 05:36, Laurent Pinchart wrote: > > >> +static int cdns_mhdp_connector_init(struct cdns_mhdp_device *mhdp) > >> +{ > >> + u32 bus_format = MEDIA_BUS_FMT_RGB121212_1X36; > >> + struct drm_connector *conn =

Re: Re: [PATCH] watchdog: Fix double-free in watchdog_cdev_register

2020-08-23 Thread dinghao . liu
> On 8/23/20 12:13 AM, Dinghao Liu wrote: > > When misc_register() fails, wd_data will be released by the > > release callback function watchdog_core_data_release(), so > > we don't need to free it again. But when watchdog_kworker is > > NULL, we should free wd_data to prevent memleak. > > > >

Re: [RFC PATCH 1/5] printk: implement pr_cont_t

2020-08-23 Thread Sergey Senozhatsky
On (20/08/20 12:16), Petr Mladek wrote: > Now that I think about it. This is the biggest problem with any temporary > buffer > for pr_cont() lines. I am more and more convinced that we should just > _keep the current behavior_. It is not ideal. But sometimes mixed > messages are always better

[PATCH] regulator: rt4801: Fix W=1 build warning when CONFIG_OF=n

2020-08-23 Thread cy_huang
From: ChiYuan Huang Fix below warning when CONFIG_OF=n: drivers/regulator/rt4801-regulator.c:206:34: warning: unused variable 'rt4801_of_id' [-Wunused-const-variable] 206 | static const struct of_device_id rt4801_of_id[] = { | ^~~~

Re: [PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Anton Altaparmakov
Hi Rustam, Thank you for the patch but it introduces an endianness bug - you have to us le32_to_cpu(m->bytes_allocated) both when doing the comparison and then printing the message. Also, please drop the square brackets. Wherever the driver prints such things it never uses brackets around

Re: [PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Rustam Kovhaev
On Mon, Aug 24, 2020 at 01:44:06AM +, Anton Altaparmakov wrote: > Hi Rustam, > > Thank you for the patch but it introduces an endianness bug - you have to us > le32_to_cpu(m->bytes_allocated) both when doing the comparison and then > printing the message. > > Also, please drop the square

Re: [PATCH] Add missing bound checks for software 842 decompressor

2020-08-23 Thread Joan Bruguera
On 19.08.20 22:19, Kees Cook wrote: On Sun, Aug 16, 2020 at 02:33:41AM +0200, Joan Bruguera wrote: Any feedback? Hi! I just happened to see this email. I think this should likely be directed to the crypto (which also handled compress/decompress APIs) list and the original author...

Re: [PATCH 1/2] KVM: LAPIC: Fix updating DFR missing apic map recalculation

2020-08-23 Thread Wanpeng Li
ping, :) On Wed, 19 Aug 2020 at 16:55, Wanpeng Li wrote: > > From: Wanpeng Li > > There is missing apic map recalculation after updating DFR, if it is > INIT RESET, in x2apic mode, local apic is software enabled before. > This patch fix it by introducing the function kvm_apic_set_dfr() to > be

[PATCH] mmc: sd: Use kobj_to_dev() instead of container_of()

2020-08-23 Thread Tian Tao
Use kobj_to_dev() instead of container_of() Signed-off-by: Tian Tao --- drivers/mmc/core/sd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c index 5a2210c..a0d2c34 100644 --- a/drivers/mmc/core/sd.c +++ b/drivers/mmc/core/sd.c

答复: [PATCH] mmc: core: Initial support for SD express card/host

2020-08-23 Thread 冯锐
Hi Hansson: If this patch will not be changed, I will post a patch for rtsx driver according your patch. > > Rui, > > On Thu, 16 Jul 2020 at 16:16, Ulf Hansson wrote: > > > > In the SD specification v7.10 the SD express card has been added. This > > new type of removable SD card, can be

Re: [PATCH v2] ARM: dts: imx7d-zii-rmu2: fix rgmii phy-mode for ksz9031 phy

2020-08-23 Thread Fabio Estevam
Hi Chris, On Sat, Aug 22, 2020 at 11:25 PM Chris Healy wrote: > > From: Chris Healy > > Since commit bcf3440c6dd7 ("net: phy: micrel: add phy-mode support for the > KSZ9031 PHY") the networking is broken on the imx7d-zii-rmu2 board. > > The end result is that network receive behaviour is

Re: [PATCH v2 2/2] KVM: LAPIC: Guarantee the timer is in tsc-deadline mode when setting

2020-08-23 Thread Wanpeng Li
ping :) On Wed, 12 Aug 2020 at 14:30, Wanpeng Li wrote: > > From: Wanpeng Li > > Check apic_lvtt_tscdeadline() mode directly instead of apic_lvtt_oneshot() > and apic_lvtt_period() to guarantee the timer is in tsc-deadline mode when > wrmsr MSR_IA32_TSCDEADLINE. > > Signed-off-by: Wanpeng Li >

[PATCH v2] KVM: LAPIC: Narrow down the kick target vCPU

2020-08-23 Thread Wanpeng Li
From: Wanpeng Li The kick after setting KVM_REQ_PENDING_TIMER is used to handle the timer fires on a different pCPU which vCPU is running on, this kick is expensive since memory barrier, rcu, preemption disable/enable operations. We don't need this kick when injecting already-expired timer,

Re: [PATCH v3 2/3] media: dt-bindings: media: xilinx: Add Xilinx UHD-SDI Receiver Subsystem

2020-08-23 Thread Laurent Pinchart
Hi Vishal, On Wed, Aug 19, 2020 at 07:40:16PM +0300, Laurent Pinchart wrote: > On Wed, Aug 19, 2020 at 01:45:34PM +, Vishal Sagar wrote: > > On Wednesday, July 15, 2020 9:59 PM, Laurent Pinchart wrote: > >> On Mon, Jul 13, 2020 at 12:54:47PM -0600, Rob Herring wrote: > >>> On Thu, Jun 18,

[tip:timers/core] BUILD SUCCESS e2d977c9f1abd1d199b412f8f83c1727808b794d

2020-08-23 Thread kernel test robot
powerpc allmodconfig powerpc allnoconfig x86_64 randconfig-a002-20200823 x86_64 randconfig-a003-20200823 x86_64 randconfig-a005-20200823 x86_64 randconfig-a001-20200823 x86_64

linux-next: manual merge of the bpf-next tree with Linus' tree

2020-08-23 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the bpf-next tree got a conflict in: tools/lib/bpf/libbpf.c between commit: 1e891e513e16 ("libbpf: Fix map index used in error message") from Linus' tree and commit: 88a82120282b ("libbpf: Factor out common ELF operations and improve logging") from

Re: [PATCH v8 1/4] power: supply: core: add quick charge type property

2020-08-23 Thread Qiwu Huang
On Tue, Aug 18, 2020 at 2:07 PM Greg KH wrote: > > On Tue, Aug 18, 2020 at 09:56:28AM +0800, Qiwu Huang wrote: > > On Fri, Aug 14, 2020 at 2:09 PM Greg KH wrote: > > > > > > On Fri, Aug 14, 2020 at 11:46:54AM +0800, Qiwu Huang wrote: > > > > From: Qiwu Huang > > > > > > > > Reports the kind of

Re: [PATCH v2] ARM: dts: r8a7742-iwg21d-q7: Add LCD support

2020-08-23 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Thu, Aug 13, 2020 at 03:00:41PM +0100, Lad Prabhakar wrote: > The iwg21d comes with a 7" capacitive touch screen, therefore > add support for it. > > Signed-off-by: Lad Prabhakar > Reviewed-by: Marian-Cristian Rotariu > Everything seems to match the

Re: ..and it built & boot without fuss!!Linux 5.9-rc2

2020-08-23 Thread Bhaskar Chowdhury
On 14:23 Sun 23 Aug 2020, Linus Torvalds wrote: It's Sunday afternoon, which means it's time for another release candidate. Nothing in particular stands out, there's a random collection of fixes and updates in here. It is perhaps a bit filesystem-heavy, because the ext4 updates came in late, so

Re: [PATCH v1 05/10] powerpc/pseries/iommu: Add iommu_pseries_alloc_table() helper

2020-08-23 Thread Alexey Kardashevskiy
On 18/08/2020 09:40, Leonardo Bras wrote: > Creates a helper to allow allocating a new iommu_table without the need > to reallocate the iommu_group. > > This will be helpful for replacing the iommu_table for the new DMA window, > after we remove the old one with iommu_tce_table_put(). > >

[PATCH] Documentation: submit-checklist: add Documentation clean builds

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Add to Documentation/process/submit-checklist.rst that patch submitters should run "make htmldocs" and verify that any Documentation/ changes (patches) are clean (no new warnings/errors). Signed-off-by: Randy Dunlap --- Documentation/process/submit-checklist.rst |4

[PATCH] Documentation/powerpc: fix malformed table in syscall64-abi

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix malformed table warning in powerpc/syscall64-abi.rst by making two tables and moving the headings. Documentation/powerpc/syscall64-abi.rst:53: WARNING: Malformed table. Text in column margin in table line 2. === =

linux-next: build failure after merge of the hwmon-staging tree

2020-08-23 Thread Stephen Rothwell
Hi all, After merging the hwmon-staging tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/hwmon/pmbus/adm1266.c: In function 'adm1266_probe': drivers/hwmon/pmbus/adm1266.c:478:31: error: passing argument 2 of 'pmbus_do_probe' from incompatible pointer type

Re: [PATCH 4/4] mm: Add PGREUSE counter

2020-08-23 Thread Peter Xu
On Sat, Aug 22, 2020 at 09:14:53AM -0700, Linus Torvalds wrote: > On Fri, Aug 21, 2020 at 4:50 PM Peter Xu wrote: > > > > This accounts for wp_page_reuse() case, where we reused a page for COW. > > If we do this, wouldn't it make more sense to also count the COW case > to see how they match up?

Re: [PATCH] KVM: LAPIC: Don't kick vCPU which is injecting already-expired timer

2020-08-23 Thread Wanpeng Li
On Sat, 22 Aug 2020 at 12:01, Sean Christopherson wrote: > > On Fri, Aug 21, 2020 at 05:17:34PM +0800, Wanpeng Li wrote: > > From: Wanpeng Li > > > > The kick after setting KVM_REQ_PENDING_TIMER is used to handle the timer > > fires on a different pCPU which vCPU is running on, we don't need

Re: [PATCH 0/8] Convert the intel iommu driver to the dma-iommu api

2020-08-23 Thread Tom Murphy
Hi Logan/All, I have added a check for the sg_dma_len == 0 : """ } __sgt_iter(struct scatterlist *sgl, bool dma) { struct sgt_iter s = { .sgp = sgl }; + if (sgl && sg_dma_len(sgl) == 0) + s.sgp = NULL; if (s.sgp) { . """ at location [1]. but it

[PATCH] Documentation: sound/cards: fix heading underline lengths for https: changes

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix documentation build warnings for underline length too short, caused by s/http/https/ and not changing the accompanying underlines. Documentation/sound/cards/audigy-mixer.rst:335: WARNING: Title underline too short. US Patents (https://www.uspto.gov/)

Re: kernel BUG at kernel/fork.c:LINE!

2020-08-23 Thread Shakeel Butt
On Sun, Aug 23, 2020 at 10:34 AM Randy Dunlap wrote: > > On 8/7/20 2:16 PM, syzbot wrote: > > Hello, > > > > syzbot found the following issue on: > > > > HEAD commit:fffe3ae0 Merge tag 'for-linus-hmm' of git://git.kernel.org.. > > git tree: upstream > > console output:

Re: [PATCH] ARM: dts: imx6sx-sdb: Add headphone detection for sound card

2020-08-23 Thread Shawn Guo
On Sun, Aug 23, 2020 at 11:12:08AM -0300, Fabio Estevam wrote: > Hi Wang Shengjiu and Shawn, > > On Sun, Aug 23, 2020 at 9:31 AM S.j. Wang wrote: > > > I would like to know your opinion, should I move headphone detect GPIO > > To audmux group? > > What about adding a dedicated pinctrl_hp for

Re: [PATCH 0/4] mm: Simplfy cow handling

2020-08-23 Thread Peter Xu
On Sat, Aug 22, 2020 at 09:05:37AM -0700, Linus Torvalds wrote: > On Fri, Aug 21, 2020 at 4:50 PM Peter Xu wrote: > > > > - Run a busy loop dirty program [1] that uses 6G of memory, restrict to 1G > > RAM + 5G swap (cgroup). A few hours later, all things still look good. > > Make sure

[PATGCH -next] LSM: SafeSetID: fix underline length warning

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix documentation build warning: Documentation/admin-guide/LSM/SafeSetID.rst:110: WARNING: Title underline too short. Note on GID policies and setgroups() == Fixes: 02e316b088df ("LSM: SafeSetID: Add GID security policy handling") Signed-off-by: Randy Dunlap

Re: [PATCH] scsi: scsi_debug: Remove superfluous close zone in resp_open_zone()

2020-08-23 Thread Damien Le Moal
On 2020/08/21 22:00, Niklas Cassel wrote: > resp_open_zone() always calls zbc_open_zone() with parameter explicit > set to true. > > If zbc_open_zone() is called with parameter explicit set to true, and > the current zone state is implicit open, it will call zbc_close_zone() > on the zone before

[PATCH] Documentation: fix dma-buf.rst underline length warning

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap /home/rdunlap/lnx/lnx-59-rc2/Documentation/driver-api/dma-buf.rst:182: WARNING: Title underline too short. Indefinite DMA Fences Fixes: 72b6ede73623 ("dma-buf.rst: Document why indefinite fences are a bad idea") Signed-off-by: Randy Dunlap Cc: Daniel

arch/powerpc/platforms/4xx/gpio.c:63:28: sparse: sparse: incorrect type in argument 1 (different base types)

2020-08-23 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d012a7190fc1fd72ed48911e77ca97ba4521bccd commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces date: 9 weeks ago config: powerpc64-randconfig-s032-20200824

Re: dt-bindings: display: xlnx: mixer: Inconsistent pixel format terminology in dt docs

2020-08-23 Thread Laurent Pinchart
Hello, On Thu, Aug 20, 2020 at 04:38:18PM -0700, Hyun Kwon wrote: > On Thursday, August 20, 2020 2:18 PM, Kenneth Sloat write: > > Hello, > > > > The Xilinx Video mixer IP uses the DRM fourcc string as a device tree > > binding in > > order to describe the format for a specific DRM layer/plane.

Re: [GIT pull] x86/urgent for v5.9-rc2

2020-08-23 Thread Andy Lutomirski
On Sun, Aug 23, 2020 at 3:35 PM Linus Torvalds wrote: > > On Sun, Aug 23, 2020 at 3:27 PM Andy Lutomirski wrote: > > > > Every interrupt is going to load the CS and SS descriptor cache lines. > > Yeah, but this isn't even sharing the same GDT cache line. Those two > are at least in the same

Re: [PATCH v2 8/8] x86/debug: Remove the historical junk

2020-08-23 Thread Andy Lutomirski
On Fri, Aug 21, 2020 at 3:21 AM Peter Zijlstra wrote: > > Remove the historical junk and replace it with a WARN and a comment. > > The problem is that even though the kernel only uses TF single-step in > kprobes and KGDB, both of which consume the event before this, > QEMU/KVM has bugs in this

Re: [PATCH v2 5/8] x86/debug: Simplify #DB signal code

2020-08-23 Thread Andy Lutomirski
On Fri, Aug 21, 2020 at 3:21 AM Peter Zijlstra wrote: > > Get rid of the two variables, avoid computing si_code when not needed > and be consistent about which dr6 value is used. > > - if (tsk->thread.debugreg6 & (DR_STEP | DR_TRAP_BITS) || user_icebp) > - send_sigtrap(regs,

Re: [PATCH] mmc: mmc_spi: fix timeout calculation

2020-08-23 Thread Tobias Schramm
Hi Uffe, > This looks good to me, but before applying just wanted to check that > you tested this on some HW, to make sure it doesn't break anything? yes, I should have mentioned that. I tested the change on a custom STM32H743 board with a microSD card connected to one of its SPIs. It does still

[PATCH v8 3/3] mm: cma: use CMA_MAX_NAME to define the length of cma name array

2020-08-23 Thread Barry Song
CMA_MAX_NAME should be visible to CMA's users as they might need it to set the name of CMA areas and avoid hardcoding the size locally. So this patch moves CMA_MAX_NAME from local header file to include/linux header file and removes the hardcode in both hugetlb.c and contiguous.c. Cc: Mike

  1   2   3   4   >