]> www.infradead.org Git - users/hch/nvmetcli.git/commitdiff
nvmetcli: remove status method for the UIPortNode
authorJohannes Thumshirn <jthumshirn@suse.de>
Thu, 30 Nov 2017 10:27:23 +0000 (11:27 +0100)
committerChristoph Hellwig <hch@lst.de>
Thu, 30 Nov 2017 14:27:17 +0000 (06:27 -0800)
nvmetcli defines a status method for the UIPortNode class but there is
no corresponding 'enable' file in configfs so typing in status will
always result in nvmetcli printing disbaled if one types 'status' in
the port node:

/> cd ports/1/
/ports/1> status
Status for /ports/1: disabled
/ports/1>

There are two possible ways for fixing this issue, provide a configfs
file to enable/disable the port node in the kernel or delete the
method in nvmetcli. Having a configfs file in the kernel is pointless,
as ports get enabled when symlinked to a namespace so zap the pointless status
method in user-space.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Reported-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>
nvmetcli

index 5378365eb91ffb2112f7deaaa0f1c77f1d576919..4fbc12ec430f465710bcf965340d7b24f5a02a24 100755 (executable)
--- a/nvmetcli
+++ b/nvmetcli
@@ -356,11 +356,6 @@ class UIPortNode(UINode):
         UIPortSubsystemsNode(self)
         UIReferralsNode(self)
 
-    def status(self):
-        if self.cfnode.get_enable():
-            return "enabled"
-        return "disabled"
-
 
 class UIPortSubsystemsNode(UINode):
     def __init__(self, parent):