If fcoeadm is going to rely on fcoemon for create, destroy, reset, and scan commands, then it should just send the command instead of trying to verify if ports exist or not. Just let fcoemon do that, instead of trying to get the behavior right in two places.
Signed-off-by: Chris Leech <[email protected]> --- fcoeadm.c | 12 +----------- 1 files changed, 1 insertions(+), 11 deletions(-) diff --git a/fcoeadm.c b/fcoeadm.c index ed8d433..779b8d1 100644 --- a/fcoeadm.c +++ b/fcoeadm.c @@ -245,17 +245,7 @@ int main(int argc, char *argv[]) } ifname = optarg; - - if (opt == 'c') { - rc = fcoe_validate_interface(ifname); - if (!rc && - !fcoe_validate_fcoe_conn(ifname)) - rc = EFCOECONN; - } else - rc = fcoe_validate_fcoe_conn(ifname); - - if (!rc) - rc = fcoeadm_action(cmd, ifname); + rc = fcoeadm_action(cmd, ifname); break; case 'i': _______________________________________________ devel mailing list [email protected] http://www.open-fcoe.org/mailman/listinfo/devel
