Thread (4 messages) 4 messages, 4 authors, 2012-08-09

Re: What happened to TRIM support for raid linear/0/1/10?

From: Shaohua Li <shli@kernel.org>
Date: 2012-08-09 05:15:03
Also in: lkml

2012/8/9 NeilBrown [off-list ref]:
On Wed, 8 Aug 2012 13:10:51 +0000 (GMT) Holger Kiehl [off-list ref]
wrote:
quoted
Hello,

I have been using the patches posted by Shaohua Li on 16th March 2012:

    http://lkml.indiana.edu/hypermail/linux/kernel/1203.2/00048.html

for several month on a very busy file server (serving 9 million files
with 5.3 TiB daily) without any problems.

Is there any chance that these patches will go into the official kernel?
Or what is the reason that these patches are no applied?
I'm trying to appear to be an incompetent maintainer so that someone will
offer to take over.  It isn't working yet.  I'm probably scuttling the
attempt just by replying to this email - drat.
quoted
I have attached the patch set in one big patch for 3.5. Please do not
use it since I am not sure if it is correct. Shaohua could you please
take a look if it is correct and maybe post a new one?

Personally, I would think that TRIM support MD would be a very good thing.
Probably.  Maybe if they get posted again I'll feel guilty and do something
about them.

One issue that needs to be fixed is that a few places assumed that the
"discard_zeros_data" flag is always set, which is not the case (else we
wouldn't have that flag).  That was only a couple of places though, not a
wide spread problem.
The big problem is SATA discard request merge is broken, while doing
MD discard will trigger discard request merge. In this form, I bet Neil will
not accept the patches. So not the problem MD discard patch itself.

Last time Martin said he is working on SCSI discard request merge
patches. That isn't happened yet. Maybe I should ping Jens
to disable discard request merge currently, it's broken for a long
time anyway.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help