search for: kunit

...est.c?????? | 16 +++++++----- > > > -- > > > - > > > ? .../gpu/drm/ttm/tests/ttm_bo_validate_test.c? |? 3 ++- > > > ? drivers/gpu/drm/ttm/tests/ttm_device_test.c?? | 16 ++++++++---- > > > -- > > > - > > > ? drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.c | 20 ++++++++---- > > > -- > > > ----- > > > ? drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.h |? 6 ++---- > > > ? drivers/gpu/drm/ttm/ttm_device.c????????????? |? 7 +++---- > > > ? drivers/gpu/drm/vmwgfx/vmwgfx_drv.c?????????? |? 4 ++-- > &gt...

...> > > -- > > > > - > > > > ? .../gpu/drm/ttm/tests/ttm_bo_validate_test.c? |? 3 ++- > > > > ? drivers/gpu/drm/ttm/tests/ttm_device_test.c?? | 16 ++++++++---- > > > > -- > > > > - > > > > ? drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.c | 20 ++++++++---- > > > > -- > > > > ----- > > > > ? drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.h |? 6 ++---- > > > > ? drivers/gpu/drm/ttm/ttm_device.c????????????? |? 7 +++---- > > > > ? drivers/gpu/drm/vmwgfx/vmwgfx_drv.c????...

The NULL initialization of the pointers assigned by kzalloc() or kunit_kzalloc() first is not necessary, because if the kzalloc() or kunit_kzalloc() failed, the pointers will be assigned NULL, otherwise it works as usual. so remove it. Ruan Jinjie (7): drm/amdkfd: Remove unnecessary NULL values drm/amd/display: Remove unnecessary NULL values drm/msm: Remove unn...

...| 2 +- drivers/gpu/drm/radeon/radeon_ttm.c | 6 +++-- drivers/gpu/drm/ttm/tests/ttm_bo_test.c | 16 ++++++------ .../gpu/drm/ttm/tests/ttm_bo_validate_test.c | 3 ++- drivers/gpu/drm/ttm/tests/ttm_device_test.c | 16 ++++++------ drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.c | 20 ++++++--------- drivers/gpu/drm/ttm/tests/ttm_kunit_helpers.h | 6 ++--- drivers/gpu/drm/ttm/ttm_bo.c | 2 +- drivers/gpu/drm/ttm/ttm_device.c | 8 +++--- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 4 +-- drivers/gpu/drm/xe/xe_device.c...

...cit declaration of function 'unxlate_dev_mem_ptr'; did you mean 'xlate_dev_mem_ptr'? [-Werror=implicit-function-declaration] drivers/mfd/max77541.c:176:18: warning: cast to smaller integer type 'enum max7754x_ids' from 'const void *' [-Wvoid-pointer-to-enum-cast] lib/kunit/executor_test.c:138:4: warning: cast from 'void (*)(const void *)' to 'kunit_action_t *' (aka 'void (*)(void *)') converts to incompatible function type [-Wcast-function-type-strict] lib/kunit/test.c:775:38: warning: cast from 'void (*)(const void *)' to 'kunit_a...

...Vetter <daniel.vetter at ffwll.ch> One thought on testing all this: I think long term some unti tests, where we have a fake driver doing fake mst branch/ports and a bunch of allocations and then checking that it all works and validates would be nice. Longer term project ofc, and maybe after Kunit has been merged ... -Daniel > --- > drivers/gpu/drm/drm_dp_mst_topology.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c > index dcfab7536914..8bb03700e199 100644 >...

...intel.com> > > > Tested-by: Donald Robson <donald.robson at imgtec.com> > > > Suggested-by: Dave Airlie <airlied at redhat.com> > > > Signed-off-by: Danilo Krummrich <dakr at redhat.com> > > > > For some reason this breaks the drm_exec kunit patches: > > Fix available here [1]. > > [1]https://lore.kernel.org/dri-devel/cbf4ccf9-8131-27a0-332c-6942866340d1 at igalia.com/T/#t Thanks for pointing it out :) Maxime -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: applicat...

...s/bindings_helper.h b/rust/bindings/bindings_helper.h index ed25b686748e..dc19cb789536 100644 --- a/rust/bindings/bindings_helper.h +++ b/rust/bindings/bindings_helper.h @@ -6,6 +6,7 @@ * Sorted alphabetically. */ +#include <drm/drm_drv.h> #include <drm/drm_ioctl.h> #include <kunit/test.h> #include <linux/errname.h> diff --git a/rust/kernel/drm/drv.rs b/rust/kernel/drm/drv.rs new file mode 100644 index 000000000000..cd594a32f9e4 --- /dev/null +++ b/rust/kernel/drm/drv.rs @@ -0,0 +1,141 @@ +// SPDX-License-Identifier: GPL-2.0 OR MIT + +//! DRM driver core. +//! +//!...

...od.rs diff --git a/rust/bindings/bindings_helper.h b/rust/bindings/bindings_helper.h index 30ad2a0e22d7..ed25b686748e 100644 --- a/rust/bindings/bindings_helper.h +++ b/rust/bindings/bindings_helper.h @@ -6,6 +6,7 @@ * Sorted alphabetically. */ +#include <drm/drm_ioctl.h> #include <kunit/test.h> #include <linux/errname.h> #include <linux/ethtool.h> diff --git a/rust/kernel/drm/ioctl.rs b/rust/kernel/drm/ioctl.rs new file mode 100644 index 000000000000..09ca7a8e7583 --- /dev/null +++ b/rust/kernel/drm/ioctl.rs @@ -0,0 +1,153 @@ +// SPDX-License-Identifier: GPL-2.0 O...

...r.h > index ed25b686748e..dc19cb789536 100644 > --- a/rust/bindings/bindings_helper.h > +++ b/rust/bindings/bindings_helper.h > @@ -6,6 +6,7 @@ > * Sorted alphabetically. > */ > > +#include <drm/drm_drv.h> > #include <drm/drm_ioctl.h> > #include <kunit/test.h> > #include <linux/errname.h> > diff --git a/rust/kernel/drm/drv.rs b/rust/kernel/drm/drv.rs > new file mode 100644 > index 000000000000..cd594a32f9e4 > --- /dev/null > +++ b/rust/kernel/drm/drv.rs > @@ -0,0 +1,141 @@ > +// SPDX-License-Identifier: GPL-2.0 O...

This patchset does some cleaning up of the atomic VCPI helpers for MST, and converts nouveau over to using them. I would have included amdgpu in this patch as well, but at the moment moving them over to the atomic helpers is nontrivial. Cc: Daniel Vetter <daniel at ffwll.ch> Lyude Paul (4): drm/dp_mst: Add some atomic state iterator macros drm/dp_mst: Start tracking per-port VCPI

...when releasing devmap pages" and rebase the other patches on top of that - fold the hmm_devmem_add_resource into the DEVICE_PUBLIC memory removal patch - remove _vm_normal_page as it isn't needed without DEVICE_PUBLIC memory - pick up various ACKs Changes since v2: - fix nvdimm kunit build - add a new memory type for device dax - fix a few issues in intermediate patches that didn't show up in the end result - incorporate feedback from Michal Hocko, including killing of the DEVICE_PUBLIC memory type entirely Changes since v1: - rebase - also switch p2pdma to the...

...o resolve. Diffstat summary: 32 files changed, 361 insertions(+), 1012 deletions(-) Git tree: git://git.infradead.org/users/hch/misc.git hmm-devmem-cleanup.3 Gitweb: http://git.infradead.org/users/hch/misc.git/shortlog/refs/heads/hmm-devmem-cleanup.3 Changes since v2: - fix nvdimm kunit build - add a new memory type for device dax - fix a few issues in intermediate patches that didn't show up in the end result - incorporate feedback from Michal Hocko, including killing of the DEVICE_PUBLIC memory type entirely Changes since v1: - rebase - also switch p2pdma to the...