linux-kernel - Re: [PATCH v4 2/2] hv_utils: implement Hyper-V PTP source

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 20 Jan 2017 02:01:24 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Vitaly Kuznetsov <vkuznets@...hat.com>
Cc:     kbuild-all@...org, devel@...uxdriverproject.org,
        Olaf Hering <olaf@...fle.de>,
        Radim Krcmar <rkrcmar@...hat.com>,
        Haiyang Zhang <haiyangz@...rosoft.com>,
        Alex Ng <alexng@...rosoft.com>, linux-kernel@...r.kernel.org,
        John Stultz <john.stultz@...aro.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Richard Cochran <richardcochran@...il.com>
Subject: Re: [PATCH v4 2/2] hv_utils: implement Hyper-V PTP source

Hi Vitaly,

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

url:    https://github.com/0day-ci/linux/commits/Vitaly-Kuznetsov/hv_util-adjust-system-time-smoothly/20170120-011342
config: x86_64-rhel (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/hv/hv_util.c: In function 'hv_timesync_init':
>> drivers/hv/hv_util.c:561:36: warning: passing argument 1 of 'ptp_clock_register' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
     hv_ptp_clock = ptp_clock_register(&ptp_hyperv_info, NULL);
                                       ^
   In file included from drivers/hv/hv_util.c:30:0:
   include/linux/ptp_clock_kernel.h:172:26: note: expected 'struct ptp_clock_info *' but argument is of type 'const struct ptp_clock_info *'
    extern struct ptp_clock *ptp_clock_register(struct ptp_clock_info *info,
                             ^~~~~~~~~~~~~~~~~~

vim +561 drivers/hv/hv_util.c

   545		.gettime64      = hv_ptp_gettime,
   546		.settime64      = hv_ptp_settime,
   547		.owner		= THIS_MODULE,
   548	};
   549	
   550	static struct ptp_clock *hv_ptp_clock;
   551	
   552	static int hv_timesync_init(struct hv_util_service *srv)
   553	{
   554		INIT_WORK(&wrk.work, hv_set_host_time);
   555	
   556		/*
   557		 * ptp_clock_register() returns NULL when CONFIG_PTP_1588_CLOCK is
   558		 * disabled but the driver is still useful without the PTP device
   559		 * as it still handles the ICTIMESYNCFLAG_SYNC case.
   560		 */
 > 561		hv_ptp_clock = ptp_clock_register(&ptp_hyperv_info, NULL);
   562		if (IS_ERR_OR_NULL(hv_ptp_clock)) {
   563			pr_err("cannot register PTP clock: %ld\n",
   564			       PTR_ERR(hv_ptp_clock));
   565			hv_ptp_clock = NULL;
   566		}
   567	
   568		return 0;
   569	}

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

Download attachment ".config.gz" of type "application/gzip" (38264 bytes)