[ti:linux-4.19.y 32/184] arch/alpha/kernel/osf_sys.c:1274:43: error: passing argument 2 of 'put_tv_to_tv32' from incompatible pointer type

[ti:linux-4.19.y 32/184] arch/alpha/kernel/osf_sys.c:1274:43: error: passing argument 2 of 'put_tv_to_tv32' from incompatible pointer type

[kernel test robot]

Hi Arnd,

FYI, the error/warning was bisected to this commit, please ignore it if it's irrelevant.

tree:   git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git linux-4.19.y
head:   33caadfa4b279da780b99fb774337c870080b894
commit: d5e38d6b84d6d21a4f8a4f555a0908b6d9ffe224 [32/184] y2038: rusage: use __kernel_old_timeval
config: alpha-allnoconfig (https://download.01.org/0day-ci/archive/20240328/202403282051.aBWYYsaT-lkp@intel.com/config)
compiler: alpha-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240328/202403282051.aBWYYsaT-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202403282051.aBWYYsaT-lkp@intel.com/

All errors (new ones prefixed by >>):

   arch/alpha/kernel/osf_sys.c: In function '__do_sys_osf_utsname':
   arch/alpha/kernel/osf_sys.c:532:13: warning: unused variable 'error' [-Wunused-variable]
     532 |         int error;
         |             ^~~~~
   arch/alpha/kernel/osf_sys.c: In function '__do_sys_osf_getdomainname':
   arch/alpha/kernel/osf_sys.c:563:18: warning: unused variable 'err' [-Wunused-variable]
     563 |         int len, err = 0;
         |                  ^~~
   arch/alpha/kernel/osf_sys.c: In function '__do_sys_old_adjtimex':
>> arch/alpha/kernel/osf_sys.c:1274:43: error: passing argument 2 of 'put_tv_to_tv32' from incompatible pointer type [-Werror=incompatible-pointer-types]
    1274 |             (put_tv_to_tv32(&txc_p->time, &txc.time)))
         |                                           ^~~~~~~~~
         |                                           |
         |                                           struct timeval *
   arch/alpha/kernel/osf_sys.c:967:73: note: expected 'struct __kernel_old_timeval *' but argument is of type 'struct timeval *'
     967 | put_tv_to_tv32(struct timeval32 __user *o, struct __kernel_old_timeval *i)
         |                                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
   cc1: some warnings being treated as errors


vim +/put_tv_to_tv32 +1274 arch/alpha/kernel/osf_sys.c

^1da177e4c3f41 Linus Torvalds  2005-04-16  1254  
e5d9a90c36e05d Ivan Kokshaysky 2009-01-29  1255  SYSCALL_DEFINE1(old_adjtimex, struct timex32 __user *, txc_p)
^1da177e4c3f41 Linus Torvalds  2005-04-16  1256  {
^1da177e4c3f41 Linus Torvalds  2005-04-16  1257          struct timex txc;
^1da177e4c3f41 Linus Torvalds  2005-04-16  1258  	int ret;
^1da177e4c3f41 Linus Torvalds  2005-04-16  1259  
^1da177e4c3f41 Linus Torvalds  2005-04-16  1260  	/* copy relevant bits of struct timex. */
^1da177e4c3f41 Linus Torvalds  2005-04-16  1261  	if (copy_from_user(&txc, txc_p, offsetof(struct timex32, time)) ||
^1da177e4c3f41 Linus Torvalds  2005-04-16  1262  	    copy_from_user(&txc.tick, &txc_p->tick, sizeof(struct timex32) - 
2b5efc089769cd Al Viro         2017-03-25  1263  			   offsetof(struct timex32, tick)))
^1da177e4c3f41 Linus Torvalds  2005-04-16  1264  	  return -EFAULT;
^1da177e4c3f41 Linus Torvalds  2005-04-16  1265  
^1da177e4c3f41 Linus Torvalds  2005-04-16  1266  	ret = do_adjtimex(&txc);	
^1da177e4c3f41 Linus Torvalds  2005-04-16  1267  	if (ret < 0)
^1da177e4c3f41 Linus Torvalds  2005-04-16  1268  	  return ret;
^1da177e4c3f41 Linus Torvalds  2005-04-16  1269  	
^1da177e4c3f41 Linus Torvalds  2005-04-16  1270  	/* copy back to timex32 */
^1da177e4c3f41 Linus Torvalds  2005-04-16  1271  	if (copy_to_user(txc_p, &txc, offsetof(struct timex32, time)) ||
^1da177e4c3f41 Linus Torvalds  2005-04-16  1272  	    (copy_to_user(&txc_p->tick, &txc.tick, sizeof(struct timex32) - 
^1da177e4c3f41 Linus Torvalds  2005-04-16  1273  			  offsetof(struct timex32, tick))) ||
ce4c253573ad18 Arnd Bergmann   2017-11-08 @1274  	    (put_tv_to_tv32(&txc_p->time, &txc.time)))
^1da177e4c3f41 Linus Torvalds  2005-04-16  1275  	  return -EFAULT;
^1da177e4c3f41 Linus Torvalds  2005-04-16  1276  
^1da177e4c3f41 Linus Torvalds  2005-04-16  1277  	return ret;
^1da177e4c3f41 Linus Torvalds  2005-04-16  1278  }
^1da177e4c3f41 Linus Torvalds  2005-04-16  1279  

:::::: The code at line 1274 was first introduced by commit
:::::: ce4c253573ad184603e0fa77876ba155b0cde46d alpha: osf_sys.c: use timespec64 where appropriate

:::::: TO: Arnd Bergmann <arnd@arndb.de>
:::::: CC: Al Viro <viro@zeniv.linux.org.uk>

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