From: Alexey Skidanov <alexey.skidanov@intel.com>
To: Laura Abbott <labbott@redhat.com>, devel@driverdev.osuosl.org
Cc: linux-kernel@vger.kernel.org
Subject: Re: staging: ion: ION allocation fall back order depends on heap linkage order
Date: Wed, 7 Feb 2018 17:10:36 +0200 [thread overview]
Message-ID: <3f77b591-6248-da0a-e20d-88ca26a60a95@intel.com> (raw)
In-Reply-To: <a210baff-4124-b0d7-bcd9-a2ee8b986c7a@redhat.com>
On 02/07/2018 04:58 PM, Laura Abbott wrote:
> On 02/06/2018 11:05 PM, Alexey Skidanov wrote:
>>
>>
>>> Yup, you've hit upon a key problem. Having fallbacks be stable
>>> was always a problem and the recommendation these days is to
>>> not rely on them. You can specify a heap at a time and fallback
>>> manually if you want that behavior.
>>>
>>> If you have a proposal to make fallbacks work reliably without
>>> overly complicating the ABI I'm happy to review it.
>>>
>>> Thanks,
>>> Laura
>>>
>> I think it's possible to "automate" the "manual fallback" behavior. But
>> the real issues is using heap id to specify the particular heap object.
>>
>> Current API (allocation IOCTL) requires to specify the particular heap
>> object by using heap id. From the other hand, the user space doesn't
>> control the heaps creation order and heap id assignment. So it may be
>> tricky, especially when more than one object of the same heap type is
>> created automatically.
>>
>> Thanks,
>> Alexey
>>
>>
>
> The query ioctl is designed to get the heap ID information without
> needing to rely on the linking order or anything else defined in
> the kernel.
>
> Thanks,
> Laura
That is true. But if we have 2 *automatically created* heaps of the same
type, how userspace can distinguish between them?
Thanks,
Alexey
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
next prev parent reply other threads:[~2018-02-07 15:10 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-28 16:24 Alexey Skidanov
2018-02-06 23:48 ` Laura Abbott
[not found] ` <CAM7Y4U_hw5anfgGyAK2uVxOxA3zPoJtOa93mrw8k7xfyOaSTUA@mail.gmail.com>
2018-02-07 7:05 ` Alexey Skidanov
2018-02-07 14:58 ` Laura Abbott
2018-02-07 15:10 ` Alexey Skidanov [this message]
2018-02-07 15:32 ` Laura Abbott
2018-02-07 15:49 ` Alexey Skidanov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3f77b591-6248-da0a-e20d-88ca26a60a95@intel.com \
--to=alexey.skidanov@intel.com \
--cc=devel@driverdev.osuosl.org \
--cc=labbott@redhat.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome