Re: [PATCH] [MTD] [UBI] add block device layer on top of UBI

[Artem Bityutskiy <dedekind@infradead.org>]

On Tue, 2008-05-13 at 12:16 +0800, Nancy wrote:
> Hi all,
>        I think it is my final edition of ubi block device layer code
> based on UBI commit e442c48f84982d0fa10c6b292018241dafca4d65
>        Finally, it can support any filesystem based on block device layer.
>        eg: FAT, ext2....
>        #modprobe ubiblk
>        #mkfs.vfat /dev/ubiblock1
>        #mount -t vfat /dev/ubiblock1 /mnt/fat
> 
>        I notice there many people need this function especially our
> Chinese. Hope this helpful :-)
>        Here's my implementation:

What is the reason you added this "generic" ubi mtd block stuff? What
for? Also, UBI should not have any explicit block device support - it
should be isolated. It is to ok to add some general support for devices
on top of UBI, like notification about volume size change and so on,
but not explicit block layer support. Gluebi should also get a separate
module, ideally.

Also, the patch suffers from too direct copy-pastes which is not
acceptable.

-- 
Best regards,
Artem Bityutskiy (Битюцкий Артём)