Re: [PATCH v3 kernel 4/7] virtio-balloon: speed up inflate/deflate process

From: kbuild test robot <lkp@intel.com>
To: Liang Li <liang.z.li@intel.com>
Cc: kbuild-all@01.org, linux-kernel@vger.kernel.org,
	virtualization@lists.linux-foundation.org, linux-mm@kvack.org,
	virtio-dev@lists.oasis-open.org, kvm@vger.kernel.org,
	qemu-devel@nongnu.org, quintela@redhat.com, dgilbert@redhat.com,
	dave.hansen@intel.com, Liang Li <liang.z.li@intel.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Cornelia Huck <cornelia.huck@de.ibm.com>,
	Amit Shah <amit.shah@redhat.com>
Subject: Re: [PATCH v3 kernel 4/7] virtio-balloon: speed up inflate/deflate process
Date: Mon, 8 Aug 2016 16:17:49 +0800	[thread overview]
Message-ID: <201608081626.EMzTs6Ov%fengguang.wu@intel.com> (raw)
In-Reply-To: <1470638134-24149-5-git-send-email-liang.z.li@intel.com>

[-- Attachment #1: Type: text/plain, Size: 3186 bytes --]

Hi Liang,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.8-rc1 next-20160805]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Liang-Li/Extend-virtio-balloon-for-fast-de-inflating-fast-live-migration/20160808-144551
config: s390-default_defconfig (attached as .config)
compiler: s390x-linux-gnu-gcc (Debian 5.4.0-6) 5.4.0 20160609
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=s390 

All warnings (new ones prefixed by >>):

   drivers/virtio/virtio_balloon.c: In function 'tell_host':
>> drivers/virtio/virtio_balloon.c:188:1: warning: the frame size of 1456 bytes is larger than 1024 bytes [-Wframe-larger-than=]
    }
    ^

vim +188 drivers/virtio/virtio_balloon.c

112d1263 Liang Li           2016-08-08  172  			msleep(2);
112d1263 Liang Li           2016-08-08  173  		if (virtqueue_add_outbuf(vq, sg2, nr_buf, vb, GFP_KERNEL) == 0)
112d1263 Liang Li           2016-08-08  174  			virtqueue_kick(vq);
6b35e407 Rusty Russell      2008-02-04  175  
112d1263 Liang Li           2016-08-08  176  	} else {
6b35e407 Rusty Russell      2008-02-04  177  		sg_init_one(&sg, vb->pfns, sizeof(vb->pfns[0]) * vb->num_pfns);
6b35e407 Rusty Russell      2008-02-04  178  
112d1263 Liang Li           2016-08-08  179  		/* We should always be able to add one buffer to an empty
112d1263 Liang Li           2016-08-08  180  		 * queue. */
4951cc90 Rusty Russell      2014-03-13  181  		virtqueue_add_outbuf(vq, &sg, 1, vb, GFP_KERNEL);
946cfe0e Michael S. Tsirkin 2010-04-12  182  		virtqueue_kick(vq);
112d1263 Liang Li           2016-08-08  183  	}
6b35e407 Rusty Russell      2008-02-04  184  
6b35e407 Rusty Russell      2008-02-04  185  	/* When host has read buffer, this completes via balloon_ack */
9c378abc Michael S. Tsirkin 2012-07-02  186  	wait_event(vb->acked, virtqueue_get_buf(vq, &len));
fd0e21c3 Petr Mladek        2016-01-25  187  
6b35e407 Rusty Russell      2008-02-04 @188  }
6b35e407 Rusty Russell      2008-02-04  189  
87c9403b Michael S. Tsirkin 2016-05-17  190  static void set_page_pfns(struct virtio_balloon *vb,
87c9403b Michael S. Tsirkin 2016-05-17  191  			  __virtio32 pfns[], struct page *page)
3ccc9372 Michael S. Tsirkin 2012-04-12  192  {
3ccc9372 Michael S. Tsirkin 2012-04-12  193  	unsigned int i;
3ccc9372 Michael S. Tsirkin 2012-04-12  194  
3ccc9372 Michael S. Tsirkin 2012-04-12  195  	/* Set balloon pfns pointing at this page.
3ccc9372 Michael S. Tsirkin 2012-04-12  196  	 * Note that the first pfn points at start of the page. */

:::::: The code at line 188 was first introduced by commit
:::::: 6b35e40767c6c1ac783330109ae8e0c09ea6bc82 virtio: balloon driver

:::::: TO: Rusty Russell <rusty@rustcorp.com.au>
:::::: CC: Rusty Russell <rusty@rustcorp.com.au>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 16210 bytes --]