On 03/21/2014 08:09 AM, Mike Snitzer wrote:
> On Fri, Mar 21 2014 at 2:32am -0400,
> Matias Bjørling <m@xxxxxxxxxxx> wrote:
>
>> LightNVM implements the internal logic of an SSD within the host system.
>> This includes logic such as translation tables for logical to physical
>> address translation, garbage collection and wear-leveling.
>>
>> It is designed to be used either standalone or with a LightNVM
>> compatible firmware. If used standalone, NVM memory can be simulated
>> by passing timings to the dm target table. If used with a LightNVM
>> compatible device, the device will be queued upon initialized for the
>> relevant values.
>>
>> The last part is still in progress and a fully working prototype will be
>> presented in upcoming patches.
>>
>> Contributions to make this possible by the following people:
>>
>> Aviad Zuck <aviadzuc@xxxxxxxxx>
>> Jesper Madsen <jmad@xxxxxx>
>>
>> Signed-off-by: Matias Bjorling <m@xxxxxxxxxxx>
> ...
>> diff --git a/drivers/md/lightnvm/core.c b/drivers/md/lightnvm/core.c
>> new file mode 100644
>> index 0000000..113fde9
>> --- /dev/null
>> +++ b/drivers/md/lightnvm/core.c
>> @@ -0,0 +1,705 @@
>> +#include "lightnvm.h"
>> +
>> +/* alloc pbd, but also decorate it with bio */
>> +static struct per_bio_data *alloc_init_pbd(struct nvmd *nvmd, struct bio *bio)
>> +{
>> + struct per_bio_data *pb = mempool_alloc(nvmd->per_bio_pool, GFP_NOIO);
>> +
>> + if (!pb) {
>> + DMERR("Couldn't allocate per_bio_data");
>> + return NULL;
>> + }
>> +
>> + pb->bi_end_io = bio->bi_end_io;
>> + pb->bi_private = bio->bi_private;
>> +
>> + bio->bi_private = pb;
>> +
>> + return pb;
>> +}
>> +
>> +static void free_pbd(struct nvmd *nvmd, struct per_bio_data *pb)
>> +{
>> + mempool_free(pb, nvmd->per_bio_pool);
>> +}
>> +
>> +/* bio to be stripped from the pbd structure */
>> +static void exit_pbd(struct per_bio_data *pb, struct bio *bio)
>> +{
>> + bio->bi_private = pb->bi_private;
>> + bio->bi_end_io = pb->bi_end_io;
>> +}
>> +
>
> Hi Matias,
>
> This looks like it'll be very interesting! But I won't have time to do
> a proper review of this code for ~1.5 weeks (traveling early next week
> and then need to finish some high priority work on dm-thin once I'm
> back).
That's great.
>
> But a couple quick things I noticed:
>
> 1) you don't need to roll your own per-bio-data allocation code any
> more. The core block layer provides per_bio_data now.
>
> And the DM targets have been converted to make use of it. See callers
> of dm_per_bio_data() and how the associated targets set
> ti->per_bio_data_size
>
> 2) Also, if you're chaining bi_end_io (like it appears you're doing)
> you'll definitely need to call atomic_inc(&bio->bi_remaining); after you
> restore bio->bi_end_io. This is a new requirement of the 3.14 kernel
> (due to the block core's immutable biovec changes).
>
> Please sort these issues out, re-test on 3.14, and post v2, thanks!
> Mike
>
Thanks. I'll get it fixed and rebasedk/tested on 3.14.
Matias
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
