From: kernel test robot <lkp@intel.com>
To: Aditya Nagesh <adityanagesh@linux.microsoft.com>,
adityanagesh@microsoft.com, kys@microsoft.com,
haiyangz@microsoft.com, wei.liu@kernel.org, decui@microsoft.com,
linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: oe-kbuild-all@lists.linux.dev,
Aditya Nagesh <adityanagesh@linux.microsoft.com>
Subject: Re: [PATCH v5] Drivers: hv: Cosmetic changes for hv.c and balloon.c
Date: Thu, 30 May 2024 16:12:07 +0800 [thread overview]
Message-ID: <202405301550.DKuS2OzK-lkp@intel.com> (raw)
In-Reply-To: <1716998695-32135-1-git-send-email-adityanagesh@linux.microsoft.com>
Hi Aditya,
kernel test robot noticed the following build errors:
[auto build test ERROR on next-20240529]
[cannot apply to linus/master v6.10-rc1 v6.9 v6.9-rc7 v6.10-rc1]
[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/Aditya-Nagesh/Drivers-hv-Cosmetic-changes-for-hv-c-and-balloon-c/20240530-000928
base: next-20240529
patch link: https://lore.kernel.org/r/1716998695-32135-1-git-send-email-adityanagesh%40linux.microsoft.com
patch subject: [PATCH v5] Drivers: hv: Cosmetic changes for hv.c and balloon.c
config: arm64-randconfig-002-20240530 (https://download.01.org/0day-ci/archive/20240530/202405301550.DKuS2OzK-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240530/202405301550.DKuS2OzK-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/202405301550.DKuS2OzK-lkp@intel.com/
All error/warnings (new ones prefixed by >>):
drivers/hv/hv_balloon.c: In function 'hot_add_req':
>> drivers/hv/hv_balloon.c:998:9: warning: "/*" within comment [-Wcomment]
998 | /*
|
>> drivers/hv/hv_balloon.c:980: error: unterminated #ifdef
980 | #ifdef CONFIG_MEMORY_HOTPLUG
|
>> drivers/hv/hv_balloon.c:978:39: error: expected declaration or statement at end of input
978 | resp.hdr.size = sizeof(struct dm_hot_add_response);
| ^~~~~~~~~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:974:34: warning: unused variable 'dm' [-Wunused-variable]
974 | struct hv_dynmem_device *dm = &dm_device;
| ^~
drivers/hv/hv_balloon.c: At top level:
>> drivers/hv/hv_balloon.c:575:13: warning: 'post_status' declared 'static' but never defined [-Wunused-function]
575 | static void post_status(struct hv_dynmem_device *dm);
| ^~~~~~~~~~~
>> drivers/hv/hv_balloon.c:577:13: warning: 'enable_page_reporting' declared 'static' but never defined [-Wunused-function]
577 | static void enable_page_reporting(void);
| ^~~~~~~~~~~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:579:13: warning: 'disable_page_reporting' declared 'static' but never defined [-Wunused-function]
579 | static void disable_page_reporting(void);
| ^~~~~~~~~~~~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:967:13: warning: 'hot_add_req' defined but not used [-Wunused-function]
967 | static void hot_add_req(struct work_struct *dummy)
| ^~~~~~~~~~~
>> drivers/hv/hv_balloon.c:496:22: warning: 'ha_pages_in_chunk' defined but not used [-Wunused-variable]
496 | static unsigned long ha_pages_in_chunk;
| ^~~~~~~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:494:13: warning: 'balloon_up_send_buffer' defined but not used [-Wunused-variable]
494 | static __u8 balloon_up_send_buffer[HV_HYP_PAGE_SIZE];
| ^~~~~~~~~~~~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:493:13: warning: 'recv_buffer' defined but not used [-Wunused-variable]
493 | static __u8 recv_buffer[HV_HYP_PAGE_SIZE];
| ^~~~~~~~~~~
>> drivers/hv/hv_balloon.c:478:12: warning: 'dm_ring_size' defined but not used [-Wunused-variable]
478 | static int dm_ring_size = VMBUS_RING_SIZE(16 * 1024);
| ^~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:476:17: warning: 'trans_id' defined but not used [-Wunused-variable]
476 | static atomic_t trans_id = ATOMIC_INIT(0);
| ^~~~~~~~
>> drivers/hv/hv_balloon.c:469:12: warning: 'hv_hypercall_multi_failure' defined but not used [-Wunused-variable]
469 | static int hv_hypercall_multi_failure;
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:467:22: warning: 'last_post_time' defined but not used [-Wunused-variable]
467 | static unsigned long last_post_time;
| ^~~~~~~~~~~~~~
>> drivers/hv/hv_balloon.c:455:13: warning: 'do_hot_add' defined but not used [-Wunused-variable]
455 | static bool do_hot_add;
| ^~~~~~~~~~
>> drivers/hv/hv_balloon.c:453:13: warning: 'allow_hibernation' defined but not used [-Wunused-variable]
453 | static bool allow_hibernation;
| ^~~~~~~~~~~~~~~~~
vim +980 drivers/hv/hv_balloon.c
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 966
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 @967 static void hot_add_req(struct work_struct *dummy)
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 968 {
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 969 struct dm_hot_add_response resp;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 970 #ifdef CONFIG_MEMORY_HOTPLUG
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 971 unsigned long pg_start, pfn_cnt;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 972 unsigned long rg_start, rg_sz;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 973 #endif
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 @974 struct hv_dynmem_device *dm = &dm_device;
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 975
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 976 memset(&resp, 0, sizeof(struct dm_hot_add_response));
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 977 resp.hdr.type = DM_MEM_HOT_ADD_RESPONSE;
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 @978 resp.hdr.size = sizeof(struct dm_hot_add_response);
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 979
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 @980 #ifdef CONFIG_MEMORY_HOTPLUG
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 981 pg_start = dm->ha_wrk.ha_page_range.finfo.start_page;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 982 pfn_cnt = dm->ha_wrk.ha_page_range.finfo.page_cnt;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 983
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 984 rg_start = dm->ha_wrk.ha_region_range.finfo.start_page;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 985 rg_sz = dm->ha_wrk.ha_region_range.finfo.page_cnt;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 986
1aa0ebde593dfb1 Aditya Nagesh 2024-05-29 987 if (rg_start == 0 && !dm->host_specified_ha_region) {
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 988 /*
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 989 * Based on the hot-add page range being specified,
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 990 }
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 991
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 992 if (do_hot_add)
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 993 resp.page_count = process_hot_add(pg_start, pfn_cnt,
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 994 rg_start, rg_sz);
549fd280b145e21 Vitaly Kuznetsov 2015-02-28 995
549fd280b145e21 Vitaly Kuznetsov 2015-02-28 996 dm->num_pages_added += resp.page_count;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 997 #endif
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 @998 /*
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 999 * The result field of the response structure has the
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1000 * following semantics:
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1001 *
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1002 * 1. If all or some pages hot-added: Guest should return success.
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1003 *
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1004 * 2. If no pages could be hot-added:
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1005 *
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1006 * If the guest returns success, then the host
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1007 * will not attempt any further hot-add operations. This
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1008 * signifies a permanent failure.
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1009 *
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1010 * If the guest returns failure, then this failure will be
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1011 * treated as a transient failure and the host may retry the
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1012 * hot-add operation after some delay.
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1013 */
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 1014 if (resp.page_count > 0)
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 1015 resp.result = 1;
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1016 else if (!do_hot_add)
7f4f2302a11173d K. Y. Srinivasan 2013-03-18 1017 resp.result = 1;
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 1018 else
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 1019 resp.result = 0;
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 1020
25bd2b2f1f05347 Dexuan Cui 2019-11-19 1021 if (!do_hot_add || resp.page_count == 0) {
25bd2b2f1f05347 Dexuan Cui 2019-11-19 1022 if (!allow_hibernation)
223e1e4d2c16fed Vitaly Kuznetsov 2018-03-04 1023 pr_err("Memory hot add failed\n");
25bd2b2f1f05347 Dexuan Cui 2019-11-19 1024 else
25bd2b2f1f05347 Dexuan Cui 2019-11-19 1025 pr_info("Ignore hot-add request!\n");
25bd2b2f1f05347 Dexuan Cui 2019-11-19 1026 }
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 1027
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 1028 dm->state = DM_INITIALIZED;
20138d6cb838aa0 K. Y. Srinivasan 2013-07-17 1029 resp.hdr.trans_id = atomic_inc_return(&trans_id);
1cac8cd4d146b60 K. Y. Srinivasan 2013-03-15 1030 vmbus_sendpacket(dm->dev->channel, &resp,
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 1031 sizeof(struct dm_hot_add_response),
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 1032 (unsigned long)NULL,
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 1033 VM_PKT_DATA_INBAND, 0);
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 1034 }
9aa8b50b2b3d3a7 K. Y. Srinivasan 2012-11-14 1035
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
prev parent reply other threads:[~2024-05-30 8:12 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-29 16:04 [PATCH v5] Drivers: hv: Cosmetic changes for hv.c and balloon.c Aditya Nagesh
2024-05-29 16:29 ` Michael Kelley
2024-05-30 2:45 ` Wei Liu
2024-05-30 1:06 ` kernel test robot
2024-05-30 8:12 ` kernel test robot [this message]
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=202405301550.DKuS2OzK-lkp@intel.com \
--to=lkp@intel.com \
--cc=adityanagesh@linux.microsoft.com \
--cc=adityanagesh@microsoft.com \
--cc=decui@microsoft.com \
--cc=haiyangz@microsoft.com \
--cc=kys@microsoft.com \
--cc=linux-hyperv@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=wei.liu@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox