From: kernel test robot <lkp@intel.com>
To: Cristian Ciocaltea <cristian.ciocaltea@collabora.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
Dave Stevenson <dave.stevenson@raspberrypi.com>,
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
kernel@collabora.com, dri-devel@lists.freedesktop.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 7/7] drm/tests: hdmi: Add max TMDS rate fallback tests for YUV420 mode
Date: Wed, 12 Mar 2025 20:02:44 +0800 [thread overview]
Message-ID: <202503121909.5IeaFEUt-lkp@intel.com> (raw)
In-Reply-To: <20250311-hdmi-conn-yuv-v2-7-fbdb94f02562@collabora.com>
Hi Cristian,
kernel test robot noticed the following build errors:
[auto build test ERROR on 4423e607ff50157aaf088854b145936cbab4d560]
url: https://github.com/intel-lab-lkp/linux/commits/Cristian-Ciocaltea/drm-connector-hdmi-Evaluate-limited-range-after-computing-format/20250311-190150
base: 4423e607ff50157aaf088854b145936cbab4d560
patch link: https://lore.kernel.org/r/20250311-hdmi-conn-yuv-v2-7-fbdb94f02562%40collabora.com
patch subject: [PATCH v2 7/7] drm/tests: hdmi: Add max TMDS rate fallback tests for YUV420 mode
config: x86_64-randconfig-004-20250312 (https://download.01.org/0day-ci/archive/20250312/202503121909.5IeaFEUt-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250312/202503121909.5IeaFEUt-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202503121909.5IeaFEUt-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from include/drm/drm_kunit_helpers.h:10,
from drivers/gpu/drm/tests/drm_hdmi_state_helper_test.c:14:
drivers/gpu/drm/tests/drm_hdmi_state_helper_test.c: In function 'drm_test_check_max_tmds_rate_bpc_fallback_yuv420':
>> include/kunit/test.h:776:29: error: SSE register return with SSE disabled
776 | const typeof(right) __right = (right); \
| ^~~~~~~
include/kunit/test.h:805:9: note: in expansion of macro 'KUNIT_BASE_BINARY_ASSERTION'
805 | KUNIT_BASE_BINARY_ASSERTION(test, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/kunit/test.h:971:9: note: in expansion of macro 'KUNIT_BINARY_INT_ASSERTION'
971 | KUNIT_BINARY_INT_ASSERTION(test, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
include/kunit/test.h:968:9: note: in expansion of macro 'KUNIT_EXPECT_EQ_MSG'
968 | KUNIT_EXPECT_EQ_MSG(test, left, right, NULL)
| ^~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/tests/drm_hdmi_state_helper_test.c:1337:9: note: in expansion of macro 'KUNIT_EXPECT_EQ'
1337 | KUNIT_EXPECT_EQ(test, conn_state->hdmi.tmds_char_rate,
| ^~~~~~~~~~~~~~~
--
In file included from include/drm/drm_kunit_helpers.h:10,
from drm_hdmi_state_helper_test.c:14:
drm_hdmi_state_helper_test.c: In function 'drm_test_check_max_tmds_rate_bpc_fallback_yuv420':
>> include/kunit/test.h:776:29: error: SSE register return with SSE disabled
776 | const typeof(right) __right = (right); \
| ^~~~~~~
include/kunit/test.h:805:9: note: in expansion of macro 'KUNIT_BASE_BINARY_ASSERTION'
805 | KUNIT_BASE_BINARY_ASSERTION(test, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/kunit/test.h:971:9: note: in expansion of macro 'KUNIT_BINARY_INT_ASSERTION'
971 | KUNIT_BINARY_INT_ASSERTION(test, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
include/kunit/test.h:968:9: note: in expansion of macro 'KUNIT_EXPECT_EQ_MSG'
968 | KUNIT_EXPECT_EQ_MSG(test, left, right, NULL)
| ^~~~~~~~~~~~~~~~~~~
drm_hdmi_state_helper_test.c:1337:9: note: in expansion of macro 'KUNIT_EXPECT_EQ'
1337 | KUNIT_EXPECT_EQ(test, conn_state->hdmi.tmds_char_rate,
| ^~~~~~~~~~~~~~~
vim +776 include/kunit/test.h
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 734
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 735 #define KUNIT_TRUE_MSG_ASSERTION(test, assert_type, condition, fmt, ...) \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 736 KUNIT_UNARY_ASSERTION(test, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 737 assert_type, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 738 condition, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 739 true, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 740 fmt, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 741 ##__VA_ARGS__)
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 742
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 743 #define KUNIT_FALSE_MSG_ASSERTION(test, assert_type, condition, fmt, ...) \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 744 KUNIT_UNARY_ASSERTION(test, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 745 assert_type, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 746 condition, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 747 false, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 748 fmt, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 749 ##__VA_ARGS__)
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 750
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 751 /*
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 752 * A factory macro for defining the assertions and expectations for the basic
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 753 * comparisons defined for the built in types.
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 754 *
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 755 * Unfortunately, there is no common type that all types can be promoted to for
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 756 * which all the binary operators behave the same way as for the actual types
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 757 * (for example, there is no type that long long and unsigned long long can
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 758 * both be cast to where the comparison result is preserved for all values). So
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 759 * the best we can do is do the comparison in the original types and then coerce
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 760 * everything to long long for printing; this way, the comparison behaves
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 761 * correctly and the printed out value usually makes sense without
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 762 * interpretation, but can always be interpreted to figure out the actual
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 763 * value.
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 764 */
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 765 #define KUNIT_BASE_BINARY_ASSERTION(test, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 766 assert_class, \
064ff292aca500d Daniel Latypov 2022-01-25 767 format_func, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 768 assert_type, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 769 left, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 770 op, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 771 right, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 772 fmt, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 773 ...) \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 774 do { \
c2741453478badf Daniel Latypov 2022-01-27 775 const typeof(left) __left = (left); \
c2741453478badf Daniel Latypov 2022-01-27 @776 const typeof(right) __right = (right); \
2b6861e2372bac6 Daniel Latypov 2022-01-25 777 static const struct kunit_binary_assert_text __text = { \
2b6861e2372bac6 Daniel Latypov 2022-01-25 778 .operation = #op, \
2b6861e2372bac6 Daniel Latypov 2022-01-25 779 .left_text = #left, \
2b6861e2372bac6 Daniel Latypov 2022-01-25 780 .right_text = #right, \
2b6861e2372bac6 Daniel Latypov 2022-01-25 781 }; \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 782 \
8bd5d74babc9255 Mickaël Salaün 2024-04-08 783 _KUNIT_SAVE_LOC(test); \
97d453bc4007d4a Daniel Latypov 2022-09-30 784 if (likely(__left op __right)) \
97d453bc4007d4a Daniel Latypov 2022-09-30 785 break; \
97d453bc4007d4a Daniel Latypov 2022-09-30 786 \
97d453bc4007d4a Daniel Latypov 2022-09-30 787 _KUNIT_FAILED(test, \
21957f90b28f6bc Daniel Latypov 2022-01-13 788 assert_type, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 789 assert_class, \
a8495ad8e973cb6 Daniel Latypov 2022-09-30 790 format_func, \
697365c08679137 Daniel Latypov 2022-09-30 791 KUNIT_INIT_ASSERT(.text = &__text, \
697365c08679137 Daniel Latypov 2022-09-30 792 .left_value = __left, \
697365c08679137 Daniel Latypov 2022-09-30 793 .right_value = __right), \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 794 fmt, \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 795 ##__VA_ARGS__); \
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 796 } while (0)
73cda7bb8bfb1d4 Brendan Higgins 2019-09-23 797
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-03-12 12:03 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-11 10:57 [PATCH v2 0/7] drm/connector: hdmi: Allow using the YUV420 output format Cristian Ciocaltea
2025-03-11 10:57 ` [PATCH v2 1/7] drm/connector: hdmi: Evaluate limited range after computing format Cristian Ciocaltea
2025-03-11 19:36 ` Dmitry Baryshkov
2025-03-11 10:57 ` [PATCH v2 2/7] drm/connector: hdmi: Add support for YUV420 format verification Cristian Ciocaltea
2025-03-11 15:30 ` Maxime Ripard
2025-03-11 17:06 ` Cristian Ciocaltea
2025-03-11 10:57 ` [PATCH v2 3/7] drm/connector: hdmi: Improve debug message for supported format Cristian Ciocaltea
2025-03-11 15:31 ` Maxime Ripard
2025-03-11 10:57 ` [PATCH v2 4/7] drm/connector: hdmi: Use YUV420 output format as an RGB fallback Cristian Ciocaltea
2025-03-11 15:55 ` Maxime Ripard
2025-03-11 18:59 ` Cristian Ciocaltea
2025-03-14 13:57 ` Maxime Ripard
2025-03-11 19:46 ` Dmitry Baryshkov
2025-03-14 13:47 ` Maxime Ripard
2025-03-14 15:00 ` Dmitry Baryshkov
2025-03-11 10:57 ` [PATCH v2 5/7] drm/tests: hdmi: Add macros to simplify EDID setup Cristian Ciocaltea
2025-03-11 16:12 ` Maxime Ripard
2025-03-11 22:44 ` Cristian Ciocaltea
2025-03-19 15:32 ` Maxime Ripard
2025-03-19 21:33 ` Cristian Ciocaltea
2025-03-11 10:57 ` [PATCH v2 6/7] drm/tests: hdmi: Add limited range tests for YUV420 mode Cristian Ciocaltea
2025-03-11 16:17 ` Maxime Ripard
2025-03-11 22:54 ` Cristian Ciocaltea
2025-03-14 13:52 ` Maxime Ripard
2025-03-11 10:57 ` [PATCH v2 7/7] drm/tests: hdmi: Add max TMDS rate fallback " Cristian Ciocaltea
2025-03-12 12:02 ` kernel test robot [this message]
2025-03-12 18:14 ` kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202503121909.5IeaFEUt-lkp@intel.com \
--to=lkp@intel.com \
--cc=airlied@gmail.com \
--cc=cristian.ciocaltea@collabora.com \
--cc=dave.stevenson@raspberrypi.com \
--cc=dmitry.baryshkov@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=kernel@collabora.com \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=simona@ffwll.ch \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.