[S390]: Adapt monreader driver to new IUCV API
authorMartin Schwidefsky <schwidefsky@de.ibm.com>
Thu, 8 Feb 2007 21:38:11 +0000 (13:38 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 8 Feb 2007 21:38:11 +0000 (13:38 -0800)
Adapt monreader character device driver to new IUCV API

Signed-off-by: Frank Pavlic <fpavlic@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/s390/char/monreader.c

index a138b1510093cc53a6538a99c47fe020da53518a..3a1a958fb5f2843a2132e1a487af739d06417915 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Character device driver for reading z/VM *MONITOR service records.
  *
- * Copyright (C) 2004 IBM Corporation, IBM Deutschland Entwicklung GmbH.
+ * Copyright 2004 IBM Corporation, IBM Deutschland Entwicklung GmbH.
  *
  * Author: Gerald Schaefer <geraldsc@de.ibm.com>
  */
@@ -22,7 +22,7 @@
 #include <asm/ebcdic.h>
 #include <asm/extmem.h>
 #include <linux/poll.h>
-#include "../net/iucv.h"
+#include <net/iucv/iucv.h>
 
 
 //#define MON_DEBUG                    /* Debug messages on/off */
@@ -50,14 +50,13 @@ static char mon_dcss_name[9] = "MONDCSS\0";
 struct mon_msg {
        u32 pos;
        u32 mca_offset;
-       iucv_MessagePending local_eib;
+       struct iucv_message msg;
        char msglim_reached;
        char replied_msglim;
 };
 
 struct mon_private {
-       u16 pathid;
-       iucv_handle_t iucv_handle;
+       struct iucv_path *path;
        struct mon_msg *msg_array[MON_MSGLIM];
        unsigned int   write_index;
        unsigned int   read_index;
@@ -75,8 +74,6 @@ static unsigned long mon_dcss_end;
 static DECLARE_WAIT_QUEUE_HEAD(mon_read_wait_queue);
 static DECLARE_WAIT_QUEUE_HEAD(mon_conn_wait_queue);
 
-static u8 iucv_host[8] = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00};
-
 static u8 user_data_connect[16] = {
        /* Version code, must be 0x01 for shared mode */
        0x01,
@@ -100,8 +97,7 @@ static u8 user_data_sever[16] = {
  * Create the 8 bytes EBCDIC DCSS segment name from
  * an ASCII name, incl. padding
  */
-static inline void
-dcss_mkname(char *ascii_name, char *ebcdic_name)
+static inline void dcss_mkname(char *ascii_name, char *ebcdic_name)
 {
        int i;
 
@@ -119,8 +115,7 @@ dcss_mkname(char *ascii_name, char *ebcdic_name)
  * print appropriate error message for segment_load()/segment_type()
  * return code
  */
-static void
-mon_segment_warn(int rc, char* seg_name)
+static void mon_segment_warn(int rc, char* seg_name)
 {
        switch (rc) {
        case -ENOENT:
@@ -166,44 +161,37 @@ mon_segment_warn(int rc, char* seg_name)
        }
 }
 
-static inline unsigned long
-mon_mca_start(struct mon_msg *monmsg)
+static inline unsigned long mon_mca_start(struct mon_msg *monmsg)
 {
-       return monmsg->local_eib.ln1msg1.iprmmsg1_u32;
+       return *(u32 *) &monmsg->msg.rmmsg;
 }
 
-static inline unsigned long
-mon_mca_end(struct mon_msg *monmsg)
+static inline unsigned long mon_mca_end(struct mon_msg *monmsg)
 {
-       return monmsg->local_eib.ln1msg2.ipbfln1f;
+       return *(u32 *) &monmsg->msg.rmmsg[4];
 }
 
-static inline u8
-mon_mca_type(struct mon_msg *monmsg, u8 index)
+static inline u8 mon_mca_type(struct mon_msg *monmsg, u8 index)
 {
        return *((u8 *) mon_mca_start(monmsg) + monmsg->mca_offset + index);
 }
 
-static inline u32
-mon_mca_size(struct mon_msg *monmsg)
+static inline u32 mon_mca_size(struct mon_msg *monmsg)
 {
        return mon_mca_end(monmsg) - mon_mca_start(monmsg) + 1;
 }
 
-static inline u32
-mon_rec_start(struct mon_msg *monmsg)
+static inline u32 mon_rec_start(struct mon_msg *monmsg)
 {
        return *((u32 *) (mon_mca_start(monmsg) + monmsg->mca_offset + 4));
 }
 
-static inline u32
-mon_rec_end(struct mon_msg *monmsg)
+static inline u32 mon_rec_end(struct mon_msg *monmsg)
 {
        return *((u32 *) (mon_mca_start(monmsg) + monmsg->mca_offset + 8));
 }
 
-static inline int
-mon_check_mca(struct mon_msg *monmsg)
+static inline int mon_check_mca(struct mon_msg *monmsg)
 {
        if ((mon_rec_end(monmsg) <= mon_rec_start(monmsg)) ||
            (mon_rec_start(monmsg) < mon_dcss_start) ||
@@ -221,20 +209,17 @@ mon_check_mca(struct mon_msg *monmsg)
        return 0;
 }
 
-static inline int
-mon_send_reply(struct mon_msg *monmsg, struct mon_private *monpriv)
+static inline int mon_send_reply(struct mon_msg *monmsg,
+                                struct mon_private *monpriv)
 {
-       u8 prmmsg[8];
        int rc;
 
        P_DEBUG("read, REPLY: pathid = 0x%04X, msgid = 0x%08X, trgcls = "
                "0x%08X\n\n",
-               monmsg->local_eib.ippathid, monmsg->local_eib.ipmsgid,
-               monmsg->local_eib.iptrgcls);
-       rc = iucv_reply_prmmsg(monmsg->local_eib.ippathid,
-                               monmsg->local_eib.ipmsgid,
-                               monmsg->local_eib.iptrgcls,
-                               0, prmmsg);
+               monpriv->path->pathid, monmsg->msg.id, monmsg->msg.class);
+
+       rc = iucv_message_reply(monpriv->path, &monmsg->msg,
+                               IUCV_IPRMDATA, NULL, 0);
        atomic_dec(&monpriv->msglim_count);
        if (likely(!monmsg->msglim_reached)) {
                monmsg->pos = 0;
@@ -251,10 +236,19 @@ mon_send_reply(struct mon_msg *monmsg, struct mon_private *monpriv)
        return 0;
 }
 
-static inline struct mon_private *
-mon_alloc_mem(void)
+static inline void mon_free_mem(struct mon_private *monpriv)
+{
+       int i;
+
+       for (i = 0; i < MON_MSGLIM; i++)
+               if (monpriv->msg_array[i])
+                       kfree(monpriv->msg_array[i]);
+       kfree(monpriv);
+}
+
+static inline struct mon_private *mon_alloc_mem(void)
 {
-       int i,j;
+       int i;
        struct mon_private *monpriv;
 
        monpriv = kzalloc(sizeof(struct mon_private), GFP_KERNEL);
@@ -267,16 +261,15 @@ mon_alloc_mem(void)
                                                    GFP_KERNEL);
                if (!monpriv->msg_array[i]) {
                        P_ERROR("open, no memory for msg_array\n");
-                       for (j = 0; j < i; j++)
-                               kfree(monpriv->msg_array[j]);
+                       mon_free_mem(monpriv);
                        return NULL;
                }
        }
        return monpriv;
 }
 
-static inline void
-mon_read_debug(struct mon_msg *monmsg, struct mon_private *monpriv)
+static inline void mon_read_debug(struct mon_msg *monmsg,
+                                 struct mon_private *monpriv)
 {
 #ifdef MON_DEBUG
        u8 msg_type[2], mca_type;
@@ -284,7 +277,7 @@ mon_read_debug(struct mon_msg *monmsg, struct mon_private *monpriv)
 
        records_len = mon_rec_end(monmsg) - mon_rec_start(monmsg) + 1;
 
-       memcpy(msg_type, &monmsg->local_eib.iptrgcls, 2);
+       memcpy(msg_type, &monmsg->msg.class, 2);
        EBCASC(msg_type, 2);
        mca_type = mon_mca_type(monmsg, 0);
        EBCASC(&mca_type, 1);
@@ -292,8 +285,7 @@ mon_read_debug(struct mon_msg *monmsg, struct mon_private *monpriv)
        P_DEBUG("read, mon_read_index = %i, mon_write_index = %i\n",
                monpriv->read_index, monpriv->write_index);
        P_DEBUG("read, pathid = 0x%04X, msgid = 0x%08X, trgcls = 0x%08X\n",
-               monmsg->local_eib.ippathid, monmsg->local_eib.ipmsgid,
-               monmsg->local_eib.iptrgcls);
+               monpriv->path->pathid, monmsg->msg.id, monmsg->msg.class);
        P_DEBUG("read, msg_type = '%c%c', mca_type = '%c' / 0x%X / 0x%X\n",
                msg_type[0], msg_type[1], mca_type ? mca_type : 'X',
                mon_mca_type(monmsg, 1), mon_mca_type(monmsg, 2));
@@ -306,8 +298,7 @@ mon_read_debug(struct mon_msg *monmsg, struct mon_private *monpriv)
 #endif
 }
 
-static inline void
-mon_next_mca(struct mon_msg *monmsg)
+static inline void mon_next_mca(struct mon_msg *monmsg)
 {
        if (likely((mon_mca_size(monmsg) - monmsg->mca_offset) == 12))
                return;
@@ -316,8 +307,7 @@ mon_next_mca(struct mon_msg *monmsg)
        monmsg->pos = 0;
 }
 
-static inline struct mon_msg *
-mon_next_message(struct mon_private *monpriv)
+static inline struct mon_msg *mon_next_message(struct mon_private *monpriv)
 {
        struct mon_msg *monmsg;
 
@@ -342,39 +332,37 @@ mon_next_message(struct mon_private *monpriv)
 /******************************************************************************
  *                               IUCV handler                                 *
  *****************************************************************************/
-static void
-mon_iucv_ConnectionComplete(iucv_ConnectionComplete *eib, void *pgm_data)
+static void mon_iucv_path_complete(struct iucv_path *path, u8 ipuser[16])
 {
-       struct mon_private *monpriv = (struct mon_private *) pgm_data;
+       struct mon_private *monpriv = path->private;
 
        P_DEBUG("IUCV connection completed\n");
        P_DEBUG("IUCV ACCEPT (from *MONITOR): Version = 0x%02X, Event = "
                "0x%02X, Sample = 0x%02X\n",
-               eib->ipuser[0], eib->ipuser[1], eib->ipuser[2]);
+               ipuser[0], ipuser[1], ipuser[2]);
        atomic_set(&monpriv->iucv_connected, 1);
        wake_up(&mon_conn_wait_queue);
 }
 
-static void
-mon_iucv_ConnectionSevered(iucv_ConnectionSevered *eib, void *pgm_data)
+static void mon_iucv_path_severed(struct iucv_path *path, u8 ipuser[16])
 {
-       struct mon_private *monpriv = (struct mon_private *) pgm_data;
+       struct mon_private *monpriv = path->private;
 
-       P_ERROR("IUCV connection severed with rc = 0x%X\n",
-               (u8) eib->ipuser[0]);
+       P_ERROR("IUCV connection severed with rc = 0x%X\n", ipuser[0]);
+       iucv_path_sever(path, NULL);
        atomic_set(&monpriv->iucv_severed, 1);
        wake_up(&mon_conn_wait_queue);
        wake_up_interruptible(&mon_read_wait_queue);
 }
 
-static void
-mon_iucv_MessagePending(iucv_MessagePending *eib, void *pgm_data)
+static void mon_iucv_message_pending(struct iucv_path *path,
+                                    struct iucv_message *msg)
 {
-       struct mon_private *monpriv = (struct mon_private *) pgm_data;
+       struct mon_private *monpriv = path->private;
 
        P_DEBUG("IUCV message pending\n");
-       memcpy(&monpriv->msg_array[monpriv->write_index]->local_eib, eib,
-              sizeof(iucv_MessagePending));
+       memcpy(&monpriv->msg_array[monpriv->write_index]->msg,
+              msg, sizeof(*msg));
        if (atomic_inc_return(&monpriv->msglim_count) == MON_MSGLIM) {
                P_WARNING("IUCV message pending, message limit (%i) reached\n",
                          MON_MSGLIM);
@@ -385,54 +373,45 @@ mon_iucv_MessagePending(iucv_MessagePending *eib, void *pgm_data)
        wake_up_interruptible(&mon_read_wait_queue);
 }
 
-static iucv_interrupt_ops_t mon_iucvops = {
-       .ConnectionComplete = mon_iucv_ConnectionComplete,
-       .ConnectionSevered  = mon_iucv_ConnectionSevered,
-       .MessagePending     = mon_iucv_MessagePending,
+static struct iucv_handler monreader_iucv_handler = {
+       .path_complete   = mon_iucv_path_complete,
+       .path_severed    = mon_iucv_path_severed,
+       .message_pending = mon_iucv_message_pending,
 };
 
 /******************************************************************************
  *                               file operations                              *
  *****************************************************************************/
-static int
-mon_open(struct inode *inode, struct file *filp)
+static int mon_open(struct inode *inode, struct file *filp)
 {
-       int rc, i;
        struct mon_private *monpriv;
+       int rc;
 
        /*
         * only one user allowed
         */
+       rc = -EBUSY;
        if (test_and_set_bit(MON_IN_USE, &mon_in_use))
-               return -EBUSY;
+               goto out;
 
+       rc = -ENOMEM;
        monpriv = mon_alloc_mem();
        if (!monpriv)
-               return -ENOMEM;
+               goto out_use;
 
        /*
-        * Register with IUCV and connect to *MONITOR service
+        * Connect to *MONITOR service
         */
-       monpriv->iucv_handle = iucv_register_program("my_monreader    ",
-                                                       MON_SERVICE,
-                                                       NULL,
-                                                       &mon_iucvops,
-                                                       monpriv);
-       if (!monpriv->iucv_handle) {
-               P_ERROR("failed to register with iucv driver\n");
-               rc = -EIO;
-               goto out_error;
-       }
-       P_INFO("open, registered with IUCV\n");
-
-       rc = iucv_connect(&monpriv->pathid, MON_MSGLIM, user_data_connect,
-                         MON_SERVICE, iucv_host, IPRMDATA, NULL, NULL,
-                         monpriv->iucv_handle, NULL);
+       monpriv->path = iucv_path_alloc(MON_MSGLIM, IUCV_IPRMDATA, GFP_KERNEL);
+       if (!monpriv->path)
+               goto out_priv;
+       rc = iucv_path_connect(monpriv->path, &monreader_iucv_handler,
+                              MON_SERVICE, NULL, user_data_connect, monpriv);
        if (rc) {
                P_ERROR("iucv connection to *MONITOR failed with "
                        "IPUSER SEVER code = %i\n", rc);
                rc = -EIO;
-               goto out_unregister;
+               goto out_path;
        }
        /*
         * Wait for connection confirmation
@@ -444,24 +423,23 @@ mon_open(struct inode *inode, struct file *filp)
                atomic_set(&monpriv->iucv_severed, 0);
                atomic_set(&monpriv->iucv_connected, 0);
                rc = -EIO;
-               goto out_unregister;
+               goto out_path;
        }
        P_INFO("open, established connection to *MONITOR service\n\n");
        filp->private_data = monpriv;
        return nonseekable_open(inode, filp);
 
-out_unregister:
-       iucv_unregister_program(monpriv->iucv_handle);
-out_error:
-       for (i = 0; i < MON_MSGLIM; i++)
-               kfree(monpriv->msg_array[i]);
-       kfree(monpriv);
+out_path:
+       kfree(monpriv->path);
+out_priv:
+       mon_free_mem(monpriv);
+out_use:
        clear_bit(MON_IN_USE, &mon_in_use);
+out:
        return rc;
 }
 
-static int
-mon_close(struct inode *inode, struct file *filp)
+static int mon_close(struct inode *inode, struct file *filp)
 {
        int rc, i;
        struct mon_private *monpriv = filp->private_data;
@@ -469,18 +447,12 @@ mon_close(struct inode *inode, struct file *filp)
        /*
         * Close IUCV connection and unregister
         */
-       rc = iucv_sever(monpriv->pathid, user_data_sever);
+       rc = iucv_path_sever(monpriv->path, user_data_sever);
        if (rc)
                P_ERROR("close, iucv_sever failed with rc = %i\n", rc);
        else
                P_INFO("close, terminated connection to *MONITOR service\n");
 
-       rc = iucv_unregister_program(monpriv->iucv_handle);
-       if (rc)
-               P_ERROR("close, iucv_unregister failed with rc = %i\n", rc);
-       else
-               P_INFO("close, unregistered with IUCV\n");
-
        atomic_set(&monpriv->iucv_severed, 0);
        atomic_set(&monpriv->iucv_connected, 0);
        atomic_set(&monpriv->read_ready, 0);
@@ -495,8 +467,8 @@ mon_close(struct inode *inode, struct file *filp)
        return 0;
 }
 
-static ssize_t
-mon_read(struct file *filp, char __user *data, size_t count, loff_t *ppos)
+static ssize_t mon_read(struct file *filp, char __user *data,
+                       size_t count, loff_t *ppos)
 {
        struct mon_private *monpriv = filp->private_data;
        struct mon_msg *monmsg;
@@ -563,8 +535,7 @@ out_copy:
        return count;
 }
 
-static unsigned int
-mon_poll(struct file *filp, struct poll_table_struct *p)
+static unsigned int mon_poll(struct file *filp, struct poll_table_struct *p)
 {
        struct mon_private *monpriv = filp->private_data;
 
@@ -593,8 +564,7 @@ static struct miscdevice mon_dev = {
 /******************************************************************************
  *                              module init/exit                              *
  *****************************************************************************/
-static int __init
-mon_init(void)
+static int __init mon_init(void)
 {
        int rc;
 
@@ -603,22 +573,34 @@ mon_init(void)
                return -ENODEV;
        }
 
+       /*
+        * Register with IUCV and connect to *MONITOR service
+        */
+       rc = iucv_register(&monreader_iucv_handler, 1);
+       if (rc) {
+               P_ERROR("failed to register with iucv driver\n");
+               return rc;
+       }
+       P_INFO("open, registered with IUCV\n");
+
        rc = segment_type(mon_dcss_name);
        if (rc < 0) {
                mon_segment_warn(rc, mon_dcss_name);
-               return rc;
+               goto out_iucv;
        }
        if (rc != SEG_TYPE_SC) {
                P_ERROR("segment %s has unsupported type, should be SC\n",
                        mon_dcss_name);
-               return -EINVAL;
+               rc = -EINVAL;
+               goto out_iucv;
        }
 
        rc = segment_load(mon_dcss_name, SEGMENT_SHARED,
                          &mon_dcss_start, &mon_dcss_end);
        if (rc < 0) {
                mon_segment_warn(rc, mon_dcss_name);
-               return -EINVAL;
+               rc = -EINVAL;
+               goto out_iucv;
        }
        dcss_mkname(mon_dcss_name, &user_data_connect[8]);
 
@@ -634,14 +616,16 @@ mon_init(void)
 
 out:
        segment_unload(mon_dcss_name);
+out_iucv:
+       iucv_unregister(&monreader_iucv_handler, 1);
        return rc;
 }
 
-static void __exit
-mon_exit(void)
+static void __exit mon_exit(void)
 {
        segment_unload(mon_dcss_name);
        WARN_ON(misc_deregister(&mon_dev) != 0);
+       iucv_unregister(&monreader_iucv_handler, 1);
        return;
 }