Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S968512AbdIZMHa (ORCPT ); Tue, 26 Sep 2017 08:07:30 -0400 Received: from mail-wm0-f45.google.com ([74.125.82.45]:48948 "EHLO mail-wm0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965484AbdIZMH1 (ORCPT ); Tue, 26 Sep 2017 08:07:27 -0400 X-Google-Smtp-Source: AOwi7QC8q4Vj7w/PgySF8MBsb6s4yafeL3YD5pugyU9kioQA+9ZQFp6KinHcpag5bfRAhqJOQMb7WQ== From: Benjamin Gaignard To: labbott@redhat.com, sumit.semwal@linaro.org, gregkh@linuxfoundation.org, arve@android.com, riandrews@android.com, broonie@kernel.org, dan.carpenter@oracle.com Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-api@vger.kernel.org, Benjamin Gaignard Subject: [PATCH v4 0/2] staging: ion: get one device per heap Date: Tue, 26 Sep 2017 14:07:03 +0200 Message-Id: <1506427625-19202-1-git-send-email-benjamin.gaignard@linaro.org> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1507 Lines: 37 version 4: - add a configuration flag to switch between legacy Ion misc device and one device per heap version. This change has been suggested after Laura talks at XDC 2017. version 3: - change ion_device_add_heap prototype to return a possible error. version 2: - simplify ioctl check like propose by Dan - make sure that we don't register more than ION_DEV_MAX heaps. Until now all ion heaps are addressing using the same device "/dev/ion". This way of working doesn't allow to give access rights (for example with SElinux rules) per heap. This series propose to have (under configuartion flag) one device "/dev/ionX" per heap. Query heaps informations will be possible on each device node but allocation request will only be possible if heap_mask_id match with device minor number. That enable thsi new flag change Ion ABI because: - device name change - allocation must be done on the correct device/heap. - device major number will not be the same and that could impact init scripts. Benjamin Gaignard (2): staging: ion: simplify ioctl args checking function staging: ion: create one device entry per heap drivers/staging/android/TODO | 1 - drivers/staging/android/ion/Kconfig | 8 ++++++++ drivers/staging/android/ion/ion-ioctl.c | 27 +++++++++++++++++++-------- drivers/staging/android/ion/ion.c | 29 +++++++++++++++++++++++++++-- drivers/staging/android/ion/ion.h | 18 +++++++++++++++++- 5 files changed, 71 insertions(+), 12 deletions(-) -- 2.7.4