From nobody Thu Dec 18 15:32:03 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BCDB2C46CA1 for ; Wed, 18 Oct 2023 12:28:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231401AbjJRM2d (ORCPT ); Wed, 18 Oct 2023 08:28:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50444 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229499AbjJRM2c (ORCPT ); Wed, 18 Oct 2023 08:28:32 -0400 Received: from mail-oi1-x233.google.com (mail-oi1-x233.google.com [IPv6:2607:f8b0:4864:20::233]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D58EEA3 for ; Wed, 18 Oct 2023 05:28:29 -0700 (PDT) Received: by mail-oi1-x233.google.com with SMTP id 5614622812f47-3af5b5d7f16so4122168b6e.0 for ; Wed, 18 Oct 2023 05:28:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1697632109; x=1698236909; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=gWJF8w+ZmKp1jb1CoVlOBJDONP0RLpMI50cE905pgWg=; b=PwLtym1ncagTWpc5zAUR0U85UKogVLCEv9n4KE0e3KzROyXqrwa7YdWwycfK9gmI7M 51WF6Pc/gE+T6N2Awb74lI382swoJtP2FC3WeI4hcDOyYiexFDDvQlqixLNHeJjSdbu7 9divqXUnpMlQDWy+jQwfuYIyRNwo9dvUyo3C001qHRLsmlJ0UgzAsh1IgaR3hYIPzffg GRNBZp9dsTBeZhEyQVwlvkx5bvORQqL8EPrai7GAFE+vBc44JR14u2Do78Xdu080k1Dz l6B6RGnMX6h5lMQIN5JAJ6FbsXIUR404HkzMSKV7HZqa5cy0EvDyCOnz6kqzyPjpUU9G gFbg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697632109; x=1698236909; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=gWJF8w+ZmKp1jb1CoVlOBJDONP0RLpMI50cE905pgWg=; b=lJYyJo+jFOYNHhr4YVJTAHw2/tcEC+eVLS7Kvd9MNwaBFbg6YkDQRwSVPuRZDOozAA JAWBLYDwiHNdn0LjcQkoJpW/Npk/WirL/vFQrbLi01ePENm9G3rgjR7mg0KgiKgHrqkb 0/80U7oWB8zoJkgxmKOqKG6jGAN8SDZ0hro/R65yG4n83BRyJc7DzH6EbLoPHMxrLK/q CZShbK7ndhqR5IF0hiH0od22hbw/ojVaw4s5iRWZk5HETrsSHY2s25TWOZaworIUJokl Q3uG+fK4ejT3y2R9zyygVwn/hrnVJICS1QZ1qM8IOoEw8es1nl47ajcWGW+QtYQByMMd AMnA== X-Gm-Message-State: AOJu0YzWQX9MOXCYlQZxYQaNYBWVvsw3DDpildy4M+ZWtWhoRIqnJoEs sAOkW1dGwNIGvtTVw4Egm/Pn X-Google-Smtp-Source: AGHT+IH+ZPOle3U+H6TZ/sMIPLF6eFrT7YLOVduG7uxDTyeVQuqGmjsQUbEA3+9fBXs2Yer4/7sE9g== X-Received: by 2002:a05:6358:e87:b0:150:8ba9:4d with SMTP id 7-20020a0563580e8700b001508ba9004dmr5613112rwg.3.1697632109011; Wed, 18 Oct 2023 05:28:29 -0700 (PDT) Received: from localhost.localdomain ([117.202.186.25]) by smtp.gmail.com with ESMTPSA id k3-20020aa79d03000000b0069102aa1918sm3180708pfp.48.2023.10.18.05.28.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Oct 2023 05:28:28 -0700 (PDT) From: Manivannan Sadhasivam To: mhi@lists.linux.dev Cc: linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Manivannan Sadhasivam , Alex Elder Subject: [PATCH] bus: mhi: ep: Use slab allocator where applicable Date: Wed, 18 Oct 2023 17:58:12 +0530 Message-Id: <20231018122812.47261-1-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Use slab allocator for allocating the memory for objects used frequently and are of fixed size. This reduces the overheard associated with kmalloc(). Suggested-by: Alex Elder Signed-off-by: Manivannan Sadhasivam --- drivers/bus/mhi/ep/main.c | 70 +++++++++++++++++++++++++++++---------- include/linux/mhi_ep.h | 3 ++ 2 files changed, 56 insertions(+), 17 deletions(-) diff --git a/drivers/bus/mhi/ep/main.c b/drivers/bus/mhi/ep/main.c index 66ca470bf302..834e7afadd64 100644 --- a/drivers/bus/mhi/ep/main.c +++ b/drivers/bus/mhi/ep/main.c @@ -71,7 +71,8 @@ static int mhi_ep_send_event(struct mhi_ep_cntrl *mhi_cnt= rl, u32 ring_idx, static int mhi_ep_send_completion_event(struct mhi_ep_cntrl *mhi_cntrl, st= ruct mhi_ep_ring *ring, struct mhi_ring_element *tre, u32 len, enum mhi_ev_ccs code) { - struct mhi_ring_element *event =3D kzalloc(sizeof(struct mhi_ring_element= ), GFP_KERNEL); + struct mhi_ring_element *event =3D kmem_cache_zalloc(mhi_cntrl->ev_ring_e= l_cache, + GFP_KERNEL | GFP_DMA); int ret; =20 event->ptr =3D cpu_to_le64(ring->rbase + ring->rd_offset * sizeof(*tre)); @@ -79,42 +80,45 @@ static int mhi_ep_send_completion_event(struct mhi_ep_c= ntrl *mhi_cntrl, struct m event->dword[1] =3D MHI_TRE_EV_DWORD1(ring->ch_id, MHI_PKT_TYPE_TX_EVENT); =20 ret =3D mhi_ep_send_event(mhi_cntrl, ring->er_index, event, MHI_TRE_DATA_= GET_BEI(tre)); - kfree(event); + kmem_cache_free(mhi_cntrl->ev_ring_el_cache, event); =20 return ret; } =20 int mhi_ep_send_state_change_event(struct mhi_ep_cntrl *mhi_cntrl, enum mh= i_state state) { - struct mhi_ring_element *event =3D kzalloc(sizeof(struct mhi_ring_element= ), GFP_KERNEL); + struct mhi_ring_element *event =3D kmem_cache_zalloc(mhi_cntrl->ev_ring_e= l_cache, + GFP_KERNEL | GFP_DMA); int ret; =20 event->dword[0] =3D MHI_SC_EV_DWORD0(state); event->dword[1] =3D MHI_SC_EV_DWORD1(MHI_PKT_TYPE_STATE_CHANGE_EVENT); =20 ret =3D mhi_ep_send_event(mhi_cntrl, 0, event, 0); - kfree(event); + kmem_cache_free(mhi_cntrl->ev_ring_el_cache, event); =20 return ret; } =20 int mhi_ep_send_ee_event(struct mhi_ep_cntrl *mhi_cntrl, enum mhi_ee_type = exec_env) { - struct mhi_ring_element *event =3D kzalloc(sizeof(struct mhi_ring_element= ), GFP_KERNEL); + struct mhi_ring_element *event =3D kmem_cache_zalloc(mhi_cntrl->ev_ring_e= l_cache, + GFP_KERNEL | GFP_DMA); int ret; =20 event->dword[0] =3D MHI_EE_EV_DWORD0(exec_env); event->dword[1] =3D MHI_SC_EV_DWORD1(MHI_PKT_TYPE_EE_EVENT); =20 ret =3D mhi_ep_send_event(mhi_cntrl, 0, event, 0); - kfree(event); + kmem_cache_free(mhi_cntrl->ev_ring_el_cache, event); =20 return ret; } =20 static int mhi_ep_send_cmd_comp_event(struct mhi_ep_cntrl *mhi_cntrl, enum= mhi_ev_ccs code) { - struct mhi_ring_element *event =3D kzalloc(sizeof(struct mhi_ring_element= ), GFP_KERNEL); + struct mhi_ring_element *event =3D kmem_cache_zalloc(mhi_cntrl->ev_ring_e= l_cache, + GFP_KERNEL | GFP_DMA); struct mhi_ep_ring *ring =3D &mhi_cntrl->mhi_cmd->ring; int ret; =20 @@ -123,7 +127,7 @@ static int mhi_ep_send_cmd_comp_event(struct mhi_ep_cnt= rl *mhi_cntrl, enum mhi_e event->dword[1] =3D MHI_CC_EV_DWORD1(MHI_PKT_TYPE_CMD_COMPLETION_EVENT); =20 ret =3D mhi_ep_send_event(mhi_cntrl, 0, event, 0); - kfree(event); + kmem_cache_free(mhi_cntrl->ev_ring_el_cache, event); =20 return ret; } @@ -435,7 +439,7 @@ static int mhi_ep_process_ch_ring(struct mhi_ep_ring *r= ing, struct mhi_ring_elem mhi_chan->xfer_cb(mhi_chan->mhi_dev, &result); } else { /* UL channel */ - result.buf_addr =3D kzalloc(len, GFP_KERNEL); + result.buf_addr =3D kmem_cache_zalloc(mhi_cntrl->tre_buf_cache, GFP_KERN= EL | GFP_DMA); if (!result.buf_addr) return -ENOMEM; =20 @@ -443,7 +447,7 @@ static int mhi_ep_process_ch_ring(struct mhi_ep_ring *r= ing, struct mhi_ring_elem ret =3D mhi_ep_read_channel(mhi_cntrl, ring, &result, len); if (ret < 0) { dev_err(&mhi_chan->mhi_dev->dev, "Failed to read channel\n"); - kfree(result.buf_addr); + kmem_cache_free(mhi_cntrl->tre_buf_cache, result.buf_addr); return ret; } =20 @@ -455,7 +459,7 @@ static int mhi_ep_process_ch_ring(struct mhi_ep_ring *r= ing, struct mhi_ring_elem /* Read until the ring becomes empty */ } while (!mhi_ep_queue_is_empty(mhi_chan->mhi_dev, DMA_TO_DEVICE)); =20 - kfree(result.buf_addr); + kmem_cache_free(mhi_cntrl->tre_buf_cache, result.buf_addr); } =20 return 0; @@ -764,14 +768,14 @@ static void mhi_ep_ch_ring_worker(struct work_struct = *work) if (ret) { dev_err(dev, "Error updating write offset for ring\n"); mutex_unlock(&chan->lock); - kfree(itr); + kmem_cache_free(mhi_cntrl->ring_item_cache, itr); continue; } =20 /* Sanity check to make sure there are elements in the ring */ if (ring->rd_offset =3D=3D ring->wr_offset) { mutex_unlock(&chan->lock); - kfree(itr); + kmem_cache_free(mhi_cntrl->ring_item_cache, itr); continue; } =20 @@ -783,12 +787,12 @@ static void mhi_ep_ch_ring_worker(struct work_struct = *work) dev_err(dev, "Error processing ring for channel (%u): %d\n", ring->ch_id, ret); mutex_unlock(&chan->lock); - kfree(itr); + kmem_cache_free(mhi_cntrl->ring_item_cache, itr); continue; } =20 mutex_unlock(&chan->lock); - kfree(itr); + kmem_cache_free(mhi_cntrl->ring_item_cache, itr); } } =20 @@ -844,7 +848,7 @@ static void mhi_ep_queue_channel_db(struct mhi_ep_cntrl= *mhi_cntrl, unsigned lon u32 ch_id =3D ch_idx + i; =20 ring =3D &mhi_cntrl->mhi_chan[ch_id].ring; - item =3D kzalloc(sizeof(*item), GFP_ATOMIC); + item =3D kmem_cache_zalloc(mhi_cntrl->ring_item_cache, GFP_ATOMIC); if (!item) return; =20 @@ -1391,6 +1395,29 @@ int mhi_ep_register_controller(struct mhi_ep_cntrl *= mhi_cntrl, goto err_free_ch; } =20 + mhi_cntrl->ev_ring_el_cache =3D kmem_cache_create("mhi_ep_event_ring_el", + sizeof(struct mhi_ring_element), 0, + SLAB_CACHE_DMA, NULL); + if (!mhi_cntrl->ev_ring_el_cache) { + ret =3D -ENOMEM; + goto err_free_cmd; + } + + mhi_cntrl->tre_buf_cache =3D kmem_cache_create("mhi_ep_tre_buf", MHI_EP_D= EFAULT_MTU, 0, + SLAB_CACHE_DMA, NULL); + if (!mhi_cntrl->tre_buf_cache) { + ret =3D -ENOMEM; + goto err_destroy_ev_ring_el_cache; + } + + mhi_cntrl->ring_item_cache =3D kmem_cache_create("mhi_ep_ring_item", + sizeof(struct mhi_ep_ring_item), 0, + 0, NULL); + if (!mhi_cntrl->ev_ring_el_cache) { + ret =3D -ENOMEM; + goto err_destroy_tre_buf_cache; + } + INIT_WORK(&mhi_cntrl->state_work, mhi_ep_state_worker); INIT_WORK(&mhi_cntrl->reset_work, mhi_ep_reset_worker); INIT_WORK(&mhi_cntrl->cmd_ring_work, mhi_ep_cmd_ring_worker); @@ -1399,7 +1426,7 @@ int mhi_ep_register_controller(struct mhi_ep_cntrl *m= hi_cntrl, mhi_cntrl->wq =3D alloc_workqueue("mhi_ep_wq", 0, 0); if (!mhi_cntrl->wq) { ret =3D -ENOMEM; - goto err_free_cmd; + goto err_destroy_ring_item_cache; } =20 INIT_LIST_HEAD(&mhi_cntrl->st_transition_list); @@ -1458,6 +1485,12 @@ int mhi_ep_register_controller(struct mhi_ep_cntrl *= mhi_cntrl, ida_free(&mhi_ep_cntrl_ida, mhi_cntrl->index); err_destroy_wq: destroy_workqueue(mhi_cntrl->wq); +err_destroy_ring_item_cache: + kmem_cache_destroy(mhi_cntrl->ring_item_cache); +err_destroy_ev_ring_el_cache: + kmem_cache_destroy(mhi_cntrl->ev_ring_el_cache); +err_destroy_tre_buf_cache: + kmem_cache_destroy(mhi_cntrl->tre_buf_cache); err_free_cmd: kfree(mhi_cntrl->mhi_cmd); err_free_ch: @@ -1479,6 +1512,9 @@ void mhi_ep_unregister_controller(struct mhi_ep_cntrl= *mhi_cntrl) =20 free_irq(mhi_cntrl->irq, mhi_cntrl); =20 + kmem_cache_destroy(mhi_cntrl->tre_buf_cache); + kmem_cache_destroy(mhi_cntrl->ev_ring_el_cache); + kmem_cache_destroy(mhi_cntrl->ring_item_cache); kfree(mhi_cntrl->mhi_cmd); kfree(mhi_cntrl->mhi_chan); =20 diff --git a/include/linux/mhi_ep.h b/include/linux/mhi_ep.h index f198a8ac7ee7..ce85d42b685d 100644 --- a/include/linux/mhi_ep.h +++ b/include/linux/mhi_ep.h @@ -128,6 +128,9 @@ struct mhi_ep_cntrl { struct work_struct reset_work; struct work_struct cmd_ring_work; struct work_struct ch_ring_work; + struct kmem_cache *ring_item_cache; + struct kmem_cache *ev_ring_el_cache; + struct kmem_cache *tre_buf_cache; =20 void (*raise_irq)(struct mhi_ep_cntrl *mhi_cntrl, u32 vector); int (*alloc_map)(struct mhi_ep_cntrl *mhi_cntrl, u64 pci_addr, phys_addr_= t *phys_ptr, --=20 2.25.1