[gustavoars-linux:for-next/kspp 5/5] kernel/debug/gdbstub.c:1049:4: warning: fallthrough annotation in unreachable code

From: kernel test robot
Date: Mon Jul 12 2021 - 06:41:27 EST

Next message:
Previous
Next in
Messages

tree: wget # install # apt-get # If you fix the issue, Reported-by: kernel
All warnings (new
>> kernel/debug/gd ^
include/linux/compiler_a # define fallthrough 1 warning generated.

vim +1049 kernel/debug/gdbstub.c

53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 2bbd9b0f2b4ec0 Bhaskar Chowdhury 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 55751145dc1e08 Jason Wessel 53197fc4954924 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 84a0bd5b283072 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 50e1499f468fd7 Andy Shevchenko 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 55751145dc1e08 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel dcc7871128e994 Jason Wessel dcc7871128e994 Jason Wessel dcc7871128e994 Jason Wessel dcc7871128e994 Jason Wessel dcc7871128e994 Jason Wessel dcc7871128e994 Jason Wessel dcc7871128e994 Jason Wessel df561f6688fef7 Gustavo 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel df561f6688fef7 Gustavo 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel df561f6688fef7 Gustavo 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel 53197fc4954924 Jason Wessel dcc7871128e994 Jason Wessel
:::::: The code at :::::: df561f6688fef775baa3
:::::: TO: Gustavo :::::: CC: Gustavo
---
0-DAY CI Kernel Test Attachment: Description: application/gzip Adam Thomson: "RE: [PATCH v5] regulator: da9063: Add support for full-current mode." message: Suzuki K Poulose: "Re: [PATCH v4 02/11] coresight: tmc-etr: Add barrier after updating AUX ring buffer" thread: Andy Shevchenko: "Re: [gustavoars-linux:for-next/kspp 5/5] kernel/debug/gdbstub.c:1049:4: warning: fallthrough annotation in unreachable code" sorted by: [ date ] [ thread ] [ subject ] [ author ] href="https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git">https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git for-next/kspp
b0764958b640642edc6ff
b0764958b640642edc6ff [5/5] Makefile: Enable -Wimplicit-fallthrough for Clang
(attached as .config)
rel="nofollow" href="https://github.com/llvm/llvm-project">https://github.com/llvm/llvm-project 8d69635ed9ecf36fd0ca85906bfde17949671cbe)
href="https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross">https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
riscv cross compiling tool for clang build
install binutils-riscv64-linux-gnu
href="https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git/commit/?id=cd013b8a781567829cbb0764958b640642edc6ff">https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git/commit/?id=cd013b8a781567829cbb0764958b640642edc6ff
add gustavoars-linux https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git
--no-tags gustavoars-linux for-next/kspp
cd013b8a781567829cbb0764958b640642edc6ff
attached .config to linux build tree
COMPILER=clang make.cross ARCH=riscv
kindly add following tag as appropriate
test robot <lkp@xxxxxxxxx>
ones prefixed by >>):
bstub.c:1049:4: warning: fallthrough annotation in unreachable code [-Wimplicit-fallthrough]
fallthrough;
ttributes.h:210:41: note: expanded from macro 'fallthrough'
__attribute__((__fallthrough__))
^
2010-04-02 953
2010-04-02 954 /*
2021-03-17 955 * This function performs all gdbserial command processing
2010-04-02 956 */
2010-04-02 957 int gdb_serial_stub(struct kgdb_state *ks)
2010-04-02 958 {
2010-04-02 959 int error = 0;
2010-04-02 960 int tmp;
2010-04-02 961
2010-08-05 962 /* Initialize comm buffer and globals. */
2010-04-02 963 memset(remcom_out_buffer, 0, sizeof(remcom_out_buffer));
2010-08-05 964 kgdb_usethread = kgdb_info[ks->cpu].task;
2010-08-05 965 ks->kgdb_usethreadid = shadow_pid(kgdb_info[ks->cpu].task->pid);
2010-08-05 966 ks->pass_exception = 0;
2010-04-02 967
2010-04-02 968 if (kgdb_connected) {
2010-08-05 969 unsigned char thref[BUF_THREAD_ID_SIZE];
2010-04-02 970 char *ptr;
2010-04-02 971
2010-04-02 972 /* Reply to host that an exception has occurred */
2010-04-02 973 ptr = remcom_out_buffer;
2010-04-02 974 *ptr++ = 'T';
2011-10-31 975 ptr = hex_byte_pack(ptr, ks->signo);
2010-04-02 976 ptr += strlen(strcpy(ptr, "thread:"));
2010-04-02 977 int_to_threadref(thref, shadow_pid(current->pid));
2010-04-02 978 ptr = pack_threadid(ptr, thref);
2010-04-02 979 *ptr++ = ';';
2010-04-02 980 put_packet(remcom_out_buffer);
2010-04-02 981 }
2010-04-02 982
2010-04-02 983 while (1) {
2010-04-02 984 error = 0;
2010-04-02 985
2010-04-02 986 /* Clear the out buffer. */
2010-04-02 987 memset(remcom_out_buffer, 0, sizeof(remcom_out_buffer));
2010-04-02 988
2010-04-02 989 get_packet(remcom_in_buffer);
2010-04-02 990
2010-04-02 991 switch (remcom_in_buffer[0]) {
2010-04-02 992 case '?': /* gdbserial status */
2010-04-02 993 gdb_cmd_status(ks);
2010-04-02 994 break;
2010-04-02 995 case 'g': /* return the value of the CPU registers */
2010-04-02 996 gdb_cmd_getregs(ks);
2010-04-02 997 break;
2010-04-02 998 case 'G': /* set the value of the CPU registers - return OK */
2010-04-02 999 gdb_cmd_setregs(ks);
2010-04-02 1000 break;
2010-04-02 1001 case 'm': /* mAA..AA,LLLL Read LLLL bytes at address AA..AA */
2010-04-02 1002 gdb_cmd_memread(ks);
2010-04-02 1003 break;
2010-04-02 1004 case 'M': /* MAA..AA,LLLL: Write LLLL bytes at address AA..AA */
2010-04-02 1005 gdb_cmd_memwrite(ks);
2010-04-02 1006 break;
2010-08-05 1007 #if DBG_MAX_REG_NUM > 0
2010-08-05 1008 case 'p': /* pXX Return gdb register XX (in hex) */
2010-08-05 1009 gdb_cmd_reg_get(ks);
2010-08-05 1010 break;
2010-08-05 1011 case 'P': /* PXX=aaaa Set gdb register XX to aaaa (in hex) */
2010-08-05 1012 gdb_cmd_reg_set(ks);
2010-08-05 1013 break;
2010-08-05 1014 #endif /* DBG_MAX_REG_NUM > 0 */
2010-04-02 1015 case 'X': /* XAA..AA,LLLL: Write LLLL bytes at address AA..AA */
2010-04-02 1016 gdb_cmd_binwrite(ks);
2010-04-02 1017 break;
2010-04-02 1018 /* kill or detach. KGDB should treat this like a
2010-04-02 1019 * continue.
2010-04-02 1020 */
2010-04-02 1021 case 'D': /* Debugger detach */
2010-04-02 1022 case 'k': /* Debugger detach via kill */
2010-04-02 1023 gdb_cmd_detachkill(ks);
2010-04-02 1024 goto default_handle;
2010-04-02 1025 case 'R': /* Reboot */
2010-04-02 1026 if (gdb_cmd_reboot(ks))
2010-04-02 1027 goto default_handle;
2010-04-02 1028 break;
2010-04-02 1029 case 'q': /* query command */
2010-04-02 1030 gdb_cmd_query(ks);
2010-04-02 1031 break;
2010-04-02 1032 case 'H': /* task related */
2010-04-02 1033 gdb_cmd_task(ks);
2010-04-02 1034 break;
2010-04-02 1035 case 'T': /* Query thread status */
2010-04-02 1036 gdb_cmd_thread(ks);
2010-04-02 1037 break;
2010-04-02 1038 case 'z': /* Break point remove */
2010-04-02 1039 case 'Z': /* Break point set */
2010-04-02 1040 gdb_cmd_break(ks);
2010-04-02 1041 break;
2010-05-20 1042 #ifdef CONFIG_KGDB_KDB
2010-05-20 1043 case '3': /* Escape into back into kdb */
2010-05-20 1044 if (remcom_in_buffer[1] == '\0') {
2010-05-20 1045 gdb_cmd_detachkill(ks);
2010-05-20 1046 return DBG_PASS_EVENT;
2010-05-20 1047 }
2010-05-20 1048 #endif
A. R. Silva 2020-08-23 @1049 fallthrough;
2010-04-02 1050 case 'C': /* Exception passing */
2010-04-02 1051 tmp = gdb_cmd_exception_pass(ks);
2010-04-02 1052 if (tmp > 0)
2010-04-02 1053 goto default_handle;
2010-04-02 1054 if (tmp == 0)
2010-04-02 1055 break;
A. R. Silva 2020-08-23 1056 fallthrough; /* on tmp < 0 */
2010-04-02 1057 case 'c': /* Continue packet */
2010-04-02 1058 case 's': /* Single step packet */
2010-04-02 1059 if (kgdb_contthread && kgdb_contthread != current) {
2010-04-02 1060 /* Can't switch threads in kgdb */
2010-04-02 1061 error_packet(remcom_out_buffer, -EINVAL);
2010-04-02 1062 break;
2010-04-02 1063 }
A. R. Silva 2020-08-23 1064 fallthrough; /* to default processing */
2010-04-02 1065 default:
2010-04-02 1066 default_handle:
2010-04-02 1067 error = kgdb_arch_handle_exception(ks->ex_vector,
2010-04-02 1068 ks->signo,
2010-04-02 1069 ks->err_code,
2010-04-02 1070 remcom_in_buffer,
2010-04-02 1071 remcom_out_buffer,
2010-04-02 1072 ks->linux_regs);
2010-04-02 1073 /*
2010-04-02 1074 * Leave cmd processing on error, detach,
2010-04-02 1075 * kill, continue, or single step.
2010-04-02 1076 */
2010-04-02 1077 if (error >= 0 || remcom_in_buffer[0] == 'D' ||
2010-04-02 1078 remcom_in_buffer[0] == 'k') {
2010-04-02 1079 error = 0;
2010-04-02 1080 goto kgdb_exit;
2010-04-02 1081 }
2010-04-02 1082
2010-04-02 1083 }
2010-04-02 1084
2010-04-02 1085 /* reply to the request */
2010-04-02 1086 put_packet(remcom_out_buffer);
2010-04-02 1087 }
2010-04-02 1088
2010-04-02 1089 kgdb_exit:
2010-04-02 1090 if (ks->pass_exception)
2010-04-02 1091 error = 1;
2010-04-02 1092 return error;
2010-04-02 1093 }
2010-05-20 1094
line 1049 was first introduced by commit
41a0f5d960becd248b11 treewide: Use fallthrough pseudo-keyword
A. R. Silva <gustavoars@xxxxxxxxxx>
A. R. Silva <gustavoars@xxxxxxxxxx>
Service, Intel Corporation
href="https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx">https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
>.config.gz

Next message: Adam Thomson: "RE: [PATCH v5] regulator: da9063: Add support for full-current mode."
Previous message: Suzuki K Poulose: "Re: [PATCH v4 02/11] coresight: tmc-etr: Add barrier after updating AUX ring buffer"
Next in thread: Andy Shevchenko: "Re: [gustavoars-linux:for-next/kspp 5/5] kernel/debug/gdbstub.c:1049:4: warning: fallthrough annotation in unreachable code"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]