From: kernel test robot <lkp@intel.com>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
Abhinav Kumar <abhinavk@codeaurora.org>
Cc: kbuild-all@lists.01.org, Jonathan Marek <jonathan@marek.ca>,
Stephen Boyd <sboyd@kernel.org>,
linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
David Airlie <airlied@linux.ie>,
freedreno@lists.freedesktop.org
Subject: Re: [PATCH 7/7] drm/msm/mdp5: provide dynamic bandwidth management
Date: Wed, 26 May 2021 01:15:22 +0800 [thread overview]
Message-ID: <202105260126.S7XpGgXI-lkp@intel.com> (raw)
In-Reply-To: <20210525131316.3117809-8-dmitry.baryshkov@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 3492 bytes --]
Hi Dmitry,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.13-rc3 next-20210525]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Dmitry-Baryshkov/drm-msm-mdp5-add-properties-and-bandwidth-management/20210525-211559
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git a050a6d2b7e80ca52b2f4141eaf3420d201b72b3
config: arm-defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/1077a27642ba4bcb15951c29bffdd94bfc378dbe
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Dmitry-Baryshkov/drm-msm-mdp5-add-properties-and-bandwidth-management/20210525-211559
git checkout 1077a27642ba4bcb15951c29bffdd94bfc378dbe
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c: In function 'mdp5_plane_calc_bw':
>> drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c:169:16: warning: variable 'vfp' set but not used [-Wunused-but-set-variable]
169 | int vbp, vpw, vfp;
| ^~~
vim +/vfp +169 drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c
156
157 /* based on _dpu_plane_calc_bw */
158 static void mdp5_plane_calc_bw(struct drm_plane_state *state, struct drm_crtc_state *crtc_state)
159 {
160 struct drm_framebuffer *fb = state->fb;
161 struct mdp5_plane_state *pstate = to_mdp5_plane_state(state);
162 struct drm_display_mode *mode = &crtc_state->mode;
163 int bpp;
164 int src_width, src_height, dst_height, fps;
165 u64 plane_bw;
166 u32 hw_latency_lines;
167 u32 prefill_div;
168 u64 scale_factor;
> 169 int vbp, vpw, vfp;
170
171 src_width = drm_rect_width(&state->src) >> 16;
172 src_height = drm_rect_height(&state->src) >> 16;
173 dst_height = drm_rect_height(&state->dst);
174 fps = drm_mode_vrefresh(mode);
175 vbp = mode->vtotal - mode->vsync_end;
176 vpw = mode->vsync_end - mode->vsync_start;
177 vfp = mode->vsync_start - mode->vdisplay;
178 scale_factor = src_height > dst_height ?
179 mult_frac(src_height, 1, dst_height) : 1;
180
181 bpp = to_mdp_format(msm_framebuffer_format(fb))->cpp;
182
183 plane_bw = src_width * mode->vtotal * fps * bpp * scale_factor;
184
185 hw_latency_lines = 21; /* or 24? */
186 prefill_div = hw_latency_lines;
187 if (vbp + vpw > hw_latency_lines)
188 prefill_div = vbp + vpw;
189 #if 0
190 else if (vbp + vpw + vfp < hw_latency_lines)
191 prefill_div = vbp + vpw + vfp;
192 #endif
193
194 pstate->plane_bw = max(plane_bw, mult_frac(plane_bw, hw_latency_lines, prefill_div));
195 }
196
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 54736 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH 7/7] drm/msm/mdp5: provide dynamic bandwidth management
Date: Wed, 26 May 2021 01:15:22 +0800 [thread overview]
Message-ID: <202105260126.S7XpGgXI-lkp@intel.com> (raw)
In-Reply-To: <20210525131316.3117809-8-dmitry.baryshkov@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 3576 bytes --]
Hi Dmitry,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.13-rc3 next-20210525]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Dmitry-Baryshkov/drm-msm-mdp5-add-properties-and-bandwidth-management/20210525-211559
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git a050a6d2b7e80ca52b2f4141eaf3420d201b72b3
config: arm-defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/1077a27642ba4bcb15951c29bffdd94bfc378dbe
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Dmitry-Baryshkov/drm-msm-mdp5-add-properties-and-bandwidth-management/20210525-211559
git checkout 1077a27642ba4bcb15951c29bffdd94bfc378dbe
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c: In function 'mdp5_plane_calc_bw':
>> drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c:169:16: warning: variable 'vfp' set but not used [-Wunused-but-set-variable]
169 | int vbp, vpw, vfp;
| ^~~
vim +/vfp +169 drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c
156
157 /* based on _dpu_plane_calc_bw */
158 static void mdp5_plane_calc_bw(struct drm_plane_state *state, struct drm_crtc_state *crtc_state)
159 {
160 struct drm_framebuffer *fb = state->fb;
161 struct mdp5_plane_state *pstate = to_mdp5_plane_state(state);
162 struct drm_display_mode *mode = &crtc_state->mode;
163 int bpp;
164 int src_width, src_height, dst_height, fps;
165 u64 plane_bw;
166 u32 hw_latency_lines;
167 u32 prefill_div;
168 u64 scale_factor;
> 169 int vbp, vpw, vfp;
170
171 src_width = drm_rect_width(&state->src) >> 16;
172 src_height = drm_rect_height(&state->src) >> 16;
173 dst_height = drm_rect_height(&state->dst);
174 fps = drm_mode_vrefresh(mode);
175 vbp = mode->vtotal - mode->vsync_end;
176 vpw = mode->vsync_end - mode->vsync_start;
177 vfp = mode->vsync_start - mode->vdisplay;
178 scale_factor = src_height > dst_height ?
179 mult_frac(src_height, 1, dst_height) : 1;
180
181 bpp = to_mdp_format(msm_framebuffer_format(fb))->cpp;
182
183 plane_bw = src_width * mode->vtotal * fps * bpp * scale_factor;
184
185 hw_latency_lines = 21; /* or 24? */
186 prefill_div = hw_latency_lines;
187 if (vbp + vpw > hw_latency_lines)
188 prefill_div = vbp + vpw;
189 #if 0
190 else if (vbp + vpw + vfp < hw_latency_lines)
191 prefill_div = vbp + vpw + vfp;
192 #endif
193
194 pstate->plane_bw = max(plane_bw, mult_frac(plane_bw, hw_latency_lines, prefill_div));
195 }
196
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 54736 bytes --]
next prev parent reply other threads:[~2021-05-25 17:16 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-25 13:13 [PATCH 0/7] drm/msm/mdp5: add properties and bandwidth management Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 13:13 ` [PATCH 1/7] drm/msm/mdp5: use drm atomic helpers to handle base drm plane state Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 13:13 ` [PATCH 2/7] drm/msm/mdp5: use drm_plane_state for storing alpha value Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 13:13 ` [PATCH 3/7] drm/msm/mdp5: use drm_plane_state for pixel blend mode Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 13:13 ` [PATCH 4/7] drm/msm/mdp5: add support for alpha/blend_mode properties Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 13:13 ` [PATCH 5/7] drm/msm/mdp5: switch to standard zpos property Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 13:13 ` [PATCH 6/7] drm/msm/mdp5: add perf blocks for holding fudge factors Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 13:13 ` [PATCH 7/7] drm/msm/mdp5: provide dynamic bandwidth management Dmitry Baryshkov
2021-05-25 13:13 ` Dmitry Baryshkov
2021-05-25 17:15 ` kernel test robot [this message]
2021-05-25 17:15 ` 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=202105260126.S7XpGgXI-lkp@intel.com \
--to=lkp@intel.com \
--cc=abhinavk@codeaurora.org \
--cc=airlied@linux.ie \
--cc=bjorn.andersson@linaro.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=freedreno@lists.freedesktop.org \
--cc=jonathan@marek.ca \
--cc=kbuild-all@lists.01.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=robdclark@gmail.com \
--cc=sboyd@kernel.org \
--cc=sean@poorly.run \
/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.