From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id F0B6CC433F5 for ; Mon, 31 Jan 2022 13:08:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Cc:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=usUj8gpOKBgOVfXRZO2oSEUMYdtgbpcxSQnF+n5lqe8=; b=QgRhlUJorOaHfe k2ke3+SD44c7VsMwjUwcx7WViX95NGFj99e1VOrEMq5IV0s8eNPnA7oCluJxZGCOtf21/6aeku077 t/keu0UvM01q6p0DWZV6xW6PdPewUVBZWveGKrY2Cy0HVsIdh0gMs79xzVTOshjO9JdHQOQGlC5/W sangK/cVU+f98VpHEN8IvyHzOQxqe4z/gBrqPpbHnfcBqENxBq+8eoIkt7yHNTouWVdaUD4dm6Auy fbXNrKkBS1oQJVlq1Hzy2SuHsMJ4MAfKhHBK7+vhMv4Se1mGRPJ+kGXa7b7nd3F0Mm3Bl/gaQ1bQs Raa2ShUey4GLLkQNJiQA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nEWOl-009Qit-0F; Mon, 31 Jan 2022 13:07:11 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nEWOh-009Qi6-NW; Mon, 31 Jan 2022 13:07:09 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 0C971611E2; Mon, 31 Jan 2022 13:07:06 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D08E4C340E8; Mon, 31 Jan 2022 13:07:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1643634425; bh=1wmFtXZO4CqFtCd7+S+KYp7Ke3fu6dCt4v93kjNLb5w=; h=Date:From:To:Subject:References:In-Reply-To:From; b=q+c+qw8v8acp2dFk7dmiu5Y0jvKJxoMvdkjMlBbqanXiJtwukiaiVMfVW9I+NafXS 3V1yIKJkY7uiFIkN7JVzDWdlDzWQ8hIe+bKPfZrd0rmEYSS9Geney4rgA3W7ysC/Ai hgH2UsqbgVIuN+E0qh76iKcHJaf+mUraMX5Niolg= Date: Mon, 31 Jan 2022 14:07:02 +0100 From: Greg Kroah-Hartman To: kernel test robot , Nguyen Dinh Phi , Nicolas Saenz Julienne , kbuild-all@lists.01.org, bcm-kernel-feedback-list@broadcom.com, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: vc04_services: make a couple of functions static Message-ID: References: <20220131071006.1764343-1-phind.uet@gmail.com> <202201311901.HT4PtVja-lkp@intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <202201311901.HT4PtVja-lkp@intel.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220131_050707_887410_A4E1E698 X-CRM114-Status: GOOD ( 19.67 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Jan 31, 2022 at 07:16:01PM +0800, kernel test robot wrote: > Hi Nguyen, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on staging/staging-testing] > > url: https://github.com/0day-ci/linux/commits/Nguyen-Dinh-Phi/staging-vc04_services-make-a-couple-of-functions-static/20220131-151209 > base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 64b2d6ffff862c0e7278198b4229e42e1abb3bb1 > config: arm-randconfig-r002-20220130 (https://download.01.org/0day-ci/archive/20220131/202201311901.HT4PtVja-lkp@intel.com/config) > compiler: arm-linux-gnueabi-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/1b64825a2b5b3312ae309c1714239df049b018ac > git remote add linux-review https://github.com/0day-ci/linux > git fetch --no-tags linux-review Nguyen-Dinh-Phi/staging-vc04_services-make-a-couple-of-functions-static/20220131-151209 > git checkout 1b64825a2b5b3312ae309c1714239df049b018ac > # 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=arm SHELL=/bin/bash drivers/staging/vc04_services/ > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot > > All warnings (new ones prefixed by >>): > > drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c: In function 'free_pagelist': > >> drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:434:25: warning: argument 2 null where non-null expected [-Wnonnull] > 434 | memcpy((char *)kmap(pages[0]) + > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 435 | pagelist->offset, > | ~~~~~~~~~~~~~~~~~ > 436 | fragments, > | ~~~~~~~~~~ > 437 | head_bytes); > | ~~~~~~~~~~~ > In file included from include/linux/string.h:20, > from include/linux/bitmap.h:11, > from include/linux/cpumask.h:12, > from include/linux/mm_types_task.h:14, > from include/linux/mm_types.h:5, > from include/linux/buildid.h:5, > from include/linux/module.h:14, > from drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:8: > arch/arm/include/asm/string.h:20:15: note: in a call to function 'memcpy' declared 'nonnull' > 20 | extern void * memcpy(void *, const void *, __kernel_size_t); > | ^~~~~~ > Oddly, your patch caused these build errors to happen, which implies that you did not test-build your patch? I remember seeing this before and being confused as to why this type of change would cause this error, but it must be for some reason, which means we can't take this patch. Always test-build your changes before sending them out. thanks, greg k-h _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel