From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S932702AbbCYQIE (ORCPT <rfc822;w@1wt.eu>);
	Wed, 25 Mar 2015 12:08:04 -0400
Received: from mailout1.w1.samsung.com ([210.118.77.11]:63832 "EHLO
	mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1753166AbbCYQH5 (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 25 Mar 2015 12:07:57 -0400
X-AuditID: cbfec7f5-b7fc86d0000066b7-7d-5512dd5aac83
Date: Wed, 25 Mar 2015 17:07:45 +0100 (CET)
From: Paul Osmialowski <p.osmialowsk@samsung.com>
X-X-Sender: p.osmialowsk@AMDC1262.digital.local
To: David Herrmann <dh.herrmann@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Daniel Mack <daniel@zonque.org>, Djalal Harouni <tixxdz@opendz.org>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        Lukasz Skalski <l.skalski@samsung.com>,
        Paul Osmialowski <p.osmialowsk@samsung.com>
Subject: Re: [RFC PATCH] kdbus: move all kdbus headers to include/linux/kdbus
In-reply-to: <CANq1E4QmJphrB86ZarcreD9OA8oOWJgYogqV1pCRxj-M1e6GNA@mail.gmail.com>
Message-id: <alpine.DEB.2.10.1503251707350.27044@AMDC1262.digital.local>
References: <1427297835-19978-1-git-send-email-p.osmialowsk@samsung.com>
 <1427297835-19978-2-git-send-email-p.osmialowsk@samsung.com>
 <CANq1E4QmJphrB86ZarcreD9OA8oOWJgYogqV1pCRxj-M1e6GNA@mail.gmail.com>
User-Agent: Alpine 2.10 (DEB 1266 2009-07-14)
MIME-version: 1.0
Content-type: TEXT/PLAIN; charset=US-ASCII; format=flowed
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrBLMWRmVeSWpSXmKPExsVy+t/xa7pRd4VCDXZ3clk0/X3FYjGv9SiL
	RfPi9WwWc841Mllc3jWHzWLCmwPMFit/b2dzYPfYOesuu8f+uWvYPVb8OsLu0bdlFaPH501y
	Htt3L2cJYIvisklJzcksSy3St0vgyrj54zJTwWL2iubzs5kbGO+xdjFyckgImEicubGYHcIW
	k7hwbz1bFyMXh5DAUkaJI/fbGUESQgIfGSVWLjAHsVkEtCUmd19mA7HZBAwlbv4/zAjRrC/x
	89dDsEEiAloSG+fNZwIZxCzQxCSxcmUrM0hCWMBXYumRTWBFnALBEi+nbwQbxCvgIbHw3ENm
	iM3nGCUWL5/MApIQFdCVOPTvD1SRoMSPyffA4swClhI9Z1YwTWAUmIUkNQtJagEj0ypG0dTS
	5ILipPRcI73ixNzi0rx0veT83E2MkND+uoNx6TGrQ4wCHIxKPLwRSkKhQqyJZcWVuYcYJTiY
	lUR4d24CCvGmJFZWpRblxxeV5qQWH2Jk4uCUamCsfar20/3L/l/HCnielp33/Cl6b//Eyi0r
	5E6G7/Nd9TNr5Yn3a1hP8K590XJTc0e9x97tZz89CzIp77GK36zQwtbUpLpej3fuucal/1Mf
	3vAOr/74wqWIUSlk8jm1Ws5J6tMYvFyeFVTGOSltN9/hpvZhXsL563P5jBY/9vsaa7ZnY/6l
	s822SizFGYmGWsxFxYkARcEshEsCAAA=
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi,

First of all, I'm sorry for this no-subject covering letter. Accidents 
never happen only in a perfect world.

On Wed, 25 Mar 2015, David Herrmann wrote:

>
> I'd prefer if we only move the required structure definitions into a
> separate header. There is no reason to export all internal entry
> points.
>

You're right, this would definitely look nicer.

>
> Anyway, this is a technicality that actually depends on the
> implemented LSM hooks. So I'd prefer if we discuss that in combination
> with the actual LSM patches. I'm not entirely sure which parts are
> needed. For testing-purposes, we can always use -I./ipc/kdbus/, right?
>

Yes we can, which makes whole this patch a bit premature.

However, we can't predict whether someone may want to use kdbus headers 
for any other purpose. As for now, I don't see immediate need for this 
apart from implementing LSM callbacks.

Thanks,
Paul