From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752126AbdKFKOw (ORCPT ); Mon, 6 Nov 2017 05:14:52 -0500 Received: from mx1.redhat.com ([209.132.183.28]:58690 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750739AbdKFKOu (ORCPT ); Mon, 6 Nov 2017 05:14:50 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com E631D4900F Authentication-Results: ext-mx09.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx09.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=kzak@redhat.com Date: Mon, 6 Nov 2017 11:14:44 +0100 From: Karel Zak To: Pali =?iso-8859-1?Q?Roh=E1r?= Cc: Andy Shevchenko , Andreas Bombe , util-linux@vger.kernel.org, "linux-kernel@vger.kernel.org" , Andrius =?utf-8?B?xaB0aWtvbmFz?= , Curtis Gedak , Pavel Machek Subject: Re: Linux & FAT32 label Message-ID: <20171106101444.m5kxevutsdxx34kl@ws.net.home> References: <20171012092113.2bsb3pzv6un4xahr@pali> <20171012101311.zfvg6edfvszlujom@ws.net.home> <20171012204931.tfd2bhmwu5b6rbpz@pali> <20171016011243.zurh5jhb2y6mczx7@amos.fritz.box> <20171105133929.7cscboxymmpkw634@pali> <20171105140745.ze4ttkazeczrqsy7@pali> <20171105143411.aynz5tr3igobuh6e@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20171105143411.aynz5tr3igobuh6e@pali> User-Agent: NeoMutt/20170912-66-504cd9 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.38]); Mon, 06 Nov 2017 10:14:50 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Nov 05, 2017 at 03:34:11PM +0100, Pali Rohár wrote: > On Sunday 05 November 2017 16:25:54 Andy Shevchenko wrote: > > On Sun, Nov 5, 2017 at 4:07 PM, Pali Rohár wrote: > > > On Sunday 05 November 2017 15:56:53 Andy Shevchenko wrote: > > >> On Sun, Nov 5, 2017 at 3:39 PM, Pali Rohár wrote: > > >> > On Tuesday 31 October 2017 10:35:48 Andy Shevchenko wrote: > > >> >> On Mon, Oct 16, 2017 at 4:12 AM, Andreas Bombe wrote: > > >> >> > On Thu, Oct 12, 2017 at 10:49:31PM +0200, Pali Rohár wrote: > > >> >> >> On Thursday 12 October 2017 12:13:11 Karel Zak wrote: > > >> > > >> >> > I was worried that there might be some scripts or programs that expect > > >> >> > > >> >> If we really care about such scripts another approach might be to > > >> >> introduce a CLI switch to "spec compatible mode" to each tool and > > >> >> suggest in documentation to use it. > > >> >> > > >> >> There are also variants: > > >> >> - spec compatible > > >> >> - WinXX compatible > > >> >> - DOS compatible > > >> >> - etc > > >> > > > >> > I did tests with MS-DOS and Windows versions (results in previous > > >> > email), and they seems to be compatible how they read label. > > >> > > > >> > Based on results I would suggest to ignore label from the boot sector > > >> > when reading label. > > >> > > >> So, for tools which are not doing that to add > > >> > > >> --ignore-boot-sector-label (or alike) [recommended] > > >> > > >> right? > > >> > > >> We don't actually know how many users (scripts) are relying on current > > >> behaviour. > > >> If there are only few, we may introduce backward compatibility switch > > >> > > >> --read-boot-sector-label The current recommended way how to get filesystem label is to read it from udev db. For example this is the way how lsblk reads labels by default. And udevd and some another tools are linked with libblkid. I don't see an elegant way how to support something like "read-boot-sector-label" switch for the library to switch it on the fly. Maybe it would be good enough to change the default behavior and add #ifdef to the library to switch to the old behavior. This is elegant way how to move the decision to downstream maintainers. They have more clue about importance of the FAT32 usage. I can imagine that for example for some embedded systems it's more important than for example for Fedora desktop. I prefer this solution. The another way is to add an option to blkid.conf, but it will make filesystems probing more complex and slow (we don't read the file during superblocks probing now). I'm sure udev guys will hate me with this solution. I'd like to avoid something like this. Karel -- Karel Zak http://karelzak.blogspot.com