[axboe-block:for-next] [block] 1122c0c1cc: aim7.jobs-per-min 22.6% improvement

2024-06-24 Thread kernel test robot



Hello,

kernel test robot noticed a 22.6% improvement of aim7.jobs-per-min on:


commit: 1122c0c1cc71f740fa4d5f14f239194e06a1d5e7 ("block: move cache control 
settings out of queue->flags")
https://git.kernel.org/cgit/linux/kernel/git/axboe/linux-block.git for-next

testcase: aim7
test machine: 96 threads 2 sockets Intel(R) Xeon(R) Platinum 8260L CPU @ 
2.40GHz (Cascade Lake) with 128G memory
parameters:

disk: 4BRD_12G
md: RAID0
fs: xfs
test: sync_disk_rw
load: 300
cpufreq_governor: performance






Details are as below:
-->


The kernel config and materials to reproduce are available at:
https://download.01.org/0day-ci/archive/20240625/202406250948.e0044f1d-oliver.s...@intel.com

=
compiler/cpufreq_governor/disk/fs/kconfig/load/md/rootfs/tbox_group/test/testcase:
  
gcc-13/performance/4BRD_12G/xfs/x86_64-rhel-8.3/300/RAID0/debian-12-x86_64-20240206.cgz/lkp-csl-2sp3/sync_disk_rw/aim7

commit: 
  70905f8706 ("block: remove blk_flush_policy")
  1122c0c1cc ("block: move cache control settings out of queue->flags")

70905f8706b62113 1122c0c1cc71f740fa4d5f14f23 
 --- 
 %stddev %change %stddev
 \  |\  
153.19   -13.3% 132.81uptime.boot
   2.8e+09   -11.9%  2.466e+09cpuidle..time
  21945319 ±  2% -40.4%   13076160cpuidle..usage
 29.31+7.8%  31.58 ±  2%  iostat.cpu.idle
 69.87-3.6%  67.35iostat.cpu.system
  0.04 ±  4%  +0.00.08 ±  5%  mpstat.cpu.all.iowait%
  0.78 ±  2%  +0.20.99 ±  2%  mpstat.cpu.all.usr%
 52860 ± 49% -78.2%  11536 ± 78%  numa-numastat.node0.other_node
 46804 ± 56% +88.4%  88190 ± 10%  numa-numastat.node1.other_node
955871 ± 10% -43.3% 542216 ± 14%  numa-meminfo.node1.Active
955871 ± 10% -43.3% 542216 ± 14%  numa-meminfo.node1.Active(anon)
   1015354 ± 10% -34.7% 662696 ± 13%  numa-meminfo.node1.Shmem
  6008   -14.3%   5146 ±  2%  perf-c2c.DRAM.remote
  7889   -12.4%   6908 ±  2%  perf-c2c.HITM.local
  3839   -16.5%   3203 ±  2%  perf-c2c.HITM.remote
 11728   -13.8%  10112 ±  2%  perf-c2c.HITM.total
695109   +20.5% 837625vmstat.io.bo
105.99 ±  7% -23.7%  80.83 ± 11%  vmstat.procs.r
803244   -30.9% 555360vmstat.system.cs
209736   -12.9% 182626vmstat.system.in
  1448 ± 89%+207.9%   4459 ±  6%  
numa-vmstat.node0.nr_page_table_pages
 52860 ± 49% -78.2%  11536 ± 78%  numa-vmstat.node0.numa_other
239214 ± 10% -43.6% 134883 ± 13%  numa-vmstat.node1.nr_active_anon
254124 ± 10% -34.9% 165421 ± 13%  numa-vmstat.node1.nr_shmem
239214 ± 10% -43.6% 134883 ± 13%  
numa-vmstat.node1.nr_zone_active_anon
 46805 ± 56% +88.4%  88190 ± 10%  numa-vmstat.node1.numa_other
 17374   +22.6%  21299aim7.jobs-per-min
103.64   -18.4%  84.58aim7.time.elapsed_time
103.64   -18.4%  84.58aim7.time.elapsed_time.max
   4641240   -83.4% 770073
aim7.time.involuntary_context_switches
 32705-4.3%  31289 ±  2%  aim7.time.minor_page_faults
  6562-3.1%   6359
aim7.time.percent_of_cpu_this_job_got
  6775   -21.0%   5351 ±  2%  aim7.time.system_time
  49095202   -38.3%   30299361
aim7.time.voluntary_context_switches
   1297567   -37.0% 817692meminfo.Active
   1297567   -37.0% 817692meminfo.Active(anon)
 97760 ±  5% -23.4%  74859 ± 20%  meminfo.AnonHugePages
   2390317   -15.3%2024905meminfo.Committed_AS
884407   +11.9% 989723meminfo.Inactive
743152 ±  2% +14.8% 853331meminfo.Inactive(anon)
159265 ±  8% +38.6% 220668 ±  3%  meminfo.Mapped
   1382079   -27.1%1007445meminfo.Shmem
324534   -37.2% 203663 ±  2%  proc-vmstat.nr_active_anon
   1165686-8.2%1070277proc-vmstat.nr_file_pages
185928 ±  2% +14.9% 213697proc-vmstat.nr_inactive_anon
 35436-2.9%  34420proc-vmstat.nr_inactive_file
 40463 ±  8% +38.2%  55918 ±  3%  proc-vmstat.nr_mapped
345824   -27.3% 251424proc-vmstat.nr_shmem
 28871-1.4%  28477proc-vmstat.nr_slab_reclaimable
324534   -37.2% 203663 ±  2%  proc-vmstat.nr_zone_active_anon

[axboe-block:for-next] [block] bd4a633b6f: fsmark.files_per_sec -64.5% regression

2024-06-24 Thread kernel test robot



Hello,

kernel test robot noticed a -64.5% regression of fsmark.files_per_sec on:


commit: bd4a633b6f7c3c6b6ebc1a07317643270e751a94 ("block: move the nonrot flag 
to queue_limits")
https://git.kernel.org/cgit/linux/kernel/git/axboe/linux-block.git for-next

testcase: fsmark
test machine: 48 threads 2 sockets Intel(R) Xeon(R) CPU E5-2697 v2 @ 2.70GHz 
(Ivy Bridge-EP) with 64G memory
parameters:

iterations: 1x
nr_threads: 32t
disk: 1SSD
fs: btrfs
fs2: nfsv4
filesize: 9B
test_size: 400M
sync_method: fsyncBeforeClose
nr_directories: 16d
nr_files_per_directory: 256fpd
cpufreq_governor: performance


In addition to that, the commit also has significant impact on the following 
tests:

+--++
| testcase: change | fsmark: fsmark.files_per_sec -54.0% regression 
|
| test machine | 48 threads 2 sockets Intel(R) Xeon(R) CPU E5-2697 v2 @ 
2.70GHz (Ivy Bridge-EP) with 64G memory |
| test parameters  | cpufreq_governor=performance   
|
|  | disk=1SSD  
|
|  | filesize=8K
|
|  | fs2=nfsv4  
|
|  | fs=btrfs   
|
|  | iterations=1x  
|
|  | nr_directories=16d 
|
|  | nr_files_per_directory=256fpd  
|
|  | nr_threads=32t 
|
|  | sync_method=fsyncBeforeClose   
|
|  | test_size=400M 
|
+--++
| testcase: change | fxmark: fxmark.ssd_btrfs_DWSL_4_directio.works/sec -75.6% 
regression   |
| test machine | 96 threads 2 sockets Intel(R) Xeon(R) Gold 6252 CPU @ 
2.10GHz (Cascade Lake) with 512G memory  |
| test parameters  | cpufreq_governor=performance   
|
|  | directio=directio  
|
|  | disk=1SSD  
|
|  | fstype=btrfs   
|
|  | media=ssd  
|
|  | test=DWSL  
|
|  | thread_nr=4
|
+--++
| testcase: change | fxmark: fxmark.ssd_btrfs_MWUM_4_directio.works/sec -45.9% 
regression   |
| test machine | 96 threads 2 sockets Intel(R) Xeon(R) Gold 6252 CPU @ 
2.10GHz (Cascade Lake) with 512G memory  |
| test parameters  | cpufreq_governor=performance   
|
|  | directio=directio  
|
|  | disk=1SSD  
|
|  | fstype=btrfs   
|
|  | media=ssd  
|
|  | test=MWUM  
|
|  | thread_nr=4
|
+--++


If you fix the issue in a separate patch/c

Re: [PATCH] xen: xen-pciback: Export a bridge and all its children as per TODO

2024-06-10 Thread kernel test robot
Hi Abhinav,

kernel test robot noticed the following build warnings:

[auto build test WARNING on xen-tip/linux-next]
[also build test WARNING on linus/master v6.10-rc3 next-20240607]
[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/Abhinav-Jain/xen-xen-pciback-Export-a-bridge-and-all-its-children-as-per-TODO/20240610-024623
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
patch link:
https://lore.kernel.org/r/20240609184410.53500-1-jain.abhinav177%40gmail.com
patch subject: [PATCH] xen: xen-pciback: Export a bridge and all its children 
as per TODO
config: x86_64-randconfig-006-20240610 
(https://download.01.org/0day-ci/archive/20240610/202406101933.49pm50ii-...@intel.com/config)
compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 
617a15a9eac96088ae5e9134248d8236e34b91b1)
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20240610/202406101933.49pm50ii-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202406101933.49pm50ii-...@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/xen/xen-pciback/xenbus.c:262:21: warning: use of logical '&&' with 
>> constant operand [-Wconstant-logical-operand]
 262 | if ((dev->hdr_type && PCI_HEADER_TYPE_MASK) == 
PCI_HEADER_TYPE_BRIDGE) {
 |^  
   drivers/xen/xen-pciback/xenbus.c:262:21: note: use '&' for a bitwise 
operation
 262 | if ((dev->hdr_type && PCI_HEADER_TYPE_MASK) == 
PCI_HEADER_TYPE_BRIDGE) {
 |^~
 |&
   drivers/xen/xen-pciback/xenbus.c:262:21: note: remove constant to silence 
this warning
 262 | if ((dev->hdr_type && PCI_HEADER_TYPE_MASK) == 
PCI_HEADER_TYPE_BRIDGE) {
 |^~~
   drivers/xen/xen-pciback/xenbus.c:270:12: error: no member named 'domain' in 
'struct pci_dev'
 270 | child->domain, child->bus->number,
 | ~  ^
   include/linux/dev_printk.h:163:47: note: expanded from macro 'dev_dbg'
 163 | dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), 
##__VA_ARGS__); \
 | 
^~~
   include/linux/dev_printk.h:129:34: note: expanded from macro 'dev_printk'
 129 | _dev_printk(level, dev, fmt, ##__VA_ARGS__); 
   \
 |^~~
   drivers/xen/xen-pciback/xenbus.c:275:20: error: no member named 'domain' in 
'struct pci_dev'
 275 |   child->domain,
 |   ~  ^
   drivers/xen/xen-pciback/xenbus.c:284:13: error: no member named 'domain' in 
'struct pci_dev'
 284 | child->domain,
 | ~  ^
   include/linux/dev_printk.h:144:65: note: expanded from macro 'dev_err'
 144 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), 
##__VA_ARGS__)
 |  
  ^~~
   include/linux/dev_printk.h:110:23: note: expanded from macro 
'dev_printk_index_wrap'
 110 | _p_func(dev, fmt, ##__VA_ARGS__);
   \
 | ^~~
   1 warning and 3 errors generated.


vim +262 drivers/xen/xen-pciback/xenbus.c

   225  
   226  static int xen_pcibk_export_device(struct xen_pcibk_device *pdev,
   227   int domain, int bus, int slot, int 
func,
   228   int devid)
   229  {
   230  struct pci_dev *dev;
   231  int err = 0;
   232  
   233  dev_dbg(>xdev->dev, "exporting dom %x bus %x slot %x func 
%x\n",
   234  domain, bus, slot, func);
   235  
   236  dev = pcistub_get_pci_dev_by_slot(pdev, domain, bus, slot, 
func);
   237  if (!dev) {
   238  err = -EINVAL;
   239  xenbus_dev_fatal(pdev->xdev, err,
   240   "Couldn't locate PCI device "
   241   "(%04x:%02x:%02x.%d)! "
   242   "perhaps

Re: [PATCH] xen: xen-pciback: Export a bridge and all its children as per TODO

2024-06-10 Thread kernel test robot
Hi Abhinav,

kernel test robot noticed the following build errors:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on linus/master v6.10-rc3 next-20240607]
[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/Abhinav-Jain/xen-xen-pciback-Export-a-bridge-and-all-its-children-as-per-TODO/20240610-024623
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
patch link:
https://lore.kernel.org/r/20240609184410.53500-1-jain.abhinav177%40gmail.com
patch subject: [PATCH] xen: xen-pciback: Export a bridge and all its children 
as per TODO
config: arm64-defconfig 
(https://download.01.org/0day-ci/archive/20240610/202406101511.hto5m855-...@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/20240610/202406101511.hto5m855-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202406101511.hto5m855-...@intel.com/

All errors (new ones prefixed by >>):

   In file included from include/linux/device.h:15,
from include/xen/xenbus.h:37,
from drivers/xen/xen-pciback/xenbus.c:15:
   drivers/xen/xen-pciback/xenbus.c: In function 'xen_pcibk_export_device':
>> drivers/xen/xen-pciback/xenbus.c:270:38: error: 'struct pci_dev' has no 
>> member named 'domain'
 270 | child->domain, child->bus->number,
 |  ^~
   include/linux/dev_printk.h:129:48: note: in definition of macro 'dev_printk'
 129 | _dev_printk(level, dev, fmt, ##__VA_ARGS__); 
   \
 |^~~
   drivers/xen/xen-pciback/xenbus.c:268:25: note: in expansion of macro 
'dev_dbg'
 268 | dev_dbg(>xdev->dev,
 | ^~~
   drivers/xen/xen-pciback/xenbus.c:275:60: error: 'struct pci_dev' has no 
member named 'domain'
 275 |   child->domain,
 |^~
   drivers/xen/xen-pciback/xenbus.c:284:46: error: 'struct pci_dev' has no 
member named 'domain'
 284 | child->domain,
 |  ^~
   include/linux/dev_printk.h:110:37: note: in definition of macro 
'dev_printk_index_wrap'
 110 | _p_func(dev, fmt, ##__VA_ARGS__);
   \
 | ^~~
   drivers/xen/xen-pciback/xenbus.c:281:33: note: in expansion of macro 
'dev_err'
 281 | dev_err(>xdev->dev,
 | ^~~


vim +270 drivers/xen/xen-pciback/xenbus.c

   225  
   226  static int xen_pcibk_export_device(struct xen_pcibk_device *pdev,
   227   int domain, int bus, int slot, int 
func,
   228   int devid)
   229  {
   230  struct pci_dev *dev;
   231  int err = 0;
   232  
   233  dev_dbg(>xdev->dev, "exporting dom %x bus %x slot %x func 
%x\n",
   234  domain, bus, slot, func);
   235  
   236  dev = pcistub_get_pci_dev_by_slot(pdev, domain, bus, slot, 
func);
   237  if (!dev) {
   238  err = -EINVAL;
   239  xenbus_dev_fatal(pdev->xdev, err,
   240   "Couldn't locate PCI device "
   241   "(%04x:%02x:%02x.%d)! "
   242   "perhaps already in-use?",
   243   domain, bus, slot, func);
   244  goto out;
   245  }
   246  
   247  err = xen_pcibk_add_pci_dev(pdev, dev, devid,
   248  xen_pcibk_publish_pci_dev);
   249  if (err)
   250  goto out;
   251  
   252  dev_info(>dev, "registering for %d\n", 
pdev->xdev->otherend_id);
   253  if (xen_register_device_domain_owner(dev,
   254   pdev->xdev->otherend_id) 
!= 0) {
   255  dev_err(>dev, "Stealing ownership from dom%d.\n",
   256  xen_find_device_domain_owner(dev));
   257  xen_unregister_device_domain_owner(dev);
   258

Re: [PATCH] x86/xen: Fix some null pointer dereference issues in smp.c

2024-01-15 Thread kernel test robot
Hi Kunwu,

kernel test robot noticed the following build warnings:

[auto build test WARNING on tip/x86/core]
[also build test WARNING on linus/master v6.7 next-20240112]
[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/Kunwu-Chan/x86-xen-Fix-some-null-pointer-dereference-issues-in-smp-c/20240115-180429
base:   tip/x86/core
patch link:
https://lore.kernel.org/r/20240115100138.34340-1-chentao%40kylinos.cn
patch subject: [PATCH] x86/xen: Fix some null pointer dereference issues in 
smp.c
config: x86_64-rhel-8.3-rust 
(https://download.01.org/0day-ci/archive/20240116/202401161119.iof6bqsf-...@intel.com/config)
compiler: ClangBuiltLinux clang version 17.0.6 
(https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20240116/202401161119.iof6bqsf-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202401161119.iof6bqsf-...@intel.com/

All warnings (new ones prefixed by >>):

>> arch/x86/xen/smp.c:68:6: warning: variable 'rc' is used uninitialized 
>> whenever 'if' condition is true [-Wsometimes-uninitialized]
  68 | if (!resched_name)
 | ^
   arch/x86/xen/smp.c:127:9: note: uninitialized use occurs here
 127 | return rc;
 |^~
   arch/x86/xen/smp.c:68:2: note: remove the 'if' if its condition is always 
false
  68 | if (!resched_name)
 | ^~
  69 | goto fail;
 | ~
   arch/x86/xen/smp.c:64:8: note: initialize the variable 'rc' to silence this 
warning
  64 | int rc;
 |   ^
 |= 0
   1 warning generated.


vim +68 arch/x86/xen/smp.c

61  
62  int xen_smp_intr_init(unsigned int cpu)
63  {
64  int rc;
65  char *resched_name, *callfunc_name, *debug_name;
66  
67  resched_name = kasprintf(GFP_KERNEL, "resched%d", cpu);
  > 68  if (!resched_name)
69  goto fail;
70  per_cpu(xen_resched_irq, cpu).name = resched_name;
71  rc = bind_ipi_to_irqhandler(XEN_RESCHEDULE_VECTOR,
72  cpu,
73  xen_reschedule_interrupt,
74  IRQF_PERCPU|IRQF_NOBALANCING,
75  resched_name,
76  NULL);
77  if (rc < 0)
78  goto fail;
79  per_cpu(xen_resched_irq, cpu).irq = rc;
80  
81  callfunc_name = kasprintf(GFP_KERNEL, "callfunc%d", cpu);
82  if (!callfunc_name)
83  goto fail;
84  per_cpu(xen_callfunc_irq, cpu).name = callfunc_name;
85  rc = bind_ipi_to_irqhandler(XEN_CALL_FUNCTION_VECTOR,
86  cpu,
87  xen_call_function_interrupt,
88  IRQF_PERCPU|IRQF_NOBALANCING,
89  callfunc_name,
90  NULL);
91  if (rc < 0)
92  goto fail;
93  per_cpu(xen_callfunc_irq, cpu).irq = rc;
94  
95  if (!xen_fifo_events) {
96  debug_name = kasprintf(GFP_KERNEL, "debug%d", cpu);
97  if (!debug_name)
98  goto fail;
99  per_cpu(xen_debug_irq, cpu).name = debug_name;
   100  rc = bind_virq_to_irqhandler(VIRQ_DEBUG, cpu,
   101   xen_debug_interrupt,
   102   IRQF_PERCPU | 
IRQF_NOBALANCING,
   103   debug_name, NULL);
   104  if (rc < 0)
   105  goto fail;
   106  per_cpu(xen_debug_irq, cpu).irq = rc;
   107  }
   108  
   109  callfunc_name = kasprintf(GFP_KERNEL, "callfuncsingle%d", cpu);
   110  if (!callfunc_name)
   111  goto fail;
   112  per_cpu(xen_callfuncsingle_irq, cpu).name = callfunc_name;
   113  rc = bind_ipi_to_irqhandler(XEN_CALL_FUNCTION_SINGLE_VECTOR,
   114  cpu,
   115  

Re: [PATCH] xen/gntdev: Fix the abuse of underlying struct page in DMA-buf import

2024-01-06 Thread kernel test robot
Hi Oleksandr,

kernel test robot noticed the following build warnings:

[auto build test WARNING on xen-tip/linux-next]
[also build test WARNING on linus/master v6.7-rc8 next-20240105]
[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/Oleksandr-Tyshchenko/xen-gntdev-Fix-the-abuse-of-underlying-struct-page-in-DMA-buf-import/20240105-025516
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
patch link:
https://lore.kernel.org/r/20240104185327.177376-1-olekstysh%40gmail.com
patch subject: [PATCH] xen/gntdev: Fix the abuse of underlying struct page in 
DMA-buf import
config: x86_64-allmodconfig 
(https://download.01.org/0day-ci/archive/20240106/202401062122.it6zvlg0-...@intel.com/config)
compiler: ClangBuiltLinux clang version 17.0.6 
(https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20240106/202401062122.it6zvlg0-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202401062122.it6zvlg0-...@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/xen/gntdev-dmabuf.c:623:6: warning: variable 'ret' is used 
>> uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
 623 | if (!gfns)
 | ^
   drivers/xen/gntdev-dmabuf.c:660:9: note: uninitialized use occurs here
 660 | return ret;
 |^~~
   drivers/xen/gntdev-dmabuf.c:623:2: note: remove the 'if' if its condition is 
always false
 623 | if (!gfns)
 | ^~
 624 | goto fail_unmap;
 | ~~~
   drivers/xen/gntdev-dmabuf.c:569:43: note: initialize the variable 'ret' to 
silence this warning
 569 | struct gntdev_dmabuf *gntdev_dmabuf, *ret;
 |  ^
 |   = NULL
   1 warning generated.


vim +623 drivers/xen/gntdev-dmabuf.c

   564  
   565  static struct gntdev_dmabuf *
   566  dmabuf_imp_to_refs(struct gntdev_dmabuf_priv *priv, struct device *dev,
   567 int fd, int count, int domid)
   568  {
   569  struct gntdev_dmabuf *gntdev_dmabuf, *ret;
   570  struct dma_buf *dma_buf;
   571  struct dma_buf_attachment *attach;
   572  struct sg_table *sgt;
   573  struct sg_dma_page_iter sg_iter;
   574  unsigned long *gfns;
   575  int i;
   576  
   577  dma_buf = dma_buf_get(fd);
   578  if (IS_ERR(dma_buf))
   579  return ERR_CAST(dma_buf);
   580  
   581  gntdev_dmabuf = dmabuf_imp_alloc_storage(count);
   582  if (IS_ERR(gntdev_dmabuf)) {
   583  ret = gntdev_dmabuf;
   584  goto fail_put;
   585  }
   586  
   587  gntdev_dmabuf->priv = priv;
   588  gntdev_dmabuf->fd = fd;
   589  
   590  attach = dma_buf_attach(dma_buf, dev);
   591  if (IS_ERR(attach)) {
   592  ret = ERR_CAST(attach);
   593  goto fail_free_obj;
   594  }
   595  
   596  gntdev_dmabuf->u.imp.attach = attach;
   597  
   598  sgt = dma_buf_map_attachment_unlocked(attach, 
DMA_BIDIRECTIONAL);
   599  if (IS_ERR(sgt)) {
   600  ret = ERR_CAST(sgt);
   601  goto fail_detach;
   602  }
   603  
   604  /* Check that we have zero offset. */
   605  if (sgt->sgl->offset) {
   606  ret = ERR_PTR(-EINVAL);
   607  pr_debug("DMA buffer has %d bytes offset, user-space 
expects 0\n",
   608   sgt->sgl->offset);
   609  goto fail_unmap;
   610  }
   611  
   612  /* Check number of pages that imported buffer has. */
   613  if (attach->dmabuf->size != gntdev_dmabuf->nr_pages << 
PAGE_SHIFT) {
   614  ret = ERR_PTR(-EINVAL);
   615  pr_debug("DMA buffer has %zu pages, user-space expects 
%d\n",
   616   attach->dmabuf->size, gntdev_dmabuf->nr_pages);
   617  goto fail_unmap;
   618  }
   619  
   620  gntdev_dmabuf->u.imp.sgt = sgt;
   621  
   622  gfns = kcalloc(count, sizeof(*gfns), GFP_KERNEL);
 > 623  if (!gfns)
   624  goto fail_unmap;
   625  
   626  /* N

[linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread kernel test robot


hi, Thomas Gleixner,

as we understand, this commit be5341eb0d doesn't introduce new WARNING, it just
converts the WARING in another type.


b82a8dbd3d2f4563 be5341eb0d43b1e754799498bd2
 ---
   fail:runs  %reproductionfail:runs
   | | |
  9:9 -100%:9 
dmesg.WARNING:CPU:#PID:#at_do_int80_syscall_32
   :9  100%   9:9 
dmesg.WARNING:CPU:#PID:#at_int80_emulation


however, we failed to bisect dmesg.WARNING:CPU:#PID:#at_do_int80_syscall_32.
at the same time, we are not sure enough if this WARNING worth to care? or it's
just due to our test config (we used a randconfig in this test which could be
got from below link).

so we just report to you what we observed in our tests, in case it's still
expose some potential problems, and maybe you could find correct owner to make
some code fix.


Hello,

kernel test robot noticed "WARNING:CPU:#PID:#at_int80_emulation" on:

commit: be5341eb0d43b1e754799498bd2e8756cc167a41 ("x86/entry: Convert INT 0x80 
emulation to IDTENTRY")
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git master

[test failed on linux-next/master 11651f8cb2e88372d4ed523d909514dc9a613ea3]

in testcase: boot

compiler: clang-16
test machine: qemu-system-x86_64 -enable-kvm -cpu SandyBridge -smp 2 -m 16G

(please refer to attached dmesg/kmsg for entire log/backtrace)



[ 13.481107][ T48] WARNING: CPU: 0 PID: 48 at int80_emulation 
(arch/x86/entry/common.c:164) 
[   13.481454][   T48] Modules linked in:
[   13.481655][   T48] CPU: 0 PID: 48 Comm: init Tainted: G N 
6.7.0-rc4-2-gbe5341eb0d43 #1
[ 13.482162][ T48] RIP: 0010:int80_emulation (arch/x86/entry/common.c:164) 
[ 13.482437][ T48] Code: 01 00 00 77 43 89 c1 48 81 f9 c9 01 00 00 48 19 c9 21 
c1 48 89 df 4c 8b 1c cd 90 12 20 9a 41 ba 27 cb d4 4f 45 03 53 fc 74 02 <0f> 0b 
41 ff d3 48 89 c1 48 89 4b 50 90 48 89 df 5b 41 5e 31 c0 31
All code

   0:   01 00   add%eax,(%rax)
   2:   00 77 43add%dh,0x43(%rdi)
   5:   89 c1   mov%eax,%ecx
   7:   48 81 f9 c9 01 00 00cmp$0x1c9,%rcx
   e:   48 19 c9sbb%rcx,%rcx
  11:   21 c1   and%eax,%ecx
  13:   48 89 dfmov%rbx,%rdi
  16:   4c 8b 1c cd 90 12 20mov-0x65dfed70(,%rcx,8),%r11
  1d:   9a 
  1e:   41 ba 27 cb d4 4f   mov$0x4fd4cb27,%r10d
  24:   45 03 53 fc add-0x4(%r11),%r10d
  28:   74 02   je 0x2c
  2a:*  0f 0b   ud2 <-- trapping instruction
  2c:   41 ff d3call   *%r11
  2f:   48 89 c1mov%rax,%rcx
  32:   48 89 4b 50 mov%rcx,0x50(%rbx)
  36:   90  nop
  37:   48 89 dfmov%rbx,%rdi
  3a:   5b  pop%rbx
  3b:   41 5e   pop%r14
  3d:   31 c0   xor%eax,%eax
  3f:   31  .byte 0x31

Code starting with the faulting instruction
===
   0:   0f 0b   ud2
   2:   41 ff d3call   *%r11
   5:   48 89 c1mov%rax,%rcx
   8:   48 89 4b 50 mov%rcx,0x50(%rbx)
   c:   90  nop
   d:   48 89 dfmov%rbx,%rdi
  10:   5b  pop%rbx
  11:   41 5e   pop%r14
  13:   31 c0   xor%eax,%eax
  15:   31  .byte 0x31
[   13.483405][   T48] RSP: :9d1ffbbd7f40 EFLAGS: 00010a92
[   13.483713][   T48] RAX: 001b RBX: 9d1ffbbd7f58 RCX: 
001b
[   13.484112][   T48] RDX:  RSI:  RDI: 
9d1ffbbd7f58
[   13.484510][   T48] RBP:  R08:  R09: 
002b
[   13.484910][   T48] R10: c26707b6 R11: 993719d0 R12: 

[   13.485314][   T48] R13:  R14: 001b R15: 

[   13.485715][   T48] FS:  () GS:9a85c000(0063) 
knlGS:f7fc7040
[   13.486168][   T48] CS:  0010 DS: 002b ES: 002b CR0: 80050033
[   13.486500][   T48] CR2: f7fc2204 CR3: 00013b76a000 CR4: 
000406f0
[   13.486901][   T48] DR0:  DR1:  DR2: 

[   13.487299][   T48] DR3:  DR6: fffe0ff0 DR7: 
0400
[   13.487697][   T48] Call Trace:
[   13.487867][   T48]  
[ 13.488018][ T48] ? __warn (kernel/panic.c:236 kernel/panic.c:677) 
[ 13.488229][ T48] ? int80_emulation (arch/x86/entry/common.c:164) 
[ 13.488474][ T48] ? report_cfi_failure (kernel/cfi.c:22) 
[ 13.488731][ T48] ? handle_cfi_failure (arch/x86/kernel/cfi.c:80) 
[ 13.488997][ T48] ? __cfi_sys_ni_posix_timers (kernel/time/p

Re: [PATCH 1/4] x86/percpu: Use explicit segment registers in lib/cmpxchg{8,16}b_emu.S

2023-10-26 Thread kernel test robot



Hello,

kernel test robot noticed "general_protection_fault:#[##]" on:

commit: 33c7952d925e905f7af1fb7628e48e03f59885da ("[PATCH 1/4] x86/percpu: Use 
explicit segment registers in lib/cmpxchg{8,16}b_emu.S")
url: 
https://github.com/intel-lab-lkp/linux/commits/Uros-Bizjak/x86-percpu-Use-explicit-segment-registers-in-lib-cmpxchg-8-16-b_emu-S/20231017-111304
base: https://git.kernel.org/cgit/linux/kernel/git/tip/tip.git 
92fe9bb77b0c9fade150350fdb0629a662f0923f
patch link: 
https://lore.kernel.org/all/20231012161237.114733-2-ubiz...@gmail.com/
patch subject: [PATCH 1/4] x86/percpu: Use explicit segment registers in 
lib/cmpxchg{8,16}b_emu.S

in testcase: boot

compiler: gcc-12
test machine: qemu-system-x86_64 -enable-kvm -cpu SandyBridge -smp 2 -m 16G

(please refer to attached dmesg/kmsg for entire log/backtrace)


+--+++
|  | 92fe9bb77b | 33c7952d92 |
+--+++
| boot_successes   | 7  | 0  |
| boot_failures| 0  | 7  |
| general_protection_fault:#[##]   | 0  | 7  |
| EIP:this_cpu_cmpxchg8b_emu   | 0  | 7  |
| Kernel_panic-not_syncing:Fatal_exception | 0  | 7  |
+--+++


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 
| Closes: 
https://lore.kernel.org/oe-lkp/202310261417.b269d37e-oliver.s...@intel.com


[0.186570][T0] stackdepot hash table entries: 65536 (order: 6, 262144 
bytes, linear)
[0.187499][T0] Initializing HighMem for node 0 (0002ebfe:000bffe0)
[1.727965][T0] Initializing Movable for node 0 (:)
[1.943274][T0] Checking if this processor honours the WP bit even in 
supervisor mode...Ok.
[1.944313][T0] Memory: 2896220K/3145208K available (16182K kernel code, 
5537K rwdata, 11756K rodata, 816K init, 9720K bss, 248988K reserved, 0K 
cma-reserved, 2379656K highmem)
[1.947172][T0] general protection fault:  [#1] PREEMPT
[1.947900][T0] CPU: 0 PID: 0 Comm: swapper Not tainted 
6.6.0-rc1-00024-g33c7952d925e #1 8d4b014f9a0a85cc9a3f6a52ed8e88f1e431f74e
[1.949317][T0] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), 
BIOS 1.16.2-debian-1.16.2-1 04/01/2014
[ 1.950480][ T0] EIP: this_cpu_cmpxchg8b_emu 
(kbuild/src/consumer/arch/x86/lib/cmpxchg8b_emu.S:73) 
[ 1.951093][ T0] Code: ff ff ff 8d b4 26 00 00 00 00 66 90 83 c6 01 3c 3d 0f 95 
c0 0f b6 c0 83 c0 01 e9 56 ff ff ff bf ff ff ff ff eb a6 cc cc 9c fa <64> 3b 06 
75 13 64 3b 56 04 75 0d 64 89 1e 64 89 4e 04 83 0c 24 40
All code

   0:   ff  (bad)
   1:   ff  (bad)
   2:   ff 8d b4 26 00 00   decl   0x26b4(%rbp)
   8:   00 00   add%al,(%rax)
   a:   66 90   xchg   %ax,%ax
   c:   83 c6 01add$0x1,%esi
   f:   3c 3d   cmp$0x3d,%al
  11:   0f 95 c0setne  %al
  14:   0f b6 c0movzbl %al,%eax
  17:   83 c0 01add$0x1,%eax
  1a:   e9 56 ff ff ff  jmp0xff75
  1f:   bf ff ff ff ff  mov$0x,%edi
  24:   eb a6   jmp0xffcc
  26:   cc  int3
  27:   cc  int3
  28:   9c  pushf
  29:   fa  cli
  2a:*  64 3b 06cmp%fs:(%rsi),%eax  <-- trapping 
instruction
  2d:   75 13   jne0x42
  2f:   64 3b 56 04 cmp%fs:0x4(%rsi),%edx
  33:   75 0d   jne0x42
  35:   64 89 1emov%ebx,%fs:(%rsi)
  38:   64 89 4e 04 mov%ecx,%fs:0x4(%rsi)
  3c:   83 0c 24 40 orl$0x40,(%rsp)

Code starting with the faulting instruction
===
   0:   64 3b 06cmp%fs:(%rsi),%eax
   3:   75 13   jne0x18
   5:   64 3b 56 04 cmp%fs:0x4(%rsi),%edx
   9:   75 0d   jne0x18
   b:   64 89 1emov%ebx,%fs:(%rsi)
   e:   64 89 4e 04 mov%ecx,%fs:0x4(%rsi)
  12:   83 0c 24 40 orl$0x40,(%rsp)
[1.953397][T0] EAX: c3c01100 EBX: c3c01180 ECX: 0004 EDX: 0003
[1.954231][T0] ESI: e52cd090 EDI: e52cd090 EBP: c2b4bf00 ESP: c2b4bec4
[1.955060][T0] DS: 007b ES: 007b FS:  GS:  SS: 0068 EFLAGS: 
00210082
[1.955949][T0] CR0: 80050033 CR2: ffdeb000 CR3: 031b5000 CR4: 0090
[1.956783][T0] DR0:  DR1:  DR2:  DR3: 
[1.9576

Re: [PATCH v2] acpi/processor: sanitize _PDC buffer bits when running as Xen dom0

2023-09-06 Thread kernel test robot
Hi Jason,

kernel test robot noticed the following build errors:

[auto build test ERROR on tip/x86/core]
[also build test ERROR on v6.5]
[cannot apply to rafael-pm/linux-next linus/master next-20230906]
[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/Jason-Andryuk/acpi-processor-sanitize-_PDC-buffer-bits-when-running-as-Xen-dom0/20230907-022235
base:   tip/x86/core
patch link:
https://lore.kernel.org/r/20230906182125.48642-1-jandryuk%40gmail.com
patch subject: [PATCH v2] acpi/processor: sanitize _PDC buffer bits when 
running as Xen dom0
config: i386-randconfig-003-20230907 
(https://download.01.org/0day-ci/archive/20230907/202309070741.2n0k8fjn-...@intel.com/config)
compiler: clang version 16.0.4 (https://github.com/llvm/llvm-project.git 
ae42196bc493ffe877a7e3dff8be32035dea4d07)
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20230907/202309070741.2n0k8fjn-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202309070741.2n0k8fjn-...@intel.com/

All errors (new ones prefixed by >>):

>> drivers/acpi/processor_pdc.c:147:3: error: call to undeclared function 
>> 'xen_sanitize_pdc'; ISO C99 and later do not support implicit function 
>> declarations [-Wimplicit-function-declaration]
   xen_sanitize_pdc(buffer);
   ^
   1 error generated.


vim +/xen_sanitize_pdc +147 drivers/acpi/processor_pdc.c

   116  
   117  /*
   118   * _PDC is required for a BIOS-OS handshake for most of the newer
   119   * ACPI processor features.
   120   */
   121  static acpi_status
   122  acpi_processor_eval_pdc(acpi_handle handle, struct acpi_object_list 
*pdc_in)
   123  {
   124  acpi_status status = AE_OK;
   125  union acpi_object *obj;
   126  u32 *buffer = NULL;
   127  
   128  obj = pdc_in->pointer;
   129  buffer = (u32 *)(obj->buffer.pointer);
   130  
   131  if (boot_option_idle_override == IDLE_NOMWAIT) {
   132  /*
   133   * If mwait is disabled for CPU C-states, the C2C3_FFH 
access
   134   * mode will be disabled in the parameter of _PDC 
object.
   135   * Of course C1_FFH access mode will also be disabled.
   136   */
   137  buffer[2] &= ~(ACPI_PDC_C_C2C3_FFH | ACPI_PDC_C_C1_FFH);
   138  }
   139  
   140  if (xen_initial_domain()) {
   141  /*
   142   * When Linux is running as Xen dom0, the hypervisor is 
the
   143   * entity in charge of the processor power management, 
and so
   144   * Xen needs to check the OS capabilities reported in 
the _PDC
   145   * buffer matches what the hypervisor driver supports.
   146   */
 > 147  xen_sanitize_pdc(buffer);
   148  }
   149  
   150  status = acpi_evaluate_object(handle, "_PDC", pdc_in, NULL);
   151  
   152  if (ACPI_FAILURE(status))
   153  acpi_handle_debug(handle,
   154  "Could not evaluate _PDC, using legacy perf 
control\n");
   155  
   156  return status;
   157  }
   158  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki



Re: [PATCH v2] acpi/processor: sanitize _PDC buffer bits when running as Xen dom0

2023-09-06 Thread kernel test robot
Hi Jason,

kernel test robot noticed the following build errors:

[auto build test ERROR on tip/x86/core]
[also build test ERROR on v6.5]
[cannot apply to rafael-pm/linux-next linus/master next-20230906]
[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/Jason-Andryuk/acpi-processor-sanitize-_PDC-buffer-bits-when-running-as-Xen-dom0/20230907-022235
base:   tip/x86/core
patch link:
https://lore.kernel.org/r/20230906182125.48642-1-jandryuk%40gmail.com
patch subject: [PATCH v2] acpi/processor: sanitize _PDC buffer bits when 
running as Xen dom0
config: x86_64-randconfig-r011-20230907 
(https://download.01.org/0day-ci/archive/20230907/202309070625.djudcgzg-...@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20230907/202309070625.djudcgzg-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202309070625.djudcgzg-...@intel.com/

All errors (new ones prefixed by >>):

   drivers/acpi/processor_pdc.c: In function 'acpi_processor_eval_pdc':
>> drivers/acpi/processor_pdc.c:147:17: error: implicit declaration of function 
>> 'xen_sanitize_pdc' [-Werror=implicit-function-declaration]
 147 | xen_sanitize_pdc(buffer);
 | ^~~~
   cc1: some warnings being treated as errors


vim +/xen_sanitize_pdc +147 drivers/acpi/processor_pdc.c

   116  
   117  /*
   118   * _PDC is required for a BIOS-OS handshake for most of the newer
   119   * ACPI processor features.
   120   */
   121  static acpi_status
   122  acpi_processor_eval_pdc(acpi_handle handle, struct acpi_object_list 
*pdc_in)
   123  {
   124  acpi_status status = AE_OK;
   125  union acpi_object *obj;
   126  u32 *buffer = NULL;
   127  
   128  obj = pdc_in->pointer;
   129  buffer = (u32 *)(obj->buffer.pointer);
   130  
   131  if (boot_option_idle_override == IDLE_NOMWAIT) {
   132  /*
   133   * If mwait is disabled for CPU C-states, the C2C3_FFH 
access
   134   * mode will be disabled in the parameter of _PDC 
object.
   135   * Of course C1_FFH access mode will also be disabled.
   136   */
   137  buffer[2] &= ~(ACPI_PDC_C_C2C3_FFH | ACPI_PDC_C_C1_FFH);
   138  }
   139  
   140  if (xen_initial_domain()) {
   141  /*
   142   * When Linux is running as Xen dom0, the hypervisor is 
the
   143   * entity in charge of the processor power management, 
and so
   144   * Xen needs to check the OS capabilities reported in 
the _PDC
   145   * buffer matches what the hypervisor driver supports.
   146   */
 > 147  xen_sanitize_pdc(buffer);
   148  }
   149  
   150  status = acpi_evaluate_object(handle, "_PDC", pdc_in, NULL);
   151  
   152  if (ACPI_FAILURE(status))
   153  acpi_handle_debug(handle,
   154  "Could not evaluate _PDC, using legacy perf 
control\n");
   155  
   156  return status;
   157  }
   158  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki



Re: [PATCH mm-unstable v7 12/31] powerpc: Convert various functions to use ptdescs

2023-07-25 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build errors:

[auto build test ERROR on akpm-mm/mm-everything]
[also build test ERROR on next-20230725]
[cannot apply to powerpc/next powerpc/fixes s390/features geert-m68k/for-next 
geert-m68k/for-linus linus/master v6.5-rc3]
[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/Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230725-122458
base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git 
mm-everything
patch link:
https://lore.kernel.org/r/20230725042051.36691-13-vishal.moola%40gmail.com
patch subject: [PATCH mm-unstable v7 12/31] powerpc: Convert various functions 
to use ptdescs
config: powerpc-randconfig-r034-20230725 
(https://download.01.org/0day-ci/archive/20230726/202307260706.qnpjsnjr-...@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 
4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a)
reproduce: 
(https://download.01.org/0day-ci/archive/20230726/202307260706.qnpjsnjr-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202307260706.qnpjsnjr-...@intel.com/

All errors (new ones prefixed by >>):

   In file included from arch/powerpc/mm/pgtable-frag.c:12:
   In file included from include/linux/hardirq.h:11:
   In file included from arch/powerpc/include/asm/hardirq.h:6:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/powerpc/include/asm/io.h:672:
   arch/powerpc/include/asm/io-defs.h:45:1: error: performing pointer 
arithmetic on a null pointer has undefined behavior 
[-Werror,-Wnull-pointer-arithmetic]
  45 | DEF_PCI_AC_NORET(insw, (unsigned long p, void *b, unsigned long c),
 | ^~~
  46 |  (p, b, c), pio, p)
 |  ~~
   arch/powerpc/include/asm/io.h:669:3: note: expanded from macro 
'DEF_PCI_AC_NORET'
 669 | __do_##name al; \
 | ^~
   :40:1: note: expanded from here
  40 | __do_insw
 | ^
   arch/powerpc/include/asm/io.h:610:56: note: expanded from macro '__do_insw'
 610 | #define __do_insw(p, b, n)  readsw((PCI_IO_ADDR)_IO_BASE+(p), 
(b), (n))
 |~^
   In file included from arch/powerpc/mm/pgtable-frag.c:12:
   In file included from include/linux/hardirq.h:11:
   In file included from arch/powerpc/include/asm/hardirq.h:6:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/powerpc/include/asm/io.h:672:
   arch/powerpc/include/asm/io-defs.h:47:1: error: performing pointer 
arithmetic on a null pointer has undefined behavior 
[-Werror,-Wnull-pointer-arithmetic]
  47 | DEF_PCI_AC_NORET(insl, (unsigned long p, void *b, unsigned long c),
 | ^~~
  48 |  (p, b, c), pio, p)
 |  ~~
   arch/powerpc/include/asm/io.h:669:3: note: expanded from macro 
'DEF_PCI_AC_NORET'
 669 | __do_##name al; \
 | ^~
   :42:1: note: expanded from here
  42 | __do_insl
 | ^
   arch/powerpc/include/asm/io.h:611:56: note: expanded from macro '__do_insl'
 611 | #define __do_insl(p, b, n)  readsl((PCI_IO_ADDR)_IO_BASE+(p), 
(b), (n))
 |~^
   In file included from arch/powerpc/mm/pgtable-frag.c:12:
   In file included from include/linux/hardirq.h:11:
   In file included from arch/powerpc/include/asm/hardirq.h:6:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/powerpc/include/asm/io.h:672:
   arch/powerpc/include/asm/io-defs.h:49:1: error: performing pointer 
arithmetic on a null pointer has undefined behavior 
[-Werror,-Wnull-pointer-arithmetic]
  49 | DEF_PCI_AC_NORET(outsb, (unsigned long p, const void *b, unsigned 
long c),
 | 
^~
  50 |  (p, b, c), pio, p)
 |  ~~
   arch/powerpc/include/asm/io.h:669:3: note: expanded from macro 
'DEF_PCI_AC_NORET'
 669 | __do_##n

Re: [PATCH V2 2/2] xen: privcmd: Add support for irqfd

2023-07-22 Thread kernel test robot
Hi Viresh,

kernel test robot noticed the following build errors:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on linus/master v6.5-rc2 next-20230721]
[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/Viresh-Kumar/xen-privcmd-Add-support-for-irqfd/20230720-173905
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
patch link:
https://lore.kernel.org/r/a25d5f01fe9b4624aa12cab77abd001044ea02d5.1689845210.git.viresh.kumar%40linaro.org
patch subject: [PATCH V2 2/2] xen: privcmd: Add support for irqfd
config: x86_64-randconfig-r091-20230723 
(https://download.01.org/0day-ci/archive/20230723/202307231237.jhgxqmdg-...@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce: 
(https://download.01.org/0day-ci/archive/20230723/202307231237.jhgxqmdg-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202307231237.jhgxqmdg-...@intel.com/

All error/warnings (new ones prefixed by >>):

   drivers/xen/privcmd.c: In function 'privcmd_irqfd_assign':
>> drivers/xen/privcmd.c:961:19: error: implicit declaration of function 
>> 'eventfd_ctx_fileget'; did you mean 'eventfd_ctx_fdget'? 
>> [-Werror=implicit-function-declaration]
 961 | eventfd = eventfd_ctx_fileget(f.file);
 |   ^~~
 |   eventfd_ctx_fdget
>> drivers/xen/privcmd.c:961:17: warning: assignment to 'struct eventfd_ctx *' 
>> from 'int' makes pointer from integer without a cast [-Wint-conversion]
 961 | eventfd = eventfd_ctx_fileget(f.file);
 | ^
   cc1: some warnings being treated as errors


vim +961 drivers/xen/privcmd.c

   936  
   937  static int privcmd_irqfd_assign(struct privcmd_irqfd *irqfd)
   938  {
   939  struct privcmd_kernel_irqfd *kirqfd, *tmp;
   940  struct eventfd_ctx *eventfd;
   941  __poll_t events;
   942  struct fd f;
   943  int ret;
   944  
   945  kirqfd = kzalloc(sizeof(*kirqfd), GFP_KERNEL);
   946  if (!kirqfd)
   947  return -ENOMEM;
   948  
   949  kirqfd->irq = irqfd->irq;
   950  kirqfd->dom = irqfd->dom;
   951  kirqfd->level = irqfd->level;
   952  INIT_LIST_HEAD(>list);
   953  INIT_WORK(>shutdown, irqfd_shutdown);
   954  
   955  f = fdget(irqfd->fd);
   956  if (!f.file) {
   957  ret = -EBADF;
   958  goto error_kfree;
   959  }
   960  
 > 961  eventfd = eventfd_ctx_fileget(f.file);
   962  if (IS_ERR(eventfd)) {
   963  ret = PTR_ERR(eventfd);
   964  goto error_fd_put;
   965  }
   966  
   967  kirqfd->eventfd = eventfd;
   968  
   969  /*
   970   * Install our own custom wake-up handling so we are notified 
via a
   971   * callback whenever someone signals the underlying eventfd.
   972   */
   973  init_waitqueue_func_entry(>wait, irqfd_wakeup);
   974  init_poll_funcptr(>pt, irqfd_poll_func);
   975  
   976  mutex_lock(_lock);
   977  
   978  list_for_each_entry(tmp, _list, list) {
   979  if (kirqfd->eventfd == tmp->eventfd) {
   980  ret = -EBUSY;
   981  mutex_unlock(_lock);
   982  goto error_eventfd;
   983  }
   984  }
   985  
   986  list_add_tail(>list, _list);
   987  mutex_unlock(_lock);
   988  
   989  /*
   990   * Check if there was an event already pending on the eventfd 
before we
   991   * registered, and trigger it as if we didn't miss it.
   992   */
   993  events = vfs_poll(f.file, >pt);
   994  if (events & EPOLLIN)
   995  irqfd_inject(kirqfd);
   996  
   997  /*
   998   * Do not drop the file until the kirqfd is fully initialized, 
otherwise
   999   * we might race against the EPOLLHUP.
  1000   */
  1001  fdput(f);
  1002  return 0;
  1003  
  1004  error_eventfd:
  1005  eventfd_ctx_put(eventfd);
  1006  
  1007  error_fd_put:
  1008  fdput(f);
  1009  
  1010  error_kfree:
  1011  kfree(kirqfd);
  1012  return ret;
  1013  }
  1014  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki



Re: [PATCH V2 2/2] xen: privcmd: Add support for irqfd

2023-07-20 Thread kernel test robot
Hi Viresh,

kernel test robot noticed the following build errors:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on linus/master v6.5-rc2 next-20230720]
[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/Viresh-Kumar/xen-privcmd-Add-support-for-irqfd/20230720-173905
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
patch link:
https://lore.kernel.org/r/a25d5f01fe9b4624aa12cab77abd001044ea02d5.1689845210.git.viresh.kumar%40linaro.org
patch subject: [PATCH V2 2/2] xen: privcmd: Add support for irqfd
config: arm64-randconfig-r026-20230720 
(https://download.01.org/0day-ci/archive/20230721/202307210852.ukq5f98v-...@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 
4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a)
reproduce: 
(https://download.01.org/0day-ci/archive/20230721/202307210852.ukq5f98v-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202307210852.ukq5f98v-...@intel.com/

All errors (new ones prefixed by >>):

>> drivers/xen/privcmd.c:961:12: error: call to undeclared function 
>> 'eventfd_ctx_fileget'; ISO C99 and later do not support implicit function 
>> declarations [-Wimplicit-function-declaration]
 961 | eventfd = eventfd_ctx_fileget(f.file);
 |   ^
   drivers/xen/privcmd.c:961:12: note: did you mean 'eventfd_ctx_fdget'?
   include/linux/eventfd.h:56:35: note: 'eventfd_ctx_fdget' declared here
  56 | static inline struct eventfd_ctx *eventfd_ctx_fdget(int fd)
 |   ^
>> drivers/xen/privcmd.c:961:10: error: incompatible integer to pointer 
>> conversion assigning to 'struct eventfd_ctx *' from 'int' [-Wint-conversion]
 961 | eventfd = eventfd_ctx_fileget(f.file);
 | ^ ~~~
   2 errors generated.


vim +/eventfd_ctx_fileget +961 drivers/xen/privcmd.c

   936  
   937  static int privcmd_irqfd_assign(struct privcmd_irqfd *irqfd)
   938  {
   939  struct privcmd_kernel_irqfd *kirqfd, *tmp;
   940  struct eventfd_ctx *eventfd;
   941  __poll_t events;
   942  struct fd f;
   943  int ret;
   944  
   945  kirqfd = kzalloc(sizeof(*kirqfd), GFP_KERNEL);
   946  if (!kirqfd)
   947  return -ENOMEM;
   948  
   949  kirqfd->irq = irqfd->irq;
   950  kirqfd->dom = irqfd->dom;
   951  kirqfd->level = irqfd->level;
   952  INIT_LIST_HEAD(>list);
   953  INIT_WORK(>shutdown, irqfd_shutdown);
   954  
   955  f = fdget(irqfd->fd);
   956  if (!f.file) {
   957  ret = -EBADF;
   958  goto error_kfree;
   959  }
   960  
 > 961  eventfd = eventfd_ctx_fileget(f.file);
   962  if (IS_ERR(eventfd)) {
   963  ret = PTR_ERR(eventfd);
   964  goto error_fd_put;
   965  }
   966  
   967  kirqfd->eventfd = eventfd;
   968  
   969  /*
   970   * Install our own custom wake-up handling so we are notified 
via a
   971   * callback whenever someone signals the underlying eventfd.
   972   */
   973  init_waitqueue_func_entry(>wait, irqfd_wakeup);
   974  init_poll_funcptr(>pt, irqfd_poll_func);
   975  
   976  mutex_lock(_lock);
   977  
   978  list_for_each_entry(tmp, _list, list) {
   979  if (kirqfd->eventfd == tmp->eventfd) {
   980  ret = -EBUSY;
   981  mutex_unlock(_lock);
   982  goto error_eventfd;
   983  }
   984  }
   985  
   986  list_add_tail(>list, _list);
   987  mutex_unlock(_lock);
   988  
   989  /*
   990   * Check if there was an event already pending on the eventfd 
before we
   991   * registered, and trigger it as if we didn't miss it.
   992   */
   993  events = vfs_poll(f.file, >pt);
   994  if (events & EPOLLIN)
   995  irqfd_inject(kirqfd);
   996  
   997  /*
   998   * Do not drop the file until the kirqfd is fully initialized, 
otherwise
   999   * we might race against the EPOLLHUP.
  1000   */
  1001  fdput(f);
  1002  return 0;
  1003  
  1004  error_eventfd:
  1005  eventfd_ctx_put(eventfd);
  1006  
  1007  error_fd_put:
  

Re: [PATCH v5 24/33] m68k: Convert various functions to use ptdescs

2023-06-25 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build errors:

[auto build test ERROR on next-20230622]
[cannot apply to akpm-mm/mm-everything powerpc/next powerpc/fixes s390/features 
geert-m68k/for-next geert-m68k/for-linus linus/master v6.4-rc7 v6.4-rc6 
v6.4-rc5 v6.4-rc7]
[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/Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230623-050011
base:   next-20230622
patch link:
https://lore.kernel.org/r/20230622205745.79707-25-vishal.moola%40gmail.com
patch subject: [PATCH v5 24/33] m68k: Convert various functions to use ptdescs
config: m68k-randconfig-s051-20230625 
(https://download.01.org/0day-ci/archive/20230625/202306251513.wvzxggxu-...@intel.com/config)
compiler: m68k-linux-gcc (GCC) 12.3.0
reproduce: 
(https://download.01.org/0day-ci/archive/20230625/202306251513.wvzxggxu-...@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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202306251513.wvzxggxu-...@intel.com/

All error/warnings (new ones prefixed by >>):

   In file included from arch/m68k/include/asm/pgalloc.h:12,
from kernel/fork.c:103:
   arch/m68k/include/asm/mcf_pgalloc.h: In function 'pgd_alloc':
>> arch/m68k/include/asm/mcf_pgalloc.h:82:60: error: 'GFP_NOWARN' undeclared 
>> (first use in this function); did you mean 'GFP_NOWAIT'?
  82 | struct ptdesc *ptdesc = pagetable_alloc((GFP_DMA | 
GFP_NOWARN) &
 |^~
 |GFP_NOWAIT
   arch/m68k/include/asm/mcf_pgalloc.h:82:60: note: each undeclared identifier 
is reported only once for each function it appears in
   arch/m68k/include/asm/mcf_pgalloc.h: At top level:
>> arch/m68k/include/asm/mcf_pgalloc.h:23:16: warning: 'ptdesc_address' is 
>> static but used in inline function 'pte_alloc_one_kernel' which is not static
  23 | return ptdesc_address(ptdesc);
 |^~
>> arch/m68k/include/asm/mcf_pgalloc.h:17:33: warning: 'pagetable_alloc' is 
>> static but used in inline function 'pte_alloc_one_kernel' which is not static
  17 | struct ptdesc *ptdesc = pagetable_alloc((GFP_DMA | 
__GFP_ZERO) &
 | ^~~
>> arch/m68k/include/asm/mcf_pgalloc.h:10:24: warning: 'virt_to_ptdesc' is 
>> static but used in inline function 'pte_free_kernel' which is not static
  10 | pagetable_free(virt_to_ptdesc(pte));
 |^~
>> arch/m68k/include/asm/mcf_pgalloc.h:10:9: warning: 'pagetable_free' is 
>> static but used in inline function 'pte_free_kernel' which is not static
  10 | pagetable_free(virt_to_ptdesc(pte));
 | ^~
--
   In file included from arch/m68k/mm/mcfmmu.c:21:
   arch/m68k/include/asm/mcf_pgalloc.h: In function 'pgd_alloc':
>> arch/m68k/include/asm/mcf_pgalloc.h:82:60: error: 'GFP_NOWARN' undeclared 
>> (first use in this function); did you mean 'GFP_NOWAIT'?
  82 | struct ptdesc *ptdesc = pagetable_alloc((GFP_DMA | 
GFP_NOWARN) &
 |^~
 |GFP_NOWAIT
   arch/m68k/include/asm/mcf_pgalloc.h:82:60: note: each undeclared identifier 
is reported only once for each function it appears in
   arch/m68k/mm/mcfmmu.c: At top level:
   arch/m68k/mm/mcfmmu.c:36:13: warning: no previous prototype for 
'paging_init' [-Wmissing-prototypes]
  36 | void __init paging_init(void)
 | ^~~
   arch/m68k/mm/mcfmmu.c: In function 'paging_init':
   arch/m68k/mm/mcfmmu.c:41:37: warning: variable 'bootmem_end' set but not 
used [-Wunused-but-set-variable]
  41 | unsigned long next_pgtable, bootmem_end;
 | ^~~
   arch/m68k/include/asm/mcf_pgalloc.h: At top level:
>> arch/m68k/include/asm/mcf_pgalloc.h:23:16: warning: 'ptdesc_address' is 
>> static but used in inline function 'pte_alloc_one_kernel' which is not static
  23 | return ptdesc_address(ptdesc);
 |^~
>> arch/m68k/include/asm/mcf_pgalloc.h:17:33: warning: 'pagetable_alloc' is 
>> static but used in inline function 'pte_alloc_one_kernel' which is not static
  17 | struct ptdesc *ptdesc = pagetable_alloc((GFP_DMA | 
__GFP_ZERO) &
 | 

Re: [PATCH v2 1/3] vsscanf(): Integer overflow is a conversion failure

2023-06-10 Thread kernel test robot
Hi Demi,

kernel test robot noticed the following build errors:

[auto build test ERROR on media-tree/master]
[also build test ERROR on lee-mfd/for-mfd-next xen-tip/linux-next linus/master 
lee-mfd/for-mfd-fixes v6.4-rc5 next-20230609]
[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/Demi-Marie-Obenour/vsscanf-Integer-overflow-is-a-conversion-failure/20230611-010926
base:   git://linuxtv.org/media_tree.git master
patch link:
https://lore.kernel.org/r/20230610170743.2510-2-demi%40invisiblethingslab.com
patch subject: [PATCH v2 1/3] vsscanf(): Integer overflow is a conversion 
failure
config: arc-randconfig-r033-20230611 
(https://download.01.org/0day-ci/archive/20230611/202306110319.q5styx6b-...@intel.com/config)
compiler: arceb-elf-gcc (GCC) 12.3.0
reproduce (this is a W=1 build):
mkdir -p ~/bin
wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
chmod +x ~/bin/make.cross
git remote add media-tree git://linuxtv.org/media_tree.git
git fetch media-tree master
git checkout media-tree/master
b4 shazam 
https://lore.kernel.org/r/20230610170743.2510-2-d...@invisiblethingslab.com
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross 
W=1 O=build_dir ARCH=arc olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross 
W=1 O=build_dir ARCH=arc SHELL=/bin/bash

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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202306110319.q5styx6b-...@intel.com/

All errors (new ones prefixed by >>):

   lib/vsprintf.c: In function 'simple_strtoll':
>> lib/vsprintf.c:163:16: error: too few arguments to function 'simple_strntoll'
 163 | return simple_strntoll(cp, INT_MAX, endp, base);
 |^~~
   lib/vsprintf.c:134:18: note: declared here
 134 | static long long simple_strntoll(const char *cp, size_t max_chars, 
char **endp,
 |  ^~~
   lib/vsprintf.c: In function 'va_format':
   lib/vsprintf.c:1687:9: warning: function 'va_format' might be a candidate 
for 'gnu_printf' format attribute [-Wsuggest-attribute=format]
1687 | buf += vsnprintf(buf, end > buf ? end - buf : 0, 
va_fmt->fmt, va);
 | ^~~
   lib/vsprintf.c: In function 'simple_strtoll':
   lib/vsprintf.c:164:1: error: control reaches end of non-void function 
[-Werror=return-type]
 164 | }
 | ^
   cc1: some warnings being treated as errors


vim +/simple_strntoll +163 lib/vsprintf.c

900fdc4573766d Richard Fitzgerald 2021-05-14  152  
^1da177e4c3f41 Linus Torvalds 2005-04-16  153  /**
^1da177e4c3f41 Linus Torvalds 2005-04-16  154   * simple_strtoll - convert 
a string to a signed long long
^1da177e4c3f41 Linus Torvalds 2005-04-16  155   * @cp: The start of the 
string
^1da177e4c3f41 Linus Torvalds 2005-04-16  156   * @endp: A pointer to the 
end of the parsed string will be placed here
^1da177e4c3f41 Linus Torvalds 2005-04-16  157   * @base: The number base to 
use
462e471107624f Eldad Zack 2012-12-17  158   *
e8cc2b97ca5aa1 Andy Shevchenko2020-02-21  159   * This function has 
caveats. Please use kstrtoll instead.
^1da177e4c3f41 Linus Torvalds 2005-04-16  160   */
^1da177e4c3f41 Linus Torvalds 2005-04-16  161  long long 
simple_strtoll(const char *cp, char **endp, unsigned int base)
^1da177e4c3f41 Linus Torvalds 2005-04-16  162  {
900fdc4573766d Richard Fitzgerald 2021-05-14 @163   return 
simple_strntoll(cp, INT_MAX, endp, base);
^1da177e4c3f41 Linus Torvalds 2005-04-16  164  }
98d5ce0d004466 Hans Verkuil   2010-04-23  165  
EXPORT_SYMBOL(simple_strtoll);
^1da177e4c3f41 Linus Torvalds 2005-04-16  166  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki



Re: [PATCH 2/4] vsscanf(): Return -ERANGE on integer overflow

2023-06-10 Thread kernel test robot
Hi Demi,

kernel test robot noticed the following build warnings:

[auto build test WARNING on lee-mfd/for-mfd-next]
[also build test WARNING on lee-leds/for-leds-next linus/master v6.4-rc5 
next-20230609]
[cannot apply to xen-tip/linux-next lee-mfd/for-mfd-fixes]
[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/Demi-Marie-Obenour/vsscanf-Return-ERANGE-on-integer-overflow/20230610-110026
base:   https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
patch link:
https://lore.kernel.org/r/20230610025759.1813-2-demi%40invisiblethingslab.com
patch subject: [PATCH 2/4] vsscanf(): Return -ERANGE on integer overflow
config: i386-allyesconfig 
(https://download.01.org/0day-ci/archive/20230610/202306102055.zsjk8xsj-...@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build):
git remote add lee-mfd 
https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git
git fetch lee-mfd for-mfd-next
git checkout lee-mfd/for-mfd-next
b4 shazam 
https://lore.kernel.org/r/20230610025759.1813-2-d...@invisiblethingslab.com
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=i386 olddefconfig
make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash 
drivers/staging/media/atomisp/

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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202306102055.zsjk8xsj-...@intel.com/

All warnings (new ones prefixed by >>):

   In file included from 
drivers/staging/media/atomisp//pci/ia_css_acc_types.h:25,
from 
drivers/staging/media/atomisp//pci/ia_css_pipe_public.h:29,
from 
drivers/staging/media/atomisp//pci/ia_css_stream_public.h:25,
from 
drivers/staging/media/atomisp//pci/runtime/binary/interface/ia_css_binary.h:23,
from 
drivers/staging/media/atomisp//pci/runtime/debug/interface/ia_css_debug.h:25,
from 
drivers/staging/media/atomisp/pci/isp/kernels/macc/macc_1.0/ia_css_macc.host.c:18:
>> drivers/staging/media/atomisp//pci/hive_isp_css_include/platform_support.h:30:
>>  warning: "UCHAR_MAX" redefined
  30 | #define UCHAR_MAX  (255)
 | 
   In file included from include/linux/limits.h:7,
from 
drivers/staging/media/atomisp//pci/hive_isp_css_include/type_support.h:37,
from drivers/staging/media/atomisp//pci/ia_css_types.h:27,
from 
drivers/staging/media/atomisp/pci/isp/kernels/macc/macc_1.0/ia_css_macc.host.c:16:
   include/vdso/limits.h:5: note: this is the location of the previous 
definition
   5 | #define UCHAR_MAX   ((unsigned char)~0U)
 | 


vim +/UCHAR_MAX +30 
drivers/staging/media/atomisp//pci/hive_isp_css_include/platform_support.h

ad85094b293e40 
drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_include/platform_support.h
 Mauro Carvalho Chehab 2020-04-19  27  
ad85094b293e40 
drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_include/platform_support.h
 Mauro Carvalho Chehab 2020-04-19  28  #define UINT16_MAX USHRT_MAX
ad85094b293e40 
drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_include/platform_support.h
 Mauro Carvalho Chehab 2020-04-19  29  #define UINT32_MAX UINT_MAX
ad85094b293e40 
drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_include/platform_support.h
 Mauro Carvalho Chehab 2020-04-19 @30  #define UCHAR_MAX  (255)
ad85094b293e40 
drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_include/platform_support.h
 Mauro Carvalho Chehab 2020-04-19  31  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki



Re: [PATCH 1/4] Rip out simple_strtoll()

2023-06-09 Thread kernel test robot
Hi Demi,

kernel test robot noticed the following build errors:

[auto build test ERROR on lee-mfd/for-mfd-next]
[also build test ERROR on lee-leds/for-leds-next linus/master v6.4-rc5 
next-20230609]
[cannot apply to xen-tip/linux-next lee-mfd/for-mfd-fixes]
[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/Demi-Marie-Obenour/vsscanf-Return-ERANGE-on-integer-overflow/20230610-110026
base:   https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
patch link:
https://lore.kernel.org/r/20230610025759.1813-1-demi%40invisiblethingslab.com
patch subject: [PATCH 1/4] Rip out simple_strtoll()
config: csky-randconfig-r011-20230610 
(https://download.01.org/0day-ci/archive/20230610/202306101317.yibrl6oz-...@intel.com/config)
compiler: csky-linux-gcc (GCC) 12.3.0
reproduce (this is a W=1 build):
mkdir -p ~/bin
wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
chmod +x ~/bin/make.cross
git remote add lee-mfd 
https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git
git fetch lee-mfd for-mfd-next
git checkout lee-mfd/for-mfd-next
b4 shazam 
https://lore.kernel.org/r/20230610025759.1813-1-d...@invisiblethingslab.com
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross 
W=1 O=build_dir ARCH=csky olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross 
W=1 O=build_dir ARCH=csky SHELL=/bin/bash drivers/md/bcache/

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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202306101317.yibrl6oz-...@intel.com/

All errors (new ones prefixed by >>):

   drivers/md/bcache/util.c: In function 'bch_strtoll_h':
>> drivers/md/bcache/util.c:28:18: error: implicit declaration of function 
>> 'simple_strtoll'; did you mean 'simple_strtoull'? 
>> [-Werror=implicit-function-declaration]
  28 | type i = simple_ ## name(cp, , 10);   \
 |  ^~~
   drivers/md/bcache/util.c:82:1: note: in expansion of macro 'STRTO_H'
  82 | STRTO_H(strtoll, long long)
 | ^~~
   cc1: some warnings being treated as errors


vim +28 drivers/md/bcache/util.c

cafe563591446c Kent Overstreet 2013-03-23  22  
cafe563591446c Kent Overstreet 2013-03-23  23  #define STRTO_H(name, type)  
\
169ef1cf6171d3 Kent Overstreet 2013-03-28  24  int bch_ ## name ## _h(const 
char *cp, type *res)\
cafe563591446c Kent Overstreet 2013-03-23  25  {
\
cafe563591446c Kent Overstreet 2013-03-23  26   int u = 0;  
\
cafe563591446c Kent Overstreet 2013-03-23  27   char *e;
\
cafe563591446c Kent Overstreet 2013-03-23 @28   type i = simple_ ## 
name(cp, , 10);   \
cafe563591446c Kent Overstreet 2013-03-23  29   
\
cafe563591446c Kent Overstreet 2013-03-23  30   switch (tolower(*e)) {  
\
cafe563591446c Kent Overstreet 2013-03-23  31   default:
\
cafe563591446c Kent Overstreet 2013-03-23  32   return -EINVAL; 
\
cafe563591446c Kent Overstreet 2013-03-23  33   case 'y':   
\
cafe563591446c Kent Overstreet 2013-03-23  34   case 'z':   
\
cafe563591446c Kent Overstreet 2013-03-23  35   u++;
\
df561f6688fef7 Gustavo A. R. Silva 2020-08-23  36   fallthrough;
\
cafe563591446c Kent Overstreet 2013-03-23  37   case 'e':   
\
cafe563591446c Kent Overstreet 2013-03-23  38   u++;
\
df561f6688fef7 Gustavo A. R. Silva 2020-08-23  39   fallthrough;
\
cafe563591446c Kent Overstreet 2013-03-23  40   case 'p':   
\
cafe563591446c Kent Overstreet 2013-03-23  41   u++;
\
df561f6688fef7 Gustavo A. R. Silva 2020-08-

Re: [PATCH 1/4] Rip out simple_strtoll()

2023-06-09 Thread kernel test robot
Hi Demi,

kernel test robot noticed the following build errors:

[auto build test ERROR on lee-mfd/for-mfd-next]
[also build test ERROR on lee-leds/for-leds-next linus/master v6.4-rc5 
next-20230609]
[cannot apply to xen-tip/linux-next lee-mfd/for-mfd-fixes]
[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/Demi-Marie-Obenour/vsscanf-Return-ERANGE-on-integer-overflow/20230610-110026
base:   https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
patch link:
https://lore.kernel.org/r/20230610025759.1813-1-demi%40invisiblethingslab.com
patch subject: [PATCH 1/4] Rip out simple_strtoll()
config: arm-randconfig-r046-20230610 
(https://download.01.org/0day-ci/archive/20230610/202306101334.ldzersko-...@intel.com/config)
compiler: clang version 15.0.7 (https://github.com/llvm/llvm-project.git 
8dfdcc7b7bf66834a761bd8de445840ef68e4d1a)
reproduce (this is a W=1 build):
mkdir -p ~/bin
wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
git remote add lee-mfd 
https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git
git fetch lee-mfd for-mfd-next
git checkout lee-mfd/for-mfd-next
b4 shazam 
https://lore.kernel.org/r/20230610025759.1813-1-d...@invisiblethingslab.com
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang ~/bin/make.cross W=1 
O=build_dir ARCH=arm olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang ~/bin/make.cross W=1 
O=build_dir ARCH=arm SHELL=/bin/bash drivers/md/bcache/

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 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202306101334.ldzersko-...@intel.com/

All errors (new ones prefixed by >>):

>> drivers/md/bcache/util.c:82:1: error: call to undeclared function 
>> 'simple_strtoll'; ISO C99 and later do not support implicit function 
>> declarations [-Werror,-Wimplicit-function-declaration]
   STRTO_H(strtoll, long long)
   ^
   drivers/md/bcache/util.c:28:11: note: expanded from macro 'STRTO_H'
   type i = simple_ ## name(cp, , 10);   \
^
   :22:1: note: expanded from here
   simple_strtoll
   ^
   1 error generated.


vim +/simple_strtoll +82 drivers/md/bcache/util.c

cafe563591446c Kent Overstreet 2013-03-23  79  
cafe563591446c Kent Overstreet 2013-03-23  80  STRTO_H(strtoint, int)
cafe563591446c Kent Overstreet 2013-03-23  81  STRTO_H(strtouint, unsigned int)
cafe563591446c Kent Overstreet 2013-03-23 @82  STRTO_H(strtoll, long long)
cafe563591446c Kent Overstreet 2013-03-23  83  STRTO_H(strtoull, unsigned long 
long)
cafe563591446c Kent Overstreet 2013-03-23  84  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki



Re: [PATCH v3 25/34] m68k: Convert various functions to use ptdescs

2023-06-01 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build errors:

[auto build test ERROR on next-20230531]
[cannot apply to akpm-mm/mm-everything s390/features powerpc/next powerpc/fixes 
geert-m68k/for-next geert-m68k/for-linus linus/master v6.4-rc4 v6.4-rc3 
v6.4-rc2 v6.4-rc4]
[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/Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230601-053454
base:   next-20230531
patch link:
https://lore.kernel.org/r/20230531213032.25338-26-vishal.moola%40gmail.com
patch subject: [PATCH v3 25/34] m68k: Convert various functions to use ptdescs
config: m68k-randconfig-r002-20230531 
(https://download.01.org/0day-ci/archive/20230601/202306011704.i8xmwkpl-...@intel.com/config)
compiler: m68k-linux-gcc (GCC) 12.3.0
reproduce (this is a W=1 build):
mkdir -p ~/bin
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/915ab62dc3315fe0a0544fccb4ee5f3ee32694b5
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230601-053454
git checkout 915ab62dc3315fe0a0544fccb4ee5f3ee32694b5
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross 
W=1 O=build_dir ARCH=m68k olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.3.0 ~/bin/make.cross 
W=1 O=build_dir ARCH=m68k SHELL=/bin/bash arch/m68k/mm/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202306011704.i8xmwkpl-...@intel.com/

All error/warnings (new ones prefixed by >>):

   In file included from arch/m68k/include/asm/pgalloc.h:12,
from arch/m68k/mm/init.c:26:
   arch/m68k/include/asm/mcf_pgalloc.h: In function 'pgd_alloc':
>> arch/m68k/include/asm/mcf_pgalloc.h:83:59: error: 'GFP_NOWARN' undeclared 
>> (first use in this function); did you mean 'GFP_NOWAIT'?
  83 | struct ptdesc *ptdesc = pagetable_alloc(GFP_DMA | 
GFP_NOWARN, 0);
 |   ^~
 |   GFP_NOWAIT
   arch/m68k/include/asm/mcf_pgalloc.h:83:59: note: each undeclared identifier 
is reported only once for each function it appears in
   arch/m68k/include/asm/mcf_pgalloc.h: At top level:
>> arch/m68k/include/asm/mcf_pgalloc.h:22:27: warning: 'ptdesc_address' is 
>> static but used in inline function 'pte_alloc_one_kernel' which is not static
  22 | return (pte_t *) (ptdesc_address(ptdesc));
 |   ^~
>> arch/m68k/include/asm/mcf_pgalloc.h:17:33: warning: 'pagetable_alloc' is 
>> static but used in inline function 'pte_alloc_one_kernel' which is not static
  17 | struct ptdesc *ptdesc = pagetable_alloc(GFP_DMA | 
__GFP_ZERO, 0);
 | ^~~
>> arch/m68k/include/asm/mcf_pgalloc.h:10:24: warning: 'virt_to_ptdesc' is 
>> static but used in inline function 'pte_free_kernel' which is not static
  10 | pagetable_free(virt_to_ptdesc(pte));
 |^~
>> arch/m68k/include/asm/mcf_pgalloc.h:10:9: warning: 'pagetable_free' is 
>> static but used in inline function 'pte_free_kernel' which is not static
  10 | pagetable_free(virt_to_ptdesc(pte));
 | ^~
--
   In file included from arch/m68k/mm/mcfmmu.c:21:
   arch/m68k/include/asm/mcf_pgalloc.h: In function 'pgd_alloc':
>> arch/m68k/include/asm/mcf_pgalloc.h:83:59: error: 'GFP_NOWARN' undeclared 
>> (first use in this function); did you mean 'GFP_NOWAIT'?
  83 | struct ptdesc *ptdesc = pagetable_alloc(GFP_DMA | 
GFP_NOWARN, 0);
 |   ^~
 |   GFP_NOWAIT
   arch/m68k/include/asm/mcf_pgalloc.h:83:59: note: each undeclared identifier 
is reported only once for each function it appears in
   arch/m68k/mm/mcfmmu.c: At top level:
   arch/m68k/mm/mcfmmu.c:36:13: warning: no previous prototype for 
'paging_init' [-Wmissing-prototypes]
  36 | void __init paging_init(void)
 | ^~~
   arch/m68k/mm/mcfmmu.c: In function 'paging_init':
   arch/m68k/mm/mcfmmu.c:41:37: warning: variable 'bootmem_end' set b

Re: [PATCH v2 21/34] arm64: Convert various functions to use ptdescs

2023-05-01 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build errors:

[auto build test ERROR on akpm-mm/mm-everything]
[also build test ERROR on linus/master next-20230428]
[cannot apply to s390/features powerpc/next powerpc/fixes geert-m68k/for-next 
geert-m68k/for-linus v6.3]
[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/Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230502-033042
base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git 
mm-everything
patch link:
https://lore.kernel.org/r/20230501192829.17086-22-vishal.moola%40gmail.com
patch subject: [PATCH v2 21/34] arm64: Convert various functions to use ptdescs
config: arm64-randconfig-r023-20230430 
(https://download.01.org/0day-ci/archive/20230502/202305021038.c9jfvdsv-...@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 
b1465cd49efcbc114a75220b153f5a055ce7911f)
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
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# 
https://github.com/intel-lab-lkp/linux/commit/8e9481b63b5773d7c914836dcd7fbec2449902bc
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230502-033042
git checkout 8e9481b63b5773d7c914836dcd7fbec2449902bc
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=arm64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=arm64 SHELL=/bin/bash arch/arm64/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202305021038.c9jfvdsv-...@intel.com/

All errors (new ones prefixed by >>):

>> arch/arm64/mm/mmu.c:440:10: error: invalid argument type 'void' to unary 
>> expression
   BUG_ON(!ptdesc_pte_dtor(ptdesc));
  ^~~~
   include/asm-generic/bug.h:71:45: note: expanded from macro 'BUG_ON'
   #define BUG_ON(condition) do { if (unlikely(condition)) BUG(); } while (0)
   ^
   include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
   # define unlikely(x)__builtin_expect(!!(x), 0)
   ^
   arch/arm64/mm/mmu.c:442:10: error: invalid argument type 'void' to unary 
expression
   BUG_ON(!ptdesc_pte_dtor(ptdesc));
  ^~~~
   include/asm-generic/bug.h:71:45: note: expanded from macro 'BUG_ON'
   #define BUG_ON(condition) do { if (unlikely(condition)) BUG(); } while (0)
   ^
   include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
   # define unlikely(x)__builtin_expect(!!(x), 0)
   ^
   2 errors generated.


vim +/void +440 arch/arm64/mm/mmu.c

   425  
   426  static phys_addr_t pgd_pgtable_alloc(int shift)
   427  {
   428  phys_addr_t pa = __pgd_pgtable_alloc(shift);
   429  struct ptdesc *ptdesc = page_ptdesc(phys_to_page(pa));
   430  
   431  /*
   432   * Call proper page table ctor in case later we need to
   433   * call core mm functions like apply_to_page_range() on
   434   * this pre-allocated page table.
   435   *
   436   * We don't select ARCH_ENABLE_SPLIT_PMD_PTLOCK if pmd is
   437   * folded, and if so ptdesc_pte_dtor() becomes nop.
   438   */
   439  if (shift == PAGE_SHIFT)
 > 440  BUG_ON(!ptdesc_pte_dtor(ptdesc));
   441  else if (shift == PMD_SHIFT)
   442  BUG_ON(!ptdesc_pte_dtor(ptdesc));
   443  
   444  return pa;
   445  }
   446  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests



Re: [PATCH v2 21/34] arm64: Convert various functions to use ptdescs

2023-05-01 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build errors:

[auto build test ERROR on akpm-mm/mm-everything]
[also build test ERROR on linus/master next-20230428]
[cannot apply to s390/features powerpc/next powerpc/fixes geert-m68k/for-next 
geert-m68k/for-linus v6.3]
[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/Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230502-033042
base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git 
mm-everything
patch link:
https://lore.kernel.org/r/20230501192829.17086-22-vishal.moola%40gmail.com
patch subject: [PATCH v2 21/34] arm64: Convert various functions to use ptdescs
config: arm64-allyesconfig 
(https://download.01.org/0day-ci/archive/20230502/202305020914.ogrwceg1-...@intel.com/config)
compiler: aarch64-linux-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/8e9481b63b5773d7c914836dcd7fbec2449902bc
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Vishal-Moola-Oracle/mm-Add-PAGE_TYPE_OP-folio-functions/20230502-033042
git checkout 8e9481b63b5773d7c914836dcd7fbec2449902bc
# 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=arm64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 
O=build_dir ARCH=arm64 SHELL=/bin/bash arch/arm64/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202305020914.ogrwceg1-...@intel.com/

All errors (new ones prefixed by >>):

   In file included from include/linux/build_bug.h:5,
from include/linux/bits.h:21,
from include/linux/bitops.h:6,
from arch/arm64/include/asm/cache.h:39,
from include/linux/cache.h:6,
from arch/arm64/mm/mmu.c:9:
   arch/arm64/mm/mmu.c: In function 'pgd_pgtable_alloc':
>> arch/arm64/mm/mmu.c:440:24: error: invalid use of void expression
 440 | BUG_ON(!ptdesc_pte_dtor(ptdesc));
 |^
   include/linux/compiler.h:78:45: note: in definition of macro 'unlikely'
  78 | # define unlikely(x)__builtin_expect(!!(x), 0)
 | ^
   arch/arm64/mm/mmu.c:440:17: note: in expansion of macro 'BUG_ON'
 440 | BUG_ON(!ptdesc_pte_dtor(ptdesc));
 | ^~
   arch/arm64/mm/mmu.c:442:24: error: invalid use of void expression
 442 | BUG_ON(!ptdesc_pte_dtor(ptdesc));
 |^
   include/linux/compiler.h:78:45: note: in definition of macro 'unlikely'
  78 | # define unlikely(x)__builtin_expect(!!(x), 0)
 | ^
   arch/arm64/mm/mmu.c:442:17: note: in expansion of macro 'BUG_ON'
 442 | BUG_ON(!ptdesc_pte_dtor(ptdesc));
 | ^~


vim +440 arch/arm64/mm/mmu.c

   425  
   426  static phys_addr_t pgd_pgtable_alloc(int shift)
   427  {
   428  phys_addr_t pa = __pgd_pgtable_alloc(shift);
   429  struct ptdesc *ptdesc = page_ptdesc(phys_to_page(pa));
   430  
   431  /*
   432   * Call proper page table ctor in case later we need to
   433   * call core mm functions like apply_to_page_range() on
   434   * this pre-allocated page table.
   435   *
   436   * We don't select ARCH_ENABLE_SPLIT_PMD_PTLOCK if pmd is
   437   * folded, and if so ptdesc_pte_dtor() becomes nop.
   438   */
   439  if (shift == PAGE_SHIFT)
 > 440  BUG_ON(!ptdesc_pte_dtor(ptdesc));
   441  else if (shift == PMD_SHIFT)
   442  BUG_ON(!ptdesc_pte_dtor(ptdesc));
   443  
   444  return pa;
   445  }
   446  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests



Re: [PATCH 12/33] mm: Create ptdesc equivalents for pgtable_{pte,pmd}_page_{ctor,dtor}

2023-04-17 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build warnings:

[auto build test WARNING on akpm-mm/mm-everything]
[also build test WARNING on next-20230417]
[cannot apply to s390/features powerpc/next powerpc/fixes geert-m68k/for-next 
geert-m68k/for-linus linus/master v6.3-rc7]
[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/Vishal-Moola-Oracle/s390-Use-_pt_s390_gaddr-for-gmap-address-tracking/20230418-045832
base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git 
mm-everything
patch link:
https://lore.kernel.org/r/20230417205048.15870-13-vishal.moola%40gmail.com
patch subject: [PATCH 12/33] mm: Create ptdesc equivalents for 
pgtable_{pte,pmd}_page_{ctor,dtor}
config: sh-allmodconfig 
(https://download.01.org/0day-ci/archive/20230418/202304180959.yfctfvkw-...@intel.com/config)
compiler: sh4-linux-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/d53de56a2dbf659b53aee1aa2eac60bcc936f10b
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Vishal-Moola-Oracle/s390-Use-_pt_s390_gaddr-for-gmap-address-tracking/20230418-045832
git checkout d53de56a2dbf659b53aee1aa2eac60bcc936f10b
# 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=sh olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 
O=build_dir ARCH=sh prepare

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202304180959.yfctfvkw-...@intel.com/

All warnings (new ones prefixed by >>):

   In file included from arch/sh/kernel/asm-offsets.c:14:
   include/linux/mm.h: In function 'virt_to_ptdesc':
   include/linux/mm.h:2723:16: error: implicit declaration of function 
'page_ptdesc' [-Werror=implicit-function-declaration]
2723 | return page_ptdesc(virt_to_head_page(x));
 |^~~
   include/linux/mm.h:2723:16: warning: returning 'int' from a function with 
return type 'struct ptdesc *' makes pointer from integer without a cast 
[-Wint-conversion]
2723 | return page_ptdesc(virt_to_head_page(x));
 |^
   In file included from arch/sh/include/asm/thread_info.h:13,
from include/linux/thread_info.h:60,
from include/asm-generic/preempt.h:5,
from ./arch/sh/include/generated/asm/preempt.h:1,
from include/linux/preempt.h:78,
from include/linux/spinlock.h:56,
from include/linux/mmzone.h:8,
from include/linux/gfp.h:7,
from include/linux/mm.h:7:
   include/linux/mm.h: In function 'ptdesc_to_virt':
   include/linux/mm.h:2728:29: error: implicit declaration of function 
'ptdesc_page'; did you mean 'pte_page'? [-Werror=implicit-function-declaration]
2728 | return page_to_virt(ptdesc_page(pt));
 | ^~~
   arch/sh/include/asm/page.h:139:27: note: in definition of macro '___va'
 139 | #define ___va(x)((x)+PAGE_OFFSET)
 |   ^
   include/linux/mm.h:117:25: note: in expansion of macro '__va'
 117 | #define page_to_virt(x) __va(PFN_PHYS(page_to_pfn(x)))
 | ^~~~
   include/linux/mm.h:117:30: note: in expansion of macro 'PFN_PHYS'
 117 | #define page_to_virt(x) __va(PFN_PHYS(page_to_pfn(x)))
 |  ^~~~
   include/asm-generic/memory_model.h:64:21: note: in expansion of macro 
'__page_to_pfn'
  64 | #define page_to_pfn __page_to_pfn
 | ^
   include/linux/mm.h:2728:16: note: in expansion of macro 'page_to_virt'
2728 | return page_to_virt(ptdesc_page(pt));
 |^~~~
   include/asm-generic/memory_model.h:46:35: warning: initialization of 'const 
struct page *' from 'int' makes pointer from integer without a cast 
[-Wint-conversion]
  46 | ({  const struct page *__pg = (pg); \
 |   ^
   arch/sh/include/asm/page.h:139:27: note: in definition of macro '___va'
 139 | #define ___va(x)((x)+PAGE_OFFSET)
 |   ^
   include/linux/mm.h:117:25: note: in expansion of macro '__va'

Re: [PATCH 04/33] mm: add utility functions for ptdesc

2023-04-17 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build errors:

[auto build test ERROR on akpm-mm/mm-everything]
[also build test ERROR on s390/features powerpc/next powerpc/fixes 
geert-m68k/for-next geert-m68k/for-linus linus/master v6.3-rc7 next-20230417]
[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/Vishal-Moola-Oracle/s390-Use-_pt_s390_gaddr-for-gmap-address-tracking/20230418-045832
base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git 
mm-everything
patch link:
https://lore.kernel.org/r/20230417205048.15870-5-vishal.moola%40gmail.com
patch subject: [PATCH 04/33] mm: add utility functions for ptdesc
config: sh-allmodconfig 
(https://download.01.org/0day-ci/archive/20230418/202304180913.p1buxrbb-...@intel.com/config)
compiler: sh4-linux-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/1b6f8137ca50a543ad2937092836635ca58c78ce
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Vishal-Moola-Oracle/s390-Use-_pt_s390_gaddr-for-gmap-address-tracking/20230418-045832
git checkout 1b6f8137ca50a543ad2937092836635ca58c78ce
# 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=sh olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 
O=build_dir ARCH=sh prepare

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202304180913.p1buxrbb-...@intel.com/

All error/warnings (new ones prefixed by >>):

   In file included from arch/sh/kernel/asm-offsets.c:14:
   include/linux/mm.h: In function 'virt_to_ptdesc':
>> include/linux/mm.h:2723:16: error: implicit declaration of function 
>> 'page_ptdesc' [-Werror=implicit-function-declaration]
2723 | return page_ptdesc(virt_to_head_page(x));
 |^~~
>> include/linux/mm.h:2723:16: warning: returning 'int' from a function with 
>> return type 'struct ptdesc *' makes pointer from integer without a cast 
>> [-Wint-conversion]
2723 | return page_ptdesc(virt_to_head_page(x));
 |^
   In file included from arch/sh/include/asm/thread_info.h:13,
from include/linux/thread_info.h:60,
from include/asm-generic/preempt.h:5,
from ./arch/sh/include/generated/asm/preempt.h:1,
from include/linux/preempt.h:78,
from include/linux/spinlock.h:56,
from include/linux/mmzone.h:8,
from include/linux/gfp.h:7,
from include/linux/mm.h:7:
   include/linux/mm.h: In function 'ptdesc_to_virt':
>> include/linux/mm.h:2728:29: error: implicit declaration of function 
>> 'ptdesc_page'; did you mean 'pte_page'? 
>> [-Werror=implicit-function-declaration]
2728 | return page_to_virt(ptdesc_page(pt));
 | ^~~
   arch/sh/include/asm/page.h:139:27: note: in definition of macro '___va'
 139 | #define ___va(x)((x)+PAGE_OFFSET)
 |   ^
   include/linux/mm.h:117:25: note: in expansion of macro '__va'
 117 | #define page_to_virt(x) __va(PFN_PHYS(page_to_pfn(x)))
 | ^~~~
   include/linux/mm.h:117:30: note: in expansion of macro 'PFN_PHYS'
 117 | #define page_to_virt(x) __va(PFN_PHYS(page_to_pfn(x)))
 |  ^~~~
   include/asm-generic/memory_model.h:64:21: note: in expansion of macro 
'__page_to_pfn'
  64 | #define page_to_pfn __page_to_pfn
 | ^
   include/linux/mm.h:2728:16: note: in expansion of macro 'page_to_virt'
2728 | return page_to_virt(ptdesc_page(pt));
 |^~~~
>> include/asm-generic/memory_model.h:46:35: warning: initialization of 'const 
>> struct page *' from 'int' makes pointer from integer without a cast 
>> [-Wint-conversion]
  46 | ({  const struct page *__pg = (pg); \
 |   ^
   arch/sh/include/asm/page.h:139:27: note: in definition of macro '___va'
 139 | #define ___va(x)((x)+PAGE_OFFSET)
 |   ^
   include/linux/mm.h:117:25: note: in expans

Re: [PATCH 24/33] m68k: Convert various functions to use ptdescs

2023-04-17 Thread kernel test robot
Hi Vishal,

kernel test robot noticed the following build warnings:

[auto build test WARNING on akpm-mm/mm-everything]
[also build test WARNING on next-20230417]
[cannot apply to s390/features powerpc/next powerpc/fixes geert-m68k/for-next 
geert-m68k/for-linus linus/master v6.3-rc7]
[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/Vishal-Moola-Oracle/s390-Use-_pt_s390_gaddr-for-gmap-address-tracking/20230418-045832
base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git 
mm-everything
patch link:
https://lore.kernel.org/r/20230417205048.15870-25-vishal.moola%40gmail.com
patch subject: [PATCH 24/33] m68k: Convert various functions to use ptdescs
config: m68k-allyesconfig 
(https://download.01.org/0day-ci/archive/20230418/202304180652.leolmanq-...@intel.com/config)
compiler: m68k-linux-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/630b38053b213e6138d3deb3e4325b24ad6dcb1f
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Vishal-Moola-Oracle/s390-Use-_pt_s390_gaddr-for-gmap-address-tracking/20230418-045832
git checkout 630b38053b213e6138d3deb3e4325b24ad6dcb1f
# 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=m68k olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 
O=build_dir ARCH=m68k SHELL=/bin/bash arch/m68k/mm/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202304180652.leolmanq-...@intel.com/

All warnings (new ones prefixed by >>):

   arch/m68k/mm/motorola.c: In function 'free_pointer_table':
>> arch/m68k/mm/motorola.c:204:56: warning: passing argument 1 of 
>> 'virt_to_ptdesc' makes pointer from integer without a cast [-Wint-conversion]
 204 | ptdesc_pte_dtor(virt_to_ptdesc(page));
 |^~~~
 ||
 |long unsigned 
int
   In file included from arch/m68k/mm/motorola.c:15:
   include/linux/mm.h:2721:57: note: expected 'const void *' but argument is of 
type 'long unsigned int'
2721 | static inline struct ptdesc *virt_to_ptdesc(const void *x)
 | ^
   arch/m68k/mm/motorola.c: At top level:
   arch/m68k/mm/motorola.c:418:13: warning: no previous prototype for 
'paging_init' [-Wmissing-prototypes]
 418 | void __init paging_init(void)
 | ^~~


vim +/virt_to_ptdesc +204 arch/m68k/mm/motorola.c

   185  
   186  int free_pointer_table(void *table, int type)
   187  {
   188  ptable_desc *dp;
   189  unsigned long ptable = (unsigned long)table;
   190  unsigned long page = ptable & PAGE_MASK;
   191  unsigned int mask = 1U << ((ptable - page)/ptable_size(type));
   192  
   193  dp = PD_PTABLE(page);
   194  if (PD_MARKBITS (dp) & mask)
   195  panic ("table already free!");
   196  
   197  PD_MARKBITS (dp) |= mask;
   198  
   199  if (PD_MARKBITS(dp) == ptable_mask(type)) {
   200  /* all tables in page are free, free page */
   201  list_del(dp);
   202  mmu_page_dtor((void *)page);
   203  if (type == TABLE_PTE)
 > 204  ptdesc_pte_dtor(virt_to_ptdesc(page));
   205  free_page (page);
   206  return 1;
   207  } else if (ptable_list[type].next != dp) {
   208  /*
   209   * move this descriptor to the front of the list, since
   210   * it has one or more free tables.
   211   */
   212  list_move(dp, _list[type]);
   213  }
   214  return 0;
   215  }
   216  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests



Re: [PATCH v5] ACPI: processor: Fix evaluating _PDC method when running as Xen dom0

2023-03-21 Thread kernel test robot
Hi Roger,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on rafael-pm/linux-next linus/master v6.3-rc3 
next-20230321]
[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/Roger-Pau-Monne/ACPI-processor-Fix-evaluating-_PDC-method-when-running-as-Xen-dom0/20230321-221950
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
patch link:
https://lore.kernel.org/r/20230321141904.49177-1-roger.pau%40citrix.com
patch subject: [PATCH v5] ACPI: processor: Fix evaluating _PDC method when 
running as Xen dom0
config: arm64-randconfig-r013-20230319 
(https://download.01.org/0day-ci/archive/20230322/202303221107.hgkqazl0-...@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 
67409911353323ca5edf2049ef0df54132fa1ca7)
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
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# 
https://github.com/intel-lab-lkp/linux/commit/4232c8b37a0415e1e828fef4ce522c93a0b925fc
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Roger-Pau-Monne/ACPI-processor-Fix-evaluating-_PDC-method-when-running-as-Xen-dom0/20230321-221950
git checkout 4232c8b37a0415e1e828fef4ce522c93a0b925fc
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=arm64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=arm64 SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202303221107.hgkqazl0-...@intel.com/

All errors (new ones prefixed by >>):

   In file included from arch/arm64/xen/../../arm/xen/enlighten.c:2:
>> include/xen/xen.h:79:2: error: call to undeclared function 'BUG'; ISO C99 
>> and later do not support implicit function declarations 
>> [-Wimplicit-function-declaration]
   BUG();
   ^
   In file included from arch/arm64/xen/../../arm/xen/enlighten.c:4:
   In file included from include/xen/grant_table.h:48:
   In file included from include/xen/page.h:28:
   In file included from arch/arm64/include/asm/xen/page.h:1:
   In file included from include/xen/arm/page.h:9:
   In file included from include/linux/dma-mapping.h:7:
   In file included from include/linux/device.h:32:
   In file included from include/linux/device/driver.h:21:
   In file included from include/linux/module.h:19:
   In file included from include/linux/elf.h:6:
   In file included from arch/arm64/include/asm/elf.h:141:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:97:11: warning: array index 3 is past the end of the 
array (that has type 'unsigned long[1]') [-Warray-bounds]
   return (set->sig[3] | set->sig[2] |
   ^~
   include/uapi/asm-generic/signal.h:62:2: note: array 'sig' declared here
   unsigned long sig[_NSIG_WORDS];
   ^
   In file included from arch/arm64/xen/../../arm/xen/enlighten.c:4:
   In file included from include/xen/grant_table.h:48:
   In file included from include/xen/page.h:28:
   In file included from arch/arm64/include/asm/xen/page.h:1:
   In file included from include/xen/arm/page.h:9:
   In file included from include/linux/dma-mapping.h:7:
   In file included from include/linux/device.h:32:
   In file included from include/linux/device/driver.h:21:
   In file included from include/linux/module.h:19:
   In file included from include/linux/elf.h:6:
   In file included from arch/arm64/include/asm/elf.h:141:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:97:25: warning: array index 2 is past the end of the 
array (that has type 'unsigned long[1]') [-Warray-bounds]
   return (set->sig[3] | set->sig[2] |
 ^~
   include/uapi/asm-generic/signal.h:62:2: note: array 'sig' declared here
   unsigned long sig[_NSIG_WORDS];
   ^
   In file included fro

Re: [PATCH v5] ACPI: processor: Fix evaluating _PDC method when running as Xen dom0

2023-03-21 Thread kernel test robot
Hi Roger,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on rafael-pm/linux-next linus/master v6.3-rc3 
next-20230321]
[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/Roger-Pau-Monne/ACPI-processor-Fix-evaluating-_PDC-method-when-running-as-Xen-dom0/20230321-221950
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
patch link:
https://lore.kernel.org/r/20230321141904.49177-1-roger.pau%40citrix.com
patch subject: [PATCH v5] ACPI: processor: Fix evaluating _PDC method when 
running as Xen dom0
config: x86_64-randconfig-a005 
(https://download.01.org/0day-ci/archive/20230322/202303221035.bfy5kyh1-...@intel.com/config)
compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project 
f28c006a5895fc0e329fe15fead81e37457cb1d1)
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/4232c8b37a0415e1e828fef4ce522c93a0b925fc
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Roger-Pau-Monne/ACPI-processor-Fix-evaluating-_PDC-method-when-running-as-Xen-dom0/20230321-221950
git checkout 4232c8b37a0415e1e828fef4ce522c93a0b925fc
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=x86_64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=x86_64 SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202303221035.bfy5kyh1-...@intel.com/

All errors (new ones prefixed by >>):

   In file included from arch/x86/xen/suspend_hvm.c:4:
>> include/xen/xen.h:79:2: error: implicit declaration of function 'BUG' is 
>> invalid in C99 [-Werror,-Wimplicit-function-declaration]
   BUG();
   ^
   1 error generated.


vim +/BUG +79 include/xen/xen.h

73  
74  #if defined(CONFIG_XEN_DOM0) && defined(CONFIG_ACPI) && 
defined(CONFIG_X86)
75  bool __init xen_processor_present(uint32_t acpi_id);
76  #else
77  static inline bool xen_processor_present(uint32_t acpi_id)
78  {
  > 79  BUG();
80  return false;
81  }
82  #endif
83  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests



Re: [PATCH v5 1/4] PCI: Introduce pci_dev_for_each_resource()

2023-03-16 Thread kernel test robot
Hi Andy,

I love your patch! Yet something to improve:

[auto build test ERROR on pci/next]
[also build test ERROR on pci/for-linus powerpc/next powerpc/fixes linus/master 
v6.3-rc2 next-20230316]
[cannot apply to soc/for-next]
[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/Andy-Shevchenko/PCI-Introduce-pci_dev_for_each_resource/20230315-032821
base:   https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git next
patch link:
https://lore.kernel.org/r/20230314192634.63531-2-andriy.shevchenko%40linux.intel.com
patch subject: [PATCH v5 1/4] PCI: Introduce pci_dev_for_each_resource()
config: powerpc-randconfig-r032-20230312 
(https://download.01.org/0day-ci/archive/20230317/202303170223.v0xqhs1v-...@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 
67409911353323ca5edf2049ef0df54132fa1ca7)
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
# install powerpc cross compiling tool for clang build
# apt-get install binutils-powerpc-linux-gnu
# 
https://github.com/intel-lab-lkp/linux/commit/85cdf4746b716f7b6c14d7dc5cd907c3c2a1fb0c
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Andy-Shevchenko/PCI-Introduce-pci_dev_for_each_resource/20230315-032821
git checkout 85cdf4746b716f7b6c14d7dc5cd907c3c2a1fb0c
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=powerpc olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=powerpc SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202303170223.v0xqhs1v-...@intel.com/

All errors (new ones prefixed by >>):

>> arch/powerpc/platforms/52xx/mpc52xx_pci.c:331:6: error: unused variable 'i' 
>> [-Werror,-Wunused-variable]
   int i;
   ^
   1 error generated.


vim +/i +331 arch/powerpc/platforms/52xx/mpc52xx_pci.c

f42963f8646540 Grant Likely2006-12-12  326  
f42963f8646540 Grant Likely2006-12-12  327  static void
f42963f8646540 Grant Likely2006-12-12  328  
mpc52xx_pci_fixup_resources(struct pci_dev *dev)
f42963f8646540 Grant Likely2006-12-12  329  {
85cdf4746b716f Mika Westerberg 2023-03-14  330  struct resource *res;
f42963f8646540 Grant Likely2006-12-12 @331  int i;
f42963f8646540 Grant Likely2006-12-12  332  
59510820fff76f Randy Dunlap2021-04-28  333  pr_debug("%s() 
%.4x:%.4x\n", __func__, dev->vendor, dev->device);
f42963f8646540 Grant Likely2006-12-12  334  
f42963f8646540 Grant Likely2006-12-12  335  /* We don't rely on 
boot loader for PCI and resets all
f42963f8646540 Grant Likely2006-12-12  336 devices */
85cdf4746b716f Mika Westerberg 2023-03-14  337  
pci_dev_for_each_resource_p(dev, res) {
f42963f8646540 Grant Likely2006-12-12  338  if (res->end > 
res->start) {/* Only valid resources */
f42963f8646540 Grant Likely2006-12-12  339  
res->end -= res->start;
f42963f8646540 Grant Likely2006-12-12  340  
res->start = 0;
f42963f8646540 Grant Likely2006-12-12  341  
res->flags |= IORESOURCE_UNSET;
f42963f8646540 Grant Likely2006-12-12  342  }
f42963f8646540 Grant Likely2006-12-12  343  }
f42963f8646540 Grant Likely2006-12-12  344  
f42963f8646540 Grant Likely2006-12-12  345  /* The PCI Host bridge 
of MPC52xx has a prefetch memory resource
f42963f8646540 Grant Likely2006-12-12  346 fixed to 1Gb. 
Doesn't fit in the resource system so we remove it */
f42963f8646540 Grant Likely2006-12-12  347  if ( (dev->vendor == 
PCI_VENDOR_ID_MOTOROLA) &&
f42963f8646540 Grant Likely2006-12-12  348   (   dev->device == 
PCI_DEVICE_ID_MOTOROLA_MPC5200
f42963f8646540 Grant Likely2006-12-12  349|| dev->device == 
PCI_DEVICE_ID_MOTOROLA_MPC5200B) ) {
f42963f8646540 Grant Likely2006-12-12  350  struct resource 
*res = >resource[1];
f42963f8646540 Grant Likely2006-12-12  351  res->start = 
res->end = res->flags = 0;
f42963f8646540 Grant Likely2006-12-12  352  }
f42963f8646540 Grant Likely2006-12-12  353  }
f42963f8646540 Grant Likely2006-12-12  354  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests



Re: [PATCH v4 1/4] PCI: Introduce pci_dev_for_each_resource()

2023-03-10 Thread kernel test robot
Hi Andy,

I love your patch! Yet something to improve:

[auto build test ERROR on pci/next]
[also build test ERROR on pci/for-linus powerpc/next powerpc/fixes linus/master 
v6.3-rc1 next-20230310]
[cannot apply to soc/for-next]
[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/Andy-Shevchenko/PCI-Introduce-pci_dev_for_each_resource/20230311-011642
base:   https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git next
patch link:
https://lore.kernel.org/r/20230310171416.23356-2-andriy.shevchenko%40linux.intel.com
patch subject: [PATCH v4 1/4] PCI: Introduce pci_dev_for_each_resource()
config: m68k-allyesconfig 
(https://download.01.org/0day-ci/archive/20230311/202303110550.blo6p5ds-...@intel.com/config)
compiler: m68k-linux-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/059b4a086017fbf2baacdbe0cc454f569f618ffd
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Andy-Shevchenko/PCI-Introduce-pci_dev_for_each_resource/20230311-011642
git checkout 059b4a086017fbf2baacdbe0cc454f569f618ffd
# 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=m68k olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 
O=build_dir ARCH=m68k SHELL=/bin/bash drivers/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot 
| Link: 
https://lore.kernel.org/oe-kbuild-all/202303110550.blo6p5ds-...@intel.com/

All errors (new ones prefixed by >>):

   drivers/pnp/quirks.c: In function 'quirk_system_pci_resources':
>> drivers/pnp/quirks.c:245:17: error: implicit declaration of function 
>> 'pci_dev_for_each_resource' [-Werror=implicit-function-declaration]
 245 | pci_dev_for_each_resource(pdev, r, i) {
 | ^
>> drivers/pnp/quirks.c:245:54: error: expected ';' before '{' token
 245 | pci_dev_for_each_resource(pdev, r, i) {
 |  ^~
 |  ;
   drivers/pnp/quirks.c:233:16: warning: unused variable 'j' [-Wunused-variable]
 233 | int i, j;
 |^
   drivers/pnp/quirks.c:232:26: warning: unused variable 'res' 
[-Wunused-variable]
 232 | struct resource *res, *r;
 |  ^~~
   cc1: some warnings being treated as errors


vim +/pci_dev_for_each_resource +245 drivers/pnp/quirks.c

   228  
   229  static void quirk_system_pci_resources(struct pnp_dev *dev)
   230  {
   231  struct pci_dev *pdev = NULL;
   232  struct resource *res, *r;
   233  int i, j;
   234  
   235  /*
   236   * Some BIOSes have PNP motherboard devices with resources that
   237   * partially overlap PCI BARs.  The PNP system driver claims 
these
   238   * motherboard resources, which prevents the normal PCI driver 
from
   239   * requesting them later.
   240   *
   241   * This patch disables the PNP resources that conflict with PCI 
BARs
   242   * so they won't be claimed by the PNP system driver.
   243   */
   244  for_each_pci_dev(pdev) {
 > 245  pci_dev_for_each_resource(pdev, r, i) {
   246  unsigned long type = resource_type(r);
   247  
   248  if (type != IORESOURCE_IO || type != 
IORESOURCE_MEM ||
   249  resource_size(r) == 0)
   250  continue;
   251  
   252  if (r->flags & IORESOURCE_UNSET)
   253  continue;
   254  
   255  for (j = 0;
   256   (res = pnp_get_resource(dev, type, j)); 
j++) {
   257  if (res->start == 0 && res->end == 0)
   258  continue;
   259  
   260  /*
   261   * If the PNP region doesn't overlap 
the PCI
   262   * region at all, there's no problem.
   263   */
   264  if (!resource_overlaps(res, r))
   265  continue;
  

Re: [PATCH] xen/privcmd: prevent integer overflow on 32 bit systems

2022-07-19 Thread kernel test robot
Hi Dan,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on xen-tip/linux-next]
[also build test WARNING on linus/master v5.19-rc7 next-20220719]
[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/Dan-Carpenter/xen-privcmd-prevent-integer-overflow-on-32-bit-systems/20220715-162307
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: x86_64-randconfig-a005 
(https://download.01.org/0day-ci/archive/20220720/202207200236.geswjpck-...@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 
fa0c7639e91fa1cd0cf2ff0445a1634a90fe850a)
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/ea22ebd83753c7181043e69251b78f0be73675ad
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Dan-Carpenter/xen-privcmd-prevent-integer-overflow-on-32-bit-systems/20220715-162307
git checkout ea22ebd83753c7181043e69251b78f0be73675ad
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/acpi/ drivers/ata/ drivers/rtc/ 
drivers/thermal/intel/ drivers/xen/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

>> drivers/xen/privcmd.c:459:13: warning: result of comparison of constant 
>> 2305843009213693951 with expression of type 'unsigned int' is always false 
>> [-Wtautological-constant-out-of-range-compare]
   if (m.num > SIZE_MAX / sizeof(*m.arr))
   ~ ^ ~
   drivers/xen/privcmd.c:469:13: warning: result of comparison of constant 
2305843009213693951 with expression of type 'unsigned int' is always false 
[-Wtautological-constant-out-of-range-compare]
   if (m.num > SIZE_MAX / sizeof(*m.arr))
   ~ ^ ~
   2 warnings generated.


vim +459 drivers/xen/privcmd.c

   441  
   442  static long privcmd_ioctl_mmap_batch(
   443  struct file *file, void __user *udata, int version)
   444  {
   445  struct privcmd_data *data = file->private_data;
   446  int ret;
   447  struct privcmd_mmapbatch_v2 m;
   448  struct mm_struct *mm = current->mm;
   449  struct vm_area_struct *vma;
   450  unsigned long nr_pages;
   451  LIST_HEAD(pagelist);
   452  struct mmap_batch_state state;
   453  
   454  switch (version) {
   455  case 1:
   456  if (copy_from_user(, udata, sizeof(struct 
privcmd_mmapbatch)))
   457  return -EFAULT;
   458  /* Returns per-frame error in m.arr. */
 > 459  if (m.num > SIZE_MAX / sizeof(*m.arr))
   460  return -EINVAL;
   461  m.err = NULL;
   462  if (!access_ok(m.arr, m.num * sizeof(*m.arr)))
   463  return -EFAULT;
   464  break;
   465  case 2:
   466  if (copy_from_user(, udata, sizeof(struct 
privcmd_mmapbatch_v2)))
   467  return -EFAULT;
   468  /* Returns per-frame error code in m.err. */
   469  if (m.num > SIZE_MAX / sizeof(*m.arr))
   470  return -EINVAL;
   471  if (!access_ok(m.err, m.num * (sizeof(*m.err
   472  return -EFAULT;
   473  break;
   474  default:
   475  return -EINVAL;
   476  }
   477  
   478  /* If restriction is in place, check the domid matches */
   479  if (data->domid != DOMID_INVALID && data->domid != m.dom)
   480  return -EPERM;
   481  
   482  nr_pages = DIV_ROUND_UP(m.num, XEN_PFN_PER_PAGE);
   483  if ((m.num <= 0) || (nr_pages > (LONG_MAX >> PAGE_SHIFT)))
   484  return -EINVAL;
   485  
   486  ret = gather_array(, m.num, sizeof(xen_pfn_t), m.arr);
   487  
   488  if (ret)
   489  goto out;
   490  if (list_empty()) {
   491  ret = -EINVAL;
   492  goto out;
   493  }
   494  
   495  if (version == 2) {
   496  /* Zero error array now to only copy back actual 
errors. 

[xen-tip:linux-next 6/10] drivers/xen/grant-dma-ops.c:278:6: warning: no previous prototype for 'xen_grant_setup_dma_ops'

2022-06-06 Thread kernel test robot
Hi Juergen,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
head:   bb1b8419ea23d8d2de3c886a540f41e39dfe82a9
commit: 6b268a48884cf8ef00477a0e652864638391587c [6/10] xen/virtio: Enable 
restricted memory access using Xen grant mappings
config: x86_64-allyesconfig 
(https://download.01.org/0day-ci/archive/20220606/202206062149.cnjvofb7-...@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-1) 11.3.0
reproduce (this is a W=1 build):
# 
https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git/commit/?id=6b268a48884cf8ef00477a0e652864638391587c
git remote add xen-tip 
https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git
git fetch --no-tags xen-tip linux-next
git checkout 6b268a48884cf8ef00477a0e652864638391587c
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/net/usb/ 
drivers/xen/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

>> drivers/xen/grant-dma-ops.c:278:6: warning: no previous prototype for 
>> 'xen_grant_setup_dma_ops' [-Wmissing-prototypes]
 278 | void xen_grant_setup_dma_ops(struct device *dev)
 |  ^~~


vim +/xen_grant_setup_dma_ops +278 drivers/xen/grant-dma-ops.c

2c73e39aceb90b Juergen Gross 2022-06-02  277  
2c73e39aceb90b Juergen Gross 2022-06-02 @278  void 
xen_grant_setup_dma_ops(struct device *dev)
2c73e39aceb90b Juergen Gross 2022-06-02  279  {
2c73e39aceb90b Juergen Gross 2022-06-02  280struct xen_grant_dma_data *data;
2c73e39aceb90b Juergen Gross 2022-06-02  281  
2c73e39aceb90b Juergen Gross 2022-06-02  282data = 
find_xen_grant_dma_data(dev);
2c73e39aceb90b Juergen Gross 2022-06-02  283if (data) {
2c73e39aceb90b Juergen Gross 2022-06-02  284dev_err(dev, "Xen grant 
DMA data is already created\n");
2c73e39aceb90b Juergen Gross 2022-06-02  285return;
2c73e39aceb90b Juergen Gross 2022-06-02  286}
2c73e39aceb90b Juergen Gross 2022-06-02  287  
2c73e39aceb90b Juergen Gross 2022-06-02  288data = devm_kzalloc(dev, 
sizeof(*data), GFP_KERNEL);
2c73e39aceb90b Juergen Gross 2022-06-02  289if (!data)
2c73e39aceb90b Juergen Gross 2022-06-02  290goto err;
2c73e39aceb90b Juergen Gross 2022-06-02  291  
2c73e39aceb90b Juergen Gross 2022-06-02  292/* XXX The dom0 is hardcoded as 
the backend domain for now */
2c73e39aceb90b Juergen Gross 2022-06-02  293data->backend_domid = 0;
2c73e39aceb90b Juergen Gross 2022-06-02  294  
2c73e39aceb90b Juergen Gross 2022-06-02  295if 
(xa_err(xa_store(_grant_dma_devices, (unsigned long)dev, data,
2c73e39aceb90b Juergen Gross 2022-06-02  296GFP_KERNEL))) {
2c73e39aceb90b Juergen Gross 2022-06-02  297dev_err(dev, "Cannot 
store Xen grant DMA data\n");
2c73e39aceb90b Juergen Gross 2022-06-02  298goto err;
2c73e39aceb90b Juergen Gross 2022-06-02  299}
2c73e39aceb90b Juergen Gross 2022-06-02  300  
2c73e39aceb90b Juergen Gross 2022-06-02  301dev->dma_ops = 
_grant_dma_ops;
2c73e39aceb90b Juergen Gross 2022-06-02  302  
2c73e39aceb90b Juergen Gross 2022-06-02  303return;
2c73e39aceb90b Juergen Gross 2022-06-02  304  
2c73e39aceb90b Juergen Gross 2022-06-02  305  err:
2c73e39aceb90b Juergen Gross 2022-06-02  306dev_err(dev, "Cannot set up Xen 
grant DMA ops, retain platform DMA ops\n");
2c73e39aceb90b Juergen Gross 2022-06-02  307  }
2c73e39aceb90b Juergen Gross 2022-06-02  308  

:: The code at line 278 was first introduced by commit
:: 2c73e39aceb90b59058cdbc497916049e798963c xen/grant-dma-ops: Add option 
to restrict memory access under Xen

:: TO: Juergen Gross 
:: CC: Juergen Gross 

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp



Re: [PATCH] xen/balloon: fix page onlining when populating new zone

2022-04-06 Thread kernel test robot
Hi Juergen,

I love your patch! Yet something to improve:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on linus/master linux/master v5.18-rc1 next-20220406]
[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/intel-lab-lkp/linux/commits/Juergen-Gross/xen-balloon-fix-page-onlining-when-populating-new-zone/20220407-000935
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: arm64-randconfig-r011-20220406 
(https://download.01.org/0day-ci/archive/20220407/202204071359.uas4tsu0-...@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 11.2.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/b3deb59d5386ade4fb227038f202a9bdb8ade4ab
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Juergen-Gross/xen-balloon-fix-page-onlining-when-populating-new-zone/20220407-000935
git checkout b3deb59d5386ade4fb227038f202a9bdb8ade4ab
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross 
O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All errors (new ones prefixed by >>):

   drivers/xen/balloon.c: In function 'decrease_reservation':
>> drivers/xen/balloon.c:518:24: error: implicit declaration of function 
>> 'alloc_page_for_balloon' [-Werror=implicit-function-declaration]
 518 | page = alloc_page_for_balloon(gfp);
 |^~
   drivers/xen/balloon.c:518:22: warning: assignment to 'struct page *' from 
'int' makes pointer from integer without a cast [-Wint-conversion]
 518 | page = alloc_page_for_balloon(gfp);
 |  ^
>> drivers/xen/balloon.c:545:17: error: implicit declaration of function 
>> 'add_page_to_balloon' [-Werror=implicit-function-declaration]
 545 | add_page_to_balloon(page);
 | ^~~
   cc1: some warnings being treated as errors


vim +/alloc_page_for_balloon +518 drivers/xen/balloon.c

   505  
   506  static enum bp_state decrease_reservation(unsigned long nr_pages, gfp_t 
gfp)
   507  {
   508  enum bp_state state = BP_DONE;
   509  unsigned long i;
   510  struct page *page, *tmp;
   511  int ret;
   512  LIST_HEAD(pages);
   513  
   514  if (nr_pages > ARRAY_SIZE(frame_list))
   515  nr_pages = ARRAY_SIZE(frame_list);
   516  
   517  for (i = 0; i < nr_pages; i++) {
 > 518  page = alloc_page_for_balloon(gfp);
   519  if (page == NULL) {
   520  nr_pages = i;
   521  state = BP_EAGAIN;
   522  break;
   523  }
   524  list_add(>lru, );
   525  }
   526  
   527  /*
   528   * Ensure that ballooned highmem pages don't have kmaps.
   529   *
   530   * Do this before changing the p2m as kmap_flush_unused()
   531   * reads PTEs to obtain pages (and hence needs the original
   532   * p2m entry).
   533   */
   534  kmap_flush_unused();
   535  
   536  /*
   537   * Setup the frame, update direct mapping, invalidate P2M,
   538   * and add to balloon.
   539   */
   540  i = 0;
   541  list_for_each_entry_safe(page, tmp, , lru) {
   542  frame_list[i++] = xen_page_to_gfn(page);
   543  
   544  list_del(>lru);
 > 545  add_page_to_balloon(page);
   546  }
   547  
   548  flush_tlb_all();
   549  
   550  ret = xenmem_reservation_decrease(nr_pages, frame_list);
   551  BUG_ON(ret != nr_pages);
   552  
   553  balloon_stats.current_pages -= nr_pages;
   554  
   555  return state;
   556  }
   557  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp



Re: [PATCH] xen/balloon: fix page onlining when populating new zone

2022-04-06 Thread kernel test robot
Hi Juergen,

I love your patch! Perhaps something to improve:

[auto build test WARNING on xen-tip/linux-next]
[also build test WARNING on linus/master linux/master v5.18-rc1 next-20220406]
[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/intel-lab-lkp/linux/commits/Juergen-Gross/xen-balloon-fix-page-onlining-when-populating-new-zone/20220407-000935
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: arm64-randconfig-r036-20220406 
(https://download.01.org/0day-ci/archive/20220407/202204070950.mzgbyw2q-...@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 
c4a1b07d0979e7ff20d7d541af666d822d66b566)
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
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# 
https://github.com/intel-lab-lkp/linux/commit/b3deb59d5386ade4fb227038f202a9bdb8ade4ab
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Juergen-Gross/xen-balloon-fix-page-onlining-when-populating-new-zone/20220407-000935
git checkout b3deb59d5386ade4fb227038f202a9bdb8ade4ab
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/media/platform/ drivers/xen/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

   drivers/xen/balloon.c:518:10: error: implicit declaration of function 
'alloc_page_for_balloon' [-Werror,-Wimplicit-function-declaration]
   page = alloc_page_for_balloon(gfp);
  ^
>> drivers/xen/balloon.c:518:8: warning: incompatible integer to pointer 
>> conversion assigning to 'struct page *' from 'int' [-Wint-conversion]
   page = alloc_page_for_balloon(gfp);
^ ~~~
   drivers/xen/balloon.c:545:3: error: implicit declaration of function 
'add_page_to_balloon' [-Werror,-Wimplicit-function-declaration]
   add_page_to_balloon(page);
   ^
   1 warning and 2 errors generated.


vim +518 drivers/xen/balloon.c

   505  
   506  static enum bp_state decrease_reservation(unsigned long nr_pages, gfp_t 
gfp)
   507  {
   508  enum bp_state state = BP_DONE;
   509  unsigned long i;
   510  struct page *page, *tmp;
   511  int ret;
   512  LIST_HEAD(pages);
   513  
   514  if (nr_pages > ARRAY_SIZE(frame_list))
   515  nr_pages = ARRAY_SIZE(frame_list);
   516  
   517  for (i = 0; i < nr_pages; i++) {
 > 518  page = alloc_page_for_balloon(gfp);
   519  if (page == NULL) {
   520  nr_pages = i;
   521  state = BP_EAGAIN;
   522  break;
   523  }
   524  list_add(>lru, );
   525  }
   526  
   527  /*
   528   * Ensure that ballooned highmem pages don't have kmaps.
   529   *
   530   * Do this before changing the p2m as kmap_flush_unused()
   531   * reads PTEs to obtain pages (and hence needs the original
   532   * p2m entry).
   533   */
   534  kmap_flush_unused();
   535  
   536  /*
   537   * Setup the frame, update direct mapping, invalidate P2M,
   538   * and add to balloon.
   539   */
   540  i = 0;
   541  list_for_each_entry_safe(page, tmp, , lru) {
   542  frame_list[i++] = xen_page_to_gfn(page);
   543  
   544  list_del(>lru);
   545  add_page_to_balloon(page);
   546  }
   547  
   548  flush_tlb_all();
   549  
   550  ret = xenmem_reservation_decrease(nr_pages, frame_list);
   551  BUG_ON(ret != nr_pages);
   552  
   553  balloon_stats.current_pages -= nr_pages;
   554  
   555  return state;
   556  }
   557  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp



Re: [PATCH] xen/balloon: fix page onlining when populating new zone

2022-04-06 Thread kernel test robot
Hi Juergen,

I love your patch! Perhaps something to improve:

[auto build test WARNING on xen-tip/linux-next]
[also build test WARNING on linus/master linux/master v5.18-rc1 next-20220406]
[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/intel-lab-lkp/linux/commits/Juergen-Gross/xen-balloon-fix-page-onlining-when-populating-new-zone/20220407-000935
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: arm64-randconfig-r011-20220406 
(https://download.01.org/0day-ci/archive/20220407/202204070706.pkz2th7l-...@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 11.2.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/b3deb59d5386ade4fb227038f202a9bdb8ade4ab
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review 
Juergen-Gross/xen-balloon-fix-page-onlining-when-populating-new-zone/20220407-000935
git checkout b3deb59d5386ade4fb227038f202a9bdb8ade4ab
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross 
O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/xen/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

   drivers/xen/balloon.c: In function 'decrease_reservation':
   drivers/xen/balloon.c:518:24: error: implicit declaration of function 
'alloc_page_for_balloon' [-Werror=implicit-function-declaration]
 518 | page = alloc_page_for_balloon(gfp);
 |^~
>> drivers/xen/balloon.c:518:22: warning: assignment to 'struct page *' from 
>> 'int' makes pointer from integer without a cast [-Wint-conversion]
 518 | page = alloc_page_for_balloon(gfp);
 |  ^
   drivers/xen/balloon.c:545:17: error: implicit declaration of function 
'add_page_to_balloon' [-Werror=implicit-function-declaration]
 545 | add_page_to_balloon(page);
 | ^~~
   cc1: some warnings being treated as errors


vim +518 drivers/xen/balloon.c

   505  
   506  static enum bp_state decrease_reservation(unsigned long nr_pages, gfp_t 
gfp)
   507  {
   508  enum bp_state state = BP_DONE;
   509  unsigned long i;
   510  struct page *page, *tmp;
   511  int ret;
   512  LIST_HEAD(pages);
   513  
   514  if (nr_pages > ARRAY_SIZE(frame_list))
   515  nr_pages = ARRAY_SIZE(frame_list);
   516  
   517  for (i = 0; i < nr_pages; i++) {
 > 518  page = alloc_page_for_balloon(gfp);
   519  if (page == NULL) {
   520  nr_pages = i;
   521  state = BP_EAGAIN;
   522  break;
   523  }
   524  list_add(>lru, );
   525  }
   526  
   527  /*
   528   * Ensure that ballooned highmem pages don't have kmaps.
   529   *
   530   * Do this before changing the p2m as kmap_flush_unused()
   531   * reads PTEs to obtain pages (and hence needs the original
   532   * p2m entry).
   533   */
   534  kmap_flush_unused();
   535  
   536  /*
   537   * Setup the frame, update direct mapping, invalidate P2M,
   538   * and add to balloon.
   539   */
   540  i = 0;
   541  list_for_each_entry_safe(page, tmp, , lru) {
   542  frame_list[i++] = xen_page_to_gfn(page);
   543  
   544  list_del(>lru);
   545  add_page_to_balloon(page);
   546  }
   547  
   548  flush_tlb_all();
   549  
   550  ret = xenmem_reservation_decrease(nr_pages, frame_list);
   551  BUG_ON(ret != nr_pages);
   552  
   553  balloon_stats.current_pages -= nr_pages;
   554  
   555  return state;
   556  }
   557  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp



Re: [PATCH 5/5] xen: flag xen_snd_front to be not essential for system boot

2021-11-06 Thread kernel test robot
Hi Juergen,

I love your patch! Yet something to improve:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on tty/tty-testing tiwai-sound/for-next hid/for-next 
linus/master v5.15 next-20211106]
[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/Juergen-Gross/xen-cleanup-detection-of-non-essential-pv-devices/20211022-145043
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: arm64-randconfig-r005-20211025 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 
a461fa64bb37cffd73f683c74f6b0780379fc2ca)
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
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# 
https://github.com/0day-ci/linux/commit/ed8f15a6cafc5414c3821aad66273c580034c3c0
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/xen-cleanup-detection-of-non-essential-pv-devices/20211022-145043
git checkout ed8f15a6cafc5414c3821aad66273c580034c3c0
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 
O=build_dir ARCH=arm64 SHELL=/bin/bash sound/xen/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All errors (new ones prefixed by >>):

>> sound/xen/xen_snd_front.c:361:23: error: unexpected ';' before '}'
   .not_essential = true;
^
   1 error generated.


vim +361 sound/xen/xen_snd_front.c

   355  
   356  static struct xenbus_driver xen_driver = {
   357  .ids = xen_drv_ids,
   358  .probe = xen_drv_probe,
   359  .remove = xen_drv_remove,
   360  .otherend_changed = sndback_changed,
 > 361  .not_essential = true;
   362  };
   363  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org


.config.gz
Description: application/gzip


Re: [PATCH] xen/balloon: rename alloc/free_xenballooned_pages

2021-10-30 Thread kernel test robot
Hi Juergen,

I love your patch! Yet something to improve:

[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on linux/master linus/master v5.15-rc7 next-20211029]
[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/Juergen-Gross/xen-balloon-rename-alloc-free_xenballooned_pages/20211029-222901
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: arm64-defconfig (attached as .config)
compiler: aarch64-linux-gcc (GCC) 11.2.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/4db6d782d7d5ef16ced6f4e54161bd8f1148d39b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/xen-balloon-rename-alloc-free_xenballooned_pages/20211029-222901
git checkout 4db6d782d7d5ef16ced6f4e54161bd8f1148d39b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross 
ARCH=arm64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All errors (new ones prefixed by >>):

>> drivers/xen/balloon.c:612:5: error: conflicting types for 
>> 'xen_alloc_unpopulated_pages'; have 'int(int,  struct page **)'
 612 | int xen_alloc_unpopulated_pages(int nr_pages, struct page **pages)
 | ^~~
   In file included from drivers/xen/balloon.c:68:
   include/xen/xen.h:55:5: note: previous declaration of 
'xen_alloc_unpopulated_pages' with type 'int(unsigned int,  struct page **)'
  55 | int xen_alloc_unpopulated_pages(unsigned int nr_pages, struct page 
**pages);
 | ^~~
   In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:8,
from include/linux/list.h:9,
from include/linux/rculist.h:10,
from include/linux/pid.h:5,
from include/linux/sched.h:14,
from include/linux/ratelimit.h:6,
from include/linux/dev_printk.h:16,
from include/linux/device.h:15,
from include/linux/node.h:18,
from include/linux/cpu.h:17,
from drivers/xen/balloon.c:41:
   drivers/xen/balloon.c:658:15: error: conflicting types for 
'xen_alloc_unpopulated_pages'; have 'int(int,  struct page **)'
 658 | EXPORT_SYMBOL(xen_alloc_unpopulated_pages);
 |   ^~~
   include/linux/export.h:98:28: note: in definition of macro '___EXPORT_SYMBOL'
  98 | extern typeof(sym) sym;  
   \
 |^~~
   include/linux/export.h:160:41: note: in expansion of macro '__EXPORT_SYMBOL'
 160 | #define _EXPORT_SYMBOL(sym, sec)__EXPORT_SYMBOL(sym, sec, "")
 | ^~~
   include/linux/export.h:163:41: note: in expansion of macro '_EXPORT_SYMBOL'
 163 | #define EXPORT_SYMBOL(sym)  _EXPORT_SYMBOL(sym, "")
 | ^~
   drivers/xen/balloon.c:658:1: note: in expansion of macro 'EXPORT_SYMBOL'
 658 | EXPORT_SYMBOL(xen_alloc_unpopulated_pages);
 | ^
   In file included from drivers/xen/balloon.c:68:
   include/xen/xen.h:55:5: note: previous declaration of 
'xen_alloc_unpopulated_pages' with type 'int(unsigned int,  struct page **)'
  55 | int xen_alloc_unpopulated_pages(unsigned int nr_pages, struct page 
**pages);
 | ^~~
>> drivers/xen/balloon.c:665:6: error: conflicting types for 
>> 'xen_free_unpopulated_pages'; have 'void(int,  struct page **)'
 665 | void xen_free_unpopulated_pages(int nr_pages, struct page **pages)
 |  ^~
   In file included from drivers/xen/balloon.c:68:
   include/xen/xen.h:56:6: note: previous declaration of 
'xen_free_unpopulated_pages' with type 'void(unsigned int,  struct page **)'
  56 | void xen_free_unpopulated_pages(unsigned int nr_pages, struct page 
**pages);
 |  ^~
   In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:8,
from include/linux/list.h:9,
from include/linux/rculist.h:10,
from include/linux/pid.h:5,
from include/linux/sched.h:14,
from include/linux/ratelimit.h:6,

Re: [PATCH v2 3/3] xen/blkfront: don't trust the backend response data blindly

2021-07-09 Thread kernel test robot
Hi Juergen,

I love your patch! Perhaps something to improve:

[auto build test WARNING on xen-tip/linux-next]
[also build test WARNING on next-20210708]
[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/Juergen-Gross/xen-harden-blkfront-against-malicious-backends/20210708-204423
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: x86_64-rhel-8.3-kselftests (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.3-341-g8af24329-dirty
# 
https://github.com/0day-ci/linux/commit/26379fb9eaab91fc62eefa414619d27072941f59
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/xen-harden-blkfront-against-malicious-backends/20210708-204423
git checkout 26379fb9eaab91fc62eefa414619d27072941f59
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir 
ARCH=x86_64 SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 


sparse warnings: (new ones prefixed by >>)
>> drivers/block/xen-blkfront.c:1568:20: sparse: sparse: context imbalance in 
>> 'blkif_interrupt' - different lock contexts for basic block

vim +/blkif_interrupt +1568 drivers/block/xen-blkfront.c

9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1567  
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17 @1568  static irqreturn_t 
blkif_interrupt(int irq, void *dev_id)
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1569  {
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1570   struct request *req;
4c0a9a02397621 Juergen Gross 2021-07-08  1571   struct blkif_response 
bret;
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1572   RING_IDX i, rp;
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1573   unsigned long flags;
81f35161577236 Bob Liu   2015-11-14  1574   struct 
blkfront_ring_info *rinfo = (struct blkfront_ring_info *)dev_id;
81f35161577236 Bob Liu   2015-11-14  1575   struct blkfront_info 
*info = rinfo->dev_info;
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1576  
11659569f7202d Bob Liu   2015-11-14  1577   if 
(unlikely(info->connected != BLKIF_STATE_CONNECTED))
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1578   return 
IRQ_HANDLED;
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1579  
11659569f7202d Bob Liu   2015-11-14  1580   
spin_lock_irqsave(>ring_lock, flags);
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1581   again:
26379fb9eaab91 Juergen Gross 2021-07-08  1582   rp = 
READ_ONCE(rinfo->ring.sring->rsp_prod);
26379fb9eaab91 Juergen Gross 2021-07-08  1583   virt_rmb(); /* Ensure 
we see queued responses up to 'rp'. */
26379fb9eaab91 Juergen Gross 2021-07-08  1584   if 
(RING_RESPONSE_PROD_OVERFLOW(>ring, rp)) {
26379fb9eaab91 Juergen Gross 2021-07-08  1585   pr_alert("%s: 
illegal number of responses %u\n",
26379fb9eaab91 Juergen Gross 2021-07-08  1586
info->gd->disk_name, rp - rinfo->ring.rsp_cons);
26379fb9eaab91 Juergen Gross 2021-07-08  1587   goto err;
26379fb9eaab91 Juergen Gross 2021-07-08  1588   }
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1589  
81f35161577236 Bob Liu   2015-11-14  1590   for (i = 
rinfo->ring.rsp_cons; i != rp; i++) {
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1591   unsigned long 
id;
26379fb9eaab91 Juergen Gross 2021-07-08  1592   unsigned int op;
9f27ee59503865 Jeremy Fitzhardinge   2007-07-17  1593  
4c0a9a02397621 Juergen Gross 2021-07-08  1594   
RING_COPY_RESPONSE(>ring, i, );
4c0a9a02397621 Juergen Gross 2021-07-08  1595   id = bret.id;
4c0a9a02397621 Juergen Gross 2021-07-08  1596  
6878c32e5cc0e4 Konrad Rzeszutek Wilk 2012-05-25  1597   /*
6878c32e5cc0e4 Konrad Rzeszutek Wilk 2012-05-25  1598* The backend 
has messed up and given us an id that we would
6878c32e5cc0e4 Konrad Rzeszutek Wilk 2012-05-25  1599* never have 
given to it (we stamp it up to BLK_RING_SIZE -
6878c32e5cc0e4 Konrad Rzeszutek Wilk 2012-05-25  1600* look in 
get_id_from_freelist.
6878c32e5cc0e4 Konrad Rzeszutek Wilk 2012-05-25  1601*/
86839c56dee28c Bob Liu   2015-06-03  1602   if (id >= 
BLK_RING_SIZE(info)) {
26379fb9eaab91 Juergen Gross 2021-07-08  1603   
pr_alert("%s: response has incorrect id (%ld)\n",
26379fb9eaab91 Juergen Gross 2021-07-08  1604   
 info-

Re: [PATCH v3 10/15] x86/paravirt: simplify paravirt macros

2020-12-17 Thread kernel test robot
Hi Juergen,

I love your patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v5.10]
[cannot apply to xen-tip/linux-next tip/x86/core tip/x86/asm next-20201217]
[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/Juergen-Gross/x86-major-paravirt-cleanup/20201217-173646
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
accefff5b547a9a1d959c7e76ad539bf2480e78b
config: x86_64-randconfig-a016-20201217 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 
cee1e7d14f4628d6174b33640d502bff3b54ae45)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# 
https://github.com/0day-ci/linux/commit/0d13a33e925f799d8487bcc597e2dc016d1fdd16
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/x86-major-paravirt-cleanup/20201217-173646
git checkout 0d13a33e925f799d8487bcc597e2dc016d1fdd16
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

   In file included from arch/x86/kernel/asm-offsets.c:13:
   In file included from include/linux/suspend.h:5:
   In file included from include/linux/swap.h:9:
   In file included from include/linux/memcontrol.h:22:
   In file included from include/linux/writeback.h:14:
   In file included from include/linux/blk-cgroup.h:23:
   In file included from include/linux/blkdev.h:26:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/x86/include/asm/io.h:244:
>> arch/x86/include/asm/paravirt.h:44:2: warning: variable '__eax' is 
>> uninitialized when used within its own initialization [-Wuninitialized]
   PVOP_VCALL0(mmu.flush_tlb_user);
   ^~~
   arch/x86/include/asm/paravirt_types.h:504:2: note: expanded from macro 
'PVOP_VCALL0'
   __PVOP_VCALL(op)
   ^~~~
   arch/x86/include/asm/paravirt_types.h:492:8: note: expanded from macro 
'__PVOP_VCALL'
   (void)PVOP_CALL(long, op, CLBR_ANY, PVOP_VCALL_CLOBBERS,\
 ^~~
   arch/x86/include/asm/paravirt_types.h:471:3: note: expanded from macro 
'PVOP_CALL'
   PVOP_CALL_ARGS; \
   ^~
   arch/x86/include/asm/paravirt_types.h:431:41: note: expanded from macro 
'PVOP_CALL_ARGS'
   __edx = __edx, __ecx = __ecx, __eax = __eax;
 ~   ^
   In file included from arch/x86/kernel/asm-offsets.c:13:
   In file included from include/linux/suspend.h:5:
   In file included from include/linux/swap.h:9:
   In file included from include/linux/memcontrol.h:22:
   In file included from include/linux/writeback.h:14:
   In file included from include/linux/blk-cgroup.h:23:
   In file included from include/linux/blkdev.h:26:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/x86/include/asm/io.h:244:
   arch/x86/include/asm/paravirt.h:49:2: warning: variable '__eax' is 
uninitialized when used within its own initialization [-Wuninitialized]
   PVOP_VCALL0(mmu.flush_tlb_kernel);
   ^
   arch/x86/include/asm/paravirt_types.h:504:2: note: expanded from macro 
'PVOP_VCALL0'
   __PVOP_VCALL(op)
   ^~~~
   arch/x86/include/asm/paravirt_types.h:492:8: note: expanded from macro 
'__PVOP_VCALL'
   (void)PVOP_CALL(long, op, CLBR_ANY, PVOP_VCALL_CLOBBERS,\
 ^~~
   arch/x86/include/asm/paravirt_types.h:471:3: note: expanded from macro 
'PVOP_CALL'
   PVOP_CALL_ARGS; \
   ^~
   arch/x86/include/asm/paravirt_types.h:431:41: note: expanded from macro 
'PVOP_CALL_ARGS'
   __edx = __edx, __ecx = __ecx, __eax = __eax;
 ~   ^
   In file included from arch/x86/kernel/asm-offsets.c:13:
   In file included from include/linux/suspend.h:5:
   In file included from include/linux/swap.h:9:
   In file included from include/linux/memcontrol.h:22:
   In 

Re: [PATCH v3 14/15] x86/paravirt: switch functions with custom code to ALTERNATIVE

2020-12-17 Thread kernel test robot
Hi Juergen,

I love your patch! Yet something to improve:

[auto build test ERROR on linus/master]
[cannot apply to xen-tip/linux-next tip/x86/core tip/x86/asm v5.10 
next-20201217]
[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/Juergen-Gross/x86-major-paravirt-cleanup/20201217-173646
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
accefff5b547a9a1d959c7e76ad539bf2480e78b
config: x86_64-randconfig-a002-20201217 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
# 
https://github.com/0day-ci/linux/commit/bc3cbe0ff1b123a4b7f48c91b32198d7dfe57797
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/x86-major-paravirt-cleanup/20201217-173646
git checkout bc3cbe0ff1b123a4b7f48c91b32198d7dfe57797
# save the attached .config to linux build tree
make W=1 ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All error/warnings (new ones prefixed by >>):

   arch/x86/entry/entry_64.S: Assembler messages:
>> arch/x86/entry/entry_64.S:1092: Error: junk at end of line, first 
>> unrecognized character is `('
>> arch/x86/entry/entry_64.S:1092: Error: backward ref to unknown label "771:"
>> arch/x86/entry/entry_64.S:1092: Error: backward ref to unknown label "771:"
>> arch/x86/entry/entry_64.S:1092: Error: junk at end of line, first 
>> unrecognized character is `,'
>> arch/x86/entry/entry_64.S:1092: Warning: missing closing '"'
>> arch/x86/entry/entry_64.S:1092: Error: expecting mnemonic; got nothing


vim +1092 arch/x86/entry/entry_64.S

ddeb8f2149de280 arch/x86/kernel/entry_64.S Alexander van Heukelum 2008-11-24  
1089  
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26  
1090  SYM_CODE_START_LOCAL(error_return)
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26  
1091  UNWIND_HINT_REGS
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26 
@1092  DEBUG_ENTRY_ASSERT_IRQS_OFF
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26  
1093  testb   $3, CS(%rsp)
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26  
1094  jz  restore_regs_and_return_to_kernel
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26  
1095  jmp swapgs_restore_regs_and_return_to_usermode
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26  
1096  SYM_CODE_END(error_return)
424c7d0a9a396ba arch/x86/entry/entry_64.S  Thomas Gleixner2020-03-26  
1097  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org


.config.gz
Description: application/gzip


Re: [PATCH v3 08/15] x86: add new features for paravirt patching

2020-12-17 Thread kernel test robot
Hi Juergen,

I love your patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.10]
[cannot apply to xen-tip/linux-next tip/x86/core tip/x86/asm next-20201217]
[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/Juergen-Gross/x86-major-paravirt-cleanup/20201217-173646
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
accefff5b547a9a1d959c7e76ad539bf2480e78b
config: i386-randconfig-r021-20201217 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
# 
https://github.com/0day-ci/linux/commit/032ee351da7a8adab17b0306cf5908b02f5728d2
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/x86-major-paravirt-cleanup/20201217-173646
git checkout 032ee351da7a8adab17b0306cf5908b02f5728d2
# save the attached .config to linux build tree
make W=1 ARCH=i386 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All errors (new ones prefixed by >>):

   ld: arch/x86/kernel/alternative.o: in function `paravirt_set_cap':
>> arch/x86/kernel/alternative.c:605: undefined reference to 
>> `pv_is_native_spin_unlock'
>> ld: arch/x86/kernel/alternative.c:608: undefined reference to 
>> `pv_is_native_vcpu_is_preempted'


vim +605 arch/x86/kernel/alternative.c

   601  
   602  #ifdef CONFIG_PARAVIRT
   603  static void __init paravirt_set_cap(void)
   604  {
 > 605  if (!pv_is_native_spin_unlock())
   606  setup_force_cpu_cap(X86_FEATURE_PVUNLOCK);
   607  
 > 608  if (!pv_is_native_vcpu_is_preempted())
   609  setup_force_cpu_cap(X86_FEATURE_VCPUPREEMPT);
   610  }
   611  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org


.config.gz
Description: application/gzip


Re: [PATCH v2 11/12] x86/paravirt: switch functions with custom code to ALTERNATIVE

2020-11-20 Thread kernel test robot
Hi Juergen,

I love your patch! Perhaps something to improve:

[auto build test WARNING on v5.10-rc4]
[also build test WARNING on next-20201120]
[cannot apply to tip/x86/core xen-tip/linux-next tip/x86/asm]
[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/Juergen-Gross/x86-major-paravirt-cleanup/20201120-194934
base:09162bc32c880a791c6c0668ce0745cf7958f576
config: x86_64-rhel-7.6-kselftests (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
# 
https://github.com/0day-ci/linux/commit/fd8d46a7a2c51313ee14c43af60ff337279384ef
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/x86-major-paravirt-cleanup/20201120-194934
git checkout fd8d46a7a2c51313ee14c43af60ff337279384ef
# save the attached .config to linux build tree
make W=1 ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

>> drivers/misc/lkdtm/bugs.o: warning: objtool: .altinstr_replacement+0x0: 
>> alternative modifies stack

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org


.config.gz
Description: application/gzip


Re: [PATCH v2 12/12] x86/paravirt: have only one paravirt patch function

2020-11-20 Thread kernel test robot
Hi Juergen,

I love your patch! Perhaps something to improve:

[auto build test WARNING on v5.10-rc4]
[also build test WARNING on next-20201120]
[cannot apply to tip/x86/core xen-tip/linux-next tip/x86/asm]
[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/Juergen-Gross/x86-major-paravirt-cleanup/20201120-194934
base:09162bc32c880a791c6c0668ce0745cf7958f576
config: x86_64-randconfig-r014-20201120 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 
3ded927cf80ac519f9f9c4664fef08787f7c537d)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# 
https://github.com/0day-ci/linux/commit/340df5e02c66ec37486a1f31e6497a22dab65059
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Juergen-Gross/x86-major-paravirt-cleanup/20201120-194934
git checkout 340df5e02c66ec37486a1f31e6497a22dab65059
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

>> arch/x86/kernel/paravirt.c:124:10: warning: no previous prototype for 
>> function 'paravirt_patch_insns' [-Wmissing-prototypes]
   unsigned paravirt_patch_insns(void *insn_buff, unsigned len,
^
   arch/x86/kernel/paravirt.c:124:1: note: declare 'static' if the function is 
not intended to be used outside of this translation unit
   unsigned paravirt_patch_insns(void *insn_buff, unsigned len,
   ^
   static 
   1 warning generated.

vim +/paravirt_patch_insns +124 arch/x86/kernel/paravirt.c

63f70270ccd981c arch/i386/kernel/paravirt.c Jeremy Fitzhardinge 2007-05-02  123 
 
1fc654cf6e04b40 arch/x86/kernel/paravirt.c  Ingo Molnar 2019-04-25 @124 
 unsigned paravirt_patch_insns(void *insn_buff, unsigned len,
63f70270ccd981c arch/i386/kernel/paravirt.c Jeremy Fitzhardinge 2007-05-02  125 
  const char *start, const char *end)
63f70270ccd981c arch/i386/kernel/paravirt.c Jeremy Fitzhardinge 2007-05-02  126 
 {
63f70270ccd981c arch/i386/kernel/paravirt.c Jeremy Fitzhardinge 2007-05-02  127 
unsigned insn_len = end - start;
63f70270ccd981c arch/i386/kernel/paravirt.c Jeremy Fitzhardinge 2007-05-02  128 
 
2777cae2b19d4a0 arch/x86/kernel/paravirt.c  Ingo Molnar 2019-04-25  129 
/* Alternative instruction is too large for the patch site and we 
cannot continue: */
2777cae2b19d4a0 arch/x86/kernel/paravirt.c  Ingo Molnar 2019-04-25  130 
BUG_ON(insn_len > len || start == NULL);
2777cae2b19d4a0 arch/x86/kernel/paravirt.c  Ingo Molnar 2019-04-25  131 
 
1fc654cf6e04b40 arch/x86/kernel/paravirt.c  Ingo Molnar 2019-04-25  132 
memcpy(insn_buff, start, insn_len);
139ec7c416248b9 arch/i386/kernel/paravirt.c Rusty Russell   2006-12-07  133 
 
139ec7c416248b9 arch/i386/kernel/paravirt.c Rusty Russell   2006-12-07  134 
return insn_len;
139ec7c416248b9 arch/i386/kernel/paravirt.c Rusty Russell   2006-12-07  135 
 }
139ec7c416248b9 arch/i386/kernel/paravirt.c Rusty Russell   2006-12-07  136 
 

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org


.config.gz
Description: application/gzip


Re: [PATCH v3 3/7] mm/memory_hotplug: prepare passing flags to add_memory() and friends

2020-09-15 Thread kernel test robot
Hi David,

I love your patch! Perhaps something to improve:

[auto build test WARNING on next-20200909]
[cannot apply to mmotm/master hnaz-linux-mm/master xen-tip/linux-next 
powerpc/next linus/master v5.9-rc4 v5.9-rc3 v5.9-rc2 v5.9-rc4]
[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/David-Hildenbrand/mm-memory_hotplug-selective-merging-of-system-ram-resources/20200910-171630
base:7204eaa2c1f509066486f488c9dcb065d7484494
:: branch date: 9 hours ago
:: commit date: 9 hours ago
config: powerpc-randconfig-r011-20200909 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 
0a5dc7effb191eff740e0e7ae7bd8e1f6bdb3ad9)
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
# install powerpc cross compiling tool for clang build
# apt-get install binutils-powerpc-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

   In file included from arch/powerpc/kernel/asm-offsets.c:14:
   In file included from include/linux/compat.h:17:
   In file included from include/linux/fs.h:15:
   In file included from include/linux/radix-tree.h:18:
   In file included from include/linux/xarray.h:14:
   In file included from include/linux/gfp.h:6:
   In file included from include/linux/mmzone.h:853:
   include/linux/memory_hotplug.h:354:55: error: unknown type name 'mhp_t'
   extern int __add_memory(int nid, u64 start, u64 size, mhp_t mhp_flags);
 ^
   include/linux/memory_hotplug.h:355:53: error: unknown type name 'mhp_t'
   extern int add_memory(int nid, u64 start, u64 size, mhp_t mhp_flags);
   ^
   include/linux/memory_hotplug.h:357:11: error: unknown type name 'mhp_t'
  mhp_t mhp_flags);
  ^
   include/linux/memory_hotplug.h:360:10: error: unknown type name 'mhp_t'
mhp_t mhp_flags);
^
   In file included from arch/powerpc/kernel/asm-offsets.c:21:
>> include/linux/mman.h:137:9: warning: division by zero is undefined 
>> [-Wdivision-by-zero]
  _calc_vm_trans(flags, MAP_LOCKED, VM_LOCKED) |
  ^~~~
   include/linux/mman.h:115:21: note: expanded from macro '_calc_vm_trans'
  : ((x) & (bit1)) / ((bit1) / (bit2
   ^ ~
   include/linux/mman.h:138:9: warning: division by zero is undefined 
[-Wdivision-by-zero]
  _calc_vm_trans(flags, MAP_SYNC,   VM_SYNC  );
  ^~~~
   include/linux/mman.h:115:21: note: expanded from macro '_calc_vm_trans'
  : ((x) & (bit1)) / ((bit1) / (bit2
   ^ ~
   2 warnings and 4 errors generated.
   make[2]: *** [scripts/Makefile.build:117: arch/powerpc/kernel/asm-offsets.s] 
Error 1
   make[2]: Target '__build' not remade because of errors.
   make[1]: *** [Makefile:1196: prepare0] Error 2
   make[1]: Target 'prepare' not remade because of errors.
   make: *** [Makefile:185: __sub-make] Error 2
   make: Target 'prepare' not remade because of errors.

# 
https://github.com/0day-ci/linux/commit/d88270d1c0783a7f99f24a85692be90fd2ae0d7d
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
David-Hildenbrand/mm-memory_hotplug-selective-merging-of-system-ram-resources/20200910-171630
git checkout d88270d1c0783a7f99f24a85692be90fd2ae0d7d
vim +137 include/linux/mman.h

^1da177e4c3f41 Linus Torvalds  2005-04-16  128  
^1da177e4c3f41 Linus Torvalds  2005-04-16  129  /*
^1da177e4c3f41 Linus Torvalds  2005-04-16  130   * Combine the mmap "flags" 
argument into "vm_flags" used internally.
^1da177e4c3f41 Linus Torvalds  2005-04-16  131   */
^1da177e4c3f41 Linus Torvalds  2005-04-16  132  static inline unsigned long
^1da177e4c3f41 Linus Torvalds  2005-04-16  133  calc_vm_flag_bits(unsigned long 
flags)
^1da177e4c3f41 Linus Torvalds  2005-04-16  134  {
^1da177e4c3f41 Linus Torvalds  2005-04-16  135  return 
_calc_vm_trans(flags, MAP_GROWSDOWN,  VM_GROWSDOWN ) |
^1da177e4c3f41 Linus Torvalds  2005-04-16  136 
_calc_vm_trans(flags, MAP_DENYWRITE,  VM_DENYWRITE ) |
b6fb293f2497a9 Jan Kara2017-11-01 @137

Re: [PATCH v3 3/7] mm/memory_hotplug: prepare passing flags to add_memory() and friends

2020-09-10 Thread kernel test robot
Hi David,

I love your patch! Yet something to improve:

[auto build test ERROR on next-20200909]
[cannot apply to mmotm/master hnaz-linux-mm/master xen-tip/linux-next 
powerpc/next linus/master v5.9-rc4 v5.9-rc3 v5.9-rc2 v5.9-rc4]
[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/David-Hildenbrand/mm-memory_hotplug-selective-merging-of-system-ram-resources/20200910-171630
base:7204eaa2c1f509066486f488c9dcb065d7484494
config: x86_64-randconfig-a016-20200909 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 
0a5dc7effb191eff740e0e7ae7bd8e1f6bdb3ad9)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All errors (new ones prefixed by >>):

   WARNING: unmet direct dependencies detected for PHY_SAMSUNG_UFS
   Depends on OF && (ARCH_EXYNOS || COMPILE_TEST
   Selected by
   - SCSI_UFS_EXYNOS && SCSI_LOWLEVEL && SCSI && SCSI_UFSHCD_PLATFORM && 
(ARCH_EXYNOS || COMPILE_TEST
   In file included from arch/x86/kernel/asm-offsets.c:9:
   In file included from include/linux/crypto.h:20:
   In file included from include/linux/slab.h:15:
   In file included from include/linux/gfp.h:6:
   In file included from include/linux/mmzone.h:853:
>> include/linux/memory_hotplug.h:354:55: error: unknown type name 'mhp_t'
   extern int __add_memory(int nid, u64 start, u64 size, mhp_t mhp_flags);
   ^
   include/linux/memory_hotplug.h:355:53: error: unknown type name 'mhp_t'
   extern int add_memory(int nid, u64 start, u64 size, mhp_t mhp_flags);
   ^
   include/linux/memory_hotplug.h:357:11: error: unknown type name 'mhp_t'
   mhp_t mhp_flags);
   ^
   include/linux/memory_hotplug.h:360:10: error: unknown type name 'mhp_t'
   mhp_t mhp_flags);
   ^
   4 errors generated.
   Makefile Module.symvers System.map arch block certs crypto drivers fs 
include init ipc kernel lib mm modules.builtin modules.builtin.modinfo 
modules.order net scripts security sound source tools usr virt vmlinux 
vmlinux.o vmlinux.symvers [scripts/Makefile.build:117: 
arch/x86/kernel/asm-offsets.s] Error 1
   Target '__build' not remade because of errors.
   Makefile Module.symvers System.map arch block certs crypto drivers fs 
include init ipc kernel lib mm modules.builtin modules.builtin.modinfo 
modules.order net scripts security sound source tools usr virt vmlinux 
vmlinux.o vmlinux.symvers [Makefile:1196: prepare0] Error 2
   Target 'prepare' not remade because of errors.
   make: Makefile Module.symvers System.map arch block certs crypto drivers fs 
include init ipc kernel lib mm modules.builtin modules.builtin.modinfo 
modules.order net scripts security sound source tools usr virt vmlinux 
vmlinux.o vmlinux.symvers [Makefile:185: __sub-make] Error 2
   make: Target 'prepare' not remade because of errors.

# 
https://github.com/0day-ci/linux/commit/d88270d1c0783a7f99f24a85692be90fd2ae0d7d
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
David-Hildenbrand/mm-memory_hotplug-selective-merging-of-system-ram-resources/20200910-171630
git checkout d88270d1c0783a7f99f24a85692be90fd2ae0d7d
vim +/mhp_t +354 include/linux/memory_hotplug.h

   352  
   353  extern void __ref free_area_init_core_hotplug(int nid);
 > 354  extern int __add_memory(int nid, u64 start, u64 size, mhp_t mhp_flags);

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org


.config.gz
Description: application/gzip


Re: [PATCH v3 3/7] x86/xen: drop tests for highmem in pv code

2020-08-07 Thread kernel test robot
Hi Juergen,

I love your patch! Perhaps something to improve:

[auto build test WARNING on tip/x86/core]
[also build test WARNING on tip/x86/asm v5.8 next-20200806]
[cannot apply to xen-tip/linux-next tip/x86/vdso]
[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/Juergen-Gross/Remove-32-bit-Xen-PV-guest-support/20200807-164058
base:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git 
ef2ff0f5d6008d325c9a068e20981c0d0acc4d6b
config: x86_64-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
# save the attached .config to linux build tree
make W=1 ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All warnings (new ones prefixed by >>):

   arch/x86/xen/enlighten_pv.c: In function 'set_aliased_prot':
>> arch/x86/xen/enlighten_pv.c:348:15: warning: variable 'page' set but not 
>> used [-Wunused-but-set-variable]
 348 |  struct page *page;
 |   ^~~~
   arch/x86/xen/enlighten_pv.c: At top level:
   arch/x86/xen/enlighten_pv.c:1198:34: warning: no previous prototype for 
'xen_start_kernel' [-Wmissing-prototypes]
1198 | asmlinkage __visible void __init xen_start_kernel(void)
 |  ^~~~

vim +/page +348 arch/x86/xen/enlighten_pv.c

e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  335  
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  336  /*
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  337   * Set the page permissions 
for a particular virtual address.  If the
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  338   * address is a vmalloc 
mapping (or other non-linear mapping), then
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  339   * find the linear mapping 
of the page and also set its protections to
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  340   * match.
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  341   */
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  342  static void 
set_aliased_prot(void *v, pgprot_t prot)
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  343  {
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  344   int level;
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  345   pte_t *ptep;
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  346   pte_t pte;
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  347   unsigned long pfn;
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14 @348   struct page *page;
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  349   unsigned char dummy;
64aef3716eab524 Juergen Gross 2020-08-07  350   void *av;
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  351  
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  352   ptep = 
lookup_address((unsigned long)v, );
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  353   BUG_ON(ptep == NULL);
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  354  
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  355   pfn = pte_pfn(*ptep);
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  356   page = pfn_to_page(pfn);
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  357  
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  358   pte = pfn_pte(pfn, 
prot);
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  359  
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  360   /*
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  361* Careful: 
update_va_mapping() will fail if the virtual address
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  362* we're poking isn't 
populated in the page tables.  We don't
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  363* need to worry about 
the direct map (that's always in the page
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  364* tables), but we need 
to be careful about vmap space.  In
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  365* particular, the top 
level page table can lazily propagate
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  366* entries between 
processes, so if we've switched mms since we
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  367* vmapped the target 
in the first place, we might not have the
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  368* top-level page table 
entry populated.
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  369*
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  370* We disable 
preemption because we want the same mm active when
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  371* we probe the target 
and when we issue the hypercall.  We'll
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  372* have the same 
nominal mm, but if we're a kernel thread, lazy
e1dab14cf68d1e0 Vitaly Kuznetsov  2017-03-14  373* mm dropping could 
change our pgd.
e1dab14cf68d1e0 Vitaly Kuznetso

Re: [Intel-gfx] [PATCH 6/6] drm/xen-front: Add support for EDID based configuration

2020-08-05 Thread kernel test robot
Hi Oleksandr,

I love your patch! Perhaps something to improve:

[auto build test WARNING on drm-exynos/exynos-drm-next]
[also build test WARNING on drm-intel/for-linux-next 
tegra-drm/drm/tegra/for-next drm-tip/drm-tip linus/master v5.8 next-20200804]
[cannot apply to xen-tip/linux-next drm/drm-next]
[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/Oleksandr-Andrushchenko/Fixes-and-improvements-for-Xen-pvdrm/20200731-205350
base:   https://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos.git 
exynos-drm-next
compiler: aarch64-linux-gcc (GCC) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 


cppcheck warnings: (new ones prefixed by >>)

>> drivers/irqchip/irq-gic.c:161:24: warning: Local variable gic_data shadows 
>> outer variable [shadowVar]
struct gic_chip_data *gic_data = irq_data_get_irq_chip_data(d);
  ^
   drivers/irqchip/irq-gic.c:123:29: note: Shadowed declaration
   static struct gic_chip_data gic_data[CONFIG_ARM_GIC_MAX_NR] __read_mostly;
   ^
   drivers/irqchip/irq-gic.c:161:24: note: Shadow variable
struct gic_chip_data *gic_data = irq_data_get_irq_chip_data(d);
  ^
   drivers/irqchip/irq-gic.c:167:24: warning: Local variable gic_data shadows 
outer variable [shadowVar]
struct gic_chip_data *gic_data = irq_data_get_irq_chip_data(d);
  ^
   drivers/irqchip/irq-gic.c:123:29: note: Shadowed declaration
   static struct gic_chip_data gic_data[CONFIG_ARM_GIC_MAX_NR] __read_mostly;
   ^
   drivers/irqchip/irq-gic.c:167:24: note: Shadow variable
struct gic_chip_data *gic_data = irq_data_get_irq_chip_data(d);
  ^
>> drivers/irqchip/irq-gic.c:400:28: warning: Local variable gic_irq shadows 
>> outer function [shadowFunction]
unsigned int cascade_irq, gic_irq;
  ^
   drivers/irqchip/irq-gic.c:171:28: note: Shadowed declaration
   static inline unsigned int gic_irq(struct irq_data *d)
  ^
   drivers/irqchip/irq-gic.c:400:28: note: Shadow variable
unsigned int cascade_irq, gic_irq;
  ^
>> drivers/irqchip/irq-gic.c:1507:14: warning: Local variable gic_cpu_base 
>> shadows outer function [shadowFunction]
phys_addr_t gic_cpu_base;
^
   drivers/irqchip/irq-gic.c:165:29: note: Shadowed declaration
   static inline void __iomem *gic_cpu_base(struct irq_data *d)
   ^
   drivers/irqchip/irq-gic.c:1507:14: note: Shadow variable
phys_addr_t gic_cpu_base;
^
>> drivers/irqchip/irq-gic-v3.c:874:71: warning: Boolean result is used in 
>> bitwise operation. Clarify expression with parentheses. [clarifyCondition]
gic_data.rdists.has_direct_lpi &= (!!(typer & GICR_TYPER_DirectLPIS) |
 ^
>> drivers/irqchip/irq-gic-v3.c:1808:6: warning: Local variable 
>> nr_redist_regions shadows outer variable [shadowVar]
u32 nr_redist_regions;
^
   drivers/irqchip/irq-gic-v3.c:1880:6: note: Shadowed declaration
u32 nr_redist_regions;
^
   drivers/irqchip/irq-gic-v3.c:1808:6: note: Shadow variable
u32 nr_redist_regions;
^
>> drivers/irqchip/irq-gic-v3.c:2042:6: warning: Local variable maint_irq_mode 
>> shadows outer variable [shadowVar]
int maint_irq_mode;
^
   drivers/irqchip/irq-gic-v3.c:1884:6: note: Shadowed declaration
int maint_irq_mode;
^
   drivers/irqchip/irq-gic-v3.c:2042:6: note: Shadow variable
int maint_irq_mode;
^
>> drivers/gpu/drm/xen/xen_drm_front_cfg.c:76:6: warning: Variable 'ret' is 
>> reassigned a value before the old one has been used. [redundantAssignment]
ret = xen_drm_front_get_edid(front_info, index, pages,
^
   drivers/gpu/drm/xen/xen_drm_front_cfg.c:61:0: note: Variable 'ret' is 
reassigned a value before the old one has been used.
int i, npages, ret = -ENOMEM;
   ^
   drivers/gpu/drm/xen/xen_drm_front_cfg.c:76:6: note: Variable 'ret' is 
reassigned a value before the old one has been used.
ret = xen_drm_front_get_edid(front_info, index, pages,
^

vim +/ret +76 drivers/gpu/drm/xen/xen_drm_front_cfg.c

54  
55  static void cfg_connector_edid(struct xen_drm_front_info *front_info,
56 struct xen_drm_front_cfg_connector 
*connector,
57 int index)
58  {
59  struct page **pages;
60  u32 edid_sz;
61  int i, npages, ret = -ENOMEM;
62  
63   

[Xen-devel] bfcf4bd96c [ 8.837757] double fault: 0000 [#1] PREEMPT SMP DEBUG_PAGEALLOC KASAN

2019-04-08 Thread kernel test robot
Greetings,

0day kernel testing robot got the below dmesg and the first bad commit is

https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.x86/stackguards

commit bfcf4bd96c127fa67275b54554bea089424601f8
Author: Andy Lutomirski 
AuthorDate: Fri Jul 13 17:27:40 2018 -0700
Commit: Thomas Gleixner 
CommitDate: Fri Apr 5 09:23:30 2019 +0200

x86/irq/64: Split the IRQ stack into its own pages

Currently the IRQ stack is hardcoded as the first page of the percpu area,
and the stack canary lives on the IRQ stack.  The former gets in the way of
adding an IRQ stack guard page, and the latter is a potential weakness in
the stack canary mechanism.

Split the IRQ stack into its own private percpu pages.

[ tglx: Make 64 and 32 bit share struct irq_stack ]

Signed-off-by: Andy Lutomirski 

1593e8b7e3  x86/irq/32: Handle irq stack allocation failure proper
bfcf4bd96c  x86/irq/64: Split the IRQ stack into its own pages
834dedd380  x86/irq/64: Remove stack overflow debug code
af9671e6ad  Merge branch 'perf/urgent'
+--+++++
|  | 1593e8b7e3 | bfcf4bd96c | 
834dedd380 | af9671e6ad |
+--+++++
| boot_successes   | 31 | 0  | 0
  | 8  |
| boot_failures| 2  | 11 | 11   
  | 3  |
| BUG:kernel_in_stage  | 2  | 2  | 0
  | 3  |
| double_fault:#[##]   | 0  | 9  |  
  ||
| RIP:interrupt_entry  | 0  | 9  | 11   
  ||
| Kernel_panic-not_syncing:Fatal_exception | 0  | 9  | 11   
  ||
| invalid_opcode:#[##] | 0  | 0  | 11   
  ||
+--+++++

[8.787903]  memory used by lock dependency info: 7899 kB
[8.794327]  per task-struct memory footprint: 1920 bytes
[8.806410] ACPI: Core revision 20190215
[8.826246] clocksource: hpet: mask: 0x max_cycles: 0x, 
max_idle_ns: 19112604467 ns
[8.836106] hpet clockevent registered
[8.837757] double fault:  [#1] PREEMPT SMP DEBUG_PAGEALLOC KASAN
[8.839882] CPU: 0 PID: 0 Comm: swapper/0 Not tainted 
5.1.0-rc3-00027-gbfcf4bd #1
[8.842976] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 
1.10.2-1 04/01/2014
[8.858753] RIP: 0010:interrupt_entry+0xbb/0xd0
[8.867595] Code: 00 02 00 00 74 02 0f 0b 58 48 8d 7c 24 08 65 ff 04 25 4c 
ee 01 00 75 1b 65 48 89 3c 25 f8 9f 00 00 65 48 8b 24 25 50 ee 01 00 <48> 3b 7c 
24 f8 74 02 0f 0b 57 ff 77 f8 e8 ca 13 60 fe c3 66 90 48
[8.902603] RSP: :888059002000 EFLAGS: 00010056
[8.912812] RAX: 10745f00 RBX:  RCX: 
[8.923404] RDX:  RSI: 82a008ca RDI: 83a07cf8
[8.926529] RBP:  R08:  R09: 
[8.929025] R10:  R11:  R12: 
[8.931162] R13:  R14:  R15: 
[8.943725] FS:  () GS:88801900() 
knlGS:
[8.959095] CS:  0010 DS:  ES:  CR0: 80050033
[8.966820] CR2: 888059001ff8 CR3: 03a26000 CR4: 06b0
[8.969912] Call Trace:
[8.970864] ---[ end trace 6dd0cce45af54f3e ]---
[8.972478] RIP: 0010:interrupt_entry+0xbb/0xd0

  # HH:MM RESULT GOOD 
BAD GOOD_BUT_DIRTY DIRTY_NOT_BAD
git bisect start a45b3649d85a7ed735803ebe50b0cefd39e202b2 
79a3aaa7b82e3106be97842dedfd8429248896e6 --
git bisect good cb030100ff36e2ca0582734ee7665e7a0560d884  # 17:16  G 11 
07   7  Merge 
'linux-review/Ming-Lei/block-clarify-that-bio_add_page-and-related-helpers-can-add-multi-pages/20190402-122152'
 into devel-hourly-2019040514
git bisect good 0b6c6764f5add44efa60ff6c62773f1b969c322e  # 17:42  G 11 
00   0  Merge 
'linux-review/Dan-Carpenter/kvm-vmx-Silence-a-shift-wrap-warning/20190325-175921'
 into devel-hourly-2019040514
git bisect good 694889d98fe7ad27139710ff6ac1ecca237cad60  # 17:54  G 11 
00   0  Merge 
'linux-review/Miquel-Raynal/usb-host-xhci-plat-Prevent-an-abnormally-restrictive-PHY-init-skipping/20190326-182118'
 into devel-hourly-2019040514
git bisect good 972b31f1074bf48850b5838e7dd07e39c2f0a28e  # 18:10  G 10 
00   0  Merge 
'linux-review/Chris-Wilson/drm-i915-Fixup-kerneldoc-for-intel_cdclk_needs_cd2x_update/20190404-162941'
 into devel-hourly-2019040514
git bisect good 

[Xen-devel] bfcf4bd96c ("x86/irq/64: Split the IRQ stack into its own pages"): double fault: 0000 [#1] PREEMPT SMP DEBUG_PAGEALLOC KASAN

2019-04-05 Thread kernel test robot
Greetings,

0day kernel testing robot got the below dmesg and the first bad commit is

https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.x86/stackguards

commit bfcf4bd96c127fa67275b54554bea089424601f8
Author: Andy Lutomirski 
AuthorDate: Fri Jul 13 17:27:40 2018 -0700
Commit: Thomas Gleixner 
CommitDate: Fri Apr 5 09:23:30 2019 +0200

x86/irq/64: Split the IRQ stack into its own pages

Currently the IRQ stack is hardcoded as the first page of the percpu area,
and the stack canary lives on the IRQ stack.  The former gets in the way of
adding an IRQ stack guard page, and the latter is a potential weakness in
the stack canary mechanism.

Split the IRQ stack into its own private percpu pages.

[ tglx: Make 64 and 32 bit share struct irq_stack ]

Signed-off-by: Andy Lutomirski 

1593e8b7e3  x86/irq/32: Handle irq stack allocation failure proper
bfcf4bd96c  x86/irq/64: Split the IRQ stack into its own pages
834dedd380  x86/irq/64: Remove stack overflow debug code
af9671e6ad  Merge branch 'perf/urgent'
+--+++++
|  | 1593e8b7e3 | bfcf4bd96c | 
834dedd380 | af9671e6ad |
+--+++++
| boot_successes   | 31 | 0  | 0
  | 8  |
| boot_failures| 2  | 11 | 11   
  | 3  |
| BUG:kernel_in_stage  | 2  | 2  | 0
  | 3  |
| double_fault:#[##]   | 0  | 9  |  
  ||
| RIP:interrupt_entry  | 0  | 9  | 11   
  ||
| Kernel_panic-not_syncing:Fatal_exception | 0  | 9  | 11   
  ||
| invalid_opcode:#[##] | 0  | 0  | 11   
  ||
+--+++++

[8.787903]  memory used by lock dependency info: 7899 kB
[8.794327]  per task-struct memory footprint: 1920 bytes
[8.806410] ACPI: Core revision 20190215
[8.826246] clocksource: hpet: mask: 0x max_cycles: 0x, 
max_idle_ns: 19112604467 ns
[8.836106] hpet clockevent registered
[8.837757] double fault:  [#1] PREEMPT SMP DEBUG_PAGEALLOC KASAN
[8.839882] CPU: 0 PID: 0 Comm: swapper/0 Not tainted 
5.1.0-rc3-00027-gbfcf4bd #1
[8.842976] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 
1.10.2-1 04/01/2014
[8.858753] RIP: 0010:interrupt_entry+0xbb/0xd0
[8.867595] Code: 00 02 00 00 74 02 0f 0b 58 48 8d 7c 24 08 65 ff 04 25 4c 
ee 01 00 75 1b 65 48 89 3c 25 f8 9f 00 00 65 48 8b 24 25 50 ee 01 00 <48> 3b 7c 
24 f8 74 02 0f 0b 57 ff 77 f8 e8 ca 13 60 fe c3 66 90 48
[8.902603] RSP: :888059002000 EFLAGS: 00010056
[8.912812] RAX: 10745f00 RBX:  RCX: 
[8.923404] RDX:  RSI: 82a008ca RDI: 83a07cf8
[8.926529] RBP:  R08:  R09: 
[8.929025] R10:  R11:  R12: 
[8.931162] R13:  R14:  R15: 
[8.943725] FS:  () GS:88801900() 
knlGS:
[8.959095] CS:  0010 DS:  ES:  CR0: 80050033
[8.966820] CR2: 888059001ff8 CR3: 03a26000 CR4: 06b0
[8.969912] Call Trace:
[8.970864] ---[ end trace 6dd0cce45af54f3e ]---
[8.972478] RIP: 0010:interrupt_entry+0xbb/0xd0

  # HH:MM RESULT GOOD 
BAD GOOD_BUT_DIRTY DIRTY_NOT_BAD
git bisect start a45b3649d85a7ed735803ebe50b0cefd39e202b2 
79a3aaa7b82e3106be97842dedfd8429248896e6 --
git bisect good cb030100ff36e2ca0582734ee7665e7a0560d884  # 17:16  G 11 
07   7  Merge 
'linux-review/Ming-Lei/block-clarify-that-bio_add_page-and-related-helpers-can-add-multi-pages/20190402-122152'
 into devel-hourly-2019040514
git bisect good 0b6c6764f5add44efa60ff6c62773f1b969c322e  # 17:42  G 11 
00   0  Merge 
'linux-review/Dan-Carpenter/kvm-vmx-Silence-a-shift-wrap-warning/20190325-175921'
 into devel-hourly-2019040514
git bisect good 694889d98fe7ad27139710ff6ac1ecca237cad60  # 17:54  G 11 
00   0  Merge 
'linux-review/Miquel-Raynal/usb-host-xhci-plat-Prevent-an-abnormally-restrictive-PHY-init-skipping/20190326-182118'
 into devel-hourly-2019040514
git bisect good 972b31f1074bf48850b5838e7dd07e39c2f0a28e  # 18:10  G 10 
00   0  Merge 
'linux-review/Chris-Wilson/drm-i915-Fixup-kerneldoc-for-intel_cdclk_needs_cd2x_update/20190404-162941'
 into devel-hourly-2019040514
git bisect good 

[Xen-devel] 92a0f81d89 ("x86/cpu_entry_area: Move it out of the fixmap"): BUG: kernel hang in boot stage

2017-12-23 Thread kernel test robot
Greetings,

0day kernel testing robot got the below dmesg and the first bad commit is

https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.x86/pti

commit 92a0f81d89571e3e8759366e050ee05cc545ef99
Author: Thomas Gleixner 
AuthorDate: Wed Dec 20 18:51:31 2017 +0100
Commit: Ingo Molnar 
CommitDate: Fri Dec 22 20:13:05 2017 +0100

x86/cpu_entry_area: Move it out of the fixmap

Put the cpu_entry_area into a separate P4D entry. The fixmap gets too big
and 0-day already hit a case where the fixmap PTEs were cleared by
cleanup_highmap().

Aside of that the fixmap API is a pain as it's all backwards.

Signed-off-by: Thomas Gleixner 
Cc: Andy Lutomirski 
Cc: Borislav Petkov 
Cc: Dave Hansen 
Cc: H. Peter Anvin 
Cc: Josh Poimboeuf 
Cc: Juergen Gross 
Cc: Linus Torvalds 
Cc: Peter Zijlstra 
Cc: linux-ker...@vger.kernel.org
Signed-off-by: Ingo Molnar 

ed1bbc40a0  x86/cpu_entry_area: Move it to a separate unit
92a0f81d89  x86/cpu_entry_area: Move it out of the fixmap
679d0580c1  x86/ldt: Make the LDT mapping RO
3056af3db3  Merge branch 'WIP.x86/pti.base'
+---+++++
|   | ed1bbc40a0 | 92a0f81d89 | 679d0580c1 | 
3056af3db3 |
+---+++++
| boot_successes| 77 | 0  | 0  | 0  
|
| boot_failures | 0  | 26 | 43 | 19 
|
| BUG:kernel_hang_in_boot_stage | 0  | 26 | 43 | 19 
|
+---+++++

[0.00] Inode-cache hash table entries: 32768 (order: 5, 131072 bytes)
[0.00] BRK [0x07cb7000, 0x07cb7fff] PGTABLE
[0.00] BRK [0x07cb8000, 0x07cb8fff] PGTABLE
[0.00] BRK [0x07cb9000, 0x07cb9fff] PGTABLE
[0.00] BRK [0x07cba000, 0x07cbafff] PGTABLE
BUG: kernel hang in boot stage


  # HH:MM RESULT GOOD 
BAD GOOD_BUT_DIRTY DIRTY_NOT_BAD
git bisect start 858ee49740dd2f7e85f1b45c2af708b6c08f0771 v4.14 --
git bisect good 7fbbd5cbebf118a9e09f5453f686656a167c3d1c  # 23:18  G 11 
00   0  x86/entry/64: Remove the SYSENTER stack canary
git bisect good 4fe2d8b11a370af286287a2661de9d4e6c9a145a  # 23:27  G 11 
00   0  x86/entry: Rename SYSENTER_stack to CPU_ENTRY_AREA_entry_stack
git bisect good dd95f1a4b5ca904c78e6a097091eb21436478abb  # 23:39  G 11 
00   0  x86/mm: Put MMU to hardware ASID translation in one place
git bisect  bad 613e396bc0d4c7604fba23256644e78454c68cf6  # 23:47  B  0 
2   16   0  init: Invoke init_espfix_bsp() from mm_init()
git bisect good ed1bbc40a0d10e0c5c74fe7bdc6298295cf40255  # 23:58  G 11 
00   0  x86/cpu_entry_area: Move it to a separate unit
git bisect  bad 92a0f81d89571e3e8759366e050ee05cc545ef99  # 00:07  B  0
11   36  11  x86/cpu_entry_area: Move it out of the fixmap
# first bad commit: [92a0f81d89571e3e8759366e050ee05cc545ef99] 
x86/cpu_entry_area: Move it out of the fixmap
git bisect good ed1bbc40a0d10e0c5c74fe7bdc6298295cf40255  # 00:19  G 31 
00   0  x86/cpu_entry_area: Move it to a separate unit
# extra tests on HEAD of tip/master
git bisect  bad 3056af3db33464f58e51ddcc9fd5552413e3a6f2  # 00:55  B  0 
5   27   8  Merge branch 'WIP.x86/pti.base'
# extra tests on tree/branch tip/WIP.x86/pti
git bisect  bad 679d0580c1655be350392a66a45cedc9f4c5e139  # 01:14  B  0 
2   42  26  x86/ldt: Make the LDT mapping RO
# extra tests on tree/branch tip/master
git bisect  bad 3056af3db33464f58e51ddcc9fd5552413e3a6f2  # 01:14  B  0
19   33   0  Merge branch 'WIP.x86/pti.base'

---
0-DAY kernel test infrastructureOpen Source Technology Center
https://lists.01.org/pipermail/lkp  Intel Corporation


dmesg-quantal-lkp-hsw01-11:20171224000613:i386-randconfig-a1-201751:4.14.0-00142-g92a0f81:2.gz
Description: application/gzip
#!/bin/bash

kernel=$1

kvm=(
qemu-system-x86_64
-enable-kvm
-cpu kvm64
-kernel $kernel
-m 512
-smp 2
-device e1000,netdev=net0
-netdev user,id=net0
-boot order=nc
-no-reboot
-watchdog i6300esb
-watchdog-action debug
-rtc base=localtime
-serial stdio
-display none
-monitor null
)

append=(
root=/dev/ram0
hung_task_panic=1
debug
apic=debug
sysrq_always_enabled
rcupdate.rcu_cpu_stall_timeout=100
net.ifnames=0