From: =?ISO-8859-15?Q?Luk=E1=A8_Czerner?= Subject: Re: [PATCH 3/3] mke2fs: check for a partition table and warn if present Date: Mon, 5 May 2014 16:11:41 +0200 (CEST) Message-ID: References: <1399295044-24489-1-git-send-email-tytso@mit.edu> <1399295044-24489-3-git-send-email-tytso@mit.edu> <20140505135804.GC22287@thunk.org> Mime-Version: 1.0 Content-Type: MULTIPART/MIXED; BOUNDARY="8323328-475970867-1399299035=:2223" Cc: Ext4 Developers List To: "Theodore Ts'o" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:6952 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932542AbaEEOLr (ORCPT ); Mon, 5 May 2014 10:11:47 -0400 In-Reply-To: <20140505135804.GC22287@thunk.org> Content-ID: Sender: linux-ext4-owner@vger.kernel.org List-ID: This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --8323328-475970867-1399299035=:2223 Content-Type: TEXT/PLAIN; CHARSET=ISO-8859-15 Content-Transfer-Encoding: 8BIT Content-ID: On Mon, 5 May 2014, Theodore Ts'o wrote: > Date: Mon, 5 May 2014 09:58:04 -0400 > From: Theodore Ts'o > To: Luk?? Czerner > Cc: Ext4 Developers List > Subject: Re: [PATCH 3/3] mke2fs: check for a partition table and warn if > present > > On Mon, May 05, 2014 at 03:52:05PM +0200, Luk?? Czerner wrote: > > > + ret = check_partition_table(device); > > > > This can be actually used to check more than just partitions. So we > > can use this approach to check for all rather than having separate > > checks for file system signatures and partitions. > > The issue is that e2fsprogs gets compiled for systems other than just > Linux. I don't want to be like the assholes who work on systemd and > GNOME that simply screw over *BSD systems. This is why I keep our > internal version of blkid in e2fsprogs, even if I do plan to use the > system blkid by default for 1.43. > > If I had infinite amounts of free time, I'd backport the the new > blkid_probe interfaces to our internal version of blkid, but since I > don't, I prefer use the old interfaces for blkid as much as possible, > since that's the path of least resistance in terms of continuing to > support non-Linux users of e2fsprogs. Fair enough. But we should still make the use of system libblkid by default if you do not have any objections. Also it'll be great to mention that in the commit description that this is the reason why we still try to use the old approach. > > > Also in your check_partition_table() you do not disable probing for > > supeblocks even though you do not look to them afterwards so it's > > not a big deal. But again I think that we can use it to check for > > all signatures. > > How do you disable probing for superblocks with the new blkid interface? blkid_probe_enable_superblocks(pr, 0); Thanks! -Lukas > > Cheers, > > - Ted > --8323328-475970867-1399299035=:2223--