Re: [PATCH] iio: dac: Rudimentary typo fix

On 3/22/21 6:22 PM, Bhaskar Chowdhury wrote: > > s/concurent/concurrent/ > > Signed-off-by: Bhaskar Chowdhury Acked-by: Randy Dunlap > --- > drivers/iio/dac/ad5766.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/dac/ad5766.c b/drivers/iio/dac/ad5766.c

Re: [PATCH v6 01/14] media: uapi: Add some RGB bus formats for i.MX8qm/qxp pixel combiner

Hi Marcel, On Tue, 2021-03-23 at 00:23 +, Marcel Ziswiler wrote: > On Wed, 2021-03-17 at 11:42 +0800, Liu Ying wrote: > > This patch adds RGB666_1X30_CPADLO, RGB888_1X30_CPADLO, RGB666_1X36_CPADLO > > and RGB888_1X36_CPADLO bus formats used by i.MX8qm/qxp pixel combiner. > > The RGB pixels

Re: [PATCH RESEND] PCI: dwc: Fix MSI not work after resume

Hi On Mon, 22 Mar 2021 20:24:41 -0500 Bjorn Helgaas wrote: > > [+cc Kishon, Richard, Lucas, Dilip] > > On Mon, Mar 01, 2021 at 11:10:31AM +0800, Jisheng Zhang wrote: > > After we move dw_pcie_msi_init() into core -- dw_pcie_host_init(), the > > MSI stops working after resume. Because

Re: [RFC 2/7] ath10k: Add support to process rx packet in thread

On 3/22/21 6:20 PM, Brian Norris wrote: On Mon, Mar 22, 2021 at 4:58 PM Ben Greear wrote: On 7/22/20 6:00 AM, Felix Fietkau wrote: On 2020-07-22 14:55, Johannes Berg wrote: On Wed, 2020-07-22 at 14:27 +0200, Felix Fietkau wrote: I'm considering testing a different approach (with mt76

RE: [PATCH 1/5] cifsd: add server handler and tranport layers

> On Mon, Mar 22, 2021 at 02:13:40PM +0900, Namjae Jeon wrote: > > +#define RESPONSE_BUF(w)((void *)(w)->response_buf) > > +#define REQUEST_BUF(w) ((void *)(w)->request_buf) > > Why do you do this obfuscation? I don't remember exactly, but back then, It looked easier... >

Re: GTE - The hardware timestamping engine

On Mon, Mar 22, 2021 at 06:53:10PM -0700, Dipen Patel wrote: > > > On 3/22/21 5:32 PM, Kent Gibson wrote: > > On Mon, Mar 22, 2021 at 01:21:46PM -0700, Dipen Patel wrote: > >> Hi Linus and Kent, > >> [snip] > > In response to all your comments above... > > > > Firstly, I'm not suggesting that

[PATCH] target: Fix a double put in transport_free_session

In transport_free_session, se_nacl is got from se_sess with the initial reference. If se_nacl->acl_sess_list is empty, se_nacl->dynamic_stop is set to true. Then the first target_put_nacl(se_nacl) will drop the initial reference and free se_nacl. Later there is a second target_put_nacl() to put

[PATCH -next] x86: Fix intel cpu unsed variable ‘l2’ warning

Fixes the following W=1 kernel build warning(s): ../arch/x86/kernel/cpu/intel.c: In function ‘init_intel’: ../arch/x86/kernel/cpu/intel.c:644:20: warning: variable ‘l2’ set but not used [-Wunused-but-set-variable] unsigned int l1, l2; ^~ Compilation command(s): make

Re: [PATCH] rcu: Fix various typos in comments

On 00:02 Tue 23 Mar 2021, Ingo Molnar wrote: Hi Paul, Was working on automation to make it a bit more straightforward to fix typos within comments (which we tend to reintroduce during development), and here are the ones it found in the RCU code. Thanks, Ingo => From: Ingo

[V2][PATCH] drm: xlnx: zynqmp: release reset to DP controller before accessing DP registers

From: Quanyang Wang When insmod zynqmp-dpsub.ko after rmmod it, system will hang with the error log as below: root@xilinx-zynqmp:~# insmod zynqmp-dpsub.ko [ 88.391289] [drm] Initialized zynqmp-dpsub 1.0.0 20130509 for fd4a.display on minor 0 [ 88.529906] Console: switching to colour

Re: [kbuild-all] Re: include/linux/compiler_types.h:315:38: error: call to '__compiletime_assert_536' declared with attribute error: BUILD_BUG_ON failed: offsetof(struct can_frame, len) != offsetof(st

On 3/23/21 12:24 AM, Oliver Hartkopp wrote: Hi Rong, On 22.03.21 09:52, Rong Chen wrote: On 3/21/21 10:19 PM, Oliver Hartkopp wrote: Two reminders in two days? ;-) Did you check my answer here? https://lore.kernel.org/lkml/afffeb73-ba4c-ca2c-75d0-9e7899e5c...@hartkopp.net/ And did

Re: [PATCH] rcu: Fix various typos in comments

On 3/22/21 4:02 PM, Ingo Molnar wrote: > > Hi Paul, > > Was working on automation to make it a bit more straightforward to fix > typos within comments (which we tend to reintroduce during > development), and here are the ones it found in the RCU code. > > Thanks, > > Ingo > >

[PATCH bpf-next v2 2/2] bpf: selftests: add tests for batched ops in LPM trie maps

From: Pedro Tammela Uses the already existing infrastructure for testing batched ops. The testing code is essentially the same, with minor tweaks for this use case. Suggested-by: Jamal Hadi Salim Signed-off-by: Pedro Tammela --- .../map_tests/lpm_trie_map_batch_ops.c (new) | 158

[PATCH bpf-next v2 0/2] add support for batched ops in LPM trie

The patch itself is straightforward thanks to the infrastructure that is already in-place. The tests follows the other '*_map_batch_ops' tests with minor tweaks. v1 -> v2: Fixes for checkpatch warnings Pedro Tammela (2): bpf: add support for batched operations in LPM trie maps bpf:

[PATCH bpf-next v2 1/2] bpf: add support for batched ops in LPM trie maps

From: Pedro Tammela Suggested-by: Jamal Hadi Salim Signed-off-by: Pedro Tammela --- kernel/bpf/lpm_trie.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/kernel/bpf/lpm_trie.c b/kernel/bpf/lpm_trie.c index cec792a17e5f..1b7b8a6f34ee 100644 --- a/kernel/bpf/lpm_trie.c +++

[PATCH -next] x86: Fix unused variable 'hi'

Fixes the following W=1 kernel build warning(s): arch/x86/hyperv/hv_apic.c:58:15: warning: variable ‘hi’ set but not used [-Wunused-but-set-variable] Compiled with CONFIG_HYPERV enabled: make allmodconfig ARCH=x86_64 CROSS_COMPILE=x86_64-linux-gnu- make W=1 arch/x86/hyperv/hv_apic.o ARCH=x86_64

[PATCH] [v2] drivers: watchdog: Remove duplicate include of kernel.h

linux/kernel.h has been included at line 22. According to alphabetic order, we remove the duplicate one at line 17. Signed-off-by: Wan Jiabing --- drivers/watchdog/dw_wdt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/watchdog/dw_wdt.c b/drivers/watchdog/dw_wdt.c index

Re: [PATCH] tracing: Fix various typos in comments

Hi Ingo, A few comments for you: On 3/22/21 3:45 PM, Ingo Molnar wrote: > > Fix ~59 single-word typos in the tracing code comments. > > Signed-off-by: Ingo Molnar > Cc: Steven Rostedt > Cc: linux-kernel@vger.kernel.org > --- > arch/microblaze/include/asm/ftrace.h | 2 +- >

Re: [PATCH] userfaultfd: Write protect when virtual memory range has no page table entry

On Mon, Mar 22, 2021 at 03:00:37PM +0200, Mike Rapoport wrote: > On Mon, Mar 22, 2021 at 11:14:37AM +0100, Michal Hocko wrote: > > Le'ts Andrea and Mike > > > > On Fri 19-03-21 22:24:28, Bui Quang Minh wrote: > > > userfaultfd_writeprotect() use change_protection() to clear write bit in > > >

[PATCH] drm/etnaviv: Remove redundant NULL check

Fix the following coccicheck warnings: ./drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c:622:2-8: WARNING: NULL check before some freeing functions is not needed. ./drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c:618:2-8: WARNING: NULL check before some freeing functions is not needed.

Re: [PATCH v2 8/8] arm64: dts: Add Mediatek SoC MT8195 and evaluation board dts and Makefile

On Fri, 2021-03-19 at 10:34 +0800, Seiya Wang wrote: > Add basic chip support for Mediatek MT8195 > > Signed-off-by: Seiya Wang > --- > arch/arm64/boot/dts/mediatek/Makefile | 1 + > arch/arm64/boot/dts/mediatek/mt8195-evb.dts | 29 ++ > arch/arm64/boot/dts/mediatek/mt8195.dtsi|

Re: [PATCH net-next] net: ipa: avoid 64-bit modulus

On 3/22/21 6:05 PM, Alex Elder wrote: > It is possible for a 32 bit x86 build to use a 64 bit DMA address. > > There are two remaining spots where the IPA driver does a modulo > operation to check alignment of a DMA address, and under certain > conditions this can lead to a build error on i386

[PATCH 3/3] w1: ds2438: adding support for accessing page1 registers

The ds2438 has a register in page1 called offset, which dictates how the current measurement is done. If the offset register is not calibrated well, pulling zero current from a battery shows a non-zero current measurement. Gives write access to the offset register. This way, a user following the

[PATCH -next] x86: Fix unused variable 'msr_val' warning

Fixes the following W=1 kernel build warning(s): arch/x86/hyperv/hv_spinlock.c:28:16: warning: variable ‘msr_val’ set but not used [-Wunused-but-set-variable] unsigned long msr_val; As Hypervisor Top-Level Functional Specification states in chapter 7.5 Virtual Processor Idle Sleep State, "A

[PATCH 2/3] w1: ds2438: fixed bug in get_page function

The get_page function always returned register values from page 0. Fixes by writing the actual page number to the bus. Signed-off-by: Luiz Sampaio --- drivers/w1/slaves/w1_ds2438.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/w1/slaves/w1_ds2438.c

[PATCH] arch: powerpc: Remove duplicate include of interrupt.h

asm/interrupt.h has been included at line 12. According to alphabetic order,we remove the duplicate one at line 10. Signed-off-by: Wan Jiabing --- arch/powerpc/kernel/interrupt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/kernel/interrupt.c

[PATCH 1/3] w1: ds2438: fixed a coding style issues

Fixed coding style issues. Signed-off-by: Luiz Sampaio --- drivers/w1/slaves/w1_ds2438.c | 26 -- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/drivers/w1/slaves/w1_ds2438.c b/drivers/w1/slaves/w1_ds2438.c index 5cfb0ae23e91..a9884fc8c726 100644 ---

Re: [PATCH 10/13] usb: mtu3: support ip-sleep wakeup for MT8183

On Mon, 2021-03-22 at 11:57 +0300, Sergei Shtylyov wrote: > Same comments as to the patch #6. Ok, will check others, thanks a lot > > MBR, Sergei

[PATCH 0/3] w1: ds2438: adding support for calibration of current measurements

The following patches aim to make a user able to calibrate the current measurement of the ds2438. This chip uses a offset register in page1, which is added to the current register to give the user the current measurement. If this value is wrong, the user will get an offset current value, even

Re: [PATCH 07/13] usb: xhci-mtk: add support ip-sleep wakeup for mT8192

On Mon, 2021-03-22 at 11:58 +0300, Sergei Shtylyov wrote: > On 22.03.2021 6:13, Chunfeng Yun wrote: > > > Add support ip-sleep wakeup for mT8192, it's a specific revision, > ^ for > > > and not follow IPM rule. > > Following? > > > Signed-off-by: Chunfeng Yun > > --- > >

Re: [PATCH 06/13] usb: xhci-mtk: support ip-sleep wakeup for MT8183

On Mon, 2021-03-22 at 11:54 +0300, Sergei Shtylyov wrote: > Hello! > > On 22.03.2021 6:13, Chunfeng Yun wrote: > > > Add support ip-sleep wakeup for MT8183, it's similar to MT8173, > ^ for > > > and it's also a specific one, but not follow IPM rule. > > Following? Ack > > >

Re: [PATCH 1/3] virtio_ring: always warn when descriptor chain exceeds queue size

在 2021/3/22 下午4:17, Michael S. Tsirkin 写道: On Mon, Mar 22, 2021 at 11:22:15AM +0800, Jason Wang wrote: 在 2021/3/18 下午9:52, Connor Kuehl 写道: From section 2.6.5.3.1 (Driver Requirements: Indirect Descriptors) of the virtio spec: "A driver MUST NOT create a descriptor chain longer than

[PATCH] KVM: VMX: Check the corresponding bits according to the intel sdm

From: Haiwei Li According to IA-32 SDM Vol.3D "A.1 BASIC VMX INFORMATION", two inspections are missing. * Bit 31 is always 0. Earlier versions of this manual specified that the VMCS revision identifier was a 32-bit field in bits 31:0 of this MSR. For all processors produced prior to this change,

[PATCH] scsi: fix a error refcount get in pcie_device_make_active

In pcie_device_make_active, pcie_device may be freed by pcie_device_put(pcie_device). After it, the refcount of pcie_device is increased by pcie_device_get(pcie_device). The pcie_device is freed but refcount is non-zero. It could cause use after free later. My patch puts the pcie_device_get()

Re: [PATCH 02/23] mm: Clear vmf->pte after pte_unmap_same() returns

Hi: On 2021/3/23 8:48, Peter Xu wrote: > pte_unmap_same() will always unmap the pte pointer. After the unmap, vmf->pte > will not be valid any more. We should clear it. > > It was safe only because no one is accessing vmf->pte after pte_unmap_same() > returns, since the only caller of

Re: [PATCH] ndtest: Remove redundant NULL check

Hi Ira, Ira Weiny writes: > On Mon, Mar 22, 2021 at 06:00:40PM +0800, Jiapeng Chong wrote: >> Fix the following coccicheck warnings: >> >> ./tools/testing/nvdimm/test/ndtest.c:491:2-7: WARNING: NULL check before >> some freeing functions is not needed. > > I don't think there is anything

[v2, 1/1]Fixes: e3af677607d9(clocksource/drivers/timer-mediatek: optimize systimer irq clear flow on shutdown)

mtk_syst_clkevt_shutdown is called after irq disabled in suspend flow, clear any pending systimer irq when shutdown to avoid suspend aborted due to timer irq pending Also as for systimer in mediatek socs, there must be firstly enable timer before clear systimer irq Signed-off-by: Fengquan Chen

[PATCH] [v2, 1/1]Fixes: e3af677607d9(clocksource/drivers/timer-mediatek: optimize systimer irq clear flow on shutdown)

mtk_syst_clkevt_shutdown is called after irq disabled in suspend flow, clear any pending systimer irq when shutdown to avoid suspend aborted due to timer irq pending Also as for systimer in mediatek socs, there must be firstly enable timer before clear systimer irq Change-Id:

Re: [PATCH] media: test-drivers: Remove duplicate include of string.h

Hi Wan, thanks for the patch! Unfortunately, however, a similar patch has already been accepted [1] [1] https://patchwork.linuxtv.org/project/linux-media/patch/20210306113255.217387-1-zhang.yun...@zte.com.cn/ -- thanks -- Daniel

[PATCH] soundwire: add slave device to linked list after device_register()

From: Pierre-Louis Bossart We currently add the slave device to a linked list before device_register(), and then remove it if device_register() fails. It's not clear why this sequence was necessary, this patch moves the linked list management to after the device_register(). Suggested-by: Keyon

[PATCH] arch: powerpc: bug.h is included twice

asm/bug.h has been included at line 12, so remove the duplicate one at line 21. Signed-off-by: Wan Jiabing --- arch/powerpc/include/asm/book3s/64/mmu-hash.h | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/include/asm/book3s/64/mmu-hash.h

Re: [PATCH] drivers: watchdog: Remove duplicate include of kernel.h

On 3/22/21 6:43 PM, Wan Jiabing wrote: > linux/kernel.h has been included at line 17, so remove the > duplicate one at line 22. > > Signed-off-by: Wan Jiabing > --- > drivers/watchdog/dw_wdt.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/watchdog/dw_wdt.c

[PATCH] thermal: ti-soc-thermal: of_device.h is included twice

linux/of_device.h has been included at line 25, so remove the duplicate one at line 35. Signed-off-by: Wan Jiabing --- drivers/thermal/ti-soc-thermal/ti-bandgap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/thermal/ti-soc-thermal/ti-bandgap.c

Re: [PATCH 07/23] mm: Introduce zap_details.zap_flags

On Mon, Mar 22, 2021 at 08:48:56PM -0400, Peter Xu wrote: > +/* Whether to check page->mapping when zapping */ > +#define ZAP_FLAG_CHECK_MAPPING BIT(0) > + > /* > * Parameter block passed down to zap_pte_range in exceptional cases. > */ > struct zap_details { > - struct

[PATCH] net: can: Remove duplicate include of regmap.h

linux/regmap.h has been included at line 13, so remove the duplicate one at line 14. Signed-off-by: Wan Jiabing --- drivers/net/can/m_can/tcan4x5x.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/can/m_can/tcan4x5x.h b/drivers/net/can/m_can/tcan4x5x.h index

Re: [PATCH v2] sched/fair: bring back select_idle_smt, but differently

On Mon, 2021-03-22 at 15:33 +, Mel Gorman wrote: > If trying that, I would put that in a separate patch. At one point > I did play with clearing prev, target and recent but hit problems. > Initialising the mask and clearing them in select_idle_sibling() hurt > the fast path and doing it later

Re: [PATCH 2/3] iommu/vt-d: Remove IOVA domain rcache flushing for CPU offlining

On 3/1/21 8:12 PM, John Garry wrote: Now that the core code handles flushing per-IOVA domain CPU rcaches, remove the handling here. Signed-off-by: John Garry --- drivers/iommu/intel/iommu.c | 31 --- include/linux/cpuhotplug.h | 1 - 2 files changed, 32

[PATCH] rapidio: Fix a use after free in mport_mm_close

In mport_mm_close, map is freed by kref_put(>ref,). But mutex_unlock(>md->buf_mutex) uses map later. Other thread could allocate the freed chunk and cause an unexpected result. My patch adds a variable buf_mutex to store the mutex before map is freed. Signed-off-by: Lv Yunlong ---

[PATCH] net: ethernet: Remove duplicate include of vhca_event.h

vhca_event.h has been included at line 4, so remove the duplicate one at line 8. Signed-off-by: Wan Jiabing --- drivers/net/ethernet/mellanox/mlx5/core/sf/hw_table.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/sf/hw_table.c

[PATCH] net/smc: Simplify the return expression

From: zuoqilin Simplify the return expression of smc_ism_signal_shutdown(). Signed-off-by: zuoqilin --- net/smc/smc_ism.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/net/smc/smc_ism.c b/net/smc/smc_ism.c index 9c6e958..c3558cc 100644 --- a/net/smc/smc_ism.c +++

[PATCH] media: staging: media: Remove redundant NULL check

Fix the following coccicheck warnings: ./drivers/staging/media/atomisp/pci/atomisp_cmd.c:4269:2-8: WARNING: NULL check before some freeing functions is not needed. Reported-by: Abaci Robot Signed-off-by: Jiapeng Chong --- drivers/staging/media/atomisp/pci/atomisp_cmd.c | 3 +-- 1 file

Re: [PATCH] keys: Allow disabling read permissions for key possessor

On Mon, Mar 22, 2021 at 12:57:26PM +0300, Andrey Ryabinin wrote: > keyctl_read_key() has a strange code which allows possessor to read > key's payload regardless of READ permission status: > > $ keyctl add user test test @u > 196773443 > $ keyctl print 196773443 > test > $ keyctl describe

[PATCH] net: ethernet: indir_table.h is included twice

indir_table.h has been included at line 41, so remove the duplicate one at line 43. Signed-off-by: Wan Jiabing --- drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c

[PATCH] arch/powerpc/kernel: Duplicate include asm/interrupt.h

asm/interrupt.h is repeatedly in the file interrupt.c. Signed-off-by: zhouchuangao --- arch/powerpc/kernel/interrupt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/kernel/interrupt.c b/arch/powerpc/kernel/interrupt.c index c475a22..6deaccc 100644 ---

Re: [RFC PATCH] arm64: dts: allwinner: a64/h5: Add CPU idle states

On Mon, 22 Mar 2021 01:25:14 -0500 Samuel Holland wrote: Hi, > Powering off idle CPUs saves about 33 mW compared to using WFI only. > Additional power savings are possible by idling the L2 and downclocking > the cluster when all CPUs are idle. > > Entry and exit latency were measured using a

Re: [v1] drm/msm/disp/dpu1: icc path needs to be set before dpu runtime resume

On Mon, Mar 22, 2021 at 5:44 PM Matthias Kaehlcke wrote: > > On Mon, Mar 22, 2021 at 02:17:12AM -0700, Kalyan Thota wrote: > > From: Kalyan Thota > > > > DPU runtime resume will request for a min vote on the AXI bus as > > it is a necessary step before turning ON the AXI clock. > > > > The

[PATCH v3] swiotlb: Make SWIOTLB_NO_FORCE perform no allocation

When SWIOTLB_NO_FORCE is used, there should really be no allocations of default_nslabs to occur since we are not going to use those slabs. If a platform was somehow setting swiotlb_no_force and a later call to swiotlb_init() was to be made we would still be proceeding with allocating the default

[PATCH] usb: gadget: Stall OS descriptor request for unsupported functions

From: Chandana Kishori Chiluveru Hosts which request "OS descriptors" from gadgets do so during the enumeration phase and before the configuration is set with SET_CONFIGURATION. Composite driver supports OS descriptor handling in composite_setup function. This requires to pass signature field,

Re: [PATCH] drivers: pinctrl: Remove duplicate include of io.h

On 2021/03/23 10:38, Wan Jiabing wrote: > linux/io.h has been included at line 6, so remove the > duplicate include at line 18. > > Signed-off-by: Wan Jiabing > --- > drivers/pinctrl/pinctrl-k210.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/pinctrl/pinctrl-k210.c

Re: GTE - The hardware timestamping engine

On 3/22/21 5:32 PM, Kent Gibson wrote: > On Mon, Mar 22, 2021 at 01:21:46PM -0700, Dipen Patel wrote: >> Hi Linus and Kent, >> >> Thanks you so much for your valuable inputs and time, Please see below, my >> follow ups. >> >> On 3/21/21 11:00 PM, Kent Gibson wrote: >>> On Sat, Mar 20, 2021 at

[PATCH] gpu/xen: Fix a use after free in xen_drm_drv_init

In function displback_changed, has the call chain displback_connect(front_info)->xen_drm_drv_init(front_info). We can see that drm_info is assigned to front_info->drm_info and drm_info is freed in fail branch in xen_drm_drv_init(). Later displback_disconnect(front_info) is called and it calls

[PATCH] media: test-drivers: Remove duplicate include of string.h

linux/string.h has been included at line 21, so remove the duplicate one at line 22. Signed-off-by: Wan Jiabing --- drivers/media/test-drivers/vidtv/vidtv_psi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/test-drivers/vidtv/vidtv_psi.c

Re: [RFC PATCH 2/2] integrity: double check iint_cache was initialized

On 2021/03/20 5:03, Mimi Zohar wrote: > The integrity's "iint_cache" is initialized at security_init(). Only > after an IMA policy is loaded, which is initialized at late_initcall, > is a file's integrity status stored in the "iint_cache". > > All integrity_inode_get() callers first verify that

[PATCH] drivers: watchdog: Remove duplicate include of kernel.h

linux/kernel.h has been included at line 17, so remove the duplicate one at line 22. Signed-off-by: Wan Jiabing --- drivers/watchdog/dw_wdt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/watchdog/dw_wdt.c b/drivers/watchdog/dw_wdt.c index 32d0e1781e63..b1642e2d9175 100644 ---

Re: [PATCH] locking/mutex: Remove repeated declaration

On 3/22/21 9:09 PM, Shaokun Zhang wrote: Commit 0cd39f4600ed ("locking/seqlock, headers: Untangle the spaghetti monster") introduces 'struct ww_acquire_ctx' again, remove the repeated declaration. Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Will Deacon Cc: Waiman Long Cc: Boqun Feng

[PATCH] drivers: pinctrl: Remove duplicate include of io.h

linux/io.h has been included at line 6, so remove the duplicate include at line 18. Signed-off-by: Wan Jiabing --- drivers/pinctrl/pinctrl-k210.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-k210.c b/drivers/pinctrl/pinctrl-k210.c index 8a733cf77ba0..f831526d06ff

[PATCH] soundwire: cadence: only prepare attached devices on clock stop

From: Pierre-Louis Bossart We sometimes see COMMAND_IGNORED responses during the clock stop sequence. It turns out we already have information if devices are present on a link, so we should only prepare those when they are attached. In addition, even when COMMAND_IGNORED are received, we should

[PATCH] integrity/ima: Add declarations to init_once void arguments.

init_once is a callback to kmem_cache_create. The parameter type of this function is void *, so it's better to give a explicit cast here. Signed-off-by: Jiele Zhao --- security/integrity/iint.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/integrity/iint.c

Re: [v8,3/7] PCI: mediatek-gen3: Add MediaTek Gen3 driver for MT8192

On Fri, 2021-03-19 at 19:53 +0100, Pali Rohár wrote: > On Thursday 18 March 2021 13:48:07 Jianjun Wang wrote: > > On Thu, 2021-03-18 at 01:02 +0100, Pali Rohár wrote: > > > On Saturday 13 March 2021 15:43:14 Jianjun Wang wrote: > > > > On Thu, 2021-03-11 at 13:38 +0100, Pali Rohár wrote: > > > > >

[PATCH] soc: litex: Remove duplicate include of errno.h

linux/errno.h has been included at line 11, so remove the duplicate include at line 16. Signed-off-by: Wan Jiabing --- drivers/soc/litex/litex_soc_ctrl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/soc/litex/litex_soc_ctrl.c b/drivers/soc/litex/litex_soc_ctrl.c index

Re: [Linuxarm] Re: [PATCH v14 07/13] iommu/smmuv3: Implement cache_invalidate

Hi Eric, 在 2021/3/22 17:05, Auger Eric 写道: Hi Chenxiang, On 3/22/21 7:40 AM, chenxiang (M) wrote: Hi Eric, 在 2021/3/20 1:36, Auger Eric 写道: Hi Chenxiang, On 3/4/21 8:55 AM, chenxiang (M) wrote: Hi Eric, 在 2021/2/24 4:56, Eric Auger 写道: Implement domain-selective, pasid selective and

Re: [RFC 2/7] ath10k: Add support to process rx packet in thread

On Mon, Mar 22, 2021 at 4:58 PM Ben Greear wrote: > On 7/22/20 6:00 AM, Felix Fietkau wrote: > > On 2020-07-22 14:55, Johannes Berg wrote: > >> On Wed, 2020-07-22 at 14:27 +0200, Felix Fietkau wrote: > >> > >>> I'm considering testing a different approach (with mt76 initially): > >>> - Add a

Re: drivers/char/rust_example fails to load

On Mon, Mar 22, 2021 at 1:20 PM Miguel Ojeda wrote: > > I'll take a look at fixing those soon. For those that may read this or run into the same issue: this started with the LLVM 12 upgrade in rustc nightlies. Therefore, please avoid the very latest nightlies (>= nightly-2021-03-05) until [1] is

Re: [PATCH RESEND] PCI: dwc: Fix MSI not work after resume

[+cc Kishon, Richard, Lucas, Dilip] On Mon, Mar 01, 2021 at 11:10:31AM +0800, Jisheng Zhang wrote: > After we move dw_pcie_msi_init() into core -- dw_pcie_host_init(), the > MSI stops working after resume. Because dw_pcie_host_init() is only > called once during probe. To fix this issue, we move

[PATCH] iio: dac: Rudimentary typo fix

s/concurent/concurrent/ Signed-off-by: Bhaskar Chowdhury --- drivers/iio/dac/ad5766.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/dac/ad5766.c b/drivers/iio/dac/ad5766.c index ef1618ea6a20..79837a4b3a41 100644 --- a/drivers/iio/dac/ad5766.c +++

Re: [PATCH v11 0/6] KASAN for powerpc64 radix

Hi Christophe, > In the discussion we had long time ago, > https://patchwork.ozlabs.org/project/linuxppc-dev/patch/20190806233827.16454-5-...@axtens.net/#2321067 > > , I challenged you on why it was not possible to implement things the same > way as other > architectures, in extenso with an

[PATCH] gpu: drm: amd: Remove duplicate includes

../hw_ddc.h, ../hw_gpio.h and ../hw_hpd.h have been included at line 32, so remove them. Signed-off-by: Wan Jiabing --- .../gpu/drm/amd/display/dc/gpio/dce110/hw_factory_dce110.c| 4 1 file changed, 4 deletions(-) diff --git

[PATCH] kernel/sys.c: Fix a typo

s/concurent/concurrent/ Signed-off-by: Bhaskar Chowdhury --- kernel/sys.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/sys.c b/kernel/sys.c index 2e2e3f378d97..5cb22eba59f6 100644 --- a/kernel/sys.c +++ b/kernel/sys.c @@ -2023,7 +2023,7 @@ static int

Re: [PATCH V2] scsi: iscsi_tcp: Fix use-after-free when do get_host_param

On 2021/3/22 4:21, michael.chris...@oracle.com wrote: On 3/21/21 1:47 AM, Wu Bo wrote: From: Wu Bo iscsid(cpu1): Logout of iscsi session, will do destroy session, tcp_sw_host->session is not set to NULL before release the iscsi session. in the iscsi_sw_tcp_session_destroy(). iscsadm(cpu2):

[PATCH] drm/i915: A typo fix

s/nothign/nothing/ Signed-off-by: Bhaskar Chowdhury --- drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c index

[PATCH 5/5] iommu/vt-d: Make unnecessarily global functions static

Make some functions static as they are only used inside pasid.c. Signed-off-by: Lu Baolu --- drivers/iommu/intel/pasid.c | 4 ++-- drivers/iommu/intel/pasid.h | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/iommu/intel/pasid.c b/drivers/iommu/intel/pasid.c index

[PATCH 2/5] iommu/vt-d: Remove svm_dev_ops

The svm_dev_ops has never been referenced in the tree, and there's no plan to have anything to use it. Remove it to make the code neat. Signed-off-by: Lu Baolu --- drivers/iommu/intel/svm.c | 15 +-- include/linux/intel-iommu.h | 3 --- include/linux/intel-svm.h | 7 --- 3

[PATCH 3/5] iommu/vt-d: Remove SVM_FLAG_PRIVATE_PASID

The SVM_FLAG_PRIVATE_PASID has never been referenced in the tree, and there's no plan to have anything to use it. So cleanup it. Signed-off-by: Lu Baolu --- drivers/iommu/intel/svm.c | 40 ++- include/linux/intel-svm.h | 16 +++- 2 files changed,

[PATCH 0/5] iommu/vt-d: Several misc cleanups

Hi Joerg et al, This series includes several cleanups in the VT-d driver. Please help to review. Best regards, baolu Lu Baolu (5): iommu/vt-d: Remove unused dma map/unmap trace events iommu/vt-d: Remove svm_dev_ops iommu/vt-d: Remove SVM_FLAG_PRIVATE_PASID iommu/vt-d: Remove unused

[PATCH 1/5] iommu/vt-d: Remove unused dma map/unmap trace events

With commit c588072bba6b5 ("iommu/vt-d: Convert intel iommu driver to the iommu ops"), the trace events for dma map/unmap have no users any more. Cleanup them to make the code neat. Signed-off-by: Lu Baolu --- include/trace/events/intel_iommu.h | 120 - 1 file

[PATCH 4/5] iommu/vt-d: Remove unused function declarations

Some functions have been deprecated. Remove the remaining function delarations. Signed-off-by: Lu Baolu --- drivers/iommu/intel/pasid.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/iommu/intel/pasid.h b/drivers/iommu/intel/pasid.h index 444c0bec221a..90a3268d7a77 100644 ---

Re: [PATCH 2/6] sched: tagging interface for core scheduling

> > +static unsigned long sched_core_alloc_task_cookie(void) > > +{ > > + struct sched_core_task_cookie *ck = > > + kmalloc(sizeof(struct sched_core_task_cookie), GFP_KERNEL); > > struct sched_core_task_cookie *ck = kmalloc(sizeof(*ck), GFP_KERNEL); > > Also, those type

[PATCH] gpu: drm: amd: Remove duplicate include of dce110_resource.h

dce110/dce110_resource.h has been included at line 58, so remove the duplicate include at line 64. Signed-off-by: Wan Jiabing --- drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c

[PATCH] locking/mutex: Remove repeated declaration

Commit 0cd39f4600ed ("locking/seqlock, headers: Untangle the spaghetti monster") introduces 'struct ww_acquire_ctx' again, remove the repeated declaration. Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Will Deacon Cc: Waiman Long Cc: Boqun Feng Signed-off-by: Shaokun Zhang ---

[PATCH] staging: rtl8723bs: Trivial typo fix

s/netowrk/network/ Signed-off-by: Bhaskar Chowdhury --- drivers/staging/rtl8723bs/core/rtw_mlme.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c index 2c9425e2a1e9..3888d3984ec0 100644

[PATCH] staging: wimax: Mundane typo fixes

s/procesing/processing/ s/comunication/communication/ Signed-off-by: Bhaskar Chowdhury --- drivers/staging/wimax/i2400m/driver.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wimax/i2400m/driver.c b/drivers/staging/wimax/i2400m/driver.c index

Re: [PATCHv2 00/38] Convert power-supply DT bindings to YAML

On Mon, Mar 22, 2021 at 3:43 PM Sebastian Reichel wrote: > > Hi, > > On Mon, Mar 22, 2021 at 10:25:49AM -0600, Rob Herring wrote: > > On Wed, Mar 17, 2021 at 7:49 AM Sebastian Reichel wrote: > > > Sebastian Reichel (38): > > > ARM: dts: motorola-cpcap-mapphone: Prepare for dtbs_check parsing >

[PATCH net-next] net: ipa: avoid 64-bit modulus

It is possible for a 32 bit x86 build to use a 64 bit DMA address. There are two remaining spots where the IPA driver does a modulo operation to check alignment of a DMA address, and under certain conditions this can lead to a build error on i386 (at least). The alignment checks we're doing are

Re: [PATCH v6 00/14] Add some DRM bridge drivers support for i.MX8qm/qxp SoCs

Hi Liu Some further discrepancy with them binding examples: arch/arm64/boot/dts/freescale/imx8qxp.dtsi:335.9-36: Warning (reg_format): /dpu@5618:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 2) arch/arm64/boot/dts/freescale/imx8qxp.dtsi:508.9-35: Warning

[PATCH 5/5] soundwire: qcom: add missing \n in dev_err()

From: Pierre-Louis Bossart We fixed a lot of warnings in 2019 but the magic of copy-paste keeps adding new ones... Signed-off-by: Pierre-Louis Bossart Reviewed-by: Rander Wang Reviewed-by: Guennadi Liakhovetski Signed-off-by: Bard Liao --- drivers/soundwire/qcom.c | 2 +- 1 file changed, 1

[PATCH 4/5] soundwire: stream: add missing \n in dev_err()

From: Pierre-Louis Bossart We fixed a lot of warnings in 2019 but the magic of copy-paste keeps adding new ones... Signed-off-by: Pierre-Louis Bossart Reviewed-by: Rander Wang Reviewed-by: Guennadi Liakhovetski Signed-off-by: Bard Liao --- drivers/soundwire/stream.c | 10 +- 1 file

[PATCH 3/5] soundwire: cadence: add missing \n in dev_err()

From: Pierre-Louis Bossart We fixed a lot of warnings in 2019 but the magic of copy-paste keeps adding new ones... Signed-off-by: Pierre-Louis Bossart Reviewed-by: Rander Wang Reviewed-by: Guennadi Liakhovetski Signed-off-by: Bard Liao --- drivers/soundwire/cadence_master.c | 2 +- 1 file

[PATCH 0/5] soundwire: add missing \n in dev_err()

We fixed a lot of warnings in 2019 but the magic of copy-paste keeps adding new ones... Pierre-Louis Bossart (5): soundwire: intel: add missing \n in dev_err() soundwire: bandwidth_allocation: add missing \n in dev_err() soundwire: cadence: add missing \n in dev_err() soundwire: stream:

[PATCH 2/5] soundwire: bandwidth_allocation: add missing \n in dev_err()

From: Pierre-Louis Bossart We fixed a lot of warnings in 2019 but the magic of copy-paste keeps adding new ones... Signed-off-by: Pierre-Louis Bossart Reviewed-by: Rander Wang Reviewed-by: Guennadi Liakhovetski Signed-off-by: Bard Liao --- drivers/soundwire/generic_bandwidth_allocation.c |

[PATCH 1/5] soundwire: intel: add missing \n in dev_err()

From: Pierre-Louis Bossart We fixed a lot of warnings in 2019 but the magic of copy-paste keeps adding new ones... Signed-off-by: Pierre-Louis Bossart Reviewed-by: Rander Wang Reviewed-by: Guennadi Liakhovetski Signed-off-by: Bard Liao --- drivers/soundwire/intel.c | 18 +-

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