Re: [PATCH 1/2] libata-eh.c: Introduce new ata port flag for controller which lockup on read log page
From: kbuild test robot
Date: Fri Dec 04 2015 - 11:20:20 EST
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
Attachment:
.config.gz
Description: Binary data