Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <6a4db1a9-70c4-3ed1-b055-c5161d021d3d@linux.ibm.com>
Date:   Wed, 27 Apr 2022 09:25:59 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.8.0
Subject: Re: [PATCH v6 10/21] KVM: s390: pci: add basic kvm_zdev structure
Content-Language: en-US
To:     kernel test robot <lkp@intel.com>, linux-s390@vger.kernel.org
Cc:     kbuild-all@lists.01.org, alex.williamson@redhat.com,
        cohuck@redhat.com, schnelle@linux.ibm.com, farman@linux.ibm.com,
        pmorel@linux.ibm.com, borntraeger@linux.ibm.com, hca@linux.ibm.com,
        gor@linux.ibm.com, gerald.schaefer@linux.ibm.com,
        agordeev@linux.ibm.com, svens@linux.ibm.com, frankja@linux.ibm.com,
        david@redhat.com, imbrenda@linux.ibm.com, vneethv@linux.ibm.com,
        oberpar@linux.ibm.com, freude@linux.ibm.com, thuth@redhat.com,
        pasic@linux.ibm.com, pbonzini@redhat.com, corbet@lwn.net,
        jgg@nvidia.com, kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-doc@vger.kernel.org
References: <20220426200842.98655-11-mjrosato@linux.ibm.com>
 <202204271653.1ZoYsV9W-lkp@intel.com>
From:   Matthew Rosato <mjrosato@linux.ibm.com>
In-Reply-To: <202204271653.1ZoYsV9W-lkp@intel.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
Precedence: bulk

On 4/27/22 4:41 AM, kernel test robot wrote:
> Hi Matthew,
> 
> I love your patch! Perhaps something to improve:
> 
> [auto build test WARNING on v5.18-rc4]
> [cannot apply to s390/features kvms390/next awilliam-vfio/next next-20220427]
> [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/Matthew-Rosato/KVM-s390-enable-zPCI-for-interpretive-execution/20220427-041853
> base:    af2d861d4cd2a4da5137f795ee3509e6f944a25b
> config: s390-defconfig (https://download.01.org/0day-ci/archive/20220427/202204271653.1ZoYsV9W-lkp@intel.com/config)
> compiler: s390-linux-gcc (GCC) 11.3.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/e6d8c620090a7b184afdf5b5123d10ac45776eaf
>          git remote add linux-review https://github.com/intel-lab-lkp/linux
>          git fetch --no-tags linux-review Matthew-Rosato/KVM-s390-enable-zPCI-for-interpretive-execution/20220427-041853
>          git checkout e6d8c620090a7b184afdf5b5123d10ac45776eaf
>          # save the config file
>          mkdir build_dir && cp config build_dir/.config
>          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=s390 SHELL=/bin/bash arch/s390/kvm/
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>):
> 
>>> arch/s390/kvm/pci.c:14:5: warning: no previous prototype for 'kvm_s390_pci_dev_open' [-Wmissing-prototypes]
>        14 | int kvm_s390_pci_dev_open(struct zpci_dev *zdev)
>           |     ^~~~~~~~~~~~~~~~~~~~~
>>> arch/s390/kvm/pci.c:29:6: warning: no previous prototype for 'kvm_s390_pci_dev_release' [-Wmissing-prototypes]
>        29 | void kvm_s390_pci_dev_release(struct zpci_dev *zdev)
>           |      ^~~~~~~~~~~~~~~~~~~~~~~~
> 

Oops, these 2 functions no longer need to be externalized and can simply 
be marked static.

> 
> vim +/kvm_s390_pci_dev_open +14 arch/s390/kvm/pci.c
> 
>      13	
>    > 14	int kvm_s390_pci_dev_open(struct zpci_dev *zdev)
>      15	{
>      16		struct kvm_zdev *kzdev;
>      17	
>      18		kzdev = kzalloc(sizeof(struct kvm_zdev), GFP_KERNEL);
>      19		if (!kzdev)
>      20			return -ENOMEM;
>      21	
>      22		kzdev->zdev = zdev;
>      23		zdev->kzdev = kzdev;
>      24	
>      25		return 0;
>      26	}
>      27	EXPORT_SYMBOL_GPL(kvm_s390_pci_dev_open);
>      28	
>    > 29	void kvm_s390_pci_dev_release(struct zpci_dev *zdev)
>