Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 730d678c authored by qctecmdr Service's avatar qctecmdr Service Committed by Gerrit - the friendly Code Review server
Browse files

Merge "diag: Allocate DCI memory using vzalloc instead of kzalloc"

parents 5eadaf73 84d618eb

drivers/char/diag/diag_dci.c

+21 −20
Original line number Diff line number Diff line
@@ -27,6 +27,7 @@ 
#include <asm/current.h>

#include <soc/qcom/restart.h>

#include <linux/sched/signal.h>

#include <linux/vmalloc.h>

#ifdef CONFIG_DIAG_OVER_USB

#include <linux/usb/usbdiag.h>

#endif
@@ -261,7 +262,7 @@ static int diag_dci_init_buffer(struct diag_dci_buffer_t *buffer, int type) 
	switch (type) {

	case DCI_BUF_PRIMARY:

		buffer->capacity = IN_BUF_SIZE;

		buffer->data = kzalloc(buffer->capacity, GFP_KERNEL);

		buffer->data = vzalloc(buffer->capacity);

		if (!buffer->data)

			return -ENOMEM;

		break;
@@ -271,7 +272,7 @@ static int diag_dci_init_buffer(struct diag_dci_buffer_t *buffer, int type) 
		break;

	case DCI_BUF_CMD:

		buffer->capacity = DIAG_MAX_REQ_SIZE + DCI_BUF_SIZE;

		buffer->data = kzalloc(buffer->capacity, GFP_KERNEL);

		buffer->data = vzalloc(buffer->capacity);

		if (!buffer->data)

			return -ENOMEM;

		break;
@@ -2728,7 +2729,7 @@ static int diag_dci_init_remote(void) 
		create_dci_event_mask_tbl(temp->event_mask_composite);

	}



	partial_pkt.data = kzalloc(MAX_DCI_PACKET_SZ, GFP_KERNEL);

	partial_pkt.data = vzalloc(MAX_DCI_PACKET_SZ);

	if (!partial_pkt.data)

		return -ENOMEM;
@@ -2782,7 +2783,7 @@ int diag_dci_init(void) 
		goto err;



	if (driver->apps_dci_buf == NULL) {

		driver->apps_dci_buf = kzalloc(DCI_BUF_SIZE, GFP_KERNEL);

		driver->apps_dci_buf = vzalloc(DCI_BUF_SIZE);

		if (driver->apps_dci_buf == NULL)

			goto err;

	}
@@ -2799,12 +2800,12 @@ int diag_dci_init(void) 
	return DIAG_DCI_NO_ERROR;

err:

	pr_err("diag: Could not initialize diag DCI buffers");

	kfree(driver->apps_dci_buf);

	vfree(driver->apps_dci_buf);

	driver->apps_dci_buf = NULL;



	if (driver->diag_dci_wq)

		destroy_workqueue(driver->diag_dci_wq);

	kfree(partial_pkt.data);

	vfree(partial_pkt.data);

	partial_pkt.data = NULL;

	mutex_destroy(&driver->dci_mutex);

	mutex_destroy(&dci_log_mask_mutex);
@@ -2824,9 +2825,9 @@ void diag_dci_channel_init(void) 


void diag_dci_exit(void)

{

	kfree(partial_pkt.data);

	vfree(partial_pkt.data);

	partial_pkt.data = NULL;

	kfree(driver->apps_dci_buf);

	vfree(driver->apps_dci_buf);

	driver->apps_dci_buf = NULL;

	mutex_destroy(&driver->dci_mutex);

	mutex_destroy(&dci_log_mask_mutex);
@@ -2964,7 +2965,7 @@ int diag_dci_register_client(struct diag_dci_reg_tbl_t *reg_entry) 
	new_entry->in_service = 0;

	INIT_LIST_HEAD(&new_entry->list_write_buf);

	mutex_init(&new_entry->write_buf_mutex);

	new_entry->dci_log_mask =  kzalloc(DCI_LOG_MASK_SIZE, GFP_KERNEL);

	new_entry->dci_log_mask =  vzalloc(DCI_LOG_MASK_SIZE);

	if (!new_entry->dci_log_mask) {

		pr_err("diag: Unable to create log mask for client, %d",

							driver->dci_client_id);
@@ -2972,7 +2973,7 @@ int diag_dci_register_client(struct diag_dci_reg_tbl_t *reg_entry) 
	}

	create_dci_log_mask_tbl(new_entry->dci_log_mask, DCI_LOG_MASK_CLEAN);



	new_entry->dci_event_mask =  kzalloc(DCI_EVENT_MASK_SIZE, GFP_KERNEL);

	new_entry->dci_event_mask =  vzalloc(DCI_EVENT_MASK_SIZE);

	if (!new_entry->dci_event_mask)

		goto fail_alloc;

	create_dci_event_mask_tbl(new_entry->dci_event_mask);
@@ -3036,7 +3037,7 @@ int diag_dci_register_client(struct diag_dci_reg_tbl_t *reg_entry) 
			if (proc_buf) {

				mutex_destroy(&proc_buf->health_mutex);

				if (proc_buf->buf_primary) {

					kfree(proc_buf->buf_primary->data);

					vfree(proc_buf->buf_primary->data);

					proc_buf->buf_primary->data = NULL;

					mutex_destroy(

					   &proc_buf->buf_primary->data_mutex);
@@ -3044,7 +3045,7 @@ int diag_dci_register_client(struct diag_dci_reg_tbl_t *reg_entry) 
				kfree(proc_buf->buf_primary);

				proc_buf->buf_primary = NULL;

				if (proc_buf->buf_cmd) {

					kfree(proc_buf->buf_cmd->data);

					vfree(proc_buf->buf_cmd->data);

					proc_buf->buf_cmd->data = NULL;

					mutex_destroy(

					   &proc_buf->buf_cmd->data_mutex);
@@ -3053,9 +3054,9 @@ int diag_dci_register_client(struct diag_dci_reg_tbl_t *reg_entry) 
				proc_buf->buf_cmd = NULL;

			}

		}

		kfree(new_entry->dci_event_mask);

		vfree(new_entry->dci_event_mask);

		new_entry->dci_event_mask = NULL;

		kfree(new_entry->dci_log_mask);

		vfree(new_entry->dci_log_mask);

		new_entry->dci_log_mask = NULL;

		kfree(new_entry->buffers);

		new_entry->buffers = NULL;
@@ -3090,7 +3091,7 @@ int diag_dci_deinit_client(struct diag_dci_client_tbl *entry) 
	 * Clear the client's log and event masks, update the cumulative

	 * masks and send the masks to peripherals

	 */

	kfree(entry->dci_log_mask);

	vfree(entry->dci_log_mask);

	entry->dci_log_mask = NULL;

	diag_dci_invalidate_cumulative_log_mask(token);

	if (token == DCI_LOCAL_PROC)
@@ -3098,7 +3099,7 @@ int diag_dci_deinit_client(struct diag_dci_client_tbl *entry) 
	ret = dci_ops_tbl[token].send_log_mask(token);

	if (ret != DIAG_DCI_NO_ERROR)

		return ret;

	kfree(entry->dci_event_mask);

	vfree(entry->dci_event_mask);

	entry->dci_event_mask = NULL;

	diag_dci_invalidate_cumulative_event_mask(token);

	if (token == DCI_LOCAL_PROC)
@@ -3161,12 +3162,12 @@ int diag_dci_deinit_client(struct diag_dci_client_tbl *entry) 
		}



		mutex_lock(&proc_buf->buf_primary->data_mutex);

		kfree(proc_buf->buf_primary->data);

		vfree(proc_buf->buf_primary->data);

		proc_buf->buf_primary->data = NULL;

		mutex_unlock(&proc_buf->buf_primary->data_mutex);



		mutex_lock(&proc_buf->buf_cmd->data_mutex);

		kfree(proc_buf->buf_cmd->data);

		vfree(proc_buf->buf_cmd->data);

		proc_buf->buf_cmd->data = NULL;

		mutex_unlock(&proc_buf->buf_cmd->data_mutex);