From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Date: Wed, 3 Feb 2016 14:19:09 -0800 Subject: [lustre-devel] [PATCH v2 00/13] staging:lustre: split kernel comm between user and kernel In-Reply-To: <1450905892-31089-1-git-send-email-jsimmons@infradead.org> References: <1450905892-31089-1-git-send-email-jsimmons@infradead.org> Message-ID: <20160203221909.GA11314@kroah.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: James Simmons Cc: devel@driverdev.osuosl.org, Oleg Drokin , Andreas Dilger , Linux Kernel Mailing List , lustre-devel@lists.lustre.org On Wed, Dec 23, 2015 at 04:24:39PM -0500, James Simmons wrote: > The kernel communication code used for HSM and changelog is > entangled. Move the user space bits into the liblustreapi. This will > also help for a possible relicensing. The kernel portion is also moved > from libcfs to obdclass. > > The original libcfs_kernelcomm.h header is split into three parts: > > * lustre_kernelcomm.h, a new header for the kernel parts; > * uapi_kernelcomm.h, a new header for the data structures shared > between userspace and kernelspace; > * lustreapi_internal.h receives the private liblustreapi prototypes. > > The original code in kernel_user_comm.c is split into two parts: > > * obdclass/kernelcomm.c for the kernel part. filp_user_write() was > moved there, and linux-fs.c deleted; > * liblustreapi_kernelconn.c for the user part. The calls to CDEBUG > have been removed, and calls to CERROR have been transformed to > llapi_err_noerrno. The type lustre_kernelcomm has been removed and > replace by struct lustre_kernelcomm. > > Various names and filenames have been harmonized to *kernelcomm*. > > The unused symbol KUC_FL_BLOCK has been removed. > > Signed-off-by: frank zago > Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 > Reviewed-on: http://review.whamcloud.com/14270 > Reviewed-by: Nathan Rutman > Reviewed-by: James Simmons > Reviewed-by: Dmitry Eremin > Reviewed-by: John L. Hammond > Reviewed-by: Oleg Drokin Note, your subject is the most difficult to deal with, as you didn't put the 'v2' before each patch, just some of them, so you can't sort them in an email client and I have to hand-edit them to get this to apply properly :( Please don't do that again... From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965581AbcBCWTN (ORCPT ); Wed, 3 Feb 2016 17:19:13 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:44074 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965509AbcBCWTK (ORCPT ); Wed, 3 Feb 2016 17:19:10 -0500 Date: Wed, 3 Feb 2016 14:19:09 -0800 From: Greg Kroah-Hartman To: James Simmons Cc: devel@driverdev.osuosl.org, Oleg Drokin , Andreas Dilger , Linux Kernel Mailing List , lustre-devel@lists.lustre.org Subject: Re: [PATCH v2 00/13] staging:lustre: split kernel comm between user and kernel Message-ID: <20160203221909.GA11314@kroah.com> References: <1450905892-31089-1-git-send-email-jsimmons@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1450905892-31089-1-git-send-email-jsimmons@infradead.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 23, 2015 at 04:24:39PM -0500, James Simmons wrote: > The kernel communication code used for HSM and changelog is > entangled. Move the user space bits into the liblustreapi. This will > also help for a possible relicensing. The kernel portion is also moved > from libcfs to obdclass. > > The original libcfs_kernelcomm.h header is split into three parts: > > * lustre_kernelcomm.h, a new header for the kernel parts; > * uapi_kernelcomm.h, a new header for the data structures shared > between userspace and kernelspace; > * lustreapi_internal.h receives the private liblustreapi prototypes. > > The original code in kernel_user_comm.c is split into two parts: > > * obdclass/kernelcomm.c for the kernel part. filp_user_write() was > moved there, and linux-fs.c deleted; > * liblustreapi_kernelconn.c for the user part. The calls to CDEBUG > have been removed, and calls to CERROR have been transformed to > llapi_err_noerrno. The type lustre_kernelcomm has been removed and > replace by struct lustre_kernelcomm. > > Various names and filenames have been harmonized to *kernelcomm*. > > The unused symbol KUC_FL_BLOCK has been removed. > > Signed-off-by: frank zago > Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 > Reviewed-on: http://review.whamcloud.com/14270 > Reviewed-by: Nathan Rutman > Reviewed-by: James Simmons > Reviewed-by: Dmitry Eremin > Reviewed-by: John L. Hammond > Reviewed-by: Oleg Drokin Note, your subject is the most difficult to deal with, as you didn't put the 'v2' before each patch, just some of them, so you can't sort them in an email client and I have to hand-edit them to get this to apply properly :( Please don't do that again...