* Re: [PATCH v7 2/2] drm: add kms driver for loongson display controller
[not found] <20230315211550.2620818-3-15330273260@189.cn>
@ 2023-03-16 9:53 ` kernel test robot
2023-03-16 10:17 ` Sui jingfeng
2023-03-17 2:19 ` Sui jingfeng
0 siblings, 2 replies; 3+ messages in thread
From: kernel test robot @ 2023-03-16 9:53 UTC (permalink / raw)
To: Sui Jingfeng, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter, suijingfeng,
Sumit Semwal, Christian Koenig
Cc: oe-kbuild-all, linaro-mm-sig, Li Yi, linux-kernel, dri-devel,
linux-media
Hi Sui,
I love your patch! Perhaps something to improve:
[auto build test WARNING on drm-misc/drm-misc-next]
[also build test WARNING on linus/master v6.3-rc2 next-20230316]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Sui-Jingfeng/MAINTAINERS-add-maintainers-for-DRM-LOONGSON-driver/20230316-051724
base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
patch link: https://lore.kernel.org/r/20230315211550.2620818-3-15330273260%40189.cn
patch subject: [PATCH v7 2/2] drm: add kms driver for loongson display controller
config: arm-allmodconfig (https://download.01.org/0day-ci/archive/20230316/202303161727.8HnBf6cW-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/ba899dba3475b9612f212e3b1daedc3d9a299458
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Sui-Jingfeng/MAINTAINERS-add-maintainers-for-DRM-LOONGSON-driver/20230316-051724
git checkout ba899dba3475b9612f212e3b1daedc3d9a299458
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/gpu/drm/loongson/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202303161727.8HnBf6cW-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/gpu/drm/loongson/lsdc_gem.c: In function 'lsdc_show_buffer_object':
>> drivers/gpu/drm/loongson/lsdc_gem.c:280:51: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
280 | seq_printf(m, "bo[%04u]: size: %8lukB %s\n",
| ~~~^
| |
| long unsigned int
| %8u
281 | i,
282 | lsdc_bo_size(tbo) >> 10,
| ~~~~~~~~~~~~~~~~~~~~~~~
| |
| size_t {aka unsigned int}
vim +280 drivers/gpu/drm/loongson/lsdc_gem.c
264
265 int lsdc_show_buffer_object(struct seq_file *m, void *arg)
266 {
267 #ifdef CONFIG_DEBUG_FS
268 struct drm_info_node *node = (struct drm_info_node *)m->private;
269 struct drm_device *ddev = node->minor->dev;
270 struct lsdc_device *ldev = to_lsdc(ddev);
271 struct lsdc_bo *lbo;
272 unsigned int i = 0;
273
274 mutex_lock(&ldev->gem.mutex);
275
276 list_for_each_entry(lbo, &ldev->gem.objects, list) {
277 struct ttm_buffer_object *tbo = &lbo->tbo;
278 struct ttm_resource *resource = tbo->resource;
279
> 280 seq_printf(m, "bo[%04u]: size: %8lukB %s\n",
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v7 2/2] drm: add kms driver for loongson display controller
2023-03-16 9:53 ` [PATCH v7 2/2] drm: add kms driver for loongson display controller kernel test robot
@ 2023-03-16 10:17 ` Sui jingfeng
2023-03-17 2:19 ` Sui jingfeng
1 sibling, 0 replies; 3+ messages in thread
From: Sui jingfeng @ 2023-03-16 10:17 UTC (permalink / raw)
To: kernel test robot, Sui Jingfeng, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
Sumit Semwal, Christian Koenig
Cc: oe-kbuild-all, linaro-mm-sig, Li Yi, linux-kernel, dri-devel,
linux-media
On 2023/3/16 17:53, kernel test robot wrote:
> Hi Sui,
>
> I love your patch! Perhaps something to improve:
>
> [auto build test WARNING on drm-misc/drm-misc-next]
> [also build test WARNING on linus/master v6.3-rc2 next-20230316]
> [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#_base_tree_information]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Sui-Jingfeng/MAINTAINERS-add-maintainers-for-DRM-LOONGSON-driver/20230316-051724
> base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
> patch link: https://lore.kernel.org/r/20230315211550.2620818-3-15330273260%40189.cn
> patch subject: [PATCH v7 2/2] drm: add kms driver for loongson display controller
> config: arm-allmodconfig (https://download.01.org/0day-ci/archive/20230316/202303161727.8HnBf6cW-lkp@intel.com/config)
> compiler: arm-linux-gnueabi-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/ba899dba3475b9612f212e3b1daedc3d9a299458
> git remote add linux-review https://github.com/intel-lab-lkp/linux
> git fetch --no-tags linux-review Sui-Jingfeng/MAINTAINERS-add-maintainers-for-DRM-LOONGSON-driver/20230316-051724
> git checkout ba899dba3475b9612f212e3b1daedc3d9a299458
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm olddefconfig
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/gpu/drm/loongson/
>
> If you fix the issue, kindly add following tag where applicable
> | Reported-by: kernel test robot <lkp@intel.com>
> | Link: https://lore.kernel.org/oe-kbuild-all/202303161727.8HnBf6cW-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
> drivers/gpu/drm/loongson/lsdc_gem.c: In function 'lsdc_show_buffer_object':
>>> drivers/gpu/drm/loongson/lsdc_gem.c:280:51: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
> 280 | seq_printf(m, "bo[%04u]: size: %8lukB %s\n",
> | ~~~^
> | |
> | long unsigned int
> | %8u
> 281 | i,
> 282 | lsdc_bo_size(tbo) >> 10,
> | ~~~~~~~~~~~~~~~~~~~~~~~
> | |
> | size_t {aka unsigned int}
>
>
> vim +280 drivers/gpu/drm/loongson/lsdc_gem.c
>
> 264
> 265 int lsdc_show_buffer_object(struct seq_file *m, void *arg)
> 266 {
> 267 #ifdef CONFIG_DEBUG_FS
> 268 struct drm_info_node *node = (struct drm_info_node *)m->private;
> 269 struct drm_device *ddev = node->minor->dev;
> 270 struct lsdc_device *ldev = to_lsdc(ddev);
> 271 struct lsdc_bo *lbo;
> 272 unsigned int i = 0;
> 273
> 274 mutex_lock(&ldev->gem.mutex);
> 275
> 276 list_for_each_entry(lbo, &ldev->gem.objects, list) {
> 277 struct ttm_buffer_object *tbo = &lbo->tbo;
> 278 struct ttm_resource *resource = tbo->resource;
> 279
> > 280 seq_printf(m, "bo[%04u]: size: %8lukB %s\n",
>
Hi,
Do i need fix this?
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v7 2/2] drm: add kms driver for loongson display controller
2023-03-16 9:53 ` [PATCH v7 2/2] drm: add kms driver for loongson display controller kernel test robot
2023-03-16 10:17 ` Sui jingfeng
@ 2023-03-17 2:19 ` Sui jingfeng
1 sibling, 0 replies; 3+ messages in thread
From: Sui jingfeng @ 2023-03-17 2:19 UTC (permalink / raw)
To: kernel test robot, Sui Jingfeng, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
Sumit Semwal, Christian Koenig
Cc: oe-kbuild-all, linaro-mm-sig, Li Yi, linux-kernel, dri-devel,
linux-media
On 2023/3/16 17:53, kernel test robot wrote:
> Hi Sui,
>
> I love your patch! Perhaps something to improve:
>
> [auto build test WARNING on drm-misc/drm-misc-next]
> [also build test WARNING on linus/master v6.3-rc2 next-20230316]
> [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#_base_tree_information]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Sui-Jingfeng/MAINTAINERS-add-maintainers-for-DRM-LOONGSON-driver/20230316-051724
> base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
> patch link: https://lore.kernel.org/r/20230315211550.2620818-3-15330273260%40189.cn
> patch subject: [PATCH v7 2/2] drm: add kms driver for loongson display controller
> config: arm-allmodconfig (https://download.01.org/0day-ci/archive/20230316/202303161727.8HnBf6cW-lkp@intel.com/config)
> compiler: arm-linux-gnueabi-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/ba899dba3475b9612f212e3b1daedc3d9a299458
> git remote add linux-review https://github.com/intel-lab-lkp/linux
> git fetch --no-tags linux-review Sui-Jingfeng/MAINTAINERS-add-maintainers-for-DRM-LOONGSON-driver/20230316-051724
> git checkout ba899dba3475b9612f212e3b1daedc3d9a299458
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm olddefconfig
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/gpu/drm/loongson/
>
> If you fix the issue, kindly add following tag where applicable
> | Reported-by: kernel test robot <lkp@intel.com>
> | Link: https://lore.kernel.org/oe-kbuild-all/202303161727.8HnBf6cW-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
> drivers/gpu/drm/loongson/lsdc_gem.c: In function 'lsdc_show_buffer_object':
>>> drivers/gpu/drm/loongson/lsdc_gem.c:280:51: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
> 280 | seq_printf(m, "bo[%04u]: size: %8lukB %s\n",
> | ~~~^
> | |
> | long unsigned int
> | %8u
> 281 | i,
> 282 | lsdc_bo_size(tbo) >> 10,
> | ~~~~~~~~~~~~~~~~~~~~~~~
> | |
> | size_t {aka unsigned int}
>
>
> vim +280 drivers/gpu/drm/loongson/lsdc_gem.c
>
> 264
> 265 int lsdc_show_buffer_object(struct seq_file *m, void *arg)
> 266 {
> 267 #ifdef CONFIG_DEBUG_FS
> 268 struct drm_info_node *node = (struct drm_info_node *)m->private;
> 269 struct drm_device *ddev = node->minor->dev;
> 270 struct lsdc_device *ldev = to_lsdc(ddev);
> 271 struct lsdc_bo *lbo;
> 272 unsigned int i = 0;
> 273
> 274 mutex_lock(&ldev->gem.mutex);
> 275
> 276 list_for_each_entry(lbo, &ldev->gem.objects, list) {
> 277 struct ttm_buffer_object *tbo = &lbo->tbo;
> 278 struct ttm_resource *resource = tbo->resource;
> 279
> > 280 seq_printf(m, "bo[%04u]: size: %8lukB %s\n",
>
Hi, this is buffer objects tracking implemented in v7
For example,
[root@fedora 0]# pwd
/sys/kernel/debug/dri/0
[root@fedora 0]# cat chip
I'm in LS7A2000, running on cpu 0xc0, cpu revison: 0x11
[root@fedora 0]# cat bos
bo[0000]: size: 8112kB VRAM
bo[0001]: size: 16208kB VRAM
bo[0002]: size: 16kB VRAM
bo[0003]: size: 16kB VRAM
When using with modesetting driver with shadowfb option enabled.
by default we have only 4 BOs create, the first one is for fbcon, the
largest one is framebuffer of double screen.
another two is hardware cursor bo. Our system page size is 16KB by default.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-03-17 2:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20230315211550.2620818-3-15330273260@189.cn>
2023-03-16 9:53 ` [PATCH v7 2/2] drm: add kms driver for loongson display controller kernel test robot
2023-03-16 10:17 ` Sui jingfeng
2023-03-17 2:19 ` Sui jingfeng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).