[U-Boot] [PATCH] UBIFS: Improve error message when reading superblock failed

Detlev Zundel dzu at denx.de
Fri Feb 17 16:00:11 CET 2012


Hi Bernhard,

> Hi Detlev,
>
> * Detlev Zundel <dzu at denx.de> [2012-02-17 15:15]:
>> 
>> > @@ -1191,7 +1191,7 @@ int ubifs_mount(char *vol_name)
>> >  	mnt = NULL;
>> >  	ret = ubifs_get_sb(&ubifs_fs_type, flags, name, data, mnt);
>> >  	if (ret) {
>> > -		printf("Error reading superblock on volume '%s'!\n", name);
>> > +		printf("Error reading superblock on volume '%s': %d!\n", name, -ret);
>> >  		return -1;
>> >  	}
>> 
>> I think this makes sense, but I think it would be more natural to print
>> the real error code, not the negative value.  I don't know how to search
>> for all such occurrences, but I cannot find any but a lot of sites
>> printing the error code as is.
>
> well, the return value is negative, so my intention was to print the
> error code as positive number. So you think we should display it as
> negative number (-12 instead of 12 for ENOMEM)?

Personally I believe that any transformation in the printing can mislead
people in the search for the cause or the responsible code.

So if the error value is -12, then we should print it.  After all, the
assignment to generate that value will very likely be "return -ENOMEM"
and people will thus know what to look for.

On the other hand I am open to the consistency argument, so if every
error printing would do such a transformation then it would be better to
also do it.  But as I said, I don't know an easy grep pattern to search
for such locations and quick searches showed that I all places I found
print the error codes unmangeld.

Cheers
  Detlev

-- 
Basically,  Barnes & Noble separates things by how old they are  -- current
stuff is "Fiction", stuff from 20 years ago is "Literature", stuff from 100
years ago is "Classics", stuff from 400 years ago is "Shakespeare" [..] and
stuff from 2000 years ago is "History".
                     -- James "Kibo" Parry in <kibo-1207032212000001 at 10.0.1.2>
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de


More information about the U-Boot mailing list