mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	lib/dynamic_debug.c: use seq_open_private() instead of seq_open()
Using seq_open_private() removes boilerplate code from ddebug_proc_open(). The resultant code is shorter and easier to follow. This patch does not change any functionality. Signed-off-by: Rob Jones <rob.jones@codethink.co.uk> Acked-by: Jason Baron <jbaron@akamai.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
		
							parent
							
								
									8b21d9ca17
								
							
						
					
					
						commit
						4bad78c550
					
				
					 1 changed files with 2 additions and 15 deletions
				
			
		| 
						 | 
				
			
			@ -819,22 +819,9 @@ static const struct seq_operations ddebug_proc_seqops = {
 | 
			
		|||
 */
 | 
			
		||||
static int ddebug_proc_open(struct inode *inode, struct file *file)
 | 
			
		||||
{
 | 
			
		||||
	struct ddebug_iter *iter;
 | 
			
		||||
	int err;
 | 
			
		||||
 | 
			
		||||
	vpr_info("called\n");
 | 
			
		||||
 | 
			
		||||
	iter = kzalloc(sizeof(*iter), GFP_KERNEL);
 | 
			
		||||
	if (iter == NULL)
 | 
			
		||||
		return -ENOMEM;
 | 
			
		||||
 | 
			
		||||
	err = seq_open(file, &ddebug_proc_seqops);
 | 
			
		||||
	if (err) {
 | 
			
		||||
		kfree(iter);
 | 
			
		||||
		return err;
 | 
			
		||||
	}
 | 
			
		||||
	((struct seq_file *)file->private_data)->private = iter;
 | 
			
		||||
	return 0;
 | 
			
		||||
	return seq_open_private(file, &ddebug_proc_seqops,
 | 
			
		||||
				sizeof(struct ddebug_iter));
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
static const struct file_operations ddebug_proc_fops = {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue