* drivers/scsi/hisi_sas/hisi_sas_main.c:445 hisi_sas_task_deliver() warn: inconsistent indenting
@ 2022-03-30 17:14 kernel test robot
0 siblings, 0 replies; 3+ messages in thread
From: kernel test robot @ 2022-03-30 17:14 UTC (permalink / raw)
To: John Garry; +Cc: kbuild-all, linux-kernel, Martin K. Petersen
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: d888c83fcec75194a8a48ccd283953bdba7b2550
commit: 095478a6e5bf590f2bbf341569eb25173c9c5f32 scsi: hisi_sas: Use libsas internal abort support
date: 2 weeks ago
config: powerpc64-randconfig-m031-20220330 (https://download.01.org/0day-ci/archive/20220331/202203310159.dWb4ZUaK-lkp@intel.com/config)
compiler: powerpc64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
smatch warnings:
drivers/scsi/hisi_sas/hisi_sas_main.c:445 hisi_sas_task_deliver() warn: inconsistent indenting
vim +445 drivers/scsi/hisi_sas/hisi_sas_main.c
b3cce125cb1e2e Xiang Chen 2019-02-06 393
dc313f6b125b09 John Garry 2021-12-15 394 static
dc313f6b125b09 John Garry 2021-12-15 395 void hisi_sas_task_deliver(struct hisi_hba *hisi_hba,
dc313f6b125b09 John Garry 2021-12-15 396 struct hisi_sas_slot *slot,
dc313f6b125b09 John Garry 2021-12-15 397 struct hisi_sas_dq *dq,
095478a6e5bf59 John Garry 2022-03-11 398 struct hisi_sas_device *sas_dev)
42e7a69368a585 John Garry 2015-11-18 399 {
42e7a69368a585 John Garry 2015-11-18 400 struct hisi_sas_cmd_hdr *cmd_hdr_base;
dc313f6b125b09 John Garry 2021-12-15 401 int dlvry_queue_slot, dlvry_queue;
dc313f6b125b09 John Garry 2021-12-15 402 struct sas_task *task = slot->task;
fa222db0b03689 Xiang Chen 2018-05-09 403 int wr_q_index;
42e7a69368a585 John Garry 2015-11-18 404
e9dc5e11c97ee9 Xiang Chen 2020-01-20 405 spin_lock(&dq->lock);
897cc769bcc092 John Garry 2019-08-05 406 wr_q_index = dq->wr_point;
897cc769bcc092 John Garry 2019-08-05 407 dq->wr_point = (dq->wr_point + 1) % HISI_SAS_QUEUE_SLOTS;
fa222db0b03689 Xiang Chen 2018-05-09 408 list_add_tail(&slot->delivery, &dq->list);
e9dc5e11c97ee9 Xiang Chen 2020-01-20 409 spin_unlock(&dq->lock);
e9dc5e11c97ee9 Xiang Chen 2020-01-20 410 spin_lock(&sas_dev->lock);
4fefe5bbf599d6 Xiang Chen 2019-02-06 411 list_add_tail(&slot->entry, &sas_dev->list);
e9dc5e11c97ee9 Xiang Chen 2020-01-20 412 spin_unlock(&sas_dev->lock);
42e7a69368a585 John Garry 2015-11-18 413
b1a49412f0aed7 Xiang Chen 2017-06-14 414 dlvry_queue = dq->id;
fa222db0b03689 Xiang Chen 2018-05-09 415 dlvry_queue_slot = wr_q_index;
42e7a69368a585 John Garry 2015-11-18 416
4fefe5bbf599d6 Xiang Chen 2019-02-06 417 slot->device_id = sas_dev->device_id;
42e7a69368a585 John Garry 2015-11-18 418 slot->dlvry_queue = dlvry_queue;
42e7a69368a585 John Garry 2015-11-18 419 slot->dlvry_queue_slot = dlvry_queue_slot;
42e7a69368a585 John Garry 2015-11-18 420 cmd_hdr_base = hisi_hba->cmd_hdr[dlvry_queue];
42e7a69368a585 John Garry 2015-11-18 421 slot->cmd_hdr = &cmd_hdr_base[dlvry_queue_slot];
dc313f6b125b09 John Garry 2021-12-15 422
42e7a69368a585 John Garry 2015-11-18 423 task->lldd_task = slot;
42e7a69368a585 John Garry 2015-11-18 424
42e7a69368a585 John Garry 2015-11-18 425 memset(slot->cmd_hdr, 0, sizeof(struct hisi_sas_cmd_hdr));
f557e32c0023ea Xiaofei Tan 2017-06-29 426 memset(hisi_sas_cmd_hdr_addr_mem(slot), 0, HISI_SAS_COMMAND_TABLE_SZ);
d380f55503ed28 Xiang Chen 2019-08-05 427 memset(hisi_sas_status_buf_addr_mem(slot), 0,
d380f55503ed28 Xiang Chen 2019-08-05 428 sizeof(struct hisi_sas_err_record));
42e7a69368a585 John Garry 2015-11-18 429
42e7a69368a585 John Garry 2015-11-18 430 switch (task->task_proto) {
66ee999b4e43e1 John Garry 2015-11-18 431 case SAS_PROTOCOL_SMP:
a2b3820bddfbff Xiang Chen 2018-05-09 432 hisi_sas_task_prep_smp(hisi_hba, slot);
66ee999b4e43e1 John Garry 2015-11-18 433 break;
42e7a69368a585 John Garry 2015-11-18 434 case SAS_PROTOCOL_SSP:
78bd2b4f6e7c05 Xiaofei Tan 2018-05-21 435 hisi_sas_task_prep_ssp(hisi_hba, slot);
42e7a69368a585 John Garry 2015-11-18 436 break;
42e7a69368a585 John Garry 2015-11-18 437 case SAS_PROTOCOL_SATA:
42e7a69368a585 John Garry 2015-11-18 438 case SAS_PROTOCOL_STP:
095478a6e5bf59 John Garry 2022-03-11 439 case SAS_PROTOCOL_STP_ALL:
a2b3820bddfbff Xiang Chen 2018-05-09 440 hisi_sas_task_prep_ata(hisi_hba, slot);
6f2ff1a1311e61 John Garry 2016-01-26 441 break;
095478a6e5bf59 John Garry 2022-03-11 442 case SAS_PROTOCOL_INTERNAL_ABORT:
095478a6e5bf59 John Garry 2022-03-11 443 hisi_sas_task_prep_abort(hisi_hba, slot);
dc313f6b125b09 John Garry 2021-12-15 444 break;
dc313f6b125b09 John Garry 2021-12-15 @445 fallthrough;
42e7a69368a585 John Garry 2015-11-18 446 default:
095478a6e5bf59 John Garry 2022-03-11 447 return;
42e7a69368a585 John Garry 2015-11-18 448 }
42e7a69368a585 John Garry 2015-11-18 449
1c09b663168bb5 Xiaofei Tan 2018-07-18 450 WRITE_ONCE(slot->ready, 1);
42e7a69368a585 John Garry 2015-11-18 451
0e4620856b8933 John Garry 2021-12-15 452 spin_lock(&dq->lock);
0e4620856b8933 John Garry 2021-12-15 453 hisi_hba->hw->start_delivery(dq);
0e4620856b8933 John Garry 2021-12-15 454 spin_unlock(&dq->lock);
42e7a69368a585 John Garry 2015-11-18 455 }
42e7a69368a585 John Garry 2015-11-18 456
:::::: The code at line 445 was first introduced by commit
:::::: dc313f6b125b095d3d2683d94d5f69c8dc9bdc36 scsi: hisi_sas: Factor out task prep and delivery code
:::::: TO: John Garry <john.garry@huawei.com>
:::::: CC: Martin K. Petersen <martin.petersen@oracle.com>
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 3+ messages in thread* drivers/scsi/hisi_sas/hisi_sas_main.c:445 hisi_sas_task_deliver() warn: inconsistent indenting
@ 2022-03-30 7:23 kernel test robot
2022-03-30 11:49 ` John Garry
0 siblings, 1 reply; 3+ messages in thread
From: kernel test robot @ 2022-03-30 7:23 UTC (permalink / raw)
To: John Garry; +Cc: kbuild-all, linux-kernel, Martin K. Petersen
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 1c24a186398f59c80adb9a967486b65c1423a59d
commit: 095478a6e5bf590f2bbf341569eb25173c9c5f32 scsi: hisi_sas: Use libsas internal abort support
date: 2 weeks ago
config: powerpc64-randconfig-m031-20220330 (https://download.01.org/0day-ci/archive/20220330/202203301533.b4Hty4C7-lkp@intel.com/config)
compiler: powerpc64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
smatch warnings:
drivers/scsi/hisi_sas/hisi_sas_main.c:445 hisi_sas_task_deliver() warn: inconsistent indenting
vim +445 drivers/scsi/hisi_sas/hisi_sas_main.c
b3cce125cb1e2e Xiang Chen 2019-02-06 393
dc313f6b125b09 John Garry 2021-12-15 394 static
dc313f6b125b09 John Garry 2021-12-15 395 void hisi_sas_task_deliver(struct hisi_hba *hisi_hba,
dc313f6b125b09 John Garry 2021-12-15 396 struct hisi_sas_slot *slot,
dc313f6b125b09 John Garry 2021-12-15 397 struct hisi_sas_dq *dq,
095478a6e5bf59 John Garry 2022-03-11 398 struct hisi_sas_device *sas_dev)
42e7a69368a585 John Garry 2015-11-18 399 {
42e7a69368a585 John Garry 2015-11-18 400 struct hisi_sas_cmd_hdr *cmd_hdr_base;
dc313f6b125b09 John Garry 2021-12-15 401 int dlvry_queue_slot, dlvry_queue;
dc313f6b125b09 John Garry 2021-12-15 402 struct sas_task *task = slot->task;
fa222db0b03689 Xiang Chen 2018-05-09 403 int wr_q_index;
42e7a69368a585 John Garry 2015-11-18 404
e9dc5e11c97ee9 Xiang Chen 2020-01-20 405 spin_lock(&dq->lock);
897cc769bcc092 John Garry 2019-08-05 406 wr_q_index = dq->wr_point;
897cc769bcc092 John Garry 2019-08-05 407 dq->wr_point = (dq->wr_point + 1) % HISI_SAS_QUEUE_SLOTS;
fa222db0b03689 Xiang Chen 2018-05-09 408 list_add_tail(&slot->delivery, &dq->list);
e9dc5e11c97ee9 Xiang Chen 2020-01-20 409 spin_unlock(&dq->lock);
e9dc5e11c97ee9 Xiang Chen 2020-01-20 410 spin_lock(&sas_dev->lock);
4fefe5bbf599d6 Xiang Chen 2019-02-06 411 list_add_tail(&slot->entry, &sas_dev->list);
e9dc5e11c97ee9 Xiang Chen 2020-01-20 412 spin_unlock(&sas_dev->lock);
42e7a69368a585 John Garry 2015-11-18 413
b1a49412f0aed7 Xiang Chen 2017-06-14 414 dlvry_queue = dq->id;
fa222db0b03689 Xiang Chen 2018-05-09 415 dlvry_queue_slot = wr_q_index;
42e7a69368a585 John Garry 2015-11-18 416
4fefe5bbf599d6 Xiang Chen 2019-02-06 417 slot->device_id = sas_dev->device_id;
42e7a69368a585 John Garry 2015-11-18 418 slot->dlvry_queue = dlvry_queue;
42e7a69368a585 John Garry 2015-11-18 419 slot->dlvry_queue_slot = dlvry_queue_slot;
42e7a69368a585 John Garry 2015-11-18 420 cmd_hdr_base = hisi_hba->cmd_hdr[dlvry_queue];
42e7a69368a585 John Garry 2015-11-18 421 slot->cmd_hdr = &cmd_hdr_base[dlvry_queue_slot];
dc313f6b125b09 John Garry 2021-12-15 422
42e7a69368a585 John Garry 2015-11-18 423 task->lldd_task = slot;
42e7a69368a585 John Garry 2015-11-18 424
42e7a69368a585 John Garry 2015-11-18 425 memset(slot->cmd_hdr, 0, sizeof(struct hisi_sas_cmd_hdr));
f557e32c0023ea Xiaofei Tan 2017-06-29 426 memset(hisi_sas_cmd_hdr_addr_mem(slot), 0, HISI_SAS_COMMAND_TABLE_SZ);
d380f55503ed28 Xiang Chen 2019-08-05 427 memset(hisi_sas_status_buf_addr_mem(slot), 0,
d380f55503ed28 Xiang Chen 2019-08-05 428 sizeof(struct hisi_sas_err_record));
42e7a69368a585 John Garry 2015-11-18 429
42e7a69368a585 John Garry 2015-11-18 430 switch (task->task_proto) {
66ee999b4e43e1 John Garry 2015-11-18 431 case SAS_PROTOCOL_SMP:
a2b3820bddfbff Xiang Chen 2018-05-09 432 hisi_sas_task_prep_smp(hisi_hba, slot);
66ee999b4e43e1 John Garry 2015-11-18 433 break;
42e7a69368a585 John Garry 2015-11-18 434 case SAS_PROTOCOL_SSP:
78bd2b4f6e7c05 Xiaofei Tan 2018-05-21 435 hisi_sas_task_prep_ssp(hisi_hba, slot);
42e7a69368a585 John Garry 2015-11-18 436 break;
42e7a69368a585 John Garry 2015-11-18 437 case SAS_PROTOCOL_SATA:
42e7a69368a585 John Garry 2015-11-18 438 case SAS_PROTOCOL_STP:
095478a6e5bf59 John Garry 2022-03-11 439 case SAS_PROTOCOL_STP_ALL:
a2b3820bddfbff Xiang Chen 2018-05-09 440 hisi_sas_task_prep_ata(hisi_hba, slot);
6f2ff1a1311e61 John Garry 2016-01-26 441 break;
095478a6e5bf59 John Garry 2022-03-11 442 case SAS_PROTOCOL_INTERNAL_ABORT:
095478a6e5bf59 John Garry 2022-03-11 443 hisi_sas_task_prep_abort(hisi_hba, slot);
dc313f6b125b09 John Garry 2021-12-15 444 break;
dc313f6b125b09 John Garry 2021-12-15 @445 fallthrough;
42e7a69368a585 John Garry 2015-11-18 446 default:
095478a6e5bf59 John Garry 2022-03-11 447 return;
42e7a69368a585 John Garry 2015-11-18 448 }
42e7a69368a585 John Garry 2015-11-18 449
1c09b663168bb5 Xiaofei Tan 2018-07-18 450 WRITE_ONCE(slot->ready, 1);
42e7a69368a585 John Garry 2015-11-18 451
0e4620856b8933 John Garry 2021-12-15 452 spin_lock(&dq->lock);
0e4620856b8933 John Garry 2021-12-15 453 hisi_hba->hw->start_delivery(dq);
0e4620856b8933 John Garry 2021-12-15 454 spin_unlock(&dq->lock);
42e7a69368a585 John Garry 2015-11-18 455 }
42e7a69368a585 John Garry 2015-11-18 456
:::::: The code at line 445 was first introduced by commit
:::::: dc313f6b125b095d3d2683d94d5f69c8dc9bdc36 scsi: hisi_sas: Factor out task prep and delivery code
:::::: TO: John Garry <john.garry@huawei.com>
:::::: CC: Martin K. Petersen <martin.petersen@oracle.com>
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: drivers/scsi/hisi_sas/hisi_sas_main.c:445 hisi_sas_task_deliver() warn: inconsistent indenting
2022-03-30 7:23 kernel test robot
@ 2022-03-30 11:49 ` John Garry
0 siblings, 0 replies; 3+ messages in thread
From: John Garry @ 2022-03-30 11:49 UTC (permalink / raw)
To: kernel test robot; +Cc: kbuild-all, linux-kernel, Martin K. Petersen
On 30/03/2022 08:23, kernel test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 1c24a186398f59c80adb9a967486b65c1423a59d
> commit: 095478a6e5bf590f2bbf341569eb25173c9c5f32 scsi: hisi_sas: Use libsas internal abort support
> date: 2 weeks ago
> config: powerpc64-randconfig-m031-20220330 (https://download.01.org/0day-ci/archive/20220330/202203301533.b4Hty4C7-lkp@intel.com/config)
> compiler: powerpc64-linux-gcc (GCC) 11.2.0
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> smatch warnings:
> drivers/scsi/hisi_sas/hisi_sas_main.c:445 hisi_sas_task_deliver() warn: inconsistent indenting
>
> vim +445 drivers/scsi/hisi_sas/hisi_sas_main.c
>
...
> 42e7a69368a585 John Garry 2015-11-18 437 case SAS_PROTOCOL_SATA:
> 42e7a69368a585 John Garry 2015-11-18 438 case SAS_PROTOCOL_STP:
> 095478a6e5bf59 John Garry 2022-03-11 439 case SAS_PROTOCOL_STP_ALL:
> a2b3820bddfbff Xiang Chen 2018-05-09 440 hisi_sas_task_prep_ata(hisi_hba, slot);
> 6f2ff1a1311e61 John Garry 2016-01-26 441 break;
> 095478a6e5bf59 John Garry 2022-03-11 442 case SAS_PROTOCOL_INTERNAL_ABORT:
> 095478a6e5bf59 John Garry 2022-03-11 443 hisi_sas_task_prep_abort(hisi_hba, slot);
> dc313f6b125b09 John Garry 2021-12-15 444 break;
> dc313f6b125b09 John Garry 2021-12-15 @445 fallthrough;
This is already queued for removal.
> 42e7a69368a585 John Garry 2015-11-18 446 default:
> 095478a6e5bf59 John Garry 2022-03-11 447 return;
> 42e7a69368a585 John Garry 2015-11-18 448 }
> 42e7a69368a585 John Garry 2015-11-18 449
> 1c09b663168bb5 Xiaofei Tan 2018-07-18 450 WRITE_ONCE(slot->ready, 1);
> 42e7a69368a585 John Garry 2015-11-18 451
> 0e4620856b8933 John Garry 2021-12-15 452 spin_lock(&dq->lock);
> 0e4620856b8933 John Garry 2021-12-15 453 hisi_hba->hw->start_delivery(dq);
> 0e4620856b8933 John Garry 2021-12-15 454 spin_unlock(&dq->lock);
> 42e7a69368a585 John Garry 2015-11-18 455 }
> 42e7a69368a585 John Garry 2015-11-18 456
>
> :::::: The code at line 445 was first introduced by commit
> :::::: dc313f6b125b095d3d2683d94d5f69c8dc9bdc36 scsi: hisi_sas: Factor out task prep and delivery code
>
> :::::: TO: John Garry <john.garry@huawei.com>
> :::::: CC: Martin K. Petersen <martin.petersen@oracle.com>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-03-30 17:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-30 17:14 drivers/scsi/hisi_sas/hisi_sas_main.c:445 hisi_sas_task_deliver() warn: inconsistent indenting kernel test robot
-- strict thread matches above, loose matches on Subject: below --
2022-03-30 7:23 kernel test robot
2022-03-30 11:49 ` John Garry
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®