Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751917AbdI1SLz (ORCPT ); Thu, 28 Sep 2017 14:11:55 -0400 Received: from mail-wr0-f196.google.com ([209.85.128.196]:44304 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750819AbdI1SLy (ORCPT ); Thu, 28 Sep 2017 14:11:54 -0400 X-Google-Smtp-Source: AOwi7QCh+TQQxUPtNnA6XX3+pdKaI8gjv19JxsnlIrPJRLiKFItUTVMHJB/W/aYXh92H1aH6l6N8DhaYnoIQG2HBwws= MIME-Version: 1.0 In-Reply-To: References: <1506449281-8790-1-git-send-email-pintu.ping@gmail.com> <403830bf-862c-6857-3e2f-5887e91c3280@redhat.com> From: Pintu Kumar Date: Thu, 28 Sep 2017 23:41:52 +0530 Message-ID: Subject: Re: [PATCH 1/1] [tools]: android/ion: userspace test utility for ion buffer sharing To: Laura Abbott , Greg Kroah-Hartman , Pintu Kumar Cc: linux-kernel@vger.kernel.org, Pintu Kumar Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2699 Lines: 74 On Wed, Sep 27, 2017 at 7:54 PM, Pintu Kumar wrote: > On Wed, Sep 27, 2017 at 12:52 AM, Laura Abbott wrote: >> On 09/26/2017 11:08 AM, Pintu Agarwal wrote: >>> >>> This is a test utility to verify ION buffer sharing in user space >>> between 2 independent processes. >>> It uses unix domain socket as IPC to transfer an FD to another process >>> and install it. >>> >>> This utility demonstrates how ION buffer sharing can be implemented >>> between >>> two user space processes, using various heap ids. >>> >>> This utility is verified on Ubuntu 32-bit machine using 2 independent >>> process such as: ionapp_export (server) and ionapp_import (client). >>> First the server needs to be run to export FD to the client. >>> This utility works only if /dev/ion interface is present. >>> >>> Here is a sample demo example: >>> >>> linux-stable/tools/android/ion$ sudo ./ionapp_export.out -i 1 -s 10 >>> heap_type: 2, heap_size: 10 >>> Fill buffer content: >>> 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd >>> Sharing fd: 6, Client fd: 5 >>> : buffer release successfully.... >>> >>> linux-stable/tools/android/ion$ sudo ./ionapp_import.out >>> Received buffer fd: 4 >>> Read buffer content: >>> 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 >>> : buffer release successfully.... >>> >>> Signed-off-by: Pintu Agarwal >> >> >> This is a good start on a test app for Ion but it needs to be >> updated to use the new ABI. > > Thanks Laura for your feedback. > Can you point out what changes are required? > Ok I got it what you mean. In the latest mainline kernel I observed that there are several changes to uapi/ion.h header file. Like this one: staging: android: ion: Break the ABI in the name of forward progress I can quickly make those changes. In fact I actually started with linux-next tree itself. But there is one problem that I am facing using linux-next or latest mainline kernel. Right now, I am running Ubuntu on Oracle virtual box 5.1 (I don't have the real Ubuntu PC as of now). This virtual box supports only 4.10 LTS kernel version. If I try to build and flash for higher kernel version, the system does not boot and goes to kernel panic during boot (even without my changes). So, I ended up using the same LTS kernel version. Now I need to figure out how to use the linux-next tree. If you have any suggestions please let me know. Thanks, Pintu >> >> Thanks, >> Laura