You can consider using max rather than ?:
See lines 178 and 180.
julia
-- Forwarded message --
Date: Wed, 2 Oct 2024 11:11:36 +0800
From: kernel test robot
To: oe-kbu...@lists.linux.dev
Cc: l...@intel.com, Julia Lawall
Subject: Re: [PATCHv2] drm/i915/dp: Guarantee a minimum
On Wed, 20 Mar 2024, Jani Nikula wrote:
> On Wed, 20 Mar 2024, Julia Lawall wrote:
> > On Wed, 20 Mar 2024, Jani Nikula wrote:
> >> Okay, I have another one wrt macros. :)
> >>
> >> I'm trying to add a completely new variadic macro, but it fails at
&
On Wed, 20 Mar 2024, Jani Nikula wrote:
> On Wed, 20 Mar 2024, Julia Lawall wrote:
> > On Wed, 20 Mar 2024, Jani Nikula wrote:
> >> Okay, I have another one wrt macros. :)
> >>
> >> I'm trying to add a completely new variadic macro, but it fails at
&
On Wed, 20 Mar 2024, Jani Nikula wrote:
> On Wed, 20 Mar 2024, Ville Syrjälä wrote:
> > On Wed, Mar 20, 2024 at 02:24:08PM +0100, Julia Lawall wrote:
> >>
> >>
> >> On Wed, 20 Mar 2024, Ville Syrjälä wrote:
> >>
> >> > Hi Julia et al
On Wed, 20 Mar 2024, Ville Syrjälä wrote:
> Hi Julia et al,
>
> In Linux drm/i915 driver (drivers/gpu/drm/i915/display/intel_pps.[ch])
> we have a magic macro like this:
>
> #define with_intel_pps_lock(dp, wf) \
Did you try declaring:
iterator name with_intel_pps_lock;
up with the metavariabl
On Wed, 12 Jul 2023, Uwe Kleine-König wrote:
> On Wed, Jul 12, 2023 at 12:46:33PM +0200, Christian König wrote:
> > Am 12.07.23 um 11:46 schrieb Uwe Kleine-König:
> > > Hello,
> > >
> > > while I debugged an issue in the imx-lcdc driver I was constantly
> > > irritated about struct drm_device po
t:ocaml(alloc) { rename alloc };
@@
(
alloc(x1*x2*x3)
|
alloc(C1 * C2)
|
alloc((sizeof(t)) * (COUNT), ...)
|
- alloc((e1) * (e2))
+ realloc(e1, e2)
|
- alloc((e1) * (COUNT))
+ realloc(COUNT, e1)
|
- alloc((E1) * (E2))
+ realloc(E1, E2)
)
//
Signed-o
The functions vmalloc_array and vcalloc were introduced in
commit a8749a35c399 ("mm: vmalloc: introduce array allocation functions")
but are not used much yet. This series introduces uses of
these functions, to protect against multiplication overflows.
The changes were done using the following
)
+ array_size(E1, E2)
,...)
)
//
Signed-off-by: Julia Lawall
---
drivers/gpu/drm/i915/gvt/gtt.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt/gtt.c b/drivers/gpu/drm/i915/gvt/gtt.c
index 4ec85308379a..df52385ad436 100644
Use array_size to protect against multiplication overflows.
This follows up on the following patches by Kees Cook from 2018.
42bc47b35320 ("treewide: Use array_size() in vmalloc()")
fad953ce0b22 ("treewide: Use array_size() in vzalloc()")
The changes were done using the following Coccinelle sema
> Hi Julia, thanks for helping here.
>
> So, my question is why this
>
> make coccicheck M=drivers/gpu/drm/i915/ MODE=context
> COCCI=./scripts/coccinelle/api/debugfs/debugfs_simple_attr.cocci
>
> didn't catch this chunck:
>
> - debugfs_create_file("i915_fbc_false_color", 0644, parent,
On Tue, 3 Jan 2023, Deepak R Varma wrote:
> On Wed, Dec 28, 2022 at 06:18:12AM -0500, Rodrigo Vivi wrote:
> > On Tue, Dec 27, 2022 at 11:36:13PM +0530, Deepak R Varma wrote:
> > > On Tue, Dec 27, 2022 at 12:13:56PM -0500, Rodrigo Vivi wrote:
> > > > On Tue, Dec 27, 2022 at 01:30:53PM +0530, Dee
Spelling mistakes (triple letters) in comments.
Detected with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
drivers/gpu/drm/i915/display/intel_color.c |2 +-
drivers/gpu/drm/i915/display/intel_pps.c |2 +-
drivers/gpu/drm/i915/gt
On Mon, 21 Mar 2022, Ville Syrjälä wrote:
> On Wed, Mar 16, 2022 at 10:00:06AM +0200, Jani Nikula wrote:
> > On Fri, 18 Feb 2022, Ville Syrjala wrote:
> > > From: Ville Syrjälä
> > >
> > > Initialize on-stack modes with drm_mode_init() to guarantee
> > > no stack garbage in the list head, or t
iomem, because vma has
already been tested.
julia
-- Forwarded message --
Date: Sat, 11 Sep 2021 09:14:18 +0800
From: kernel test robot
To: kbu...@lists.01.org
Cc: l...@intel.com, Julia Lawall
Subject: Re: [Intel-gfx] [PATCH] drm/i915: Add ww context to intel_dpt_pin
CC: k
See line 81.
julia
-- Forwarded message --
Date: Sun, 15 Nov 2020 06:48:16 +0800
From: kernel test robot
To: kbu...@lists.01.org
Cc: l...@intel.com, Julia Lawall
Subject: Re: [Intel-gfx] [PATCH 18/27] drm/i915/pxp: Implement funcs to create
the TEE channel
CC: kbuild
Hello,
Line 203 is clearly incorrect. Please check lines 97, 123, and 171 as
well.
julia
-- Forwarded message --
Date: Sun, 15 Nov 2020 01:12:07 +0800
From: kernel test robot
To: kbu...@lists.01.org
Cc: l...@intel.com, Julia Lawall
Subject: Re: [Intel-gfx] [PATCH 08/27] drm
Hello,
There is a problem on line 21 if i915 can actually be NULL.
julia
-- Forwarded message --
Date: Sat, 14 Nov 2020 19:03:47 +0800
From: kernel test robot
To: kbu...@lists.01.org
Cc: l...@intel.com, Julia Lawall
Subject: Re: [Intel-gfx] [PATCH 05/27] drm/i915/pxp: Enable
Hello,
It looks like an unlock is missing on line 1279.
julia
-- Forwarded message --
Date: Sun, 14 Apr 2019 03:36:18 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [PATCH] RFT drm/i915: Try setting the engine wa_list
from the
On Wed, 28 Nov 2018, Manasi Navare wrote:
> On Wed, Nov 28, 2018 at 11:46:26AM +0000, Julia Lawall wrote:
> > Hello,
> >
> > row_index and column_index are unsigned, so in the last line shown
> > they will not be less than 0.
> >
>
> Row_index and column_i
Hello,
row_index and column_index are unsigned, so in the last line shown
they will not be less than 0.
julia
-- Forwarded message --
Date: Wed, 28 Nov 2018 19:43:30 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [CI v12 10/23] drm
On Fri, 26 Oct 2018, Rodrigo Vivi wrote:
> On Thu, Oct 25, 2018 at 12:11:57PM +0100, Julia Lawall wrote:
> >
> >
> > On Thu, 25 Oct 2018, Ville Syrjälä wrote:
> >
> > > On Wed, Oct 24, 2018 at 04:41:06PM -0700, Rodrigo Vivi wrote:
> > > > On Wed,
On Thu, 25 Oct 2018, Ville Syrjälä wrote:
> On Wed, Oct 24, 2018 at 04:41:06PM -0700, Rodrigo Vivi wrote:
> > On Wed, Oct 24, 2018 at 01:22:57PM +0300, Ville Syrjälä wrote:
> > > On Tue, Oct 23, 2018 at 04:36:19PM -0700, Rodrigo Vivi wrote:
> > > > Whenever possible we should stick with IS_GEN c
The containing function is called with a spin_lock held, so GFP_KERNEL
can't be used.
julia
-- Forwarded message --
Date: Tue, 23 Oct 2018 17:14:25 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: [PATCH] drm: fix call_kern.cocci warnings
CC: k
> I wish it would look something more like this:
>
> @@
> identifier old=~ "^INTEL_GEN$";
> expression exp;
> constant gen;
> fresh identifier new = "IS_GEN" ## gen;
> @@
> - old(exp) == gen
> + new(exp)
>
> But coccinelle doesn't seem to accept the constant in the fresh
> identifier thing.
+0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [PATCH 1/6] drm/dp_mst: Introduce
drm_dp_mst_connector_atomic_check()
CC: kbuild-...@01.org
In-Reply-To: <20180918230637.20700-2-ly...@redhat.com>
References: <20180918230637.20700-2-ly...@redhat.com>
T
Level is unsigned, so not less than 0 (line 230).
julia
-- Forwarded message --
Date: Tue, 20 Mar 2018 16:39:16 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: [drm-intel:drm-intel-next-queued 13/14]
drivers/gpu/drm/i915/intel_guc.c:230:12-17
On Wed, 20 Dec 2017, Joe Perches wrote:
> On Wed, 2017-12-20 at 10:59 +0100, Greg Kroah-Hartman wrote:
> > > > Why you didn't send that patch to the sysfs maintainer is a bit odd...
> > > > :)
> > >
> > > So here's an opportunity for you:
> > >
> > > The sysfs maintainer hasn't added include/l
An unlock may be missing before line 2568.
julia
-- Forwarded message --
Date: Fri, 30 Jun 2017 01:26:56 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [PATCH 10/13] drm/fb-helper: Support deferred setup
CC: kbuild-...@01.org
In-Reply
Hello,
There are two tests on IS_KBL_ULT on line 423. Please check on whether
this is intentional.
thanks,
julia
-- Forwarded message --
Date: Mon, 12 Jun 2017 01:20:56 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [PATCH] drm/i915
It looks like &udev->cmdr_lock should be released at line 512 if it has
not been released otherwise. The lock was taken at line 438.
julia
-- Forwarded message --
Date: Fri, 14 Apr 2017 22:21:44 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subj
The kfree on line 926 would only be a problem for the references to e on
lines 933 and 937 if the return value of drm_event_reserve_init can be
-EDEADLK.
julia
-- Forwarded message --
Date: Thu, 30 Mar 2017 00:48:54 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia
Remove unneeded variable used to store return value.
Generated by: scripts/coccinelle/misc/returnvar.cocci
CC: Jerome Anand
Signed-off-by: Julia Lawall
Signed-off-by: Fengguang Wu
---
In-Reply-To: <20170124225753.9045-5-jerome.an...@intel.com>
url:
https://github.com/0day-ci/linux/c
Line 427 is under the if on line 422 and the subsequent lines are not. It
is not clear at all what is intended.
julia
-- Forwarded message --
Date: Fri, 13 Jan 2017 02:13:07 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [GLK MIPI DSI
An extra tab would help avoid misunderstandings.
julia
-- Forwarded message --
Date: Thu, 4 Aug 2016 19:33:05 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [PATCH] drm/i915: Disable stolen on 865G
CC: kbuild-...@01.org
In-Reply-To
It looks like lines 1133 and 1134 should be indented more.
julia
-- Forwarded message --
Date: Sat, 18 Jun 2016 01:21:21 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [PATCH] drm/i915/gen9: Add WaInPlaceDecompressionHang
CC: kbuild
On Tue, 12 Apr 2016, Emil Velikov wrote:
> On 30 March 2016 at 10:51, Daniel Vetter wrote:
> > No need to confuse userspace like this.
> >
> > Cc: Gerd Hoffmann
> > Cc: Dave Airlie
> > Signed-off-by: Daniel Vetter
> > ---
> > drivers/gpu/drm/virtio/virtgpu_display.c | 9 -
> > 1 fil
On Mon, 18 Jan 2016, Daniel Vetter wrote:
> On Mon, Jan 18, 2016 at 04:49:06PM +0100, Julia Lawall wrote:
> > List_for_each entry binds its first argument to an offset from the list
> > pointer, so this should not be NULL.
> >
> > Generated by: scripts/coccin
List_for_each entry binds its first argument to an offset from the list
pointer, so this should not be NULL.
Generated by: scripts/coccinelle/iterators/itnull.cocci
Signed-off-by: Fengguang Wu
---
Please take the patch only if it's a positive warning. Thanks!
intel_display.c |2 +-
1 file
Please check on line 690.
julia
On Fri, 8 Jan 2016, kbuild test robot wrote:
> CC: kbuild-...@01.org
> In-Reply-To:
> <1452252592-24803-9-git-send-email-tvrtko.ursu...@linux.intel.com>
> TO: Tvrtko Ursulin
> CC: Intel-gfx@lists.freedesktop.org
> CC:
>
> Hi Tvrtko,
>
> [auto build test WARNING
The drm_dp_mst_topology_cbs structures are never modified, so declare them
as const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
drivers/gpu/drm/i915/intel_dp_mst.c|2 +-
drivers/gpu/drm/radeon/radeon_dp_mst.c |2 +-
include/drm/drm_dp_mst_helper.h
It looks like at least the call to drm_dp_dpcd_writeb should be indented.
Please check.
julia
-- Forwarded message --
Date: Wed, 16 Dec 2015 07:15:44 +0800
From: kbuild test robot
To: kbu...@01.org
Cc: Julia Lawall
Subject: Re: [Intel-gfx] [PATCH] drm/i915: Add Backlight
The intel_dvo_dev_ops structures are never modified, so declare them as
const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
drivers/gpu/drm/i915/dvo.h| 12 ++--
drivers/gpu/drm/i915/dvo_ch7017.c |2 +-
drivers/gpu/drm/i915/dvo_ch7xxx.c |2
On Fri, 2 Oct 2015, Daniel Vetter wrote:
> We chase pointers/lists without taking the locks protecting them,
> which isn't that good.
>
> Fix it.
>
> v2: Actually unlock properly, spotted by Julia.
The unlock is still on top of the unlock label?
julia
>
> Cc:
Maybe line 294 should become an unlock and should be moved under 295?
julia
On Tue, 29 Sep 2015, kbuild test robot wrote:
> CC: kbuild-...@01.org
> In-Reply-To: <1443513993-5228-2-git-send-email-daniel.vet...@ffwll.ch>
> TO: Daniel Vetter
> CC: DRI Development
> CC: Daniel Vetter , Intel Graph
Recent commits to kernel/git/torvalds/linux.git have made the following
functions able to tolerate NULL arguments:
kmem_cache_destroy (commit 3942d29918522)
mempool_destroy (commit 4e3ca3e033d1)
dma_pool_destroy (commit 44d7175da6ea)
These patches remove the associated NULL tests for the files th
Remove unneeded NULL test.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@ expression x; @@
-if (x != NULL)
\(kmem_cache_destroy\|mempool_destroy\|dma_pool_destroy\)(x);
//
Signed-off-by: Julia Lawall
---
drivers/gpu/drm/i915/i915_dma.c | 19
On Mon, 8 Dec 2014, Julian Calaby wrote:
> Hi Julia,
>
> On Mon, Dec 8, 2014 at 6:20 AM, Julia Lawall wrote:
> > These patches replace what appears to be a reference to the name of the
> > current function but is misspelled in some way by either the name of the
> > fu
Replace a misspelled function name by %s and then __func__.
This was done using Coccinelle, including the use of Levenshtein distance,
as proposed by Rasmus Villemoes.
Signed-off-by: Julia Lawall
---
The semantic patch is difficult to summarize, but is available in the cover
letter of this
These patches replace what appears to be a reference to the name of the
current function but is misspelled in some way by either the name of the
function itself, or by %s and then __func__ in an argument list.
//
// sudo apt-get install python-pip
// sudo pip install python-Levenshtein
// spatch
. This keeps static analysers happy
> and the reader sane.
>
> Suggested-by: Julia Lawall
> Signed-off-by: Chris Wilson
> Cc: Julia Lawall
> ---
> drivers/gpu/drm/i915/i915_gem_userptr.c | 17 -
> 1 file changed, 8 insertions(+), 9 deletions(-)
>
&g
It's not actually Dan's coccinelle checker... He is just the (much
appreciated) messenger.
julia
On Mon, 5 Aug 2013, Daniel Vetter wrote:
Thanks for the report, I'll squash in the following fixup:
commit 36bf63c19eb46631fa4556ae54d91a5320a198b2
Author: Daniel Vetter
Date: Mon Aug 5 12:28
52 matches
Mail list logo