diff for duplicates of <202609081301.6ARYAElY-lkp@intel.com> diff --git a/a/1.txt b/N1/1.txt index 383025a..928a373 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,15 +1,6 @@ -BCC: lkp@intel.com -CC: oe-kbuild-all@lists.linux.dev -CC: linux-kernel@vger.kernel.org -TO: Jason Gunthorpe <jgg@nvidia.com> -CC: "Christian König" <christian.koenig@amd.com> - tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 28924df2a08f440c73991b83028032c901de2ae4 commit: 6055c9e333cfbb5af3eabe204caea92757094d19 dma-buf: Change st-dma-fence-chain.c to use kunit -date: 5 months ago -:::::: branch date: 13 hours ago -:::::: commit date: 5 months ago config: arm-randconfig-r071-20260908 (https://download.01.org/0day-ci/archive/20260908/202609081301.6ARYAElY-lkp@intel.com/config) compiler: clang version 24.0.0git (https://github.com/llvm/llvm-project 2362eeb5f75560740146975e9eabe84c6bd25a0d) smatch: v0.5.0-9187-g5189e3fb @@ -34,7 +25,6 @@ drivers/dma-buf/st-dma-fence-chain.c:346 test_find_gap() warn: passing freed mem vim +/fence +207 drivers/dma-buf/st-dma-fence-chain.c -dc2f7e67a28a5c Chris Wilson 2020-04-09 173 6055c9e333cfbb Jason Gunthorpe 2026-03-01 174 static void test_find_seqno(struct kunit *test) dc2f7e67a28a5c Chris Wilson 2020-04-09 175 { dc2f7e67a28a5c Chris Wilson 2020-04-09 176 struct fence_chains fc; @@ -48,6 +38,16 @@ dc2f7e67a28a5c Chris Wilson 2020-04-09 183 dc2f7e67a28a5c Chris Wilson 2020-04-09 184 fence = dma_fence_get(fc.tail); dc2f7e67a28a5c Chris Wilson 2020-04-09 185 err = dma_fence_chain_find_seqno(&fence, 0); dc2f7e67a28a5c Chris Wilson 2020-04-09 186 dma_fence_put(fence); + +The zero day bot is warning about this code because it moved, but +calling dma_fence_put() and then calling dma_fence_get() doesn't +make much sense. Why even bother with refcounting it at that point +if we know it's unnecessary? + +fence is the same as fc.tail... + +I don't think there is any cleanup.h magic for dma fences? + dc2f7e67a28a5c Chris Wilson 2020-04-09 187 if (err) { 6055c9e333cfbb Jason Gunthorpe 2026-03-01 188 KUNIT_FAIL(test, "Reported %d for find_seqno(0)!", err); dc2f7e67a28a5c Chris Wilson 2020-04-09 189 goto err; @@ -225,12 +225,6 @@ dc2f7e67a28a5c Chris Wilson 2020-04-09 360 fence_chains_fini(&fc); dc2f7e67a28a5c Chris Wilson 2020-04-09 361 } dc2f7e67a28a5c Chris Wilson 2020-04-09 362 -:::::: The code at line 207 was first introduced by commit -:::::: dc2f7e67a28a5cbe4ea905dafaa6903df670014a dma-buf: Exercise dma-fence-chain under selftests - -:::::: TO: Chris Wilson <chris@chris-wilson.co.uk> -:::::: CC: Chris Wilson <chris@chris-wilson.co.uk> - -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki diff --git a/a/content_digest b/N1/content_digest index 6fdb3ce..cec5832 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,23 +1,17 @@ - "From\0kernel test robot <lkp@intel.com>\0" + "From\0Dan Carpenter <error27@gmail.com>\0" "Subject\0drivers/dma-buf/st-dma-fence-chain.c:207 test_find_seqno() warn: passing freed memory 'fence' (line 195)\0" - "Date\0Tue, 08 Sep 2026 14:06:27 +0800\0" - "To\0oe-kbuild@lists.linux.dev\0" + "Date\0Thu, 10 Sep 2026 17:15:00 +0300\0" + "To\0oe-kbuild@lists.linux.dev" + " Jason Gunthorpe <jgg@nvidia.com>\0" "Cc\0lkp@intel.com" - " Dan Carpenter <error27@gmail.com>\0" + oe-kbuild-all@lists.linux.dev + linux-kernel@vger.kernel.org + " Christian K\303\266nig <christian.koenig@amd.com>\0" "\00:1\0" "b\0" - "BCC: lkp@intel.com\n" - "CC: oe-kbuild-all@lists.linux.dev\n" - "CC: linux-kernel@vger.kernel.org\n" - "TO: Jason Gunthorpe <jgg@nvidia.com>\n" - "CC: \"Christian K\303\266nig\" <christian.koenig@amd.com>\n" - "\n" "tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master\n" "head: 28924df2a08f440c73991b83028032c901de2ae4\n" "commit: 6055c9e333cfbb5af3eabe204caea92757094d19 dma-buf: Change st-dma-fence-chain.c to use kunit\n" - "date: 5 months ago\n" - ":::::: branch date: 13 hours ago\n" - ":::::: commit date: 5 months ago\n" "config: arm-randconfig-r071-20260908 (https://download.01.org/0day-ci/archive/20260908/202609081301.6ARYAElY-lkp@intel.com/config)\n" "compiler: clang version 24.0.0git (https://github.com/llvm/llvm-project 2362eeb5f75560740146975e9eabe84c6bd25a0d)\n" "smatch: v0.5.0-9187-g5189e3fb\n" @@ -42,7 +36,6 @@ "\n" "vim +/fence +207 drivers/dma-buf/st-dma-fence-chain.c\n" "\n" - "dc2f7e67a28a5c Chris Wilson 2020-04-09 173 \n" "6055c9e333cfbb Jason Gunthorpe 2026-03-01 174 static void test_find_seqno(struct kunit *test)\n" "dc2f7e67a28a5c Chris Wilson 2020-04-09 175 {\n" "dc2f7e67a28a5c Chris Wilson 2020-04-09 176 \tstruct fence_chains fc;\n" @@ -56,6 +49,16 @@ "dc2f7e67a28a5c Chris Wilson 2020-04-09 184 \tfence = dma_fence_get(fc.tail);\n" "dc2f7e67a28a5c Chris Wilson 2020-04-09 185 \terr = dma_fence_chain_find_seqno(&fence, 0);\n" "dc2f7e67a28a5c Chris Wilson 2020-04-09 186 \tdma_fence_put(fence);\n" + "\n" + "The zero day bot is warning about this code because it moved, but\n" + "calling dma_fence_put() and then calling dma_fence_get() doesn't\n" + "make much sense. Why even bother with refcounting it at that point\n" + "if we know it's unnecessary?\n" + "\n" + "fence is the same as fc.tail...\n" + "\n" + "I don't think there is any cleanup.h magic for dma fences?\n" + "\n" "dc2f7e67a28a5c Chris Wilson 2020-04-09 187 \tif (err) {\n" "6055c9e333cfbb Jason Gunthorpe 2026-03-01 188 \t\tKUNIT_FAIL(test, \"Reported %d for find_seqno(0)!\", err);\n" "dc2f7e67a28a5c Chris Wilson 2020-04-09 189 \t\tgoto err;\n" @@ -233,14 +236,8 @@ "dc2f7e67a28a5c Chris Wilson 2020-04-09 361 }\n" "dc2f7e67a28a5c Chris Wilson 2020-04-09 362 \n" "\n" - ":::::: The code at line 207 was first introduced by commit\n" - ":::::: dc2f7e67a28a5cbe4ea905dafaa6903df670014a dma-buf: Exercise dma-fence-chain under selftests\n" - "\n" - ":::::: TO: Chris Wilson <chris@chris-wilson.co.uk>\n" - ":::::: CC: Chris Wilson <chris@chris-wilson.co.uk>\n" - "\n" "--\n" "0-DAY CI Kernel Test Service\n" https://github.com/intel/lkp-tests/wiki -29061c212e10303cd4a20d3ff5a91c2577c7a1c4cae7a4a26d7ddafead6b72a5 +591ff8f6c83379b00c2ec38b4051047035d3217c9c7b6a8c6eee722d24527793
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.