Re: [PATCH v5 0/1] ubi: Introduce block devices for UBI volumes

[Artem Bityutskiy <dedekind1@gmail.com>]

Thanks Ezequiel,

On Thu, 2014-02-13 at 16:57 -0300, Ezequiel Garcia wrote:
> As soon as this gets merged into UBI kernel core, I'll submit the mtd-utils
> userspace tools, as well as start preparing some documentation for the mtd
> website, and the docbook.

Ideally the docs should come along with the driver.

> I'd be willing to drop read-support is that makes this feature acceptable;
> but I guess that would make it pretty useless :-)

Well, it was your decision.

I only asked to remove the compile-time option. And if there are
limitations, document them.

I pointed that there is a standard 'blkdev' tool which has '--setro' and
'--setrw' options, and you could try to use that for _run-time_ RO/RW
toggling.

>   * Dropped write-support

Fine with me too, thanks, but please, do not allude you was pushed to do
this.

>   * Dropped cached access

Thanks. This little tiny compile options make more harm than benefit.
They break from time, because people usually use one of the
configurations, and rarely test the changed.

If you want to save people 128KiB of ram or something, do it a better
way. For example, we have memory shrinker in the kernel. Register your
your shrinker and free that buffer in case of memory pressure. Allocate
it back if needed later using "gentle" allocation techniques (NORETRY,
NOFS, etc).

Anyway, thanks for this too.


-- 
Best Regards,
Artem Bityutskiy