From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752690AbdJ3PVN (ORCPT ); Mon, 30 Oct 2017 11:21:13 -0400 Received: from aserp1040.oracle.com ([141.146.126.69]:35767 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752064AbdJ3PVM (ORCPT ); Mon, 30 Oct 2017 11:21:12 -0400 Subject: Re: [PATCH v7 13/13] xen: introduce a Kconfig option to enable the pvcalls frontend To: Stefano Stabellini References: <1509045112-9807-1-git-send-email-sstabellini@kernel.org> <1509045112-9807-13-git-send-email-sstabellini@kernel.org> <0d760e9e-c97a-dae3-dab3-d1c82ceead96@oracle.com> Cc: xen-devel@lists.xen.org, linux-kernel@vger.kernel.org, jgross@suse.com, Stefano Stabellini From: Boris Ostrovsky Message-ID: <0ffb5e07-1db5-dcc6-786c-ef1ef1451d76@oracle.com> Date: Mon, 30 Oct 2017 11:22:05 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <0d760e9e-c97a-dae3-dab3-d1c82ceead96@oracle.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/26/2017 04:45 PM, Boris Ostrovsky wrote: > On 10/26/2017 04:16 PM, Stefano Stabellini wrote: >> On Thu, 26 Oct 2017, Boris Ostrovsky wrote: >>> On 10/26/2017 03:11 PM, Stefano Stabellini wrote: >>>> Also add pvcalls-front to the Makefile. >>>> >>>> Signed-off-by: Stefano Stabellini >>>> CC: boris.ostrovsky@oracle.com >>>> CC: jgross@suse.com >>> Reviewed-by: Boris Ostrovsky >> Thank you!! >> >> The series is fully acked now. I guess it could be added to xentip? >> Maybe for v4.15? > > Yes, that's the plan unless other reviews come in. I will probably > create the branch on Monday (assuming rc7 will be the last rc for 4.14). > It's later than usual but we haven't had anything for 4.15. > > -boris Build warnings (gcc (GCC) 6.3.1 20161221 (Red Hat 6.3.1-1)) /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘__write_ring’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:459:3: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out + masked_prod, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:462:4: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out + masked_prod, ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ array_size - masked_prod, msg_iter); ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:464:4: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out, ^~~~~~~~~~~~~~~~~~~~~~~~~ len - (array_size - masked_prod), ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ msg_iter); ~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:468:4: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out + masked_prod, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘__read_ring’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:560:3: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in + masked_cons, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:563:4: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in + masked_cons, ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ array_size - masked_cons, msg_iter); ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:565:4: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in, ^~~~~~~~~~~~~~~~~~~~~~ len - (array_size - masked_cons), ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ msg_iter); ~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:569:4: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in + masked_cons, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Slightly different on 32 bit: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_event_handler’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:160:31: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] struct sock_mapping *map = (struct sock_mapping *) ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_socket’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:283:21: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.socket.id = (uint64_t) map; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_connect’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:405:22: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.connect.id = (uint64_t)map; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_bind’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:660:19: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.bind.id = (uint64_t)map; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_listen’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:722:21: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.listen.id = (uint64_t) map; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_accept’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:826:21: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.accept.id = (uint64_t) map; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:828:25: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.accept.id_new = (uint64_t) map2; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_poll_passive’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:922:19: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.poll.id = (uint64_t) map; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘pvcalls_front_release’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:1020:22: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] req->u.release.id = (uint64_t)map; ^ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘__write_ring’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:459:3: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out + masked_prod, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:462:4: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out + masked_prod, ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ array_size - masked_prod, msg_iter); ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:464:4: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out, ^~~~~~~~~~~~~~~~~~~~~~~~~ len - (array_size - masked_prod), ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ msg_iter); ~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:468:4: warning: ignoring return value of ‘copy_from_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_from_iter(data->out + masked_prod, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c: In function ‘__read_ring’: /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:560:3: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in + masked_cons, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:563:4: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in + masked_cons, ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ array_size - masked_cons, msg_iter); ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:565:4: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in, ^~~~~~~~~~~~~~~~~~~~~~ len - (array_size - masked_cons), ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ msg_iter); ~~~~~~~~~ /data/upstream/linux-xen/drivers/xen/pvcalls-front.c:569:4: warning: ignoring return value of ‘copy_to_iter’, declared with attribute warn_unused_result [-Wunused-result] copy_to_iter(data->in + masked_cons, len, msg_iter); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -boris