From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:44564 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726096AbgJGKpF (ORCPT ); Wed, 7 Oct 2020 06:45:05 -0400 Subject: Re: [PATCH v4 2/2] s390/dasd: remove ioctl_by_bdev calls References: <20200519142259.102279-1-sth@linux.ibm.com> <20200519142259.102279-3-sth@linux.ibm.com> <5c815b8a-7d77-5c69-9191-d09cc433f5ff@de.ibm.com> <20201007103936.GA24327@lst.de> From: Christian Borntraeger Message-ID: Date: Wed, 7 Oct 2020 12:44:55 +0200 MIME-Version: 1.0 In-Reply-To: <20201007103936.GA24327@lst.de> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit List-ID: To: Christoph Hellwig Cc: Stefan Haberland , axboe@kernel.dk, hoeppner@linux.ibm.com, linux-s390@vger.kernel.org, heiko.carstens@de.ibm.com, gor@linux.ibm.com, linux-kernel@vger.kernel.org On 07.10.20 12:39, Christoph Hellwig wrote: > On Wed, Oct 07, 2020 at 11:34:17AM +0200, Christian Borntraeger wrote: >> >> On 19.05.20 16:22, Stefan Haberland wrote: >>> The IBM partition parser requires device type specific information only >>> available to the DASD driver to correctly register partitions. The >>> current approach of using ioctl_by_bdev with a fake user space pointer >>> is discouraged. >>> >>> Fix this by replacing IOCTL calls with direct in-kernel function calls. >>> >>> Suggested-by: Christoph Hellwig >>> Signed-off-by: Stefan Haberland >>> Reviewed-by: Jan Hoeppner >>> Reviewed-by: Peter Oberparleiter >> >> FWIW, this broken the ibm-partition code for virtio-blk, when CONFIG_DASD=m. > > What are the symptoms? During boot I normally have [ 0.930231] virtio_blk virtio1: [vda] 5409180 4096-byte logical blocks (22.2 GB/20.6 GiB) [ 0.930233] vda: detected capacity change from 0 to 22156001280 [ 0.932806] vda:VOL1/ 0X3333: vda1 vda2 vda3 With this change, the last line is no longer there (if CONFIG_DASD=m) and this also reflects itself in /proc/partitions. The partitions are no longer detected.