simplify statistics' debugfs write function

Index: linux/lib/statistic.c
===================================================================
@@ -64,8 +64,8 @@
 
 struct statistic_file_private {
 	struct list_head read_seg_lh;
-	struct list_head write_seg_lh;
-	size_t write_seg_total_size;
+	size_t w_offset;
+	char *w_buf;
 };
 
 struct statistic_merge_private {

@@ -576,30 +576,6 @@ static void statistic_parse_line(struct 
 	kfree(name);
 }
 
-static void statistic_parse(struct statistic_interface *interface,
-			    struct list_head *line_lh, size_t line_size)
-{
-	struct sgrb_seg *seg, *tmp;
-	char *buf;
-	int offset = 0;
-
-	if (unlikely(!line_size))
-		return;
-	buf = kmalloc(line_size + 2, GFP_KERNEL);
-	if (unlikely(!buf))
-		return;
-	buf[line_size] = ' ';
-	buf[line_size + 1] = '\0';
-	list_for_each_entry_safe(seg, tmp, line_lh, list) {
-		memcpy(buf + offset, seg->address, seg->size);
-		offset += seg->size;
-		list_del(&seg->list);
-		kfree(seg);
-	}
-	statistic_parse_line(interface, buf);
-	kfree(buf);
-}
-
 /* sequential files comprising user interface */
 
 static int statistic_generic_open(struct inode *inode,

@@ -612,7 +588,6 @@ static int statistic_generic_open(struct
 	if (unlikely(!*private))
 		return -ENOMEM;
 	INIT_LIST_HEAD(&(*private)->read_seg_lh);
-	INIT_LIST_HEAD(&(*private)->write_seg_lh);
 	file->private_data = *private;
 	return 0;
 }

@@ -622,7 +597,6 @@ static int statistic_generic_close(struc
 	struct statistic_file_private *private = file->private_data;
 	BUG_ON(!private);
 	sgrb_seg_release_all(&private->read_seg_lh);
-	sgrb_seg_release_all(&private->write_seg_lh);
 	kfree(private);
 	return 0;
 }

@@ -658,72 +632,39 @@ static ssize_t statistic_generic_read(st
 	return transfered;
 }
 
-static ssize_t statistic_generic_write(struct file *file,
-			const char __user *buf, size_t len, loff_t *offset)
+static ssize_t statistic_def_write(struct file *file, const char __user *buf,
+				   size_t len, loff_t *offset)
 {
 	struct statistic_file_private *private = file->private_data;
-	struct sgrb_seg *seg;
-	size_t seg_residual, seg_transfer;
-	size_t transfered = 0;
+	char *larger;
 
-	BUG_ON(!private);
-	if (unlikely(*offset != private->write_seg_total_size))
+	if (unlikely(*offset != private->w_offset))
 		return -EPIPE;
-	while (len) {
-		seg = sgrb_seg_find(&private->write_seg_lh, 1);
-		if (unlikely(!seg))
-			return -ENOMEM;
-		seg_residual = seg->size - seg->offset;
-		seg_transfer = min(len, seg_residual);
-		if (unlikely(copy_from_user(seg->address + seg->offset,
-					    buf + transfered, seg_transfer)))
-			return -EFAULT;
-		private->write_seg_total_size += seg_transfer;
-		seg->offset += seg_transfer;
-		transfered += seg_transfer;
-		*offset += seg_transfer;
-		len -= seg_transfer;
-	}
-	return transfered;
+	if (*offset + len > 16 * PAGE_SIZE)
+		return -ENOMEM;
+	larger = kmalloc(*offset + len, GFP_KERNEL);
+	if (!larger)
+		return -ENOMEM;
+	memcpy(larger, private->w_buf, *offset);
+	if (copy_from_user(larger + *offset, buf, len))
+		return -EFAULT;
+	*offset += len;
+	kfree(private->w_buf);
+	private->w_buf = larger;
+	private->w_offset = *offset;
+	return len;
 }
 
 static int statistic_def_close(struct inode *inode, struct file *file)
 {
 	struct statistic_interface *interface = inode->i_private;
 	struct statistic_file_private *private = file->private_data;
-	struct sgrb_seg *seg, *seg_nl;
-	int offset;
-	LIST_HEAD(line_lh);
-	char *nl;
-	size_t line_size = 0;
-
-	list_for_each_entry(seg, &private->write_seg_lh, list) {
-		for (offset = 0; offset < seg->offset; offset += seg_nl->size) {
-			seg_nl = kmalloc(sizeof(struct sgrb_seg), GFP_KERNEL);
-			if (unlikely(!seg_nl))
-				goto out;
-			seg_nl->address = seg->address + offset;
-			nl = strnchr(seg_nl->address,
-				     seg->offset - offset, '\n');
-			if (nl) {
-				seg_nl->offset = nl - seg_nl->address;
-				if (seg_nl->offset)
-					seg_nl->offset--;
-			} else
-				seg_nl->offset = seg->offset - offset;
-			seg_nl->size = seg_nl->offset + 1;
-			line_size += seg_nl->size;
-			list_add_tail(&seg_nl->list, &line_lh);
-			if (nl) {
-				statistic_parse(interface, &line_lh, line_size);
-				line_size = 0;
-			}
-		}
-	}
-out:
-	if (!list_empty(&line_lh))
-		statistic_parse(interface, &line_lh, line_size);
-	return statistic_generic_close(inode, file);
+	char *p, *q = private->w_buf;
+
+	while ((p = strsep(&q, "\n")))
+		statistic_parse_line(interface, p);
+	kfree(private->w_buf);
+	return 0;
 }
 
 static int statistic_def_open(struct inode *inode, struct file *file)

@@ -771,7 +712,7 @@ static int statistic_data_open(struct in
 static struct file_operations statistic_def_fops = {
 	.owner		= THIS_MODULE,
 	.read		= statistic_generic_read,
-	.write		= statistic_generic_write,
+	.write		= statistic_def_write,
 	.open		= statistic_def_open,
 	.release	= statistic_def_close,
 };