Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755971AbbHGHsn (ORCPT ); Fri, 7 Aug 2015 03:48:43 -0400 Received: from verein.lst.de ([213.95.11.211]:53558 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755623AbbHGHsk (ORCPT ); Fri, 7 Aug 2015 03:48:40 -0400 Date: Fri, 7 Aug 2015 09:48:39 +0200 From: Christoph Hellwig To: Ming Lei Cc: Hannes Reinecke , Jens Axboe , Alexander Graf , Linux Kernel Mailing List Subject: Re: [PATCH 0/4] loop: enable different physical blocksizes Message-ID: <20150807074839.GA17851@lst.de> References: <1437118027-94602-1-git-send-email-hare@suse.de> <20150807073330.GB17618@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1112 Lines: 22 On Fri, Aug 07, 2015 at 03:44:58AM -0400, Ming Lei wrote: > There shouldn't be any problem about looping over DASP which has > 4k sector size. Also for debugging purpose, we can easily emulate 4k > sector size disk by QEMU/virtio-blk. > > We can support 4k sector size on loop for debugging purpose too, but > the side effect is that some images can't be loop mounted any more > after its secror size is become larger, then people might complain that. Have you read the patches? There isn't any change in default behavior, but it allows you to expose a non-512 byte sector size _optionally_. So no images will stop being loop mountable, quite to the contrary - you can now loop mount an image copied off from the 4k disk which otherwise would have been unusable because the file system geometry didn't match what's returned by the block layer as the sector size. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/