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-...@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 <l...@intel.com>
| Link: 
https://lore.kernel.org/oe-kbuild-all/202303161727.8hnbf6cw-...@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.

Reply via email to