Re: rbd import from stdin and rbd_concurrent_management_ops

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

 



On Wed, Mar 11, 2020 at 12:31 PM Marian Krucina
<marian.krucina@xxxxxxxxx> wrote:
>
> What is different between read from stdin and file? I don't see any.
> Is safe remove only the condition "fd == STDIN_FILENO" ?
> Will data be ok?

Yes, I believe you can just drop the special casing for the max number
of ops if "fd == STDIN_FILENO". You'll need to keep the other special
cases for avoiding seeks and resizing the image (since we won't know
the final size until we are done).

> On Wed, Mar 11, 2020 at 1:25 PM Jason Dillaman <jdillama@xxxxxxxxxx> wrote:
>>
>> On Wed, Mar 11, 2020 at 7:59 AM <marian.krucina@xxxxxxxxx> wrote:
>> >
>> > Hi,
>> >
>> > why rbd import diff from stdin sets rbd_concurrent_management_ops to 1?
>> >
>> > https://github.com/ceph/ceph/blob/master/src/tools/rbd/action/Import.cc#L39
>> > https://github.com/ceph/ceph/blob/master/src/tools/rbd/action/Import.cc#L740
>> >
>> > Can you remove this limitation?
>>
>> The code would need to be restructured a bit to support that, but
>> definitely feel free to submit a PR to remove that limitation.
>>
>> > Thanks & Regards,
>> > Marian
>> > _______________________________________________
>> > Dev mailing list -- dev@xxxxxxx
>> > To unsubscribe send an email to dev-leave@xxxxxxx
>> >
>>
>>
>> --
>> Jason
>>


-- 
Jason
_______________________________________________
Dev mailing list -- dev@xxxxxxx
To unsubscribe send an email to dev-leave@xxxxxxx



[Index of Archives]     [CEPH Users]     [Ceph Devel]     [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