Re: hybrid raid1 with trim support [REGRESSION]

2013-04-28 Thread Shaohua Li
On Sun, Apr 28, 2013 at 11:40:42AM +0200, Markus wrote: > Hi! > > Thanks for your work. The patch seems to work for me on a vanilla 3.8.10, at > least the hdds are no longer dropped from the raid. > The code now ignores some request? What was the reason the disks fell off the > raid? The discard

Re: hybrid raid1 with trim support [REGRESSION]

2013-04-28 Thread Markus
Hi! Thanks for your work. The patch seems to work for me on a vanilla 3.8.10, at least the hdds are no longer dropped from the raid. The code now ignores some request? What was the reason the disks fell off the raid? The discards are still passed to the ssd? Thanks, Markus Shaohua Li schrieb

Re: hybrid raid1 with trim support [REGRESSION]

2013-04-27 Thread Shaohua Li
On Sun, Apr 28, 2013 at 08:54:46AM +0800, Shaohua Li wrote: > On Sat, Apr 27, 2013 at 06:29:49PM +0200, Markus wrote: > > Hi! > > > > Now I had the time to bisect, started with 3.7 as good and 3.8 as bad. > > 0cfbcafcae8b7364b5fa96c2b26ccde7a3a296a9 is the bad commit. [1] > > block: add plug for b

Re: hybrid raid1 with trim support [REGRESSION]

2013-04-27 Thread Shaohua Li
On Sat, Apr 27, 2013 at 06:29:49PM +0200, Markus wrote: > Hi! > > Now I had the time to bisect, started with 3.7 as good and 3.8 as bad. > 0cfbcafcae8b7364b5fa96c2b26ccde7a3a296a9 is the bad commit. [1] > block: add plug for blkdev_issue_discard > > While 3.8.10 was still bad, the same kernel wit

Re: hybrid raid1 with trim support [REGRESSION]

2013-04-27 Thread Markus
Hi! Now I had the time to bisect, started with 3.7 as good and 3.8 as bad. 0cfbcafcae8b7364b5fa96c2b26ccde7a3a296a9 is the bad commit. [1] block: add plug for blkdev_issue_discard While 3.8.10 was still bad, the same kernel with the reverted patch applied is fine. I found another report. [2]