Re: [PATCH V2 0/4] Add MMC erase and secure erase

From: Christoph Hellwig
Date: Mon Jun 14 2010 - 07:22:15 EST


On Mon, Jun 14, 2010 at 02:10:06PM +0300, Adrian Hunter wrote:
> Needs a bio flag, a request flag, setup the request flag based on the
> bio flag, prevent merging secure and non-secure discards, prevent drivers
> doing non-secure discards for secure discards.
>
> Seems like a lot of little changes for something that no one wants.
> Shouldn't it wait for someone to need it first?

I know that some people have asked for this. But if you want to keep
it in the driver for now that's fine to me. But in that case please:

(a) make it entirely driver private and leave no traces of it in the
block layer
(b) split it into a separate patch from the discard support
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/