linux-kernel - Re: [PATCH v2 15/15] MAINTAINERS: Add entry for RDA Micro SoC architecture

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <201811241627.2eQKZPBL%fengguang.wu@intel.com>
Date:   Sat, 24 Nov 2018 16:30:06 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Cc:     kbuild-all@...org, olof@...om.net, arnd@...db.de,
        robh+dt@...nel.org, tglx@...utronix.de, jason@...edaemon.net,
        marc.zyngier@....com, daniel.lezcano@...aro.org,
        gregkh@...uxfoundation.org, jslaby@...e.com, afaerber@...e.de,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        devicetree@...r.kernel.org, linux-serial@...r.kernel.org,
        amit.kucheria@...aro.org, linus.walleij@...aro.org,
        zhao_steven@....net, overseas.sales@...soc.com,
        Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Subject: Re: [PATCH v2 15/15] MAINTAINERS: Add entry for RDA Micro SoC
 architecture

Hi Manivannan,

I love your patch! Perhaps something to improve:

[auto build test WARNING on robh/for-next]
[also build test WARNING on v4.20-rc3 next-20181123]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Manivannan-Sadhasivam/Add-initial-RDA8810PL-SoC-and-Orange-Pi-boards-support/20181123-125507
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: x86_64-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

>> drivers/tty/serial/rda-uart.c:619:28: warning: context imbalance in 'rda_uart_port_write' - unexpected unlock

vim +/rda_uart_port_write +619 drivers/tty/serial/rda-uart.c

efb19753 Manivannan Sadhasivam 2018-11-21  588  
efb19753 Manivannan Sadhasivam 2018-11-21  589  static void rda_uart_port_write(struct uart_port *port, const char *s,
efb19753 Manivannan Sadhasivam 2018-11-21  590  				u_int count)
efb19753 Manivannan Sadhasivam 2018-11-21  591  {
efb19753 Manivannan Sadhasivam 2018-11-21  592  	u32 old_irq_mask;
efb19753 Manivannan Sadhasivam 2018-11-21  593  	unsigned long flags;
efb19753 Manivannan Sadhasivam 2018-11-21  594  	int locked;
efb19753 Manivannan Sadhasivam 2018-11-21  595  
efb19753 Manivannan Sadhasivam 2018-11-21  596  	local_irq_save(flags);
efb19753 Manivannan Sadhasivam 2018-11-21  597  
efb19753 Manivannan Sadhasivam 2018-11-21  598  	if (port->sysrq) {
efb19753 Manivannan Sadhasivam 2018-11-21  599  		locked = 0;
efb19753 Manivannan Sadhasivam 2018-11-21  600  	} else if (oops_in_progress) {
efb19753 Manivannan Sadhasivam 2018-11-21  601  		locked = spin_trylock(&port->lock);
efb19753 Manivannan Sadhasivam 2018-11-21  602  	} else {
efb19753 Manivannan Sadhasivam 2018-11-21  603  		spin_lock(&port->lock);
efb19753 Manivannan Sadhasivam 2018-11-21  604  		locked = 1;
efb19753 Manivannan Sadhasivam 2018-11-21  605  	}
efb19753 Manivannan Sadhasivam 2018-11-21  606  
efb19753 Manivannan Sadhasivam 2018-11-21  607  	old_irq_mask = rda_uart_read(port, RDA_UART_IRQ_MASK);
efb19753 Manivannan Sadhasivam 2018-11-21  608  	rda_uart_write(port, 0, RDA_UART_IRQ_MASK);
efb19753 Manivannan Sadhasivam 2018-11-21  609  
efb19753 Manivannan Sadhasivam 2018-11-21  610  	uart_console_write(port, s, count, rda_console_putchar);
efb19753 Manivannan Sadhasivam 2018-11-21  611  
efb19753 Manivannan Sadhasivam 2018-11-21  612  	/* wait until all contents have been sent out */
efb19753 Manivannan Sadhasivam 2018-11-21  613  	while (!(rda_uart_read(port, RDA_UART_STATUS) & RDA_UART_TX_FIFO_MASK))
efb19753 Manivannan Sadhasivam 2018-11-21  614  		cpu_relax();
efb19753 Manivannan Sadhasivam 2018-11-21  615  
efb19753 Manivannan Sadhasivam 2018-11-21  616  	rda_uart_write(port, old_irq_mask, RDA_UART_IRQ_MASK);
efb19753 Manivannan Sadhasivam 2018-11-21  617  
efb19753 Manivannan Sadhasivam 2018-11-21  618  	if (locked)
efb19753 Manivannan Sadhasivam 2018-11-21 @619  		spin_unlock(&port->lock);
efb19753 Manivannan Sadhasivam 2018-11-21  620  
efb19753 Manivannan Sadhasivam 2018-11-21  621  	local_irq_restore(flags);
efb19753 Manivannan Sadhasivam 2018-11-21  622  }
efb19753 Manivannan Sadhasivam 2018-11-21  623  

:::::: The code at line 619 was first introduced by commit
:::::: efb197538a7e148635e096850a818e3b29b1c4b2 tty: serial: Add RDA8810PL UART driver

:::::: TO: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
:::::: CC: 0day robot <lkp@...el.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/gzip" (66615 bytes)