Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-19 Thread Mike Snitzer
On Tue, Mar 18 2014 at 9:45pm -0400, Shaohua Li wrote: > On Tue, Mar 18, 2014 at 05:28:43PM -0400, Mike Snitzer wrote: > > On Tue, Mar 18 2014 at 3:41am -0400, > > Shaohua Li wrote: > > > > > On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: > > > > > > > > I folded your changes

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-19 Thread Mike Snitzer
On Tue, Mar 18 2014 at 9:45pm -0400, Shaohua Li s...@kernel.org wrote: On Tue, Mar 18, 2014 at 05:28:43PM -0400, Mike Snitzer wrote: On Tue, Mar 18 2014 at 3:41am -0400, Shaohua Li s...@kernel.org wrote: On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: I folded

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-18 Thread Shaohua Li
On Tue, Mar 18, 2014 at 05:28:43PM -0400, Mike Snitzer wrote: > On Tue, Mar 18 2014 at 3:41am -0400, > Shaohua Li wrote: > > > On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: > > > > > > I folded your changes in, and then committed a patch ontop that cleans > > > some code up.

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-18 Thread Mike Snitzer
On Tue, Mar 18 2014 at 3:41am -0400, Shaohua Li wrote: > On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: > > > > I folded your changes in, and then committed a patch ontop that cleans > > some code up. But added 2 FIXMEs that still speak to pretty fundamental > > problems with

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-18 Thread Shaohua Li
On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: > On Mon, Mar 17 2014 at 5:56am -0400, > Shaohua Li wrote: > > > On Fri, Mar 14, 2014 at 06:44:45PM -0400, Mike Snitzer wrote: > > > On Fri, Mar 14 2014 at 5:40am -0400, > > > Shaohua Li wrote: > > > > > > > On Mon, Mar 10, 2014

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-18 Thread Shaohua Li
On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: On Mon, Mar 17 2014 at 5:56am -0400, Shaohua Li s...@kernel.org wrote: On Fri, Mar 14, 2014 at 06:44:45PM -0400, Mike Snitzer wrote: On Fri, Mar 14 2014 at 5:40am -0400, Shaohua Li s...@kernel.org wrote: On Mon,

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-18 Thread Mike Snitzer
On Tue, Mar 18 2014 at 3:41am -0400, Shaohua Li s...@kernel.org wrote: On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: I folded your changes in, and then committed a patch ontop that cleans some code up. But added 2 FIXMEs that still speak to pretty fundamental problems

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-18 Thread Shaohua Li
On Tue, Mar 18, 2014 at 05:28:43PM -0400, Mike Snitzer wrote: On Tue, Mar 18 2014 at 3:41am -0400, Shaohua Li s...@kernel.org wrote: On Mon, Mar 17, 2014 at 04:00:40PM -0400, Mike Snitzer wrote: I folded your changes in, and then committed a patch ontop that cleans some code up.

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-17 Thread Mike Snitzer
On Mon, Mar 17 2014 at 5:56am -0400, Shaohua Li wrote: > On Fri, Mar 14, 2014 at 06:44:45PM -0400, Mike Snitzer wrote: > > On Fri, Mar 14 2014 at 5:40am -0400, > > Shaohua Li wrote: > > > > > On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: > > > > On Fri, Mar 07 2014 at 2:57am

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-17 Thread Shaohua Li
On Fri, Mar 14, 2014 at 06:44:45PM -0400, Mike Snitzer wrote: > On Fri, Mar 14 2014 at 5:40am -0400, > Shaohua Li wrote: > > > On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: > > > On Fri, Mar 07 2014 at 2:57am -0500, > > > Shaohua Li wrote: > > > > > > > ping! > > > > > > Hi,

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-17 Thread Shaohua Li
On Fri, Mar 14, 2014 at 06:44:45PM -0400, Mike Snitzer wrote: On Fri, Mar 14 2014 at 5:40am -0400, Shaohua Li s...@kernel.org wrote: On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: On Fri, Mar 07 2014 at 2:57am -0500, Shaohua Li s...@kernel.org wrote: ping!

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-17 Thread Mike Snitzer
On Mon, Mar 17 2014 at 5:56am -0400, Shaohua Li s...@kernel.org wrote: On Fri, Mar 14, 2014 at 06:44:45PM -0400, Mike Snitzer wrote: On Fri, Mar 14 2014 at 5:40am -0400, Shaohua Li s...@kernel.org wrote: On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: On Fri, Mar 07

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-14 Thread Mike Snitzer
On Fri, Mar 14 2014 at 5:40am -0400, Shaohua Li wrote: > On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: > > On Fri, Mar 07 2014 at 2:57am -0500, > > Shaohua Li wrote: > > > > > ping! > > > > Hi, > > > > I intend to get dm-insitu-comp reviewed for 3.15. Sorry I haven't > >

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-14 Thread Shaohua Li
On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: > On Fri, Mar 07 2014 at 2:57am -0500, > Shaohua Li wrote: > > > ping! > > Hi, > > I intend to get dm-insitu-comp reviewed for 3.15. Sorry I haven't > gotten back with you before now, been busy tending to 3.14-rc issues. > > I

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-14 Thread Shaohua Li
On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: On Fri, Mar 07 2014 at 2:57am -0500, Shaohua Li s...@kernel.org wrote: ping! Hi, I intend to get dm-insitu-comp reviewed for 3.15. Sorry I haven't gotten back with you before now, been busy tending to 3.14-rc issues. I

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-14 Thread Mike Snitzer
On Fri, Mar 14 2014 at 5:40am -0400, Shaohua Li s...@kernel.org wrote: On Mon, Mar 10, 2014 at 09:52:56AM -0400, Mike Snitzer wrote: On Fri, Mar 07 2014 at 2:57am -0500, Shaohua Li s...@kernel.org wrote: ping! Hi, I intend to get dm-insitu-comp reviewed for 3.15. Sorry I

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-10 Thread Mike Snitzer
On Fri, Mar 07 2014 at 2:57am -0500, Shaohua Li wrote: > ping! Hi, I intend to get dm-insitu-comp reviewed for 3.15. Sorry I haven't gotten back with you before now, been busy tending to 3.14-rc issues. I took a quick first pass over your code a couple weeks ago. Looks to be in great shape

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-10 Thread Mike Snitzer
On Fri, Mar 07 2014 at 2:57am -0500, Shaohua Li s...@kernel.org wrote: ping! Hi, I intend to get dm-insitu-comp reviewed for 3.15. Sorry I haven't gotten back with you before now, been busy tending to 3.14-rc issues. I took a quick first pass over your code a couple weeks ago. Looks to be

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-06 Thread Shaohua Li
ping! On Tue, Feb 18, 2014 at 06:13:04PM +0800, Shaohua Li wrote: > > This is a simple DM target supporting compression for SSD only. Under layer > SSD > must support 512B sector size, the target only supports 4k sector size. > > Disk layout: > |super|...meta...|..data...| > > Store unit is

Re: [patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-03-06 Thread Shaohua Li
ping! On Tue, Feb 18, 2014 at 06:13:04PM +0800, Shaohua Li wrote: This is a simple DM target supporting compression for SSD only. Under layer SSD must support 512B sector size, the target only supports 4k sector size. Disk layout: |super|...meta...|..data...| Store unit is 4k (a

[patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-02-18 Thread Shaohua Li
This is a simple DM target supporting compression for SSD only. Under layer SSD must support 512B sector size, the target only supports 4k sector size. Disk layout: |super|...meta...|..data...| Store unit is 4k (a block). Super is 1 block, which stores meta and data size and compression

[patch v3]DM: dm-insitu-comp: a compressed DM target for SSD

2014-02-18 Thread Shaohua Li
This is a simple DM target supporting compression for SSD only. Under layer SSD must support 512B sector size, the target only supports 4k sector size. Disk layout: |super|...meta...|..data...| Store unit is 4k (a block). Super is 1 block, which stores meta and data size and compression