diff options
author | Sebastian Ott <sebott@linux.vnet.ibm.com> | 2018-01-29 10:12:30 -0500 |
---|---|---|
committer | Christian Borntraeger <borntraeger@de.ibm.com> | 2018-03-14 15:21:16 -0400 |
commit | 4ed8773b5055363f1666345dd5ae5af12c7c6268 (patch) | |
tree | c980518d66394479e19ab8fe9b478e9e5738b3a0 | |
parent | a5e0acea9ed644d622281a4a8fde94a2e24ec7e8 (diff) |
KVM: s390: fix fallthrough annotation
A case statement in kvm_s390_shadow_tables uses fallthrough annotations
which are not recognized by gcc because they are hidden within a block.
Move these annotations out of the block to fix (W=1) warnings like below:
arch/s390/kvm/gaccess.c: In function 'kvm_s390_shadow_tables':
arch/s390/kvm/gaccess.c:1029:26: warning: this statement may fall through [-Wimplicit-fallthrough=]
case ASCE_TYPE_REGION1: {
^
Signed-off-by: Sebastian Ott <sebott@linux.vnet.ibm.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
-rw-r--r-- | arch/s390/kvm/gaccess.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/arch/s390/kvm/gaccess.c b/arch/s390/kvm/gaccess.c index c24bfa72baf7..8e2b8647ee12 100644 --- a/arch/s390/kvm/gaccess.c +++ b/arch/s390/kvm/gaccess.c | |||
@@ -1050,8 +1050,7 @@ shadow_r2t: | |||
1050 | rc = gmap_shadow_r2t(sg, saddr, rfte.val, *fake); | 1050 | rc = gmap_shadow_r2t(sg, saddr, rfte.val, *fake); |
1051 | if (rc) | 1051 | if (rc) |
1052 | return rc; | 1052 | return rc; |
1053 | /* fallthrough */ | 1053 | } /* fallthrough */ |
1054 | } | ||
1055 | case ASCE_TYPE_REGION2: { | 1054 | case ASCE_TYPE_REGION2: { |
1056 | union region2_table_entry rste; | 1055 | union region2_table_entry rste; |
1057 | 1056 | ||
@@ -1077,8 +1076,7 @@ shadow_r3t: | |||
1077 | rc = gmap_shadow_r3t(sg, saddr, rste.val, *fake); | 1076 | rc = gmap_shadow_r3t(sg, saddr, rste.val, *fake); |
1078 | if (rc) | 1077 | if (rc) |
1079 | return rc; | 1078 | return rc; |
1080 | /* fallthrough */ | 1079 | } /* fallthrough */ |
1081 | } | ||
1082 | case ASCE_TYPE_REGION3: { | 1080 | case ASCE_TYPE_REGION3: { |
1083 | union region3_table_entry rtte; | 1081 | union region3_table_entry rtte; |
1084 | 1082 | ||
@@ -1113,8 +1111,7 @@ shadow_sgt: | |||
1113 | rc = gmap_shadow_sgt(sg, saddr, rtte.val, *fake); | 1111 | rc = gmap_shadow_sgt(sg, saddr, rtte.val, *fake); |
1114 | if (rc) | 1112 | if (rc) |
1115 | return rc; | 1113 | return rc; |
1116 | /* fallthrough */ | 1114 | } /* fallthrough */ |
1117 | } | ||
1118 | case ASCE_TYPE_SEGMENT: { | 1115 | case ASCE_TYPE_SEGMENT: { |
1119 | union segment_table_entry ste; | 1116 | union segment_table_entry ste; |
1120 | 1117 | ||