Whamcloud - gitweb
- move the peter branch changes to the head
[fs/lustre-release.git] / lustre / llite / file.c
index f43bfa0..41ff0b8 100644 (file)
@@ -36,21 +36,45 @@ static int ll_file_open(struct inode *inode, struct file *file)
         int rc;
         struct ptlrpc_request *req = NULL;
         struct ll_file_data *fd;
-        struct obdo *oa;
+        struct obdo *oa = NULL;
+        struct lov_stripe_md *md = NULL; 
         struct ll_sb_info *sbi = ll_i2sbi(inode);
+        struct ll_inode_info *lli = ll_i2info(inode);
         ENTRY;
 
         if (file->private_data)
                 LBUG();
 
+        /*  delayed create of object (intent created inode) */
+        /*  XXX object needs to be cleaned up if mdc_open fails */
+        /*  XXX error handling appropriate here? */
+        if (lli->lli_smd == NULL || lli->lli_smd->lmd_object_id == 0) {
+                struct client_obd *mdc = sbi2mdc(ll_s2sbi(inode->i_sb));
+                struct inode * inode = file->f_dentry->d_inode;
+
+                lli->lli_smd = NULL; 
+                oa =  obdo_alloc();
+                if (!oa) { 
+                        RETURN(-ENOMEM);
+                }
+                oa->o_valid = OBD_MD_FLMODE;
+                oa->o_mode = S_IFREG | 0600;
+                oa->o_easize = mdc->cl_max_mdsize;
+                rc = obd_create(ll_i2obdconn(inode), oa, &lli->lli_smd);
+                if (rc)
+                        RETURN(rc);
+                md = lli->lli_smd;
+                lli->lli_flags &= ~OBD_FL_CREATEONOPEN;
+        }
+
         fd = kmem_cache_alloc(ll_file_data_slab, SLAB_KERNEL);
         if (!fd)
                 GOTO(out, rc = -ENOMEM);
         memset(fd, 0, sizeof(*fd));
 
-        rc = mdc_open(&sbi->ll_mds_client, sbi->ll_mds_conn, inode->i_ino,
-                      S_IFREG, file->f_flags, (__u64)(unsigned long)file, 
-                      &fd->fd_mdshandle, &req); 
+        rc = mdc_open(&sbi->ll_mdc_conn, inode->i_ino, S_IFREG | inode->i_mode,
+                      file->f_flags, md,
+                      (__u64)(unsigned long)file, &fd->fd_mdshandle, &req);
         fd->fd_req = req;
         ptlrpc_req_finished(req);
         if (rc)
@@ -62,15 +86,19 @@ static int ll_file_open(struct inode *inode, struct file *file)
         if (!fd->fd_mdshandle)
                 CERROR("mdc_open didn't assign fd_mdshandle\n");
 
-        oa = ll_i2info(inode)->lli_obdo;
         if (oa == NULL) {
-                LBUG();
-                GOTO(out_mdc, rc = -ENOMEM);
+                oa =  obdo_alloc();
+                if (!oa)
+                        GOTO(out_mdc, rc = -EINVAL);
         }
-        rc = obd_open(ll_i2obdconn(inode), oa);
-        if (rc) {
+        oa->o_id = lli->lli_smd->lmd_object_id;
+        oa->o_mode = S_IFREG | inode->i_mode;
+        rc = obd_open(ll_i2obdconn(inode), oa, lli->lli_smd);
+        obdo_free(oa);
+        oa = NULL;
+
+        if (rc)
                 GOTO(out_mdc, rc = -abs(rc));
-        }
 
         file->private_data = fd;
 
@@ -78,11 +106,13 @@ static int ll_file_open(struct inode *inode, struct file *file)
 
         return 0;
 out_mdc:
-        mdc_close(&sbi->ll_mds_client, sbi->ll_mds_conn, inode->i_ino,
+        mdc_close(&sbi->ll_mdc_conn, inode->i_ino,
                   S_IFREG, fd->fd_mdshandle, &req);
 out_req:
         ptlrpc_free_req(req);
 //out_fd:
+        if (oa)
+                obdo_free(oa);
         kmem_cache_free(ll_file_data_slab, fd);
         file->private_data = NULL;
 out:
@@ -96,6 +126,7 @@ static int ll_file_release(struct inode *inode, struct file *file)
         struct ll_file_data *fd;
         struct obdo *oa;
         struct ll_sb_info *sbi = ll_i2sbi(inode);
+        struct ll_inode_info *lli = ll_i2info(inode);
 
         ENTRY;
 
@@ -105,12 +136,13 @@ static int ll_file_release(struct inode *inode, struct file *file)
                 GOTO(out, rc = -EINVAL);
         }
 
-        oa = ll_i2info(inode)->lli_obdo;
+        oa = obdo_alloc();
         if (oa == NULL) {
                 LBUG();
                 GOTO(out_fd, rc = -ENOENT);
         }
-        rc = obd_close(ll_i2obdconn(inode), oa);
+        rc = obd_close(ll_i2obdconn(inode), oa, lli->lli_smd);
+        obdo_free(oa); 
         if (rc)
                 GOTO(out_fd, abs(rc));
 
@@ -130,11 +162,11 @@ static int ll_file_release(struct inode *inode, struct file *file)
                 }
         }
 
-        rc = mdc_close(&sbi->ll_mds_client, sbi->ll_mds_conn, inode->i_ino,
+        rc = mdc_close(&sbi->ll_mdc_conn, inode->i_ino,
                        S_IFREG, fd->fd_mdshandle, &req);
         ptlrpc_req_finished(req);
-        if (rc) { 
-                if (rc > 0) 
+        if (rc) {
+                if (rc > 0)
                         rc = -rc;
                 GOTO(out, rc);
         }
@@ -180,6 +212,35 @@ static void ll_update_atime(struct inode *inode)
         ll_inode_setattr(inode, &attr, 0);
 }
 
+static int ll_lock_callback(struct lustre_handle *lockh,
+                            struct ldlm_lock_desc *new,
+                            void *data, __u32 data_len,
+                            struct ptlrpc_request **reqp)
+{
+        struct inode *inode = data;
+        ENTRY;
+
+        if (new == NULL) {
+                /* Completion AST.  Do nothing. */
+                RETURN(0);
+        }
+
+        if (data_len != sizeof(struct inode))
+                LBUG();
+
+        /* FIXME: do something better than throwing away everything */
+        if (inode == NULL)
+                LBUG();
+        down(&inode->i_sem);
+        CDEBUG(D_INODE, "invalidating obdo/inode %ld\n", inode->i_ino);
+        invalidate_inode_pages(inode);
+        up(&inode->i_sem);
+
+        if (ldlm_cli_cancel(lockh) < 0)
+                LBUG();
+        RETURN(0);
+}
+
 static ssize_t ll_file_read(struct file *filp, char *buf, size_t count,
                             loff_t *ppos)
 {
@@ -187,7 +248,7 @@ static ssize_t ll_file_read(struct file *filp, char *buf, size_t count,
         struct inode *inode = filp->f_dentry->d_inode;
         struct ll_sb_info *sbi = ll_i2sbi(inode);
         struct ldlm_extent extent;
-        struct ldlm_handle lockh;
+        struct lustre_handle lockh;
         __u64 res_id[RES_NAME_SIZE] = {inode->i_ino};
         int flags = 0;
         ldlm_error_t err;
@@ -200,24 +261,30 @@ static ssize_t ll_file_read(struct file *filp, char *buf, size_t count,
                 CDEBUG(D_INFO, "Locking inode %ld, start %Lu end %Lu\n",
                        inode->i_ino, extent.start, extent.end);
 
-                err = obd_enqueue(&sbi->ll_conn, sbi->ll_namespace, NULL,
-                                  res_id, LDLM_EXTENT, &extent, LCK_PR, &flags,
-                                  inode, sizeof(*inode), &lockh);
-                if (err != ELDLM_OK)
+                err = obd_enqueue(&sbi->ll_osc_conn, NULL, res_id, LDLM_EXTENT,
+                                  &extent, sizeof(extent), LCK_PR, &flags,
+                                  ll_lock_callback, inode, sizeof(*inode),
+                                  &lockh);
+                if (err != ELDLM_OK) {
                         CERROR("lock enqueue: err: %d\n", err);
+                        RETURN(err);
+                }
                 ldlm_lock_dump((void *)(unsigned long)lockh.addr);
         }
 
         CDEBUG(D_INFO, "Reading inode %ld, %d bytes, offset %Ld\n",
                inode->i_ino, count, *ppos);
         retval = generic_file_read(filp, buf, count, ppos);
+
         if (retval > 0)
                 ll_update_atime(inode);
 
         if (!(fd->fd_flags & LL_FILE_IGNORE_LOCK)) {
-                err = obd_cancel(&sbi->ll_conn, LCK_PR, &lockh);
-                if (err != ELDLM_OK)
+                err = obd_cancel(&sbi->ll_osc_conn, LCK_PR, &lockh);
+                if (err != ELDLM_OK) {
                         CERROR("lock cancel: err: %d\n", err);
+                        RETURN(err);
+                }
         }
 
         RETURN(retval);
@@ -233,7 +300,7 @@ ll_file_write(struct file *file, const char *buf, size_t count, loff_t *ppos)
         struct inode *inode = file->f_dentry->d_inode;
         struct ll_sb_info *sbi = ll_i2sbi(inode);
         struct ldlm_extent extent;
-        struct ldlm_handle lockh;
+        struct lustre_handle lockh;
         __u64 res_id[RES_NAME_SIZE] = {inode->i_ino};
         int flags = 0;
         ldlm_error_t err;
@@ -241,16 +308,21 @@ ll_file_write(struct file *file, const char *buf, size_t count, loff_t *ppos)
         ENTRY;
 
         if (!(fd->fd_flags & LL_FILE_IGNORE_LOCK)) {
+                /* FIXME: this should check whether O_APPEND is set and adjust
+                 * extent.start accordingly */
                 extent.start = *ppos;
                 extent.end = *ppos + count;
                 CDEBUG(D_INFO, "Locking inode %ld, start %Lu end %Lu\n",
                        inode->i_ino, extent.start, extent.end);
 
-                err = obd_enqueue(&sbi->ll_conn, sbi->ll_namespace, NULL,
-                                  res_id, LDLM_EXTENT, &extent, LCK_PW, &flags,
-                                  inode, sizeof(*inode), &lockh);
-                if (err != ELDLM_OK)
+                err = obd_enqueue(&sbi->ll_osc_conn, NULL, res_id, LDLM_EXTENT,
+                                  &extent, sizeof(extent), LCK_PW, &flags,
+                                  ll_lock_callback, inode, sizeof(*inode),
+                                  &lockh);
+                if (err != ELDLM_OK) {
                         CERROR("lock enqueue: err: %d\n", err);
+                        RETURN(err);
+                }
                 ldlm_lock_dump((void *)(unsigned long)lockh.addr);
         }
 
@@ -260,9 +332,11 @@ ll_file_write(struct file *file, const char *buf, size_t count, loff_t *ppos)
         retval = generic_file_write(file, buf, count, ppos);
 
         if (!(fd->fd_flags & LL_FILE_IGNORE_LOCK)) {
-                err = obd_cancel(&sbi->ll_conn, LCK_PW, &lockh);
-                if (err != ELDLM_OK)
+                err = obd_cancel(&sbi->ll_osc_conn, LCK_PW, &lockh);
+                if (err != ELDLM_OK) {
                         CERROR("lock cancel: err: %d\n", err);
+                        RETURN(err);
+                }
         }
 
         RETURN(retval);