On Thu, Sep 17, 2020 at 11:55:07AM +0200, Jan Kara wrote:
> On Thu 10-09-20 16:48:23, Christoph Hellwig wrote:
> > This case isn't ever used.
> >
> > Signed-off-by: Christoph Hellwig
> > Reviewed-by: Johannes Thumshirn
>
> Are you sure it's never used? As far as I'm reading drdb code the conten
On Thu, Sep 17, 2020 at 11:25:24AM +0200, Jan Kara wrote:
> On Thu 10-09-20 16:48:30, Christoph Hellwig wrote:
> > The BDI_CAP_STABLE_WRITES is one of the few bits of information in the
> > backing_dev_info shared between the block drivers and the writeback code.
> > To help untangling the dependen
v1 --> v2:
1. add binding for SD5203 SoC, Patch 1
2. select DW_APB_ICTL instead of HISILICON_SD5203_VIC in Patch 2.
Meanwhile, change the compatible of interrupt-controller to
"snps,dw-apb-ictl" in Patch 4.
3. Fix the errors detected by dtbs_check. For example: add "reg" for cpu node,
use lowe
From: Kefeng Wang
Enable support for the Hisilicon SD5203 SoC. The core is ARM926EJ-S.
Signed-off-by: Kefeng Wang
Signed-off-by: Zhen Lei
---
arch/arm/mach-hisi/Kconfig | 16 ++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/arch/arm/mach-hisi/Kconfig b/arch/arm/
Add devicetree binding for Hisilicon SD5203 SoC.
Signed-off-by: Zhen Lei
---
Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt | 5 +
1 file changed, 5 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt
b/Documentation/devicetree/bindings/a
From: Kefeng Wang
Add support of early console for SD5203.
Signed-off-by: Kefeng Wang
Signed-off-by: Zhen Lei
---
arch/arm/Kconfig.debug | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 8a66a4e3549..d27a7
From: Kefeng Wang
Add sd5203.dts for Hisilicon SD5203 SoC platform.
Signed-off-by: Kefeng Wang
Signed-off-by: Zhen Lei
---
arch/arm/boot/dts/Makefile | 2 +
arch/arm/boot/dts/sd5203.dts | 96
2 files changed, 98 insertions(+)
create mode 100644
On Fri, 18 Sep 2020 at 15:36, George-Aurelian Popescu
wrote:
>
> From: George Popescu
>
[...]
> Suggested-by: Marco Elver
> Reviewed-by: David Brazdil
> Signed-off-by: George Popescu
There's a mismatch between From/Author and Signed-off-by email address.
> ---
> lib/Kconfig.ubsan | 14
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.
This is a behavior seen with clang, from the oldest version capable of
building the mainline x64_64 kernel (9.0), to
On Fri, Sep 18, 2020 at 03:19:05PM +0300, Leon Romanovsky wrote:
> > So we do have an open-source library called hl-thunk, which uses our
> > driver and indeed that was part of the requirement.
> > It is similar to libdrm.
> > Here is the link:
> > https://github.com/HabanaAI/hl-thunk
>
> Are you
aarch64-linux-gnu-ld: drivers/tty/hvc/hvc_dcc.o: in function `dcc_early_write':
hvc_dcc.c:(.text+0x164): undefined reference to `uart_console_write'
The driver uses the uart_console_write(), but SERIAL_CORE_CONSOLE is not
selected, so uart_console_write is not defined, then we get the error.
Fix t
Remove the loop and use the generic ffs instead.
Signed-off-by: Joe Perches
---
Just saw one by happenstance, might as well fix them all.
drivers/net/wireless/realtek/rtlwifi/rtl8188ee/phy.c | 18 ++
.../net/wireless/realtek/rtlwifi/rtl8192c/phy_common.c | 8 ++--
drivers
aarch64-linux-gnu-ld: drivers/tty/serial/imx_earlycon.o: in function
`imx_uart_console_early_write':
imx_earlycon.c:(.text+0x84): undefined reference to `uart_console_write'
The driver uses the uart_console_write(), but SERIAL_CORE_CONSOLE is not
selected, so uart_console_write is not defined, th
On Sat, Sep 19, 2020 at 09:01:12AM +0800, tiantao (H) wrote:
>
>
> 在 2020/9/19 5:09, w...@kernel.org 写道:
> >
> > > static inline struct i2c_client *kobj_to_i2c_client(struct kobject
> > > *kobj)
> > > {
> > > - struct device * const dev = container_of(kobj, struct device, kobj);
> > > + str
On Sat, Sep 19, 2020 at 6:50 AM Pierre-Louis Bossart
wrote:
>
> From: Filipe Brandenburger
>
> Set KBUILD_MODPOST_FAIL_ON_WARNINGS to a non-empty value to make the
> kbuild fail when modpost generates any warnings. This will avoid
> misses such as [1] where the SOF CI did not catch a missing modu
On Wed, Sep 16, 2020 at 01:40:43PM -0700, Joe Perches wrote:
> Convert the unbound sprintf in hugetlb_report_node_meminfo to use
> sysfs_emit_at so that no possible overrun of a PAGE_SIZE buf can occur.
>
> Signed-off-by: Joe Perches
> ---
> drivers/base/node.c | 2 +-
> include/linux/huget
On 2020/9/18 21:54, Gao Xiang wrote:
Let's add REQ_RAHEAD flag so it'd be easier to identify
readahead I/O requests in blktrace.
Signed-off-by: Gao Xiang
Reviewed-by: Chao Yu
Thanks,
On 2020/9/18 21:54, Gao Xiang wrote:
should_decompress_synchronously() has one single condition
for now, so fold it instead.
Signed-off-by: Gao Xiang
Reviewed-by: Chao Yu
Thanks,
On 2020/9/18 21:54, Gao Xiang wrote:
variable `err' in z_erofs_submit_queue() isn't useful
here, remove it instead.
Signed-off-by: Gao Xiang
Reviewed-by: Chao Yu
Thanks,
On Fri, Sep 18, 2020 at 11:14:08AM -0700, Hemant Kumar wrote:
> Hi Greg,
>
> On 9/17/20 9:44 AM, Greg KH wrote:
> > On Wed, Sep 16, 2020 at 12:56:07PM -0700, Hemant Kumar wrote:
> ...
> ...
> > > +
> > > +static int mhi_uci_open(struct inode *inode, struct file *filp)
> > > +{
> > > + struct uci_d
> diff --git a/include/uapi/linux/if.h b/include/uapi/linux/if.h
> index 797ba2c1562a..a332d6ae4dc6 100644
> --- a/include/uapi/linux/if.h
> +++ b/include/uapi/linux/if.h
> @@ -247,7 +247,13 @@ struct ifreq {
> short ifru_flags;
> int ifru_ivalue;
> i
Since commit 141f15c66d94 ("leds: pwm: remove header") that platform
interface is not usable from outside and there seems to be no in tree
user anymore. All in-tree users of the leds-pwm driver seem to use DT
currently. Getting rid of the old platform interface will allow the
leds-pwm driver to s
Hei hei,
for leds-gpio you can use the properties 'function' and 'color' in the
devicetree node and omit 'label', the label is constructed
automatically. This is a common feature supposed to be working for all
LED drivers. However it did not yet work for the 'leds-pwm' driver.
This series remov
> + if (ethtool_translate_compat()) {
> + struct compat_ethtool_rxnfc crxnfc = {};
> +
> + if (copy_from_user(&crxnfc, useraddr,
> +min(size, sizeof(crxnfc
> + return -EFAULT;
> +
> + *rxnfc = (struct et
On Fri, Sep 18, 2020 at 03:24:39PM +0200, Arnd Bergmann wrote:
> The compat implementations for mbind, get_mempolicy, set_mempolicy
> and migrate_pages are just there to handle the subtly different
> layout of bitmaps on 32-bit hosts.
>
> The compat implementation however lacks some of the checks
On Sat, Sep 19, 2020 at 10:26:38AM +0800, Yang Yingliang wrote:
> Hi,
>
> On 2020/9/18 19:16, Greg KH wrote:
> > On Fri, Sep 18, 2020 at 05:13:05PM +0800, Yang Yingliang wrote:
> > > Fix the link error by selecting SERIAL_CORE_CONSOLE.
> > >
> > > aarch64-linux-gnu-ld: drivers/tty/serial/imx_earl
On Sat, Sep 19, 2020 at 10:48:41AM +0800, Yang Yingliang wrote:
>
> On 2020/9/18 19:17, Greg KH wrote:
> > On Fri, Sep 18, 2020 at 05:20:30PM +0800, Yang Yingliang wrote:
> > > Fix the link error by selecting SERIAL_CORE_CONSOLE.
> > >
> > > aarch64-linux-gnu-ld: drivers/tty/hvc/hvc_dcc.o: in fun
If LEDs are configured through device tree and the property 'label' is
omitted, the label is supposed to be generated from the properties
'function' and 'color' if present. While this works fine for e.g. the
'leds-gpio' driver, it did not for 'leds-pwm'.
The reason is, you get this label naming m
On Fri, Sep 18, 2020 at 03:24:38PM +0200, Arnd Bergmann wrote:
> The compat move_pages() implementation uses compat_alloc_user_space()
> for converting the pointer array. Moving the compat handling into
> the function itself is a bit simpler and lets us avoid the
> compat_alloc_user_space() call.
>
The example was adapted slightly to make use of the 'function' and
'color' properties. License discussed with the original author.
Suggested-by: Jacek Anaszewski
Signed-off-by: Alexander Dahl
Cc: Peter Ujfalusi
---
Notes:
v4 -> v5:
* updated based on feedback by Rob Herring
*
On Fri, Sep 18, 2020 at 03:24:37PM +0200, Arnd Bergmann wrote:
> The compat version of sys_kexec_load() uses compat_alloc_user_space to
> convert the user-provided arguments into the native format.
>
> Move the conversion into the regular implementation with
> an in_compat_syscall() check to simpl
On Fri, Sep 18, 2020 at 03:24:36PM +0200, Arnd Bergmann wrote:
> sys_move_pages() is an optional syscall, and once we remove
> the compat version of it in favor of the native one with an
> in_compat_syscall() check, the x32 syscall table refers to
> a __x32_sys_move_pages symbol that may not exist
> index 855aa7cc9b8e..156880943c16 100644
> --- a/arch/arm/include/asm/syscall.h
> +++ b/arch/arm/include/asm/syscall.h
> @@ -28,6 +28,17 @@ static inline int syscall_get_nr(struct task_struct *task,
> return task_thread_info(task)->syscall & ~__NR_OABI_SYSCALL_BASE;
> }
>
> +static inline
On 9/18/20 12:18 AM, xiao.mx.ma wrote:
> Add delta.rst in Documentation/hwmon
> Tristate "DELTA" in Kconfig is changed to "DELTA_POWER_SUPPLIES"
> Modify code: drop the excessive empty lines, correct the comment content,
> adjust indent, remove extra brackets.
>
Description goes here.
> Signed-of
On Fri, Sep 18, 2020 at 02:46:17PM +0200, Arnd Bergmann wrote:
> ARM uses set_fs() and __get_user() to allow the stack dumping code to
> access possibly invalid pointers carefully. These can be changed to the
> simpler get_kernel_nofault(), and allow the eventual removal of set_fs().
>
> dump_inst
On Fri, Sep 18, 2020 at 02:46:15PM +0200, Arnd Bergmann wrote:
> Hi Christoph, Russell,
>
> Here is an updated series for removing set_fs() from arch/arm,
> based on the previous feedback.
>
> I have tested the oabi-compat changes using the LTP tests for the three
> modified syscalls using an Arm
On Fri, Sep 18, 2020 at 02:15:43PM +0200, Arnd Bergmann wrote:
> There have been several attempts to fix serious problems
> in the compat handling in megasas_mgmt_compat_ioctl_fw(),
> and it also uses the compat_alloc_user_space() function.
>
> Folding the compat handling into the regular ioctl
>
On Fri, Sep 18, 2020 at 02:15:22PM +0200, Arnd Bergmann wrote:
> It sounds unwise to let user space pass an unchecked 32-bit
> offset into a kernel structure in an ioctl. This is an unsigned
> variable, so checking the upper bound for the size of the structure
> it points into is sufficient to avoi
On Fri, Sep 18, 2020 at 11:52:39AM +0300, Sergei Shtylyov wrote:
> Hello!
>
> On 17.09.2020 19:57, Christoph Hellwig wrote:
>
>> Use blkdev_get_by_dev instead of open coding it using bdget_disk +
>> blkdev_get.
>
>I don't see where you are removing bdget_disk() call (situated just before
> the
On Fri, Sep 18, 2020 at 02:49:19PM -0700, Kees Cook wrote:
> In response to my recent bug fix for splice vs sysfs binary handler[1],
> I added splice testing for other pseudo filesystems[2], for which the
> test output is seen above.
>
> What is the final verdict on the "should splice have a fallb
On Fri, Sep 18, 2020 at 09:54:35PM +0800, Gao Xiang wrote:
> Use more accurate compressed page count
> instead of BIO_MAX_PAGES unconditionally.
>
> Signed-off-by: Gao Xiang
Found by ro_fsstress, the submission chain could be extended
by other threads, so this patch wouldn't work with the tail
m
From: zhuguangqing
The injected idle duration must be greater than the wake up latency
(entry + exit latency) or the idle state target residency (or min
residency), otherwise we end up consuming more energy and potentially
invert the mitigation effect.
In function idle_inject_fn(), we call play
Hi Arnaldo,
On 9/18/2020 4:26 AM, Arnaldo Carvalho de Melo wrote:
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
On Mon, 14 Sep 2020 20:58:13 PDT (-0700), greentime...@sifive.com wrote:
Palmer Dabbelt 於 2020年8月5日 週三 上午10:03寫道：
On Mon, 03 Aug 2020 20:29:32 PDT (-0700), a...@brainfault.org wrote:
> On Tue, Aug 4, 2020 at 8:32 AM Greentime Hu wrote:
>>
>> This patch addes local_flush_tlb_page(addr) to use
On Thu, 17 Sep 2020 at 07:34, Andreas Schwab wrote:
>
> On Sep 17 2020, Arnd Bergmann wrote:
>
> > The errno man page says they are supposed to be synonyms,
> > and glibc defines it that way, while musl uses the numbers
> > from the kernel.
>
> glibc also uses whatever the kernel defines.
>
That's
On Sat, Sep 19, 2020 at 12:46 AM Steven Rostedt wrote:
>
> On Sat, 5 Sep 2020 14:04:12 +0800
> Muchun Song wrote:
>
> > Introduce tracepoints for tasklets just like softirq does. In this case,
> > we can calculate tasklet latency and know what tasklet run.
> >
> > Signed-off-by: Muchun Song
> >
Hi, Jinyang,
On Fri, Sep 18, 2020 at 2:20 PM Jinyang He wrote:
>
> On 09/17/2020 09:52 PM, Zhou Yanjie wrote:
> > Hello,
> >
> > 在 2020/9/17 下午8:41, Jinyang He 写道:
> >> Hi, Huacai,
> >>
> >>
> >> On 09/16/2020 01:39 PM, Huacai Chen wrote:
> >>> Hi, Jinyang,
> >>>
> >>> On Tue, Sep 15, 2020 at 10:
The variable ret has been initialized with '-ENOMEM'. The assignment
in the if branch is redundant. So remove it.
Signed-off-by: Jing Xiangfeng
---
arch/arm/mach-omap2/omap_device.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_device.c
b/arch
> -Original Message-
> From: Dan Williams
> Sent: Saturday, September 19, 2020 3:51 AM
> To: dm-de...@redhat.com
> Cc: sta...@vger.kernel.org; Jan Kara ; Alasdair Kergon
> ; Mike Snitzer ; Adrian Huang12
> ; linux-nvd...@lists.01.org; linux-
> ker...@vger.kernel.org
> Subject: [External] [
On Sat, Sep 19, 2020 at 03:45:56AM +0100, Matthew Wilcox wrote:
> On Fri, Sep 18, 2020 at 06:39:57PM -0400, Arvind Sankar wrote:
> > On Fri, Sep 18, 2020 at 02:18:20PM -0700, Linus Torvalds wrote:
> > > On Fri, Sep 18, 2020 at 2:00 PM Arvind Sankar
> > > wrote:
> > > >
> > > > You could just asse
Fix following warnings caused by mismatch bewteen function parameters
and comments.
mm/workingset.c:228: warning: Function parameter or member 'lruvec' not
described in 'workingset_age_nonresident'
mm/workingset.c:228: warning: Excess function parameter 'memcg' description in
'workingset_age_nonr
On powerpc, access_ok() succeeds for the NULL pointer. This breaks the
dynamic check in futex_detect_cmpxchg(), which expects -EFAULT. As a
result, robust futex operations are not functional on powerpc.
Since the architecture's futex_atomic_cmpxchg_inatomic() implementation
requires no runtime fea
Hi Paul,
On 2020/9/19 1:59, Paul E. McKenney wrote:
> On Fri, Sep 18, 2020 at 07:44:24PM +0800, Hou Tao wrote:
>> When do locktorture for exclusive lock which doesn't have readlock
>> support, the following module parameters will be considered as valid:
>>
>> torture_type=mutex_lock nwriters_stre
Hi Keith,
Thanks a lot for your comments,
I will try to figure out a safe fix for this issue, then for you review:)
-Original Message-
From: Keith Busch [mailto:kbu...@kernel.org]
Sent: Saturday, September 19, 2020 3:21 AM
To: tianxianting (RD)
Cc: ax...@fb.com; h...@lst.de; s...@grimbe
After commit 9f51c05dc41a ("pvcalls-front: Avoid
get_free_pages(GFP_KERNEL) under spinlock"), the variable ret is being
initialized with '-ENOMEM' that is meaningless. So remove it.
Signed-off-by: Jing Xiangfeng
---
drivers/xen/pvcalls-front.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-
Adjust some tiny coding problems and fix a bug of DH algorithm.
Changes since v1:
Remove 'u64' cast in 'cpu_to_le64';
Meng Yu (3):
crypto: hisilicon/hpre - remove useless code
crypto: hisilicon/hpre - adjust some coding style
crypto: hisilicon/hpre - fix a bug in dh algorithm
drivers/cryp
From: Meng Yu
1. Remove unused member 'debug_root' in 'struct hpre_debug';
2. The u64 cast is redundant in 'cpu_to_le64'.
Fixes: 848974151618("crypto: hisilicon - Add debugfs for HPRE")
Fixes: dadbe4c11753("crypto: hisilicon/hpre - update debugfs ...")
Signed-off-by: Meng Yu
Reviewed-by: Zaibo
From: Meng Yu
Using 'g' not equal to 2 in dh algorithm may cause an error like this:
arm-smmu-v3 arm-smmu-v3.1.auto: event 0x10 received:
dh: Party A: generate public key test failed. err -22
11375.065672] dh alg: dh: test failed on vector 1, err=-22
arm-smmu-v3 arm-smmu-v3.1.auto: 0x79
From: Meng Yu
Adjust some coding style to make code aligned.
Signed-off-by: Meng Yu
Reviewed-by: Zaibo Xu
---
drivers/crypto/hisilicon/hpre/hpre_crypto.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/crypto/hisilicon/hpre/hpre_crypto.c
b/drivers/cr
This is the bad patch, my fault, I forget to check patch title, please ignore
it, thank you very much.
-邮件原件-
发件人: zhengyongjun
发送时间: 2020年9月19日 10:39
收件人: bryan.whiteh...@microchip.com; unglinuxdri...@microchip.com;
da...@davemloft.net; k...@kernel.org; net...@vger.kernel.org;
linux-k
The pull request you sent on Sat, 19 Sep 2020 01:27:25 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git tags/s390-5.9-6
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/eb5f95f1593f7c22dac681b19e815828e2af3efd
Thank you!
--
Deet-doot-dot, I am a b
On Fri, Sep 18, 2020 at 7:53 PM Arvind Sankar wrote:
>
> Is it ever necessary to allocate _at least_ sizeof() even if
> offsetof()+size is smaller?
Not that I can tell.
Obviously all allocators tend to have their own alignment concerns, so
they'll all align things up internally anyway.
But why
On Fri, Sep 18, 2020 at 10:53:36PM -0400, Arvind Sankar wrote:
> I think you can't do this in standard C. It's a GCC extension.
>
> A structure containing a flexible array member, or a union
> containing such a structure (possibly recursively), may not be a
> member of a structur
This is the bad patch, please ignore it, thank you very much.
-邮件原件-
发件人: zhengyongjun
发送时间: 2020年9月19日 10:38
收件人: bryan.whiteh...@microchip.com; unglinuxdri...@microchip.com;
da...@davemloft.net; k...@kernel.org; net...@vger.kernel.org;
linux-kernel@vger.kernel.org
抄送: zhengyongjun
主题
On Fri, Sep 18, 2020 at 7:20 AM Heikki Krogerus
wrote:
>
> On Thu, Sep 17, 2020 at 03:18:56AM -0700, Badhri Jagan Sridharan wrote:
> > Programs VBUS_SINK_DISCONNECT_THRESHOLD based on the power_role,
> > voltage requested as sink, mode of operation.
> >
> > The programmed threshold is based on vSi
On Fri, Sep 18, 2020 at 06:28:30PM -0700, Linus Torvalds wrote:
> On Fri, Sep 18, 2020 at 3:40 PM Arvind Sankar wrote:
> >
> > Ouch, offsetof() and sizeof() will give different results in the
> > presence of alignment padding.
>
> Indeed. But from an allocation standpoint, the offsetof()+size is
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
arch/mips/cavium-octeon/oct_ilm.c | 16 +++-
arch/mips/kernel/segment.c| 14 ++
arch/mips/ral
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
arch/sh/mm/asids-debugfs.c | 15 ++-
arch/sh/mm/cache-debugfs.c | 15 ++-
arch/sh/mm/pmb.c
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/usb/gadget/udc/lpc32xx_udc.c | 17 +++--
1 file changed, 3 insertions(+), 14 deletions(-)
diff --git a/d
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
arch/x86/xen/p2m.c | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git a/arch/x86/xen/p2m.c b/arch
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/scsi/qla2xxx/qla_dfs.c | 68 --
1 file changed, 8 insertions(+), 60 deletions(-)
dif
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/soc/ti/knav_dma.c| 16 +++-
drivers/soc/ti/knav_qmss_queue.c | 14 ++
2 files changed
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/visorbus/visorbus_main.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/vi
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/block/nbd.c | 28
1 file changed, 4 insertions(+), 24 deletions(-)
diff --git a/drivers
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
arch/arm/mm/ptdump_debugfs.c | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git a/arch/arm/mm/ptd
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/bus/mvebu-mbus.c | 24 ++--
1 file changed, 2 insertions(+), 22 deletions(-)
diff --git a/driver
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/block/aoe/aoeblk.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/block/
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c | 15 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 29 ++---
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/scsi/snic/snic_debugfs.c | 20 +---
1 file changed, 1 insertion(+), 19 deletions(-)
diff --git a/dri
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/sh/intc/virq-debugfs.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/sh
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
arch/c6x/platforms/pll.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/arch/c6x/platfor
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/block/pktcdvd.c | 19 ---
1 file changed, 4 insertions(+), 15 deletions(-)
diff --git a/drivers/bloc
在 2020/7/16 23:20, Jakub Kicinski 写道:
On Thu, 16 Jul 2020 16:57:49 +0800 Qinglang Miao wrote:
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
Acked-by: Jakub Kicinski
Hi Jakub,
I noticed that this patch has been acked by you and not patched into
lin
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
.../net/wireless/mediatek/mt7601u/debugfs.c | 34 ---
1 file changed, 6 insertions(+), 28 deletions(-)
diff --git a/drivers/net/wireless/mediatek/mt7601u/debugfs.c
b/drivers/net/wireless/me
On 2020/9/18 19:17, Greg KH wrote:
On Fri, Sep 18, 2020 at 05:20:30PM +0800, Yang Yingliang wrote:
Fix the link error by selecting SERIAL_CORE_CONSOLE.
aarch64-linux-gnu-ld: drivers/tty/hvc/hvc_dcc.o: in function `dcc_early_write':
hvc_dcc.c:(.text+0x164): undefined reference to `uart_console
On Fri, Sep 18, 2020 at 06:39:57PM -0400, Arvind Sankar wrote:
> On Fri, Sep 18, 2020 at 02:18:20PM -0700, Linus Torvalds wrote:
> > On Fri, Sep 18, 2020 at 2:00 PM Arvind Sankar wrote:
> > >
> > > You could just assert that offsetof(typeof(s),flex) == sizeof(s), no?
> >
> > No, because the whole
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/net/ethernet/natsemi/ns83820.c: In function ns83820_get_link_ksettings:
drivers/net/ethernet/natsemi/ns83820.c:1210:11: warning: variable ‘tanar’ set
but not used [-Wunused-but-set-variable]
`tanar` is never used, so remove it.
Signed-off-b
On 2020/9/19 10:11, Alex Williamson wrote:
On Sat, 19 Sep 2020 09:54:00 +0800
Zenghui Yu wrote:
Hi Alex,
On 2020/9/18 6:07, Alex Williamson wrote:
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 purpo
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Qinglang Miao
---
v2: based on linux-next(20200917), and can be applied to
mainline cleanly now.
drivers/scsi/hisi_sas/hisi_sas_main.c | 137 ++
1 file changed, 10 insertions(+), 127 deletions(-)
d
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/net/ethernet/microchip/lan743x_main.c: In function lan743x_pm_suspend:
drivers/net/ethernet/microchip/lan743x_main.c:3041:6: warning: variable ‘ret’
set but not used [-Wunused-but-set-variable]
`ret` is set but not used, so check it's value.
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/net/ethernet/microchip/lan743x_main.c: In function lan743x_pm_suspend:
drivers/net/ethernet/microchip/lan743x_main.c:3041:6: warning: variable ‘ret’
set but not used [-Wunused-but-set-variable]
`ret` is never used, so remove it.
Signed-off-
On 2020-09-17 16:16, Shuah Khan wrote:
While looking at this file for an unrelated issue, I happen to notice
there is a double locking on mhi_cntrl->pm_lock in the mhi_device_put()
when it gets called from mhi_driver_remove()
The other two calls from mhi_driver_probe() don't hold the pm_lock.
I
在 2020/7/16 20:39, luojiaxing 写道:
Hi, Qinglang
On 2020/7/16 16:47, Qinglang Miao wrote:
From: Yongqiang Liu
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by: Yongqiang Liu
---
drivers/scsi/hisi_sas/hisi_sas_main.c | 137 ++
1 file changed, 10
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/net/ethernet/micrel/ksz884x.c: In function rx_proc:
drivers/net/ethernet/micrel/ksz884x.c:4981:6: warning: variable ‘rx_status’ set
but not used [-Wunused-but-set-variable]
drivers/net/ethernet/micrel/ksz884x.c: In function netdev_get_ethtoo
When budget is non-zero, skb_unref() has already handled the
NULL checking.
When budget is zero, the dev_consume_skb_any() has handled NULL
checking in __dev_kfree_skb_irq(), or dev_kfree_skb() which also
ultimately call skb_unref().
So remove the unnecessary checking in napi_consume_skb().
Sign
Fixes the following W=1 kernel build warning(s):
drivers/net/ethernet/marvell/mvneta.c:754:6: warning:
variable 'dummy' set but not used [-Wunused-but-set-variable]
754 | u32 dummy;
| ^
This variable is not used in function mvneta_mib_counters_clear(), so
remove it to avoid bui
Hi Jiri,
On 2020/9/18 15:52, Jiri Slaby wrote:
> On 18. 09. 20, 9:47, Xiaofei Tan wrote:
>> Fix following warnings caused by mismatch bewteen function parameters
>> and comments.
>> drivers/tty/vt/vt.c:3994: warning: Function parameter or member 'vc' not
>> described in 'con_debug_enter'
>> drive
Hi,
On 2020/9/18 19:16, Greg KH wrote:
On Fri, Sep 18, 2020 at 05:13:05PM +0800, Yang Yingliang wrote:
Fix the link error by selecting SERIAL_CORE_CONSOLE.
aarch64-linux-gnu-ld: drivers/tty/serial/imx_earlycon.o: in function
`imx_uart_console_early_write':
imx_earlycon.c:(.text+0x84): undefin
Please ignore this patch, the title is not right, sorry for the noise.
On 2020/9/19 10:16, Yunsheng Lin wrote:
> When budget is non-zero, skb_unref() has already handled the
> NULL checking.
>
> When budget is zero, the dev_consume_skb_any() has handled NULL
> checking in __dev_kfree_skb_irq(), o
When budget is non-zero, skb_unref() has already handled the
NULL checking.
When budget is zero, the dev_consume_skb_any() has handled NULL
checking in __dev_kfree_skb_irq(), or dev_kfree_skb() which also
ultimately call skb_unref().
So remove the unnecessary checking in napi_consume_skb().
Sign
1 - 100 of 1343 matches
Mail list logo
