Commit 8e905217 authored by Carlos Llamas's avatar Carlos Llamas Committed by Greg Kroah-Hartman
Browse files

binder: collapse print_binder_buffer() into caller 



The code in print_binder_buffer() is quite small so it can be collapsed
into its single caller binder_alloc_print_allocated().

No functional change in this patch.

Signed-off-by: default avatarCarlos Llamas <cmllamas@google.com>
Reviewed-by: default avatarAlice Ryhl <aliceryhl@google.com>
Link: https://lore.kernel.org/r/20231201172212.1813387-25-cmllamas@google.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 67dcc880

drivers/android/binder_alloc.c

+9 −13
Original line number Diff line number Diff line
@@ -978,16 +978,6 @@ void binder_alloc_deferred_release(struct binder_alloc *alloc) 
		     __func__, alloc->pid, buffers, page_count);

}



static void print_binder_buffer(struct seq_file *m, const char *prefix,

				struct binder_buffer *buffer)

{

	seq_printf(m, "%s %d: %lx size %zd:%zd:%zd %s\n",

		   prefix, buffer->debug_id, buffer->user_data,

		   buffer->data_size, buffer->offsets_size,

		   buffer->extra_buffers_size,

		   buffer->transaction ? "active" : "delivered");

}



/**

 * binder_alloc_print_allocated() - print buffer info

 * @m:     seq_file for output via seq_printf()
@@ -999,12 +989,18 @@ static void print_binder_buffer(struct seq_file *m, const char *prefix, 
void binder_alloc_print_allocated(struct seq_file *m,

				  struct binder_alloc *alloc)

{

	struct binder_buffer *buffer;

	struct rb_node *n;



	mutex_lock(&alloc->mutex);

	for (n = rb_first(&alloc->allocated_buffers); n != NULL; n = rb_next(n))

		print_binder_buffer(m, "  buffer",

				    rb_entry(n, struct binder_buffer, rb_node));

	for (n = rb_first(&alloc->allocated_buffers); n; n = rb_next(n)) {

		buffer = rb_entry(n, struct binder_buffer, rb_node);

		seq_printf(m, "  buffer %d: %lx size %zd:%zd:%zd %s\n",

			   buffer->debug_id, buffer->user_data,

			   buffer->data_size, buffer->offsets_size,

			   buffer->extra_buffers_size,

			   buffer->transaction ? "active" : "delivered");

	}

	mutex_unlock(&alloc->mutex);

}