[kbuild] drivers/net/wireless/ath/ath11k/qmi.c:2031 ath11k_qmi_load_file_target_mem() error: uninitialized symbol 'ret'.
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Anilkumar Kolli
CC: Kalle Valo
CC: Jouni Malinen
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: 336e7b53c82fc74d261024773a0fab43623a94fb ath11k: clean up BDF download
functions
date: 4 months ago
:: branch date: 2 days ago
:: commit date: 4 months ago
config: microblaze-randconfig-m031-20220127
(https://download.01.org/0day-ci/archive/20220127/202201271555.orgxsjdc-...@intel.com/config)
compiler: microblaze-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
smatch warnings:
drivers/net/wireless/ath/ath11k/qmi.c:2031 ath11k_qmi_load_file_target_mem()
error: uninitialized symbol 'ret'.
vim +/ret +2031 drivers/net/wireless/ath/ath11k/qmi.c
d5c65159f289537 Kalle Valo 2019-11-23 1919
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1920 static int
ath11k_qmi_load_file_target_mem(struct ath11k_base *ab,
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1921
const u8 *data, u32 len, u8 type)
d5c65159f289537 Kalle Valo 2019-11-23 1922 {
d5c65159f289537 Kalle Valo 2019-11-23 1923struct
qmi_wlanfw_bdf_download_req_msg_v01 *req;
d5c65159f289537 Kalle Valo 2019-11-23 1924struct
qmi_wlanfw_bdf_download_resp_msg_v01 resp;
d5c65159f289537 Kalle Valo 2019-11-23 1925struct qmi_txn txn = {};
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1926const u8 *temp = data;
d5c65159f289537 Kalle Valo 2019-11-23 1927void __iomem *bdf_addr
= NULL;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1928int ret;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1929u32 remaining = len;
d5c65159f289537 Kalle Valo 2019-11-23 1930
d5c65159f289537 Kalle Valo 2019-11-23 1931req =
kzalloc(sizeof(*req), GFP_KERNEL);
d5c65159f289537 Kalle Valo 2019-11-23 1932if (!req)
d5c65159f289537 Kalle Valo 2019-11-23 1933return -ENOMEM;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1934
d5c65159f289537 Kalle Valo 2019-11-23 1935memset(&resp, 0,
sizeof(resp));
d5c65159f289537 Kalle Valo 2019-11-23 1936
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1937if
(ab->bus_params.fixed_bdf_addr) {
c72aa32d6d1c04f Anilkumar Kolli 2021-09-28 1938bdf_addr =
ioremap(ab->hw_params.bdf_addr, ab->hw_params.fw.board_size);
d5c65159f289537 Kalle Valo 2019-11-23 1939if (!bdf_addr) {
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1940
ath11k_warn(ab, "qmi ioremap error for bdf_addr\n");
d5c65159f289537 Kalle Valo 2019-11-23 1941ret =
-EIO;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1942goto
err_free_req;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1943}
d5c65159f289537 Kalle Valo 2019-11-23 1944}
d5c65159f289537 Kalle Valo 2019-11-23 1945
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1946while (remaining) {
d5c65159f289537 Kalle Valo 2019-11-23 1947req->valid = 1;
d5c65159f289537 Kalle Valo 2019-11-23 1948
req->file_id_valid = 1;
d5c65159f289537 Kalle Valo 2019-11-23 1949req->file_id =
ab->qmi.target.board_id;
d5c65159f289537 Kalle Valo 2019-11-23 1950
req->total_size_valid = 1;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1951req->total_size
= remaining;
d5c65159f289537 Kalle Valo 2019-11-23 1952
req->seg_id_valid = 1;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1953req->data_valid
= 1;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1954req->bdf_type =
type;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1955
req->bdf_type_valid = 1;
d5c65159f289537 Kalle Valo 2019-11-23 1956req->end_valid
= 1;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1957req->end = 0;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1958
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1959if (remaining >
QMI_WLANFW_MAX_DATA_SIZE_V01) {
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1960
req->data_len = QMI_WLANFW_MAX_DATA_SIZE_V01;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1961} else {
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1962
req->data_len = remaining;
d5c65159f289537 Kalle Valo 2019-11-23 1963
req->end = 1;
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1964}
d5c65159f289537 Kalle Valo 2019-11-23 1965
336e7b53c82fc74 Anilkumar Kolli 2021-09-28 1966if
(ab->bus_params.fixed_bdf_addr) {
336e7b53c82fc74 Anilkumar Kolli 2021-09-28
[kbuild] [rt-devel:linux-5.17.y-rt-rebase 79/103] kernel/trace/trace_output.c:483 trace_print_lat_fmt() warn: bitwise AND condition is false here
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Thomas Gleixner
CC: Sebastian Andrzej Siewior
tree: https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git
linux-5.17.y-rt-rebase
head: 7ea0aa3ea9d10a340b9fe4cb19f8a47948e4f562
commit: 4e8c1def1137805b1a9d576c3ac092bd85177088 [79/103] sched: Add support
for lazy preemption
:: branch date: 15 hours ago
:: commit date: 15 hours ago
config: x86_64-randconfig-m001
(https://download.01.org/0day-ci/archive/20220127/202201271543.pm2tcsvw-...@intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
smatch warnings:
kernel/trace/trace_output.c:483 trace_print_lat_fmt() warn: bitwise AND
condition is false here
vim +483 kernel/trace/trace_output.c
f0868d1e23a8ef Steven Rostedt2008-12-23 432
f81c972d27c367 Steven Rostedt2009-09-11 433 /**
f81c972d27c367 Steven Rostedt2009-09-11 434 *
trace_print_lat_fmt - print the irq, preempt and lockdep fields
f81c972d27c367 Steven Rostedt2009-09-11 435 * @s: trace seq
struct to write to
f81c972d27c367 Steven Rostedt2009-09-11 436 * @entry: The trace
entry field from the ring buffer
f81c972d27c367 Steven Rostedt2009-09-11 437 *
f81c972d27c367 Steven Rostedt2009-09-11 438 * Prints the generic
fields of irqs off, in hard or softirq, preempt
e6e1e2593592a8 Steven Rostedt2011-03-09 439 * count.
f81c972d27c367 Steven Rostedt2009-09-11 440 */
f81c972d27c367 Steven Rostedt2009-09-11 441 int
trace_print_lat_fmt(struct trace_seq *s, struct trace_entry *entry)
c4a8e8be2d43cc Frederic Weisbecker 2009-02-02 442 {
10da37a645b5e9 David Sharp 2010-12-03 443char
hardsoft_irq;
10da37a645b5e9 David Sharp 2010-12-03 444char
need_resched;
4e8c1def113780 Thomas Gleixner 2012-10-26 445char
need_resched_lazy;
10da37a645b5e9 David Sharp 2010-12-03 446char irqs_off;
10da37a645b5e9 David Sharp 2010-12-03 447int hardirq;
10da37a645b5e9 David Sharp 2010-12-03 448int softirq;
289e7b0f7eb47b Sebastian Andrzej Siewior 2021-12-13 449int bh_off;
7e6867bf831c71 Peter Zijlstra2016-03-18 450int nmi;
c4a8e8be2d43cc Frederic Weisbecker 2009-02-02 451
7e6867bf831c71 Peter Zijlstra2016-03-18 452nmi =
entry->flags & TRACE_FLAG_NMI;
c4a8e8be2d43cc Frederic Weisbecker 2009-02-02 453hardirq =
entry->flags & TRACE_FLAG_HARDIRQ;
c4a8e8be2d43cc Frederic Weisbecker 2009-02-02 454softirq =
entry->flags & TRACE_FLAG_SOFTIRQ;
289e7b0f7eb47b Sebastian Andrzej Siewior 2021-12-13 455bh_off =
entry->flags & TRACE_FLAG_BH_OFF;
d9793bd8018f83 Arnaldo Carvalho de Melo 2009-02-03 456
10da37a645b5e9 David Sharp 2010-12-03 457irqs_off =
289e7b0f7eb47b Sebastian Andrzej Siewior 2021-12-13 458
(entry->flags & TRACE_FLAG_IRQS_OFF && bh_off) ? 'D' :
d9793bd8018f83 Arnaldo Carvalho de Melo 2009-02-03 459
(entry->flags & TRACE_FLAG_IRQS_OFF) ? 'd' :
289e7b0f7eb47b Sebastian Andrzej Siewior 2021-12-13 460bh_off
? 'b' :
10da37a645b5e9 David Sharp 2010-12-03 461
(entry->flags & TRACE_FLAG_IRQS_NOSUPPORT) ? 'X' :
10da37a645b5e9 David Sharp 2010-12-03 462'.';
e5137b50a06400 Peter Zijlstra2013-10-04 463
e5137b50a06400 Peter Zijlstra2013-10-04 464switch
(entry->flags & (TRACE_FLAG_NEED_RESCHED |
e5137b50a06400 Peter Zijlstra2013-10-04 465
TRACE_FLAG_PREEMPT_RESCHED)) {
e5137b50a06400 Peter Zijlstra2013-10-04 466case
TRACE_FLAG_NEED_RESCHED | TRACE_FLAG_PREEMPT_RESCHED:
e5137b50a06400 Peter Zijlstra2013-10-04 467
need_resched = 'N';
e5137b50a06400 Peter Zijlstra2013-10-04 468break;
4e8c1def113780 Thomas Gleixner 2012-10-26 469 #ifndef
CONFIG_PREEMPT_LAZY
e5137b50a06400 Peter Zijlstra2013-10-04 470case
TRACE_FLAG_NEED_RESCHED:
e5137b50a06400 Peter Zijlstra2013-10-04 471
need_resched = 'n';
e5137b50a06400 Peter Zijlstra2013-10-04 472break;
4e8c1def113780 Thomas Gleixner 2012-10-26 473 #endif
e5137b50a06400 Peter Zijlstra2013-10-04 474case
TRACE_FLAG_PREEMPT_RESCHED:
e5137b50a06400 Peter Zijlstra2013-10-04 475
need_resched = 'p';
e5137b50a06400 Peter Zijlstra2013-10-04 476break;
e5137b50a06400 Peter Zijlstra2013-10-04 477default:
e5137b50a064
[kbuild] drivers/power/supply/goldfish_battery.c:225:2-9: line 225 is redundant because platform_get_irq() already prints an error
CC: kbuild-...@lists.01.org CC: linux-ker...@vger.kernel.org TO: Roman Kiryanov CC: Sebastian Reichel Hi Roman, First bad commit (maybe != root cause): tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8 commit: 570b7c0ea20c0156411394bc215114f7b1dc18ff power: supply: goldfish: Remove the GOLDFISH dependency date: 10 months ago :: branch date: 35 hours ago :: commit date: 10 months ago config: x86_64-randconfig-c022-20220117 (https://download.01.org/0day-ci/archive/20220127/202201271217.e8qbqtnv-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Julia Lawall cocci warnings: (new ones prefixed by >>) >> drivers/power/supply/goldfish_battery.c:225:2-9: line 225 is redundant >> because platform_get_irq() already prints an error Please review and possibly fold the followup patch. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] [PATCH] power: supply: goldfish: fix platform_get_irq.cocci warnings
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Roman Kiryanov
CC: Sebastian Reichel
CC: linux...@vger.kernel.org
CC: linux-ker...@vger.kernel.org
From: kernel test robot
drivers/power/supply/goldfish_battery.c:225:2-9: line 225 is redundant because
platform_get_irq() already prints an error
Remove dev_err() messages after platform_get_irq*() failures
Generated by: scripts/coccinelle/api/platform_get_irq.cocci
CC: Roman Kiryanov
Reported-by: kernel test robot
Signed-off-by: kernel test robot
---
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: 570b7c0ea20c0156411394bc215114f7b1dc18ff power: supply: goldfish:
Remove the GOLDFISH dependency
:: branch date: 35 hours ago
:: commit date: 10 months ago
Please take the patch only if it's a positive warning. Thanks!
goldfish_battery.c |1 -
1 file changed, 1 deletion(-)
--- a/drivers/power/supply/goldfish_battery.c
+++ b/drivers/power/supply/goldfish_battery.c
@@ -222,7 +222,6 @@ static int goldfish_battery_probe(struct
data->irq = platform_get_irq(pdev, 0);
if (data->irq < 0) {
- dev_err(&pdev->dev, "platform_get_irq failed\n");
return -ENODEV;
}
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] arch/riscv/include/asm/spinlock.h:24:2: warning: Dereference of null pointer [clang-analyzer-core.NullDereference]
CC: l...@lists.linux.dev CC: kbuild-...@lists.01.org CC: linux-ker...@vger.kernel.org TO: Thomas Gleixner CC: Ingo Molnar CC: "Peter Zijlstra (Intel)" tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8 commit: 531ae4b06a737ed5539cd75dc6f6b9a28f900bba locking/rtmutex: Split API from implementation date: 5 months ago :: branch date: 34 hours ago :: commit date: 5 months ago config: riscv-randconfig-c006-20220121 (https://download.01.org/0day-ci/archive/20220127/202201271222.3crzpgzj-...@intel.com/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project d4baf3b1322b84816aa623d8e8cb45a49cb68b84) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install riscv cross compiling tool for clang build # apt-get install binutils-riscv64-linux-gnu # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=531ae4b06a737ed5539cd75dc6f6b9a28f900bba git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout 531ae4b06a737ed5539cd75dc6f6b9a28f900bba # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv clang-analyzer If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot clang-analyzer warnings: (new ones prefixed by >>) ^~ fs/orangefs/orangefs-debugfs.c:752:5: note: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 strcat(kernel_debug_string, ","); ^~ fs/orangefs/orangefs-debugfs.c:777:5: warning: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 [clang-analyzer-security.insecureAPI.strcpy] strcat(client_debug_string, ^~ fs/orangefs/orangefs-debugfs.c:777:5: note: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 strcat(client_debug_string, ^~ fs/orangefs/orangefs-debugfs.c:779:5: warning: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 [clang-analyzer-security.insecureAPI.strcpy] strcat(client_debug_string, ","); ^~ fs/orangefs/orangefs-debugfs.c:779:5: note: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 strcat(client_debug_string, ","); ^~ Suppressed 14 warnings (7 in non-user code, 7 with check filters). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 4 warnings generated. Suppressed 4 warnings (4 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 6 warnings generated. lib/glob.c:48:32: warning: Assigned value is garbage or undefined [clang-analyzer-core.uninitialized.Assign] char const *back_pat = NULL, *back_str = back_str; ^ lib/glob.c:48:32: note: Assigned value is garbage or undefined char const *back_pat = NULL, *back_str = back_str; ^ Suppressed 5 warnings (5 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 5 warnings generated. Suppressed 5 warnings (5 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 12 warnings generated. Suppressed 12 warnings (5 in non-user code, 7 with check filters). Use -header-filter=.* to display
[kbuild] drivers/power/supply/goldfish_battery.c:225:2-9: line 225 is redundant because platform_get_irq() already prints an error
CC: kbuild-...@lists.01.org CC: linux-ker...@vger.kernel.org TO: Roman Kiryanov CC: Sebastian Reichel Hi Roman, First bad commit (maybe != root cause): tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8 commit: 570b7c0ea20c0156411394bc215114f7b1dc18ff power: supply: goldfish: Remove the GOLDFISH dependency date: 10 months ago :: branch date: 32 hours ago :: commit date: 10 months ago config: x86_64-randconfig-c022-20220117 (https://download.01.org/0day-ci/archive/20220127/202201271038.2qj68gln-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Julia Lawall cocci warnings: (new ones prefixed by >>) >> drivers/power/supply/goldfish_battery.c:225:2-9: line 225 is redundant >> because platform_get_irq() already prints an error Please review and possibly fold the followup patch. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] [PATCH] power: supply: goldfish: fix platform_get_irq.cocci warnings
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Roman Kiryanov
CC: Sebastian Reichel
CC: linux...@vger.kernel.org
CC: linux-ker...@vger.kernel.org
From: kernel test robot
drivers/power/supply/goldfish_battery.c:225:2-9: line 225 is redundant because
platform_get_irq() already prints an error
Remove dev_err() messages after platform_get_irq*() failures
Generated by: scripts/coccinelle/api/platform_get_irq.cocci
CC: Roman Kiryanov
Reported-by: kernel test robot
Signed-off-by: kernel test robot
---
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: 570b7c0ea20c0156411394bc215114f7b1dc18ff power: supply: goldfish:
Remove the GOLDFISH dependency
:: branch date: 32 hours ago
:: commit date: 10 months ago
Please take the patch only if it's a positive warning. Thanks!
goldfish_battery.c |1 -
1 file changed, 1 deletion(-)
--- a/drivers/power/supply/goldfish_battery.c
+++ b/drivers/power/supply/goldfish_battery.c
@@ -222,7 +222,6 @@ static int goldfish_battery_probe(struct
data->irq = platform_get_irq(pdev, 0);
if (data->irq < 0) {
- dev_err(&pdev->dev, "platform_get_irq failed\n");
return -ENODEV;
}
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] drivers/net/ipa/ipa_qmi.c:128:14: warning: Value stored to 'ipa' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
CC: l...@lists.linux.dev
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Alex Elder
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: 99e75a37bd0af8eb8a0560f48091672b1b6d9218 net: ipa: relax 64-bit build
requirement
date: 10 months ago
:: branch date: 31 hours ago
:: commit date: 10 months ago
config: arm-randconfig-c002-20220125
(https://download.01.org/0day-ci/archive/20220127/202201270925.f8g9fenj-...@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
997e128e2a78f5a5434fc75997441ae1ee76f8a4)
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
#
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=99e75a37bd0af8eb8a0560f48091672b1b6d9218
git remote add linus
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 99e75a37bd0af8eb8a0560f48091672b1b6d9218
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
clang-analyzer warnings: (new ones prefixed by >>)
drivers/mtd/chips/cfi_util.c:317:2: note: Loop condition is false. Exiting
loop
local_irq_disable();
^
include/linux/irqflags.h:205:2: note: expanded from macro 'local_irq_disable'
do {\
^
drivers/mtd/chips/cfi_util.c:321:2: note: Calling 'cfi_qry_mode_on'
cfi_qry_mode_on(base, map, cfi);
^~~
drivers/mtd/chips/cfi_util.c:251:2: note: Calling 'cfi_send_gen_cmd'
cfi_send_gen_cmd(0xF0, 0, base, map, cfi, cfi->device_type, NULL);
^
drivers/mtd/chips/cfi_util.c:209:8: note: Calling 'cfi_build_cmd'
val = cfi_build_cmd(cmd, map, cfi);
^~~~
drivers/mtd/chips/cfi_util.c:78:6: note: Assuming the condition is false
if (map_bankwidth_is_large(map)) {
^
include/linux/mtd/map.h:115:39: note: expanded from macro
'map_bankwidth_is_large'
# define map_bankwidth_is_large(map) (map_bankwidth(map) > BITS_PER_LONG/8)
^~~~
include/linux/mtd/map.h:113:29: note: expanded from macro 'map_bankwidth'
# define map_bankwidth(map) ((map)->bankwidth)
^
drivers/mtd/chips/cfi_util.c:78:2: note: Taking false branch
if (map_bankwidth_is_large(map)) {
^
drivers/mtd/chips/cfi_util.c:91:2: note: Control jumps to 'case 4:' at line
99
switch (chip_mode) {
^
drivers/mtd/chips/cfi_util.c:100:12: note: '?' condition is true
onecmd = cpu_to_cfi32(map, cmd);
^
include/linux/mtd/cfi_endian.h:30:30: note: expanded from macro
'cpu_to_cfi32'
#define cpu_to_cfi32(map, x) _cpu_to_cfi(32, (map)->swap, (x))
^
include/linux/mtd/cfi_endian.h:36:31: note: expanded from macro '_cpu_to_cfi'
#define _cpu_to_cfi(w, s, x) (cfi_host(s)?(x):_swap_to_cfi(w, s, x))
^
include/linux/mtd/cfi_endian.h:25:22: note: expanded from macro 'cfi_host'
#define cfi_host(s) (cfi_default(s) == CFI_HOST_ENDIAN)
^
include/linux/mtd/cfi_endian.h:22:25: note: expanded from macro 'cfi_default'
#define cfi_default(s) ((s)?:CFI_DEFAULT_ENDIAN)
^
drivers/mtd/chips/cfi_util.c:100:12: note: Assuming the condition is true
onecmd = cpu_to_cfi32(map, cmd);
^
include/linux/mtd/cfi_endian.h:30:30: note: expanded from macro
'cpu_to_cfi32'
#define cpu_to_cfi32(map, x) _cpu_to_cfi(32, (map)->swap, (x))
^
include/linux/mtd/cfi_endian.h:36:31: note: expanded from macro '_cpu_to_cfi'
#define _cpu_to_cfi(w, s, x) (cfi_host(s)?(x):_swap_to_cfi(w, s, x))
^~~
include/linux/mtd/cfi_endian.h:25:22: note: expanded from macro 'cfi_host'
#define cfi_host(s) (cfi_default(s) == CFI_HOST_ENDIAN)
^
include/linux/mtd/cfi_endian.h:22:24: note: expanded from macro 'cfi_default'
#define cfi_default(s) ((s)?:CFI_DEFAULT_ENDIAN)
^
drivers/mtd/chips/cfi_util.
[kbuild] [osandov:btrfs-send-encoded 15/17] fs/btrfs/send.c:7806 btrfs_ioctl_send() error: we previously assumed 'sctx->send_buf_pages' could be null (see line 7593)
CC: kbuild-...@lists.01.org
CC: Omar Sandoval
CC: linux-ker...@vger.kernel.org
TO: Omar Sandoval
CC: Nikolay Borisov
tree: https://github.com/osandov/linux.git btrfs-send-encoded
head: ec062ce505d22f204218f91a940b55fb38d66b07
commit: 57cd63d9f970d529c3cc9b80612cffd71b3186b9 [15/17] btrfs: send: allocate
send buffer with alloc_page() and vmap() for v2
:: branch date: 27 hours ago
:: commit date: 27 hours ago
config: nios2-randconfig-m031-20220124
(https://download.01.org/0day-ci/archive/20220127/202201270814.2gid9lnj-...@intel.com/config)
compiler: nios2-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
New smatch warnings:
fs/btrfs/send.c:7806 btrfs_ioctl_send() error: we previously assumed
'sctx->send_buf_pages' could be null (see line 7593)
Old smatch warnings:
arch/nios2/include/asm/thread_info.h:71 current_thread_info() error:
uninitialized symbol 'sp'.
vim +7806 fs/btrfs/send.c
62d54f3a7fa27e Filipe Manana 2019-04-22 7488
786e6838d278ed Sahil Kang2022-01-15 7489 long
btrfs_ioctl_send(struct inode *inode, struct btrfs_ioctl_send_args *arg)
31db9f7c23fbf7 Alexander Block 2012-07-25 7490 {
31db9f7c23fbf7 Alexander Block 2012-07-25 7491 int ret = 0;
786e6838d278ed Sahil Kang2022-01-15 7492 struct
btrfs_root *send_root = BTRFS_I(inode)->root;
0b246afa62b0cf Jeff Mahoney 2016-06-22 7493 struct
btrfs_fs_info *fs_info = send_root->fs_info;
31db9f7c23fbf7 Alexander Block 2012-07-25 7494 struct
btrfs_root *clone_root;
31db9f7c23fbf7 Alexander Block 2012-07-25 7495 struct send_ctx
*sctx = NULL;
31db9f7c23fbf7 Alexander Block 2012-07-25 7496 u32 i;
57cd63d9f970d5 Omar Sandoval 2020-05-01 7497 u32
send_buf_num_pages = 0;
31db9f7c23fbf7 Alexander Block 2012-07-25 7498 u64
*clone_sources_tmp = NULL;
2c68653787f91c David Sterba 2013-12-16 7499 int
clone_sources_to_rollback = 0;
bae12df966f0e1 Denis Efremov 2020-09-21 7500 size_t
alloc_size;
896c14f97f700a Wang Shilong 2014-01-07 7501 int
sort_clone_roots = 0;
31db9f7c23fbf7 Alexander Block 2012-07-25 7502
31db9f7c23fbf7 Alexander Block 2012-07-25 7503 if
(!capable(CAP_SYS_ADMIN))
31db9f7c23fbf7 Alexander Block 2012-07-25 7504 return
-EPERM;
31db9f7c23fbf7 Alexander Block 2012-07-25 7505
2c68653787f91c David Sterba 2013-12-16 7506 /*
2c68653787f91c David Sterba 2013-12-16 7507* The
subvolume must remain read-only during send, protect against
521e0546c970c3 David Sterba 2014-04-15 7508* making it
RW. This also protects against deletion.
2c68653787f91c David Sterba 2013-12-16 7509*/
2c68653787f91c David Sterba 2013-12-16 7510
spin_lock(&send_root->root_item_lock);
62d54f3a7fa27e Filipe Manana 2019-04-22 7511 if
(btrfs_root_readonly(send_root) && send_root->dedupe_in_progress) {
62d54f3a7fa27e Filipe Manana 2019-04-22 7512
dedupe_in_progress_warn(send_root);
62d54f3a7fa27e Filipe Manana 2019-04-22 7513
spin_unlock(&send_root->root_item_lock);
62d54f3a7fa27e Filipe Manana 2019-04-22 7514 return
-EAGAIN;
62d54f3a7fa27e Filipe Manana 2019-04-22 7515 }
2c68653787f91c David Sterba 2013-12-16 7516
send_root->send_in_progress++;
2c68653787f91c David Sterba 2013-12-16 7517
spin_unlock(&send_root->root_item_lock);
2c68653787f91c David Sterba 2013-12-16 7518
2c68653787f91c David Sterba 2013-12-16 7519 /*
2c68653787f91c David Sterba 2013-12-16 7520* Userspace
tools do the checks and warn the user if it's
2c68653787f91c David Sterba 2013-12-16 7521* not RO.
2c68653787f91c David Sterba 2013-12-16 7522*/
2c68653787f91c David Sterba 2013-12-16 7523 if
(!btrfs_root_readonly(send_root)) {
2c68653787f91c David Sterba 2013-12-16 7524 ret =
-EPERM;
2c68653787f91c David Sterba 2013-12-16 7525 goto
out;
2c68653787f91c David Sterba 2013-12-16 7526 }
2c68653787f91c David Sterba 2013-12-16 7527
457ae7268b29c3 Dan Carpenter 2017-03-17 7528 /*
457ae7268b29c3 Dan Carpenter 2017-03-17 7529* Check that
we don't overflow at later allocations, we request
457ae7268b29c3 Dan Carpenter 2017-03-17 7530*
clone_sources_count + 1 items, and compare to unsigned long inside
457ae7268b29c3 Dan Carpenter 2017-03-17
[kbuild] Re: [RFC PATCH 2/5] driver/net/tun: Added features for USO.
CC: kbuild-...@lists.01.org
In-Reply-To: <20220125084702.3636253-3-and...@daynix.com>
References: <20220125084702.3636253-3-and...@daynix.com>
TO: Andrew Melnychenko
Hi Andrew,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on net-next/master]
[also build test WARNING on net/master mst-vhost/linux-next linus/master
v5.17-rc1 next-20220125]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url:
https://github.com/0day-ci/linux/commits/Andrew-Melnychenko/TUN-VirtioNet-USO-features-support/20220125-171057
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
6e667749271e58d34238cf700e543beabdbe6184
:: branch date: 2 days ago
:: commit date: 2 days ago
config: openrisc-randconfig-m031-20220124
(https://download.01.org/0day-ci/archive/20220127/202201270710.tzxkgcwt-...@intel.com/config)
compiler: or1k-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
New smatch warnings:
drivers/net/tap.c:945 set_offload() warn: compare has higher precedence than
mask
drivers/net/tap.c:945 set_offload() warn: add some parenthesis here?
Old smatch warnings:
drivers/net/tap.c:958 set_offload() warn: compare has higher precedence than
mask
drivers/net/tap.c:958 set_offload() warn: add some parenthesis here?
vim +945 drivers/net/tap.c
815f236d622721b drivers/net/macvtap.c Jason Wang 2013-06-05 919
635b8c8ecdd2714 drivers/net/tap.c Sainath Grandhi2017-02-10 920
static int set_offload(struct tap_queue *q, unsigned long arg)
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 921 {
6fe3faf86757eb7 drivers/net/tap.c Sainath Grandhi2017-02-10 922
struct tap_dev *tap;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 923
netdev_features_t features;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 924
netdev_features_t feature_mask = 0;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 925
6fe3faf86757eb7 drivers/net/tap.c Sainath Grandhi2017-02-10 926
tap = rtnl_dereference(q->tap);
6fe3faf86757eb7 drivers/net/tap.c Sainath Grandhi2017-02-10 927
if (!tap)
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 928
return -ENOLINK;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 929
6fe3faf86757eb7 drivers/net/tap.c Sainath Grandhi2017-02-10 930
features = tap->dev->features;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 931
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 932
if (arg & TUN_F_CSUM) {
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 933
feature_mask = NETIF_F_HW_CSUM;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 934
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 935
if (arg & (TUN_F_TSO4 | TUN_F_TSO6)) {
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 936
if (arg & TUN_F_TSO_ECN)
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 937
feature_mask |= NETIF_F_TSO_ECN;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 938
if (arg & TUN_F_TSO4)
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 939
feature_mask |= NETIF_F_TSO;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 940
if (arg & TUN_F_TSO6)
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 941
feature_mask |= NETIF_F_TSO6;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 942
}
3c1c2daa10c8eac drivers/net/tap.c Andrew Melnychenko 2022-01-25 943
3c1c2daa10c8eac drivers/net/tap.c Andrew Melnychenko 2022-01-25 944
/* TODO: for now USO4 and USO6 should work simultaneously */
3c1c2daa10c8eac drivers/net/tap.c Andrew Melnychenko 2022-01-25 @945
if (arg & (TUN_F_USO4 | TUN_F_USO6) == (TUN_F_USO4 | TUN_F_USO6))
3c1c2daa10c8eac drivers/net/tap.c Andrew Melnychenko 2022-01-25 946
features |= NETIF_F_GSO_UDP_L4;
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 947
}
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 948
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 949
/* tun/tap driver inverts the usage for TSO offloads, where
2be5c76794b0e57 drivers/net/macvtap.c Vlad Yasevich 2013-06-25 950
* setting the TSO b
[kbuild] drivers/pinctrl/pinctrl-starfive.c:640:17: warning: Value stored to 'dev' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
CC: l...@lists.linux.dev
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Emil Renner Berthing
CC: Andy Shevchenko
CC: Linus Walleij
CC: Huan Feng
CC: Drew Fustini
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: ec648f6b7686b716424e8e73eebb4c11ae199187 pinctrl: starfive: Add pinctrl
driver for StarFive SoCs
date: 6 weeks ago
:: branch date: 28 hours ago
:: commit date: 6 weeks ago
config: riscv-randconfig-c006-20220126
(https://download.01.org/0day-ci/archive/20220127/202201270546.qg2icis2-...@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
2a1b7aa016c0f4b5598806205bdfbab1ea2d92c4)
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
#
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ec648f6b7686b716424e8e73eebb4c11ae199187
git remote add linus
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout ec648f6b7686b716424e8e73eebb4c11ae199187
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv
clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
clang-analyzer warnings: (new ones prefixed by >>)
^
include/linux/percpu-defs.h:421:34: note: expanded from macro 'raw_cpu_write'
#define raw_cpu_write(pcp, val) __pcpu_size_call(raw_cpu_write_,
pcp, val)
^
include/linux/percpu-defs.h:379:42: note: expanded from macro
'__pcpu_size_call'
case 4: stem##4(variable, __VA_ARGS__);break; \
^
kernel/softirq.c:540:2: note: Loop condition is false. Exiting loop
set_softirq_pending(0);
^
include/linux/interrupt.h:494:33: note: expanded from macro
'set_softirq_pending'
#define set_softirq_pending(x) (__this_cpu_write(local_softirq_pending_ref,
(x)))
^
include/linux/percpu-defs.h:452:2: note: expanded from macro
'__this_cpu_write'
raw_cpu_write(pcp, val);\
^
include/linux/percpu-defs.h:421:34: note: expanded from macro 'raw_cpu_write'
#define raw_cpu_write(pcp, val) __pcpu_size_call(raw_cpu_write_,
pcp, val)
^
include/linux/percpu-defs.h:373:50: note: expanded from macro
'__pcpu_size_call'
#define __pcpu_size_call(stem, variable, ...) \
^
kernel/softirq.c:542:2: note: Loop condition is false. Exiting loop
local_irq_enable();
^
include/linux/irqflags.h:235:28: note: expanded from macro 'local_irq_enable'
#define local_irq_enable() do { raw_local_irq_enable(); } while (0)
^
kernel/softirq.c:546:2: note: Loop condition is true. Entering loop body
while ((softirq_bit = ffs(pending))) {
^
kernel/softirq.c:552:3: note: The value 16 is assigned to 'vec_nr'
vec_nr = h - softirq_vec;
^~~~
kernel/softirq.c:560:16: note: Assuming the condition is true
if (unlikely(prev_count != preempt_count())) {
^
include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
# define unlikely(x)__builtin_expect(!!(x), 0)
^
kernel/softirq.c:560:3: note: Taking true branch
if (unlikely(prev_count != preempt_count())) {
^
kernel/softirq.c:561:4: note: Loop condition is false. Exiting loop
pr_err("huh, entered softirq %u %s %p with
preempt_count %08x, exited with %08x?\n",
^
include/linux/printk.h:489:2: note: expanded from macro 'pr_err'
printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
^
include/linux/printk.h:446:26: note: expanded from macro 'printk'
#define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
^
include/linux/printk.h:417:3: note: expanded from macro 'printk_index_wrap'
__printk_index_emit(_fmt, NULL, NULL); \
[kbuild] arch/x86/include/asm/atomic.h:29:9: warning: Use of memory after it is freed [clang-analyzer-unix.Malloc]
CC: l...@lists.linux.dev
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: David Howells
CC: Jeff Layton
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: 78525c74d9e7d1a6ce69bd4388f045f6e474a20b netfs, 9p, afs, ceph: Use
folios
date: 3 months ago
:: branch date: 26 hours ago
:: commit date: 3 months ago
config: x86_64-randconfig-c007-20220124
(https://download.01.org/0day-ci/archive/20220127/202201270357.mq4gky77-...@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
7b3d30728816403d1fd73cc5082e9fb761262bce)
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
#
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=78525c74d9e7d1a6ce69bd4388f045f6e474a20b
git remote add linus
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 78525c74d9e7d1a6ce69bd4388f045f6e474a20b
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
clang-analyzer warnings: (new ones prefixed by >>)
include/asm-generic/rwonce.h:36:21: note: expanded from macro
'compiletime_assert_rwonce_type'
compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long
long), \
^
include/linux/compiler_types.h:302:3: note: expanded from macro
'__native_word'
(sizeof(t) == sizeof(char) || sizeof(t) == sizeof(short) || \
^
kernel/time/posix-cpu-timers.c:979:7: note: Left side of '||' is true
if (!READ_ONCE(pct->timers_active) || pct->expiry_active)
^
include/asm-generic/rwonce.h:49:2: note: expanded from macro 'READ_ONCE'
compiletime_assert_rwonce_type(x); \
^
include/asm-generic/rwonce.h:36:21: note: expanded from macro
'compiletime_assert_rwonce_type'
compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long
long), \
^
include/linux/compiler_types.h:303:28: note: expanded from macro
'__native_word'
sizeof(t) == sizeof(int) || sizeof(t) == sizeof(long))
^
kernel/time/posix-cpu-timers.c:979:7: note: Taking false branch
if (!READ_ONCE(pct->timers_active) || pct->expiry_active)
^
include/asm-generic/rwonce.h:49:2: note: expanded from macro 'READ_ONCE'
compiletime_assert_rwonce_type(x); \
^
include/asm-generic/rwonce.h:36:2: note: expanded from macro
'compiletime_assert_rwonce_type'
compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long
long), \
^
include/linux/compiler_types.h:335:2: note: expanded from macro
'compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_,
__COUNTER__)
^
include/linux/compiler_types.h:323:2: note: expanded from macro
'_compiletime_assert'
__compiletime_assert(condition, msg, prefix, suffix)
^
include/linux/compiler_types.h:315:3: note: expanded from macro
'__compiletime_assert'
if (!(condition)) \
^
kernel/time/posix-cpu-timers.c:979:7: note: Loop condition is false.
Exiting loop
if (!READ_ONCE(pct->timers_active) || pct->expiry_active)
^
include/asm-generic/rwonce.h:49:2: note: expanded from macro 'READ_ONCE'
compiletime_assert_rwonce_type(x); \
^
include/asm-generic/rwonce.h:36:2: note: expanded from macro
'compiletime_assert_rwonce_type'
compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long
long), \
^
include/linux/compiler_types.h:335:2: note: expanded from macro
'compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_,
__COUNTER__)
^
include/linux/compiler_types.h:323:2: note: expanded from macro
'_compiletime_assert'
__compiletime_assert(condition, msg, prefix, suffix)
^
include/linux/compiler_types.h:307:2: note: expanded from macro
'__compiletime_assert'
do {\
^
kernel/time/posix-cpu-timers.c:979:6: note: Assuming the condition is true
if (!READ_ONCE(pct->timers_active) || pct->expiry_active)
^~
kernel/time/posix-cpu-timers.c:979:37: note: Left side of '||' is true
[kbuild] drivers/staging/vc04_services/interface/vchiq_arm/vchiq_dev.c:129:7: warning: Branch condition evaluates to a garbage value [clang-analyzer-core.uninitialized.Branch]
CC: l...@lists.linux.dev
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Kees Cook
CC: Miguel Ojeda
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: c80d92fbb67b2c80b8eeb8759ee79d676eb33520 compiler_types.h: Remove
__compiletime_object_size()
date: 4 months ago
:: branch date: 24 hours ago
:: commit date: 4 months ago
config: riscv-randconfig-c006-20220125
(https://download.01.org/0day-ci/archive/20220127/202201270239.y4uwkzbp-...@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
997e128e2a78f5a5434fc75997441ae1ee76f8a4)
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
#
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=c80d92fbb67b2c80b8eeb8759ee79d676eb33520
git remote add linus
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout c80d92fbb67b2c80b8eeb8759ee79d676eb33520
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv
clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
clang-analyzer warnings: (new ones prefixed by >>)
5 warnings generated.
drivers/iio/dac/ad5755.c:88:8: warning: Excessive padding in 'struct
ad5755_state' (100 padding bytes, where 36 is optimal).
Optimal fields order:
data,
spi,
chip_info,
pwr_down,
ctrl,
lock,
channels,
consider reordering the fields or adding explicit padding members
[clang-analyzer-optin.performance.Padding]
struct ad5755_state {
~~~^~
drivers/iio/dac/ad5755.c:88:8: note: Excessive padding in 'struct
ad5755_state' (100 padding bytes, where 36 is optimal). Optimal fields order:
data, spi, chip_info, pwr_down, ctrl, lock, channels, consider reordering the
fields or adding explicit padding members
struct ad5755_state {
~~~^~
drivers/iio/dac/ad5755.c:729:37: warning: Value stored to 'pdata' during its
initialization is never read [clang-analyzer-deadcode.DeadStores]
const struct ad5755_platform_data *pdata =
dev_get_platdata(&spi->dev);
^
~~~
drivers/iio/dac/ad5755.c:729:37: note: Value stored to 'pdata' during its
initialization is never read
const struct ad5755_platform_data *pdata =
dev_get_platdata(&spi->dev);
^
~~~
Suppressed 3 warnings (2 in non-user code, 1 with check filters).
Use -header-filter=.* to display errors from all non-system headers. Use
-system-headers to display errors from system headers as well.
2 warnings generated.
Suppressed 2 warnings (2 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use
-system-headers to display errors from system headers as well.
4 warnings generated.
drivers/iio/dac/ad5761.c:63:8: warning: Excessive padding in 'struct
ad5761_state' (75 padding bytes, where 11 is optimal).
Optimal fields order:
data,
spi,
vref_reg,
vref,
range,
lock,
use_intref,
consider reordering the fields or adding explicit padding members
[clang-analyzer-optin.performance.Padding]
struct ad5761_state {
~~~^~
drivers/iio/dac/ad5761.c:63:8: note: Excessive padding in 'struct
ad5761_state' (75 padding bytes, where 11 is optimal). Optimal fields order:
data, spi, vref_reg, vref, range, lock, use_intref, consider reordering the
fields or adding explicit padding members
struct ad5761_state {
~~~^~
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use
-system-headers to display errors from system headers as well.
4 warnings generated.
drivers/input/keyboard/lm8323.c:333:3: warning: Value stored to 'active' is
never read [clang-analyzer-deadcode.DeadStores]
active = debounce + 3;
^
drivers/input/keyboard/lm8323.c:333:3: note: Value stored to 'active' is
never read
active = debounce + 3;
^
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use
-system-headers to display errors from system headers as well.
3 warnings generated.
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from al
[kbuild] drivers/ata/pata_atp867x.c:281:3: warning: Value stored to 'start' is never read [clang-analyzer-deadcode.DeadStores]
CC: l...@lists.linux.dev CC: kbuild-...@lists.01.org CC: linux-ker...@vger.kernel.org TO: Hannes Reinecke CC: Damien Le Moal tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8 commit: f2f01a52f28121770c5cd48352a60b87e1fa204b ata: pata_atp867x: convert printk() calls date: 3 weeks ago :: branch date: 23 hours ago :: commit date: 3 weeks ago config: x86_64-randconfig-c007-20220124 (https://download.01.org/0day-ci/archive/20220127/202201270026.kqffvnql-...@intel.com/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 7b3d30728816403d1fd73cc5082e9fb761262bce) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f2f01a52f28121770c5cd48352a60b87e1fa204b git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout f2f01a52f28121770c5cd48352a60b87e1fa204b # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 clang-analyzer If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot clang-analyzer warnings: (new ones prefixed by >>) ^ drivers/misc/mei/client.c:73:3: note: Returning; memory was released kref_put(&me_cl->refcnt, mei_me_cl_release); ^~~ drivers/misc/mei/client.c:91:2: note: Returning; memory was released via 1st parameter mei_me_cl_put(me_cl); ^~~~ drivers/misc/mei/client.c:291:2: note: Returning; memory was released via 2nd parameter __mei_me_cl_del(dev, me_cl); ^~~ drivers/misc/mei/client.c:292:2: note: Use of memory after it is freed mei_me_cl_put(me_cl); ^ ~ drivers/misc/mei/client.c:577:2: warning: Value stored to 'dev' is never read [clang-analyzer-deadcode.DeadStores] dev = cl->dev; ^ ~~~ drivers/misc/mei/client.c:577:2: note: Value stored to 'dev' is never read dev = cl->dev; ^ ~~~ drivers/misc/mei/client.c:766:21: warning: Value stored to 'dev' during its initialization is never read [clang-analyzer-deadcode.DeadStores] struct mei_device *dev = cl->dev; ^~~ ~~~ drivers/misc/mei/client.c:766:21: note: Value stored to 'dev' during its initialization is never read struct mei_device *dev = cl->dev; ^~~ ~~~ drivers/misc/mei/client.c:1593:2: warning: Value stored to 'dev' is never read [clang-analyzer-deadcode.DeadStores] dev = cl->dev; ^ ~~~ drivers/misc/mei/client.c:1593:2: note: Value stored to 'dev' is never read dev = cl->dev; ^ ~~~ drivers/misc/mei/client.c:1990:3: warning: Value stored to 'rets' is never read [clang-analyzer-deadcode.DeadStores] rets = buf_len; ^ ~~~ drivers/misc/mei/client.c:1990:3: note: Value stored to 'rets' is never read rets = buf_len; ^ ~~~ drivers/misc/mei/client.c:1996:3: warning: Value stored to 'rets' is never read [clang-analyzer-deadcode.DeadStores] rets = buf_len; ^ ~~~ drivers/misc/mei/client.c:1996:3: note: Value stored to 'rets' is never read rets = buf_len; ^ ~~~ drivers/misc/mei/client.c:2002:3: warning: Value stored to 'rets' is never read [clang-analyzer-deadcode.DeadStores] rets = -EOVERFLOW; ^ ~~ drivers/misc/mei/client.c:2002:3: note: Value stored to 'rets' is never read rets = -EOVERFLOW; ^ ~~ Suppressed 1 warnings (1 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 4 warnings generated. drivers/cxl/core/mbox.c:326:17: warning: Value stored to 'dev' during its initialization is never read [clang-analyzer-deadcode.DeadStores] struct device *dev = &cxlmd->dev; ^~~ ~~~ drivers/cxl/core/mbox.c:326:17: note: Value stored to 'dev' during its initialization is never read struct device *dev = &cxlmd->dev; ^~~ ~~~ drivers/cxl/core/mbox.c:451:17: warning: Value stored to 'dev' during its initialization is never read [clang-analyze
[kbuild] [linux-next:master 1591/1734] fs/btrfs/scrub.c:3678 scrub_stripe() error: uninitialized symbol 'offset'.
CC: kbuild-...@lists.01.org
CC: Linux Memory Management List
TO: Qu Wenruo
CC: David Sterba
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
master
head: d25ee88530253138d0b20d43511ca5acbda4e9f7
commit: fa676286533c1c414da23d5ef8ae452c7f66e9b7 [1591/1734] btrfs: use
scrub_simple_mirror() to handle RAID56 data stripe scrub
:: branch date: 2 days ago
:: commit date: 2 days ago
config: openrisc-randconfig-m031-20220124
(https://download.01.org/0day-ci/archive/20220127/202201270045.ixlgja5t-...@intel.com/config)
compiler: or1k-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
New smatch warnings:
fs/btrfs/scrub.c:3678 scrub_stripe() error: uninitialized symbol 'offset'.
fs/btrfs/scrub.c:3680 scrub_stripe() error: uninitialized symbol 'physical_end'.
Old smatch warnings:
fs/btrfs/scrub.c:3439 scrub_simple_mirror() error: uninitialized symbol 'ret'.
vim +/offset +3678 fs/btrfs/scrub.c
38d2dda468be859 Qu Wenruo 2022-01-07 3512
d9d181c1ba7aa09 Stefan Behrens 2012-11-02 3513 static noinline_for_stack int
scrub_stripe(struct scrub_ctx *sctx,
2ae8ae3d3def4c3 Qu Wenruo 2021-12-15 3514
struct btrfs_block_group *bg,
a36cf8b8933e4a7 Stefan Behrens 2012-11-02 3515
struct map_lookup *map,
a36cf8b8933e4a7 Stefan Behrens 2012-11-02 3516
struct btrfs_device *scrub_dev,
2ae8ae3d3def4c3 Qu Wenruo 2021-12-15 3517
int stripe_index, u64 dev_extent_len)
a2de733c78fa7af Arne Jansen 2011-03-08 3518 {
2522dbe86b54ff0 Qu Wenruo 2021-12-14 3519struct btrfs_path *path;
fb456252d3d9c05 Jeff Mahoney2016-06-22 3520struct btrfs_fs_info
*fs_info = sctx->fs_info;
29cbcf401793f4e Josef Bacik 2021-11-05 3521struct btrfs_root *root;
fc28b25e1f42865 Josef Bacik 2021-11-05 3522struct btrfs_root
*csum_root;
e7786c3ae517b2c Arne Jansen 2011-05-28 3523struct blk_plug plug;
b5b99b1e02969dd Qu Wenruo 2022-01-07 3524const u64 profile =
map->type & BTRFS_BLOCK_GROUP_PROFILE_MASK;
2ae8ae3d3def4c3 Qu Wenruo 2021-12-15 3525const u64 chunk_logical
= bg->start;
a2de733c78fa7af Arne Jansen 2011-03-08 3526int ret;
a2de733c78fa7af Arne Jansen 2011-03-08 3527u64 nstripes;
a2de733c78fa7af Arne Jansen 2011-03-08 3528u64 physical;
a2de733c78fa7af Arne Jansen 2011-03-08 3529u64 logical;
625f1c8dc66d778 Liu Bo 2013-04-27 3530u64 logic_end;
3b080b2564287be Wang Shilong2014-04-01 3531u64 physical_end;
fa676286533c1c4 Qu Wenruo 2022-01-07 3532u64 increment; /* The
logical increment after finishing one stripe */
fa676286533c1c4 Qu Wenruo 2022-01-07 3533u64 offset; /*
Offset inside the chunk */
5a6ac9eacb49143 Miao Xie2014-11-06 3534u64 stripe_logical;
5a6ac9eacb49143 Miao Xie2014-11-06 3535u64 stripe_end;
3b080b2564287be Wang Shilong2014-04-01 3536int stop_loop = 0;
53b381b3abeb86f David Woodhouse 2013-01-29 3537
a2de733c78fa7af Arne Jansen 2011-03-08 3538path =
btrfs_alloc_path();
a2de733c78fa7af Arne Jansen 2011-03-08 3539if (!path)
a2de733c78fa7af Arne Jansen 2011-03-08 3540return -ENOMEM;
a2de733c78fa7af Arne Jansen 2011-03-08 3541
b5d67f64f9bc656 Stefan Behrens 2012-03-27 3542/*
fa676286533c1c4 Qu Wenruo 2022-01-07 3543 * Work on commit root.
The related disk blocks are static as
b5d67f64f9bc656 Stefan Behrens 2012-03-27 3544 * long as COW is
applied. This means, it is save to rewrite
b5d67f64f9bc656 Stefan Behrens 2012-03-27 3545 * them to repair disk
errors without any race conditions
b5d67f64f9bc656 Stefan Behrens 2012-03-27 3546 */
a2de733c78fa7af Arne Jansen 2011-03-08 3547
path->search_commit_root = 1;
a2de733c78fa7af Arne Jansen 2011-03-08 3548path->skip_locking = 1;
dcf62b204c06ac2 Qu Wenruo 2021-12-14 3549path->reada =
READA_FORWARD;
a2de733c78fa7af Arne Jansen 2011-03-08 3550
d9d181c1ba7aa09 Stefan Behrens 2012-11-02 3551
wait_event(sctx->list_wait,
b6bfebc13218f1f Stefan Behrens 2012-11-02 3552
atomic_read(&sctx->bios_in_flight) == 0);
cb7ab02156e4ba9 Wang Shilong2013-12-04 3553
scrub_blocked_if_needed(fs_info);
a2de733c78fa7af Arne Jansen 2011-03-08 3554
fa676286533c1c4 Qu Wenruo 2022-01-07 3555root =
btrfs_extent_root(fs_info, bg->start);
fa676286533c1c4 Qu Wenruo 2022-01-07 3556csum_root =
btrfs_csum_root(fs_info, bg->start);
fc28b25e1f42865 Josef Bacik 2021-11-05 3557
a2de733c78fa7af Arne Janse
[kbuild] Re: kernel/trace/trace_events_hist.c:6174 event_hist_trigger_parse() error: we previously assumed 'glob' could be null (see line 6166)
On Wed, Jan 26, 2022 at 09:39:18AM -0500, Steven Rostedt wrote:
> > 9ec5a7d16899ed Tom Zanussi 2022-01-10 6149 static int
> > event_hist_trigger_parse(struct event_command *cmd_ops,
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6150
> > struct trace_event_file *file,
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6151
> > char *glob, char *cmd, char *param)
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6152 {
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6153 unsigned int
> > hist_trigger_bits = TRACING_MAP_BITS_DEFAULT;
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6154 struct
> > event_trigger_data *trigger_data;
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6155 struct
> > hist_trigger_attrs *attrs;
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6156 struct
> > event_trigger_ops *trigger_ops;
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6157 struct
> > hist_trigger_data *hist_data;
> > 4b147936fa5096 Tom Zanussi 2018-01-15 6158 struct
> > synth_event *se;
> > 4b147936fa5096 Tom Zanussi 2018-01-15 6159 const char
> > *se_name;
> > 30350d65ac5676 Tom Zanussi 2018-01-15 6160 bool remove =
> > false;
> > c5eac6ee8bc5d3 Kalesh Singh2021-10-25 6161 char *trigger,
> > *p, *start;
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6162 int ret = 0;
> > 7ef224d1d0e3a1 Tom Zanussi 2016-03-03 6163
> > 0e2b81f7b52a1c Masami Hiramatsu2018-11-05 6164
> > lockdep_assert_held(&event_mutex);
> > 0e2b81f7b52a1c Masami Hiramatsu2018-11-05 6165
> > f404da6e1d46ce Tom Zanussi 2018-01-15 @6166 if (glob &&
> > strlen(glob)) {
>
> I just reviewed the code, and it looks like the logic should keep glob from
> ever being NULL.
Smatch can also figure that out, and does not warn, if you have the
cross function DB. Unfortunately, that's not feasible for the zero day
bot because it takes too long to build the DB.
I was puzzled why this warning showed up now when the code is from 2018.
regards,
dan carpenter
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] fs/ext4/inline.c:1386:13: warning: Assigned value is garbage or undefined [clang-analyzer-core.uninitialized.Assign]
CC: l...@lists.linux.dev
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: NeilBrown
CC: Andrew Morton
CC: Linux Memory Management List
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: 4034247a0d6ab281ba3293798ce67af494d86129 mm: introduce
memalloc_retry_wait()
date: 11 days ago
:: branch date: 20 hours ago
:: commit date: 11 days ago
config: i386-randconfig-c001
(https://download.01.org/0day-ci/archive/20220126/202201262135.p6zvkkzw-...@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project
f7b7138a62648f4019c55e4671682af1f851f295)
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
#
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4034247a0d6ab281ba3293798ce67af494d86129
git remote add linus
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 4034247a0d6ab281ba3293798ce67af494d86129
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=i386
clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
clang-analyzer warnings: (new ones prefixed by >>)
^
fs/ext4/inline.c:1234:2: note: Taking false branch
if (!data_bh) {
^
include/linux/compiler.h:56:23: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^
fs/ext4/inline.c:1242:2: note: Assuming 'error' is 0
if (error) {
^
include/linux/compiler.h:56:45: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
~^~
include/linux/compiler.h:58:52: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) :
__trace_if_value(cond))
^~~~
fs/ext4/inline.c:1242:2: note: '?' condition is false
if (error) {
^
include/linux/compiler.h:56:28: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^
include/linux/compiler.h:58:31: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) :
__trace_if_value(cond))
^
fs/ext4/inline.c:1242:6: note: 'error' is 0
if (error) {
^
include/linux/compiler.h:56:47: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~
include/linux/compiler.h:58:86: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) :
__trace_if_value(cond))
^~~~
include/linux/compiler.h:69:3: note: expanded from macro '__trace_if_value'
(cond) ?\
^~~~
fs/ext4/inline.c:1242:2: note: '?' condition is false
if (error) {
^
include/linux/compiler.h:56:28: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^
include/linux/compiler.h:58:69: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) :
__trace_if_value(cond))
^
include/linux/compiler.h:69:2: note: expanded from macro '__trace_if_value'
(cond) ?\
^
fs/ext4/inline.c:1242:2: note: Taking false branch
if (error) {
^
include/linux/compiler.h:56:23: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^
fs/ext4/inline.c:1249:7: note: Assuming the condition is true
if (!S_ISDIR(inode->i_mode)) {
^
include/uapi/linux/stat.h:23:21: note: expanded from macro 'S_ISDIR'
#define S_ISDIR(m) (((m) & S_IFMT) == S_IFDIR)
^
include/linux/compiler.h:56:47: note: expanded from macro 'if'
#define if(cond, ...) if ( __t
[kbuild] [linux-next:master 1589/1734] fs/btrfs/scrub.c:3439 scrub_simple_mirror() error: uninitialized symbol 'ret'.
CC: kbuild-...@lists.01.org
CC: Linux Memory Management List
TO: Qu Wenruo
CC: David Sterba
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
master
head: d25ee88530253138d0b20d43511ca5acbda4e9f7
commit: b5b99b1e02969ddd0950356ae1561a73bc377021 [1589/1734] btrfs: introduce
dedicated helper to scrub simple-mirror based range
:: branch date: 33 hours ago
:: commit date: 2 days ago
config: openrisc-randconfig-m031-20220124
(https://download.01.org/0day-ci/archive/20220126/202201262100.ak94yosk-...@intel.com/config)
compiler: or1k-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Dan Carpenter
smatch warnings:
fs/btrfs/scrub.c:3439 scrub_simple_mirror() error: uninitialized symbol 'ret'.
vim +/ret +3439 fs/btrfs/scrub.c
b5b99b1e02969d Qu Wenruo 2022-01-07 3304
b5b99b1e02969d Qu Wenruo 2022-01-07 3305 /*
b5b99b1e02969d Qu Wenruo 2022-01-07 3306 * Scrub one range which can only
has simple mirror based profile.
b5b99b1e02969d Qu Wenruo 2022-01-07 3307 * (Including all range in
SINGLE/DUP/RAID1/RAID1C*, and each stripe in
b5b99b1e02969d Qu Wenruo 2022-01-07 3308 * RAID0/RAID10).
b5b99b1e02969d Qu Wenruo 2022-01-07 3309 *
b5b99b1e02969d Qu Wenruo 2022-01-07 3310 * Since we may need to handle a
subset of block group, we need @logical_start
b5b99b1e02969d Qu Wenruo 2022-01-07 3311 * and @logical_length parameter.
b5b99b1e02969d Qu Wenruo 2022-01-07 3312 */
b5b99b1e02969d Qu Wenruo 2022-01-07 3313 static int
scrub_simple_mirror(struct scrub_ctx *sctx,
b5b99b1e02969d Qu Wenruo 2022-01-07 3314 struct
btrfs_root *extent_root,
b5b99b1e02969d Qu Wenruo 2022-01-07 3315 struct
btrfs_root *csum_root,
b5b99b1e02969d Qu Wenruo 2022-01-07 3316 struct
btrfs_block_group *bg,
b5b99b1e02969d Qu Wenruo 2022-01-07 3317 struct
map_lookup *map,
b5b99b1e02969d Qu Wenruo 2022-01-07 3318 u64
logical_start, u64 logical_length,
b5b99b1e02969d Qu Wenruo 2022-01-07 3319 struct
btrfs_device *device,
b5b99b1e02969d Qu Wenruo 2022-01-07 3320 u64
physical, int mirror_num)
b5b99b1e02969d Qu Wenruo 2022-01-07 3321 {
b5b99b1e02969d Qu Wenruo 2022-01-07 3322 struct btrfs_fs_info *fs_info =
sctx->fs_info;
b5b99b1e02969d Qu Wenruo 2022-01-07 3323 const u64 logical_end =
logical_start + logical_length;
b5b99b1e02969d Qu Wenruo 2022-01-07 3324 /* An artificial limit, inherit
from old scrub behavior */
b5b99b1e02969d Qu Wenruo 2022-01-07 3325 const u32 max_length = SZ_64K;
b5b99b1e02969d Qu Wenruo 2022-01-07 3326 struct btrfs_path path = {};
b5b99b1e02969d Qu Wenruo 2022-01-07 3327 u64 cur_logical = logical_start;
b5b99b1e02969d Qu Wenruo 2022-01-07 3328 int ret;
b5b99b1e02969d Qu Wenruo 2022-01-07 3329
b5b99b1e02969d Qu Wenruo 2022-01-07 3330 /* The range must be inside the
bg */
b5b99b1e02969d Qu Wenruo 2022-01-07 3331 ASSERT(logical_start >=
bg->start &&
b5b99b1e02969d Qu Wenruo 2022-01-07 3332 logical_end <= bg->start
+ bg->length);
b5b99b1e02969d Qu Wenruo 2022-01-07
b5b99b1e02969d Qu Wenruo 2022-01-07 3334 path.search_commit_root = 1;
b5b99b1e02969d Qu Wenruo 2022-01-07 3335 path.skip_locking = 1;
b5b99b1e02969d Qu Wenruo 2022-01-07 3336 /* Go through each */
b5b99b1e02969d Qu Wenruo 2022-01-07 3337 while (cur_logical <
logical_end) {
b5b99b1e02969d Qu Wenruo 2022-01-07 3338 int cur_mirror =
mirror_num;
b5b99b1e02969d Qu Wenruo 2022-01-07 3339 struct btrfs_device
*target_dev = device;
b5b99b1e02969d Qu Wenruo 2022-01-07 3340 u64 extent_start;
b5b99b1e02969d Qu Wenruo 2022-01-07 3341 u64 extent_len;
b5b99b1e02969d Qu Wenruo 2022-01-07 3342 u64 extent_flags;
b5b99b1e02969d Qu Wenruo 2022-01-07 3343 u64 extent_gen;
b5b99b1e02969d Qu Wenruo 2022-01-07 3344 u64 scrub_len;
b5b99b1e02969d Qu Wenruo 2022-01-07 3345 u64 cur_physical;
b5b99b1e02969d Qu Wenruo 2022-01-07 3346
b5b99b1e02969d Qu Wenruo 2022-01-07 3347 /* Canceled ? */
b5b99b1e02969d Qu Wenruo 2022-01-07 3348 if
(atomic_read(&fs_info->scrub_cancel_req) ||
b5b99b1e02969d Qu Wenruo 2022-01-07 3349
atomic_read(&sctx->cancel_req)) {
b5b99b1e02969d Qu Wenruo 2022-01-07 3350 ret =
-ECANCELED;
b5b99b1e02969d Qu Wenruo 2022-01-07 3351 break;
b5b99b1e02969d Qu Wenruo 2022-01-07 3352 }
b5b99b1e02969d Qu Wenruo 2022-01-07 3353 /* Paused ? */
b5b99b1e02969d Qu Wenruo 2022-01-07 3354 if
(ato
[kbuild] drivers/rtc/rtc-lp8788.c:277:8-33: WARNING: Threaded IRQ with no primary handler requested without IRQF_ONESHOT (unless it is nested IRQ)
CC: kbuild-...@lists.01.org CC: linux-ker...@vger.kernel.org TO: Krzysztof Kozlowski CC: Julia Lawall tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8 commit: 5d2db9bb5f8a850d037983f0df72ad59cefa9e3d coccinelle: irqf_oneshot: reduce the severity due to false positives date: 9 months ago :: branch date: 18 hours ago :: commit date: 9 months ago config: x86_64-randconfig-c022-20220124 (https://download.01.org/0day-ci/archive/20220126/202201262026.pkq71lt4-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Julia Lawall cocci warnings: (new ones prefixed by >>) >> drivers/rtc/rtc-lp8788.c:277:8-33: WARNING: Threaded IRQ with no primary >> handler requested without IRQF_ONESHOT (unless it is nested IRQ) Please review and possibly fold the followup patch. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org ___ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] [PATCH] coccinelle: irqf_oneshot: fix irqf_oneshot.cocci warnings
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Krzysztof Kozlowski
CC: Julia Lawall
CC: Alessandro Zummo
CC: Alexandre Belloni
CC: linux-...@vger.kernel.org
CC: linux-ker...@vger.kernel.org
From: kernel test robot
drivers/rtc/rtc-lp8788.c:277:8-33: WARNING: Threaded IRQ with no primary
handler requested without IRQF_ONESHOT (unless it is nested IRQ)
Since commit 1c6c69525b40 ("genirq: Reject bogus threaded irq requests")
threaded IRQs without a primary handler need to be requested with
IRQF_ONESHOT, otherwise the request will fail.
So pass the IRQF_ONESHOT flag in this case.
Generated by: scripts/coccinelle/misc/irqf_oneshot.cocci
CC: Krzysztof Kozlowski
Reported-by: kernel test robot
Signed-off-by: kernel test robot
---
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8
commit: 5d2db9bb5f8a850d037983f0df72ad59cefa9e3d coccinelle: irqf_oneshot:
reduce the severity due to false positives
:: branch date: 18 hours ago
:: commit date: 9 months ago
Please take the patch only if it's a positive warning. Thanks!
rtc-lp8788.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/drivers/rtc/rtc-lp8788.c
+++ b/drivers/rtc/rtc-lp8788.c
@@ -276,7 +276,7 @@ static int lp8788_alarm_irq_register(str
return devm_request_threaded_irq(&pdev->dev, rtc->irq, NULL,
lp8788_alarm_irq_handler,
- 0, LP8788_ALM_IRQ, rtc);
+ IRQF_ONESHOT, LP8788_ALM_IRQ, rtc);
}
static int lp8788_rtc_probe(struct platform_device *pdev)
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org
[kbuild] [agd5f:drm-next 124/190] drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:877:51: warning: Uninitialized variable: reg_access_ctrl [uninitvar]
CC: kbuild-...@lists.01.org
CC: linux-ker...@vger.kernel.org
TO: Hawking Zhang
CC: Alex Deucher
CC: "Zhou, Peng Ju"
CC: Lijo Lazar
tree: https://gitlab.freedesktop.org/agd5f/linux.git drm-next
head: 63e583c8843f305a3e334a96d5de00435f4d38a2
commit: 5d447e296701484f3df5b31a7a078cbf1e3a9cc9 [124/190] drm/amdgpu: add
helper for rlcg indirect reg access
:: branch date: 13 hours ago
:: commit date: 13 hours ago
compiler: riscv32-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/dc_helper.c:
>> drivers/gpu/drm/amd/display/dc/dc_helper.c:450:48: warning: Function
>> 'generic_reg_wait' argument order different: declaration 'ctx, addr, mask,
>> shift, condition_value, delay_between_poll_us, time_out_num_tries,
>> func_name, line' definition 'ctx, addr, shift, mask, condition_value,
>> delay_between_poll_us, time_out_num_tries, func_name, line'
>> [funcArgOrderDifferent]
void generic_reg_wait(const struct dc_context *ctx,
^
drivers/gpu/drm/amd/display/dc/dm_services.h:140:48: note: Function
'generic_reg_wait' argument order different: declaration 'ctx, addr, mask,
shift, condition_value, delay_between_poll_us, time_out_num_tries, func_name,
line' definition 'ctx, addr, shift, mask, condition_value,
delay_between_poll_us, time_out_num_tries, func_name, line'
void generic_reg_wait(const struct dc_context *ctx,
^
drivers/gpu/drm/amd/display/dc/dc_helper.c:450:48: note: Function
'generic_reg_wait' argument order different: declaration 'ctx, addr, mask,
shift, condition_value, delay_between_poll_us, time_out_num_tries, func_name,
line' definition 'ctx, addr, shift, mask, condition_value,
delay_between_poll_us, time_out_num_tries, func_name, line'
void generic_reg_wait(const struct dc_context *ctx,
^
--
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:877:51: warning: Uninitialized
>> variable: reg_access_ctrl [uninitvar]
scratch_reg0 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg0;
^
drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:878:51: warning: Uninitialized
variable: reg_access_ctrl [uninitvar]
scratch_reg1 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg1;
^
drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:879:51: warning: Uninitialized
variable: reg_access_ctrl [uninitvar]
scratch_reg2 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg2;
^
drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:880:51: warning: Uninitialized
variable: reg_access_ctrl [uninitvar]
scratch_reg3 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg3;
^
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:877:51: warning: Uninitialized
>> struct member: reg_access_ctrl.scratch_reg0 [uninitStructMember]
scratch_reg0 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg0;
^
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:878:51: warning: Uninitialized
>> struct member: reg_access_ctrl.scratch_reg1 [uninitStructMember]
scratch_reg1 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg1;
^
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:879:51: warning: Uninitialized
>> struct member: reg_access_ctrl.scratch_reg2 [uninitStructMember]
scratch_reg2 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg2;
^
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:880:51: warning: Uninitialized
>> struct member: reg_access_ctrl.scratch_reg3 [uninitStructMember]
scratch_reg3 = (void __iomem *)adev->rmmio + 4 *
reg_access_ctrl->scratch_reg3;
^
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:884:16: warning: Uninitialized
>> struct member: reg_access_ctrl.grbm_cntl [uninitStructMember]
if (offset == reg_access_ctrl->grbm_cntl) {
^
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:888:23: warning: Uninitialized
>> struct member: reg_access_ctrl.grbm_idx [uninitStructMember]
} else if (offset == reg_access_ctrl->grbm_idx) {
^
>> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:881:6: warning: Uninitialized
>> struct member: reg_access_ctrl.spare_int [uninitStructMember]
if (reg_access_ctrl->spare_int)
^
drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c:882:49: warning: Uninitialized
struct member: reg_access_
[kbuild] Re: [PATCH 1/7] crypto: DRBG - remove internal reseeding operation
CC: kbuild-...@lists.01.org
In-Reply-To: <2450379.h6ri2rz...@positron.chronox.de>
References: <2450379.h6ri2rz...@positron.chronox.de>
TO: "Stephan Müller"
TO: herb...@gondor.apana.org.au
CC: linux-cry...@vger.kernel.org
CC: s...@redhat.com
CC: Nicolai Stange
Hi "Stephan,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on herbert-cryptodev-2.6/master]
[also build test WARNING on herbert-crypto-2.6/master linus/master v5.17-rc1
next-20220125]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url:
https://github.com/0day-ci/linux/commits/Stephan-M-ller/Common-entropy-source-and-DRNG-management/20220126-150911
base:
https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git master
:: branch date: 5 hours ago
:: commit date: 5 hours ago
config: arc-randconfig-c004-20220124
(https://download.01.org/0day-ci/archive/20220126/202201261919.fuyjinho-...@intel.com/config)
compiler: arc-elf-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
Reported-by: Julia Lawall
cocci warnings: (new ones prefixed by >>)
>> crypto/drbg.c:1315:3-9: preceding lock on line 1303
crypto/drbg.c:1322:3-9: preceding lock on line 1303
vim +1315 crypto/drbg.c
57225e6797885e Stephan Mueller 2015-06-09 1277
541af946fe1360 Stephan Mueller 2014-05-31 1278 /*
541af946fe1360 Stephan Mueller 2014-05-31 1279 * DRBG instantiation function
as required by SP800-90A - this function
14ec08bbd20e04 Stephan Müller 2022-01-26 1280 * sets up the DRBG handle if
needed and seeds the DRBG with entropy. If the
14ec08bbd20e04 Stephan Müller 2022-01-26 1281 * DRBG is already
instantiated, the DRBG is simply reseeded.
541af946fe1360 Stephan Mueller 2014-05-31 1282 *
14ec08bbd20e04 Stephan Müller 2022-01-26 1283 * @tfm: tfm cipher handle
with DRBG state (may be uninitialized)
14ec08bbd20e04 Stephan Müller 2022-01-26 1284 * @seed: buffer with the
entropy data to (re)seed the DRBG
14ec08bbd20e04 Stephan Müller 2022-01-26 1285 * @slen: length of seed buffer
541af946fe1360 Stephan Mueller 2014-05-31 1286 *
541af946fe1360 Stephan Mueller 2014-05-31 1287 * return
541af946fe1360 Stephan Mueller 2014-05-31 1288 * 0 on success
541af946fe1360 Stephan Mueller 2014-05-31 1289 * error value otherwise
541af946fe1360 Stephan Mueller 2014-05-31 1290 */
14ec08bbd20e04 Stephan Müller 2022-01-26 1291 static int
drbg_instantiate(struct crypto_rng *tfm,
14ec08bbd20e04 Stephan Müller 2022-01-26 1292
const u8 *seed, unsigned int slen)
541af946fe1360 Stephan Mueller 2014-05-31 1293 {
14ec08bbd20e04 Stephan Müller 2022-01-26 1294 struct drbg_state *drbg
= crypto_rng_ctx(tfm);
14ec08bbd20e04 Stephan Müller 2022-01-26 1295 struct drbg_string
seeddata;
14ec08bbd20e04 Stephan Müller 2022-01-26 1296 LIST_HEAD(seedlist);
2a57e4241ec9a1 Herbert Xu 2015-04-20 1297 int ret;
2a57e4241ec9a1 Herbert Xu 2015-04-20 1298 bool reseed = true;
541af946fe1360 Stephan Mueller 2014-05-31 1299
14ec08bbd20e04 Stephan Müller 2022-01-26 1300
drbg_string_fill(&seeddata, seed, slen);
14ec08bbd20e04 Stephan Müller 2022-01-26 1301
list_add_tail(&seeddata.list, &seedlist);
14ec08bbd20e04 Stephan Müller 2022-01-26 1302
76899a41f830d1 Stephan Mueller 2015-04-18 @1303
mutex_lock(&drbg->drbg_mutex);
541af946fe1360 Stephan Mueller 2014-05-31 1304
541af946fe1360 Stephan Mueller 2014-05-31 1305 /* 9.1 step 1 is
implicit with the selected DRBG type */
14ec08bbd20e04 Stephan Müller 2022-01-26 1306 /* 9.1 step 2 is
implicit as no prediction resistance is supported */
541af946fe1360 Stephan Mueller 2014-05-31 1307 /* 9.1 step 4 is
implicit in drbg_sec_strength */
541af946fe1360 Stephan Mueller 2014-05-31 1308
2a57e4241ec9a1 Herbert Xu 2015-04-20 1309 if (!drbg->core) {
14ec08bbd20e04 Stephan Müller 2022-01-26 1310 struct
crypto_tfm *tfm_base = crypto_rng_tfm(tfm);
14ec08bbd20e04 Stephan Müller 2022-01-26 1311 int coreref = 0;
14ec08bbd20e04 Stephan Müller 2022-01-26 1312
14ec08bbd20e04 Stephan Müller 2022-01-26 1313 if (!slen) {
14ec08bbd20e04 Stephan Müller 2022-01-26 1314
pr_warn("DRBG: initial seed missing\n");
14ec08bbd20e04 Stephan Müller 2022-01-26 @1315 return
-EINVAL;
14ec08bbd20e04 Stephan Müller 2022-01-26 1316 }
14ec08bbd20e04 Stephan Müller 2022-01-26 1317
14ec08bbd20e04 Stephan Müller 2022-01-26 1318 pr_devel("DRBG:
Initializing DRBG core %d\n", coreref);
1
[kbuild] net/mptcp/ctrl.c:89:3: warning: Argument to kfree() is the address of the global variable 'mptcp_sysctl_table', which is not memory allocated by malloc() [clang-analyzer-unix.Malloc]
CC: l...@lists.linux.dev CC: kbuild-...@lists.01.org CC: linux-ker...@vger.kernel.org TO: Andy Shevchenko CC: Pavel Machek tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8 commit: 2cbbe9c50d13b6417e0baf8e8475ed73d4d12c2d leds: lgm-sso: Remove unneeded of_match_ptr() date: 8 months ago :: branch date: 17 hours ago :: commit date: 8 months ago config: x86_64-randconfig-c007-20220124 (https://download.01.org/0day-ci/archive/20220126/202201261913.99wp2cm3-...@intel.com/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 997e128e2a78f5a5434fc75997441ae1ee76f8a4) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2cbbe9c50d13b6417e0baf8e8475ed73d4d12c2d git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout 2cbbe9c50d13b6417e0baf8e8475ed73d4d12c2d # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 clang-analyzer If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot clang-analyzer warnings: (new ones prefixed by >>) drivers/most/configfs.c:446:2: note: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119 strcpy(mdev_link->name, name); ^~ drivers/most/configfs.c:535:2: warning: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119 [clang-analyzer-security.insecureAPI.strcpy] strcpy(mdev_link->name, name); ^~ drivers/most/configfs.c:535:2: note: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119 strcpy(mdev_link->name, name); ^~ 3 warnings generated. fs/cifs/smb2ops.c:4020:3: warning: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 [clang-analyzer-security.insecureAPI.strcpy] strcat(message, "R"); ^~ fs/cifs/smb2ops.c:4020:3: note: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 strcat(message, "R"); ^~ fs/cifs/smb2ops.c:4024:3: warning: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 [clang-analyzer-security.insecureAPI.strcpy] strcat(message, "H"); ^~ fs/cifs/smb2ops.c:4024:3: note: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 strcat(message, "H"); ^~ fs/cifs/smb2ops.c:4028:3: warning: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 [clang-analyzer-security.insecureAPI.strcpy] strcat(message, "W"); ^~ fs/cifs/smb2ops.c:4028:3: note: Call to function 'strcat' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcat'. CWE-119 strcat(message, "W"); ^~ 4 warnings generated. net/sunrpc/svcauth_unix.c:131:2: warning: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions
[kbuild] fs/ext4/super.c:5649:3: warning: Value stored to 'ret' is never read [clang-analyzer-deadcode.DeadStores]
CC: l...@lists.linux.dev CC: kbuild-...@lists.01.org CC: linux-ker...@vger.kernel.org TO: Lukas Czerner CC: "Theodore Ts'o" CC: Carlos Maiolino tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0280e3c58f92b2fe0e8fbbdf8d386449168de4a8 commit: cebe85d570cf84804e848332d6721bc9e5300e07 ext4: switch to the new mount api date: 7 weeks ago :: branch date: 14 hours ago :: commit date: 7 weeks ago config: x86_64-randconfig-c007 (https://download.01.org/0day-ci/archive/20220126/202201261659.dzzorgxb-...@intel.com/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project f7b7138a62648f4019c55e4671682af1f851f295) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=cebe85d570cf84804e848332d6721bc9e5300e07 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout cebe85d570cf84804e848332d6721bc9e5300e07 # save the config file to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 clang-analyzer If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot clang-analyzer warnings: (new ones prefixed by >>) ^ net/core/filter.c:1760:15: note: Assuming field 'skb' is non-null if (unlikely(!ctx->skb)) ^ include/linux/compiler.h:78:42: note: expanded from macro 'unlikely' # define unlikely(x)__builtin_expect(!!(x), 0) ^ net/core/filter.c:1760:2: note: Taking false branch if (unlikely(!ctx->skb)) ^ net/core/filter.c:1764:15: note: Assuming 'ptr' is null if (unlikely(!ptr)) ^ include/linux/compiler.h:78:42: note: expanded from macro 'unlikely' # define unlikely(x)__builtin_expect(!!(x), 0) ^ net/core/filter.c:1764:2: note: Taking true branch if (unlikely(!ptr)) ^ net/core/filter.c:1765:3: note: Control jumps to line 1771 goto err_clear; ^ net/core/filter.c:1771:2: note: Null pointer passed as 1st argument to memory set function memset(to, 0, len); ^ ~~ Suppressed 11 warnings (11 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 10 warnings generated. Suppressed 10 warnings (10 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 15 warnings generated. net/mac80211/ibss.c:934:6: warning: Value stored to 'reason' during its initialization is never read [clang-analyzer-deadcode.DeadStores] u16 reason = le16_to_cpu(mgmt->u.deauth.reason_code); ^~ net/mac80211/ibss.c:934:6: note: Value stored to 'reason' during its initialization is never read u16 reason = le16_to_cpu(mgmt->u.deauth.reason_code); ^~ Suppressed 14 warnings (14 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 14 warnings generated. Suppressed 14 warnings (14 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 14 warnings generated. Suppressed 14 warnings (14 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 9 warnings generated. Suppressed 9 warnings (9 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 5 warnings generated. Suppressed 5 warnings (5 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 4 warnings generated. Suppressed 4 warnings (4 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 4 warnings generated. Suppressed 4 warnings (4 in non-user code). Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well. 5 warnings generated.
