Re: [PATCH v2] staging: lustre: remove IOC_LIBCFS_PING_TEST ioctl

From: Dilger, Andreas
Date: Thu Nov 26 2015 - 23:31:46 EST


On 2015/11/08, 09:34, "James Simmons" <jsimmons@xxxxxxxxxxxxx> wrote:

>The ioctl IOC_LIBCFS_PING_TEST has not been used in
>ages. The recent nidstring changes which moved all
>the nidstring operations from libcfs to the LNet
>layer but this ioctl code was still using an
>nidstring operation that was causing an circular
>dependency loop between libcfs and LNet:

Hi Greg,
are you planning on pushing this patch to Linus for 4.4? It was resent
on 11/08 per your request on 11/07 but I don't see it in staging or
staging-next yet. Since it fixes the depmod dependency cycle for
allmodconfig builds on mainline it seems worthwhile to include into
4.4-rc3 rather than waiting for 4.5.

Cheers, Andreas


>Signed-off-by: James Simmons <jsimmons@xxxxxxxxxxxxx>
>---
> .../lustre/include/linux/libcfs/libcfs_ioctl.h | 1 -
> drivers/staging/lustre/lustre/libcfs/module.c | 17
>-----------------
> 2 files changed, 0 insertions(+), 18 deletions(-)
>
>diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_ioctl.h
>b/drivers/staging/lustre/include/linux/libcfs/libcfs_ioctl.h
>index f5d741f..485ab26 100644
>--- a/drivers/staging/lustre/include/linux/libcfs/libcfs_ioctl.h
>+++ b/drivers/staging/lustre/include/linux/libcfs/libcfs_ioctl.h
>@@ -110,7 +110,6 @@ struct libcfs_ioctl_handler {
> #define IOC_LIBCFS_CLEAR_DEBUG _IOWR('e', 31, long)
> #define IOC_LIBCFS_MARK_DEBUG _IOWR('e', 32, long)
> #define IOC_LIBCFS_MEMHOG _IOWR('e', 36, long)
>-#define IOC_LIBCFS_PING_TEST _IOWR('e', 37, long)
> /* lnet ioctls */
> #define IOC_LIBCFS_GET_NI _IOWR('e', 50, long)
> #define IOC_LIBCFS_FAIL_NID _IOWR('e', 51, long)
>diff --git a/drivers/staging/lustre/lustre/libcfs/module.c
>b/drivers/staging/lustre/lustre/libcfs/module.c
>index 570f05c..89038ed 100644
>--- a/drivers/staging/lustre/lustre/libcfs/module.c
>+++ b/drivers/staging/lustre/lustre/libcfs/module.c
>@@ -274,23 +274,6 @@ static int libcfs_ioctl_int(struct cfs_psdev_file
> }
> break;
>
>- case IOC_LIBCFS_PING_TEST: {
>- extern void (kping_client)(struct libcfs_ioctl_data *);
>- void (*ping)(struct libcfs_ioctl_data *);
>-
>- CDEBUG(D_IOCTL, "doing %d pings to nid %s (%s)\n",
>- data->ioc_count, libcfs_nid2str(data->ioc_nid),
>- libcfs_nid2str(data->ioc_nid));
>- ping = symbol_get(kping_client);
>- if (!ping)
>- CERROR("symbol_get failed\n");
>- else {
>- ping(data);
>- symbol_put(kping_client);
>- }
>- return 0;
>- }
>-
> default: {
> struct libcfs_ioctl_handler *hand;
>
>
>--
>1.7.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/