From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kay Sievers Date: Tue, 07 Mar 2006 17:15:04 +0000 Subject: Re: Libvolume incorrectly detects FAT32 Message-Id: <20060307171504.GA7809@vrfy.org> List-Id: References: <20060306215704.35f12840@silver> In-Reply-To: <20060306215704.35f12840@silver> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org On Tue, Mar 07, 2006 at 06:01:17PM +0100, iSteve wrote: > On Mon, 6 Mar 2006 23:37:20 +0100 > Kay Sievers wrote: > > > On Mon, Mar 06, 2006 at 10:59:31PM +0100, Michael Buesch wrote: > > > Yes, I fear users of stupid digicams may get hit by this. > > > The Formating option on cheap digicams may sometimes be buggy implemented. > > > I think you should instead do a > > > dd if=/dev/zero of=/dev/partition bsQ2 count=1 > > > and reformat your partition. > > > > Yes, just use a sane formatting program like everything which is > > libparted based. The usual Linux commandline formatters are horribly > > broken and I gave up to talk to the ignorant maintainers. Just zero-out > > the device at the beginning _and_ the end of the device if you reformat it > > with a new filesystem type. Or use a libparted or similar based program. > > > > Thanks, > > Kay > > I would have another suggestion then; how about a scoring system? > > Probes would be given OPTIONAL scoring system, let's say in percents, eg. in > case of fat, lack of magic would make it lose 20%; some other filesystem would > return just 0 or 100% hit (ie. just like it is now). Then the probe with > greatest score wins. > > I understand this design is for working with semibroken partitions, and I fear > it won't be welcome warmly; but I still got to propose it:) > > I am willing to code it, I've written about ten probes for my own detection > system until I noticed libvolumeid, but I want approval of design first. I'm all for fixing the stupid formatters. Seems the problems are really rare these days anyway, with Windows doing it right now and almost all distros use HAL today which has the same code and we don't get a lot of error reports anymore. The right way I think, is to add more checks to the probing code, if the filesytem that is recognized is actually "valid". But that is impossible for some situations like mkswap (which is the silliest formatting tool on earth that I've ever seen) on top of fat. You can even mount the corrupted fat volume with the kernel and write to it. So, I'm not sure if a score will give us something valuable. You basically push the problem to the user of the lib, which is a slippery road. But if we detect that we can't be sure what kind of filesystem we detect, we could throw an error and don't return any type, that would be at least a safe way to handle that and would fix the real problem, which are in almost all cases the too simple and broken Linux command line tools. Thanks, Kay ------------------------------------------------------- This SF.Net email is sponsored by xPML, a groundbreaking scripting language that extends applications into web and mobile media. Attend the live webcast and join the prime developer group breaking into this new coding territory! http://sel.as-us.falkag.net/sel?cmd=lnk&kid0944&bid$1720&dat1642 _______________________________________________ Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net Linux-hotplug-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel