From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: James Simmons <jsimmons@infradead.org>
Cc: devel@driverdev.osuosl.org, Oleg Drokin <oleg.drokin@intel.com>,
Andreas Dilger <andreas.dilger@intel.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
lustre-devel@lists.lustre.org
Subject: [lustre-devel] [PATCH v2 00/13] staging:lustre: split kernel comm between user and kernel
Date: Wed, 3 Feb 2016 14:19:09 -0800 [thread overview]
Message-ID: <20160203221909.GA11314@kroah.com> (raw)
In-Reply-To: <1450905892-31089-1-git-send-email-jsimmons@infradead.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 <fzago@cray.com>
> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
> Reviewed-on: http://review.whamcloud.com/14270
> Reviewed-by: Nathan Rutman <nathan.rutman@seagate.com>
> Reviewed-by: James Simmons <uja.ornl@yahoo.com>
> Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
> Reviewed-by: John L. Hammond <john.hammond@intel.com>
> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
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...
WARNING: multiple messages have this Message-ID (diff)
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: James Simmons <jsimmons@infradead.org>
Cc: devel@driverdev.osuosl.org, Oleg Drokin <oleg.drokin@intel.com>,
Andreas Dilger <andreas.dilger@intel.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
lustre-devel@lists.lustre.org
Subject: Re: [PATCH v2 00/13] staging:lustre: split kernel comm between user and kernel
Date: Wed, 3 Feb 2016 14:19:09 -0800 [thread overview]
Message-ID: <20160203221909.GA11314@kroah.com> (raw)
In-Reply-To: <1450905892-31089-1-git-send-email-jsimmons@infradead.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 <fzago at cray.com>
> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
> Reviewed-on: http://review.whamcloud.com/14270
> Reviewed-by: Nathan Rutman <nathan.rutman at seagate.com>
> Reviewed-by: James Simmons <uja.ornl at yahoo.com>
> Reviewed-by: Dmitry Eremin <dmitry.eremin at intel.com>
> Reviewed-by: John L. Hammond <john.hammond at intel.com>
> Reviewed-by: Oleg Drokin <oleg.drokin at intel.com>
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...
next prev parent reply other threads:[~2016-02-03 22:19 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-23 21:24 [PATCH v2 00/13] staging:lustre: split kernel comm between user and kernel James Simmons
2015-12-23 21:24 ` [PATCH v2 01/13] staging: lustre: kg_sem semaphore handling is incorrectly James Simmons
2015-12-23 21:24 ` [PATCH v2 02/13] staging: lustre: Prevent duplicate CT registrations James Simmons
2015-12-23 21:24 ` [PATCH v2 03/13] staging: lustre: move kernel_user_comm.c from libcfs to lustre James Simmons
2015-12-23 21:24 ` [PATCH v2 04/13] staging: lustre: split kernel comm between user and kernel James Simmons
2015-12-23 21:24 ` [PATCH v2 05/13] staging: lustre: embed kr_data into kkuc_reg James Simmons
2015-12-23 21:24 ` [PATCH v2 06/13] staging: lustre: convert kernelcomm group to unsigned int James Simmons
2015-12-23 21:24 ` [PATCH v2 07/13] staging: lustre: remove unnecessary NULL checks in kernel_comm.c James Simmons
2015-12-23 21:24 ` [PATCH v2 08/13] staging: lustre: cleanup block comment style in kernel_comm code James Simmons
2015-12-23 21:24 ` [PATCH 09/13] staging: lustre: add space around '+' " James Simmons
2015-12-23 21:24 ` [PATCH 10/13] staging: lustre: use proper braces in libcfs_kkuc_group_put James Simmons
2015-12-23 21:24 ` [PATCH 11/13] staging: lustre: return proper error code for libcfs_kkuc_msg_put James Simmons
2015-12-23 21:24 ` [PATCH 12/13] staging: lustre: fix all conditional comparison to zero for kernelcomm.c James Simmons
2015-12-23 21:24 ` [PATCH 13/13] staging: lustre: cleanup white space in kernel comm code James Simmons
2016-02-03 22:19 ` Greg Kroah-Hartman [this message]
2016-02-03 22:19 ` [PATCH v2 00/13] staging:lustre: split kernel comm between user and kernel Greg Kroah-Hartman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160203221909.GA11314@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=andreas.dilger@intel.com \
--cc=devel@driverdev.osuosl.org \
--cc=jsimmons@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lustre-devel@lists.lustre.org \
--cc=oleg.drokin@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.