Re: [PATCH 1/5] [BLOCK] Add 'discard' request handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2008-08-08 at 12:49 +0200, Jens Axboe wrote:
> This is what I came up with. You don't need the request discard flag,
> since you are using a specific request type for this. Additionally, we
> can move the request init where it is done for fs requests
> (init_req_from_bio) and get it set there. 

Hmmm. (qv)

> I also added the small bits for merging.

Nice. I was _hoping_ it would be that easy, but didn't want to get
involved with that yet. One tweak we might want to make to that would be
to allow discard requests to be _dropped_ if there's a subsequent write
to the same sectors -- if we expect it to be common that we deallocate
sectors then re-use them again before the discard request hits the disk,
that might be an interesting optimisation.

-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse@xxxxxxxxx                              Intel Corporation



--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux