linux-kernel - Re: [PATCHv2 1/1] [tools]: android/ion: userspace test utility for ion buffer sharing

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Mon, 9 Oct 2017 19:47:19 +0530
From:   Pintu Kumar <pintu.ping@...il.com>
To:     Laura Abbott <labbott@...hat.com>
Cc:     linux-kernel@...r.kernel.org,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Pintu Kumar <pintu_agarwal@...oo.com>
Subject: Re: [PATCHv2 1/1] [tools]: android/ion: userspace test utility for
 ion buffer sharing

On Fri, Oct 6, 2017 at 7:58 PM, Pintu Kumar <pintu.ping@...il.com> wrote:
>>> +
>>> +     heap_type = ion_info->heap_type;
>>> +     heap_size = ion_info->heap_size;
>>> +     flag_type = ion_info->flag_type;
>>> +     alloc_data.len = heap_size;
>>> +     alloc_data.heap_id_mask = heap_type;
>>
>> The heap type and heap ID are not the same thing. You need
>> to determine the heap id from using the new query ioctl.
>>
>
> Hello Laura,
> Can you point out some example on how to retrieve heap_id from query ioctl.
> Sorry, I am not able to figure out, how can I get a specific heap_id
> if I know the heap_type.
> Do, I need to query all the heaps? Then how do I map to heap_data for each heap?
> Here is what I am doing.
>
> struct ion_heap_query query;
> struct ion_heap_data heap_data[5];
>
>         query.cnt = 5;
>         /* Query ION heap_id_mask from ION heap */
>         ret = ioctl(ionfd, ION_IOC_HEAP_QUERY, &query);
>         if (ret < 0) {
>                 fprintf(stderr, "<%s>: Failed: ION_IOC_HEAP_QUERY: %s\n",
>                         __func__, strerror(errno));
>                 goto err_heap;
>         }
>
> After this how to map each heap_data[..]  to query.heaps?
> Do you have any examples?

Dear Laura,
I figured out myself.
Now I am able to query heap_id from the new QUERY ioctl.
Also I have added a script to test all the heaps from single terminal.
Things are working fine.
I will post the new patch set soon.
However I have few questions to be clarified.
1) I have moved the test setup under: tools/testing/selftests/android/ion/
    So that it can be included easily in selftests framework later.
    Hope this is fine.

2) I observed that ION_HEAP_TYPE_DMA heap type is returned as
"reserved" heap name.
    However allocation goes through CMA area. Hope this is expected behavior.

3) For me CHUNK & CARVEDOUT heap is not working, on my Ubuntu PC.
    So I could not check it.
    I know these heaps needs to be declared first, may be in device
tree or somewhere else.
    Do you know how to verify this on Ubuntu x86 PC?

4) Please see the snapshots of output from the test results.

linux/tools/testing/selftests/android/ion$ sudo ./ion_test.sh
heap_type: 0, heap_size: 10
--------------------------------------
heap type: 0
  heap id: 1
heap name: ion_system_heap
--------------------------------------
Fill buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
Sharing fd: 6, Client fd: 5
<ion_close_buffer_fd>: buffer release successfully....
Received buffer fd: 4
Read buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0x0 0x0 0x0 0x0 0x0
0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0
Fill buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd
<ion_close_buffer_fd>: buffer release successfully....
ion_test.sh: heap_type: 0 - [PASS]

heap_type: 1, heap_size: 10
--------------------------------------
heap type: 1
  heap id: 0
heap name: ion_system_contig_heap
--------------------------------------
Fill buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
Sharing fd: 6, Client fd: 5
<ion_close_buffer_fd>: buffer release successfully....
Received buffer fd: 4
Read buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0x0 0x0 0x0 0x0 0x0
0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0
Fill buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd
<ion_close_buffer_fd>: buffer release successfully....
ion_test.sh: heap_type: 1 - [PASS]

heap_type: 2, heap_size: 10
<ion_export_buffer_fd>: ERROR: heap type does not exists
FAILED: ion_get_buffer_fd
<ion_close_buffer_fd>: buffer release successfully....
Received buffer fd: 0
ERROR: improper buf fd
ion_test.sh: heap_type: 2 - [FAIL]

heap_type: 3, heap_size: 10
<ion_export_buffer_fd>: ERROR: heap type does not exists
FAILED: ion_get_buffer_fd
<ion_close_buffer_fd>: buffer release successfully....
Received buffer fd: 0
ERROR: improper buf fd
ion_test.sh: heap_type: 3 - [FAIL]

heap_type: 4, heap_size: 10
--------------------------------------
heap type: 4
  heap id: 2
heap name: reserved
--------------------------------------
Fill buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
Sharing fd: 6, Client fd: 5
<ion_close_buffer_fd>: buffer release successfully....
Received buffer fd: 4
Read buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd
Fill buffer content:
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd
0xfd 0xfd 0xfd 0xfd
<ion_close_buffer_fd>: buffer release successfully....
ion_test.sh: heap_type: 4 - [PASS]

done....




Thanks,
Pintu