linux-kernel - Re: [PATCH 1/2] libata-eh.c: Introduce new ata port flag for controller which lockup on read log page

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] [day] [month] [year] [list]

Message-ID: <201512050022.mZp5ScP0%fengguang.wu@intel.com>
Date:	Sat, 5 Dec 2015 00:19:38 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Andreas Werner <andreas.werner@....de>
Cc:	kbuild-all@...org, linux-kernel@...r.kernel.org,
	linux-ide@...r.kernel.org, tj@...nel.org
Subject: Re: [PATCH 1/2] libata-eh.c: Introduce new ata port flag for
 controller which lockup on read log page

Hi Andreas,

[auto build test ERROR on tj-libata/for-next]
[also build test ERROR on v4.4-rc3 next-20151203]

url:    https://github.com/0day-ci/linux/commits/Andreas-Werner/libata-introduce-ATA_FLAG_NO_LOG_PAGE/20151204-235659
base:   https://git.kernel.org/pub/scm/linux/kernel/git/tj/libata for-next
config: i386-randconfig-x008-12041200 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All error/warnings (new ones prefixed by >>):

   drivers/ata/libata-eh.c: In function 'ata_read_log_page':
>> drivers/ata/libata-eh.c:1520:6: error: 'flags' undeclared (first use in this function)
     if (flags & ATA_FLAG_NO_LOG_PAGE)
         ^
   drivers/ata/libata-eh.c:1520:6: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/ata/libata-eh.c:1508:16: warning: unused variable 'ap_flags' [-Wunused-variable]
     unsigned long ap_flags = dev->link->ap->flags;
                   ^

vim +/flags +1520 drivers/ata/libata-eh.c

  1502	 *	RETURNS:
  1503	 *	0 on success, AC_ERR_* mask otherwise.
  1504	 */
  1505	unsigned int ata_read_log_page(struct ata_device *dev, u8 log,
  1506				       u8 page, void *buf, unsigned int sectors)
  1507	{
> 1508		unsigned long ap_flags = dev->link->ap->flags;
  1509		struct ata_taskfile tf;
  1510		unsigned int err_mask;
  1511		bool dma = false;
  1512	
  1513		DPRINTK("read log page - log 0x%x, page 0x%x\n", log, page);
  1514	
  1515		/*
  1516		 * Return error without actually issuing the
  1517		 * command on controller which e.g. lockup
  1518		 * on a read log page.
  1519		 */
> 1520		if (flags & ATA_FLAG_NO_LOG_PAGE)
  1521			return AC_ERR_DEV;
  1522	
  1523	retry:

---
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/octet-stream" (29818 bytes)