- Subject: Re: [PATCH v2 08/14] block: Kill bi_destructor
- From: Boaz Harrosh <bharrosh@xxxxxxxxxxx>
- Date: Fri, 25 May 2012 09:43:03 +0300
- Cc: axboe@xxxxxxxxx, dm-devel@xxxxxxxxxx, Kent Overstreet <koverstreet@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, tj@xxxxxxxxxx, linux-bcache@xxxxxxxxxxxxxxx, mpatocka@xxxxxxxxxx, yehuda@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, sage@xxxxxxxxxxxx, agk@xxxxxxxxxx, drbd-dev@xxxxxxxxxxxxxxxx
- In-reply-to: <20120524195202.GG27550@redhat.com>
- References: <1337817771-25038-1-git-send-email-koverstreet@google.com> <1337817771-25038-9-git-send-email-koverstreet@google.com> <20120524195202.GG27550@redhat.com>
- Reply-to: device-mapper development <dm-devel@xxxxxxxxxx>
- User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111113 Thunderbird/8.0
On 05/24/2012 10:52 PM, Vivek Goyal wrote:
> In fact I am not even sure that for one driver we should introduce
> bio_reset() in generic block layer. So to me we should get rid of bio_reset()
> and let all the gory details remain in driver.
>
I disagree. The kind of trick we do where up to BIO_RESET_BYTES
the struct is memset **must** stay at header, very close to the
structure.
I would though make bio_reset() inline at header, it's a single
memset. Even the memset is inlined by the compiler.
> Thanks
> Vivek
Thanks
Boaz
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
[DM Crypt]
[Fedora Desktop]
[ATA RAID]
[Fedora Marketing]
[Fedora Packaging]
[Fedora SELinux]
[Yosemite Discussion]
[Yosemite Photos]
[KDE Users]
[Fedora Tools]
[Fedora Docs]