Hi Dan,
On Tue, Jan 19, 2021 at 11:29:37AM +0300, Dan Carpenter wrote:
Hi Imre,
url:
https://github.com/0day-ci/linux/commits/Imre-Deak/drm-i915-gen12-Add-dis...
...
d5de4516a9de74 drivers/gpu/drm/i915/display/intel_display.c Radhakrishna Sripada
2021-01-14 15296 */
d5de4516a9de74 drivers/gpu/drm/i915/display/intel_display.c Radhakrishna Sripada
2021-01-14 15297 ret = i915_gem_object_read_from_page(obj,
d5de4516a9de74 drivers/gpu/drm/i915/display/intel_display.c Radhakrishna Sripada
2021-01-14 15298 fb->offsets[2] + 16,
d5de4516a9de74 drivers/gpu/drm/i915/display/intel_display.c Radhakrishna Sripada
2021-01-14 @15299 sizeof(&plane_state->ccval),
d5de4516a9de74 drivers/gpu/drm/i915/display/intel_display.c Radhakrishna Sripada
2021-01-14 15300 &plane_state->ccval);
This is a bug in Smatch and it's quite complicated to fix... But the
sizeof() is still probably wrong. It should be the size of the u64
instead of the address.
Yes, thanks for the note, I typoed the sizeof(). It's fixed in:
https://patchwork.freedesktop.org/patch/414552/?series=85877&rev=4
--Imre