From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Jurgens Subject: Re: [PATCH 03/12] selinux: Implement Infiniband flush callback Date: Thu, 30 Jun 2016 15:44:45 +0000 Message-ID: References: <1466711578-64398-1-git-send-email-danielj@mellanox.com> <1466711578-64398-4-git-send-email-danielj@mellanox.com> <20160630151005.GC22107@yuval-lap.uk.oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Language: en-US List-Post: List-Help: Errors-To: selinux-bounces-+05T5uksL2qpZYMLLGbcSA@public.gmane.org Sender: "Selinux" To: Yuval Shaia Cc: "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-security-module-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "chrisw-69jw2NvuJkxg9hUCZPvPmw@public.gmane.org" , "dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org" , "selinux-+05T5uksL2qpZYMLLGbcSA@public.gmane.org" , "sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org" , "sds-+05T5uksL2qpZYMLLGbcSA@public.gmane.org" , "hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org" List-Id: linux-rdma@vger.kernel.org On 6/30/2016 10:10 AM, Yuval Shaia wrote: > On Thu, Jun 23, 2016 at 10:52:49PM +0300, Dan Jurgens wrote: > >> +static void (*ib_flush_callback)(void); > Do we really want to have such ib_ prefix in security/ directory? > >> + if (ib_flush_callback) >> + ib_flush_callback(); > How about some generic mechanism (such as a list) in case more > modules/drivers would like to register callbacks? > ( assuming this is no longer RFC :) ) > Paul Moore and I were having a higher level discussion about this in the 00/12 thread. I think your suggestion makes sense, perhaps Paul will weigh in when he reaches this patch. _______________________________________________ Selinux mailing list Selinux-+05T5uksL2qpZYMLLGbcSA@public.gmane.org To unsubscribe, send email to Selinux-leave-+05T5uksL2pAGbPMOrvdOA@public.gmane.org To get help, send an email containing "help" to Selinux-request-+05T5uksL2pAGbPMOrvdOA@public.gmane.org From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Jurgens To: Yuval Shaia CC: "chrisw@sous-sol.org" , "paul@paul-moore.com" , "sds@tycho.nsa.gov" , "eparis@parisplace.org" , "dledford@redhat.com" , "sean.hefty@intel.com" , "hal.rosenstock@gmail.com" , "selinux@tycho.nsa.gov" , "linux-security-module@vger.kernel.org" , "linux-rdma@vger.kernel.org" , Yevgeny Petrilin Subject: Re: [PATCH 03/12] selinux: Implement Infiniband flush callback Date: Thu, 30 Jun 2016 15:44:45 +0000 Message-ID: References: <1466711578-64398-1-git-send-email-danielj@mellanox.com> <1466711578-64398-4-git-send-email-danielj@mellanox.com> <20160630151005.GC22107@yuval-lap.uk.oracle.com> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 List-Id: "Security-Enhanced Linux \(SELinux\) mailing list" List-Post: List-Help: On 6/30/2016 10:10 AM, Yuval Shaia wrote: > On Thu, Jun 23, 2016 at 10:52:49PM +0300, Dan Jurgens wrote: > >> +static void (*ib_flush_callback)(void); > Do we really want to have such ib_ prefix in security/ directory? > >> + if (ib_flush_callback) >> + ib_flush_callback(); > How about some generic mechanism (such as a list) in case more > modules/drivers would like to register callbacks? > ( assuming this is no longer RFC :) ) > Paul Moore and I were having a higher level discussion about this in the 00/12 thread. I think your suggestion makes sense, perhaps Paul will weigh in when he reaches this patch.