Re: [PATCH v2 2/6] libceph: introduce reference counted string

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

 



On Tue, Mar 29, 2016 at 11:56 AM, Yan, Zheng <zyan@xxxxxxxxxx> wrote:
> Here is code that share i_layout and embed namespace in osd_request.  But the code is more complex and less less efficient. I strongly doubt it’s good design.
>
> https://github.com/ceph/ceph-client/commits/wip-file-layout2 <https://github.com/ceph/ceph-client/commits/wip-file-layout2>

When I suggested it, I assumed namespace updates were more synchronous
than they actually are.  I said in one of my previous emails a few days
ago that it seemed less attractive, at least compared to the posted
patches, so I agree.

The string-sharing approach reaches deeply into osd_client though.
I have a *big* set of changes in the works for it - let me try to finish
it, prepare the ground for reference-counting for object locators and
ping you in couple of weeks?  Discussing particulars like locking and
RCU usage should be easier then.

Thanks,

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



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux