scsi: blkcg: Fix application ID config options
authorMartin K. Petersen <martin.petersen@oracle.com>
Sat, 3 Jul 2021 15:58:33 +0000 (11:58 -0400)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sun, 4 Jul 2021 18:44:22 +0000 (11:44 -0700)
Commit d2bcbeab4200 ("scsi: blkcg: Add app identifier support for
blkcg") introduced an FC_APPID config option under SCSI. However, the
added config option is not used anywhere. Simply remove it.

The block layer BLK_CGROUP_FC_APPID config option is what actually
controls whether the application ID code should be built or not. Make
this option dependent on NVMe over FC since that is currently the only
transport which supports the capability.

Fixes: d2bcbeab4200 ("scsi: blkcg: Add app identifier support for blkcg")
Reported-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
block/Kconfig
drivers/scsi/Kconfig

index 7885a27..fd732ae 100644 (file)
@@ -153,7 +153,7 @@ config BLK_CGROUP_IOLATENCY
 
 config BLK_CGROUP_FC_APPID
        bool "Enable support to track FC I/O Traffic across cgroup applications"
-       depends on BLK_CGROUP=y
+       depends on BLK_CGROUP && NVME_FC
        help
          Enabling this option enables the support to track FC I/O traffic across
          cgroup applications. It enables the Fabric and the storage targets to
index 4dc42a8..8f44d43 100644 (file)
@@ -235,19 +235,6 @@ config SCSI_FC_ATTRS
          each attached FiberChannel device to sysfs, say Y.
          Otherwise, say N.
 
-config FC_APPID
-       bool "Enable support to track FC I/O Traffic"
-       depends on BLOCK && BLK_CGROUP
-       depends on SCSI
-       select BLK_CGROUP_FC_APPID
-       default y
-       help
-         If you say Y here, it enables the support to track
-         FC I/O traffic over fabric. It enables the Fabric and the
-         storage targets to identify, monitor, and handle FC traffic
-         based on VM tags by inserting application specific
-         identification into the FC frame.
-
 config SCSI_ISCSI_ATTRS
        tristate "iSCSI Transport Attributes"
        depends on SCSI && NET