Re: [PATCH v2 1/2] block: Add new BLK_STS_SELFTEST status

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> +	[BLK_STS_UCLEAN]	= { -EUCLEAN,	"structure needs cleaning" },

The subject line doesn't mention this new error code.  That being said
while this sounds slightly less bad than the original name it still
sounds weird..

The various filesystems really use EFSCORRUPTED which is just mapped
to EUCLEAN, so maybe this really should be

	[BLK_STS_FSCORRUPTED]	=
		{ -EUCLEAN, "file system corruption detected" },

But then again I really wonder why you need to pass this information
through a blk_status_t to start with.  In general these kinds of error
should be passed through file system specific errno fields.



[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux