* [PATCH] elx: efct: Fix void-pointer-to-enum-cast warning for efc_nport_topology
@ 2021-08-30 23:10 James Smart
2021-09-14 3:43 ` Martin K. Petersen
0 siblings, 1 reply; 2+ messages in thread
From: James Smart @ 2021-08-30 23:10 UTC (permalink / raw)
To: linux-scsi; +Cc: James Smart, kernel test robot, Ram Vegesna
the kernel test robot flagged an warning for ".../efc_device.c:932:6:
warning: cast to smaller integer type 'enum efc_nport_topology' from
'void *'"
For the topology events, the "arg" field is generically defined as a
void * and is used to pass different arguments. Most of the arguments
are pointers to data structures. But for the
EFC_EVT_NPORT_TOPOLOGY_NOTIFY event, the argument is an enum value, and
the code is typecasting the void * to an enum generating the warning.
Fix by converting the EFC_EVT_NPORT_TOPOLOGY_NOTIFY event to pass a
pointer to the enum, thus it's a straight-forward pointer dereference
in the event handler.
Fixes: 202bfdffae27 ("scsi: elx: libefc: FC node ELS and state handling")
Reported-by: kernel test robot <lkp@intel.com>
Co-developed-by: Ram Vegesna <ram.vegesna@broadcom.com>
Signed-off-by: Ram Vegesna <ram.vegesna@broadcom.com>
Signed-off-by: James Smart <jsmart2021@gmail.com>
---
drivers/scsi/elx/libefc/efc_device.c | 7 +++----
drivers/scsi/elx/libefc/efc_fabric.c | 3 +--
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/scsi/elx/libefc/efc_device.c b/drivers/scsi/elx/libefc/efc_device.c
index 725ca2a23fb2..52be01333c6e 100644
--- a/drivers/scsi/elx/libefc/efc_device.c
+++ b/drivers/scsi/elx/libefc/efc_device.c
@@ -928,22 +928,21 @@ __efc_d_wait_topology_notify(struct efc_sm_ctx *ctx,
break;
case EFC_EVT_NPORT_TOPOLOGY_NOTIFY: {
- enum efc_nport_topology topology =
- (enum efc_nport_topology)arg;
+ enum efc_nport_topology *topology = arg;
WARN_ON(node->nport->domain->attached);
WARN_ON(node->send_ls_acc != EFC_NODE_SEND_LS_ACC_PLOGI);
node_printf(node, "topology notification, topology=%d\n",
- topology);
+ *topology);
/* At the time the PLOGI was received, the topology was unknown,
* so we didn't know which node would perform the domain attach:
* 1. The node from which the PLOGI was sent (p2p) or
* 2. The node to which the FLOGI was sent (fabric).
*/
- if (topology == EFC_NPORT_TOPO_P2P) {
+ if (*topology == EFC_NPORT_TOPO_P2P) {
/* if this is p2p, need to attach to the domain using
* the d_id from the PLOGI received
*/
diff --git a/drivers/scsi/elx/libefc/efc_fabric.c b/drivers/scsi/elx/libefc/efc_fabric.c
index d397220d9e54..3270ce40196c 100644
--- a/drivers/scsi/elx/libefc/efc_fabric.c
+++ b/drivers/scsi/elx/libefc/efc_fabric.c
@@ -107,7 +107,6 @@ void
efc_fabric_notify_topology(struct efc_node *node)
{
struct efc_node *tmp_node;
- enum efc_nport_topology topology = node->nport->topology;
unsigned long index;
/*
@@ -118,7 +117,7 @@ efc_fabric_notify_topology(struct efc_node *node)
if (tmp_node != node) {
efc_node_post_event(tmp_node,
EFC_EVT_NPORT_TOPOLOGY_NOTIFY,
- (void *)topology);
+ &node->nport->topology);
}
}
}
--
2.26.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] elx: efct: Fix void-pointer-to-enum-cast warning for efc_nport_topology
2021-08-30 23:10 [PATCH] elx: efct: Fix void-pointer-to-enum-cast warning for efc_nport_topology James Smart
@ 2021-09-14 3:43 ` Martin K. Petersen
0 siblings, 0 replies; 2+ messages in thread
From: Martin K. Petersen @ 2021-09-14 3:43 UTC (permalink / raw)
To: James Smart, linux-scsi
Cc: Martin K . Petersen, kernel test robot, Ram Vegesna
On Mon, 30 Aug 2021 16:10:50 -0700, James Smart wrote:
> the kernel test robot flagged an warning for ".../efc_device.c:932:6:
> warning: cast to smaller integer type 'enum efc_nport_topology' from
> 'void *'"
>
> For the topology events, the "arg" field is generically defined as a
> void * and is used to pass different arguments. Most of the arguments
> are pointers to data structures. But for the
> EFC_EVT_NPORT_TOPOLOGY_NOTIFY event, the argument is an enum value, and
> the code is typecasting the void * to an enum generating the warning.
>
> [...]
Applied to 5.15/scsi-fixes, thanks!
[1/1] elx: efct: Fix void-pointer-to-enum-cast warning for efc_nport_topology
https://git.kernel.org/mkp/scsi/c/96fafe7c6523
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-09-14 3:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-30 23:10 [PATCH] elx: efct: Fix void-pointer-to-enum-cast warning for efc_nport_topology James Smart
2021-09-14 3:43 ` Martin K. Petersen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.