linux-kernel - Re: [PATCH v1] bus: mhi: host: pci_generic: Add support for EM929x and set MRU to 32768 for better performance.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <3e97885af5323dec728f40dc0176f7f82cdabf14.camel@bootlin.com>
Date: Fri, 20 Jun 2025 09:20:16 +0200
From: Thomas Perrot <thomas.perrot@...tlin.com>
To: Aleksander Morgado <aleksander.morgado@...il.com>, Manivannan Sadhasivam
	 <mani@...nel.org>
Cc: thomas.perrot@...tlin.com, Adam Xue <zxue@...tech.com>, Aleksander
 Morgado	 <aleksander@...ksander.es>, manivannan.sadhasivam@...aro.org, 
	slark_xiao@....com, johan+linaro@...nel.org, quic_vpernami@...cinc.com, 
	tglx@...utronix.de, fabio.porcedda@...il.com, quic_msarkar@...cinc.com, 
	mhi@...ts.linux.dev, linux-arm-msm@...r.kernel.org,
 linux-kernel@...r.kernel.org, 	imocanu@...tech.com
Subject: Re: [PATCH v1] bus: mhi: host: pci_generic: Add support for EM929x
 and set MRU to 32768 for better performance.

Hello,

On Tue, 2025-06-17 at 16:58 +0200, Aleksander Morgado wrote:
> On Tue, Jun 17, 2025 at 11:27 AM Manivannan Sadhasivam
> <mani@...nel.org> wrote:
> > 
> > On Wed, May 28, 2025 at 10:59:43AM -0700, Adam Xue wrote:
> > 
> > + Thomas and Aleksander (for EM919X related question)
> > 
> > > Add MHI controller config for EM929x. It uses the same
> > > configuration
> > > as EM919x. Also set the MRU to 32768 to improve downlink
> > > throughput.
> > > 
> > 
> > This also affects the EM919X modem. So I want either Thomas or
> > Aleksander to
> > confirm that it doesn't cause any regression.

Sorry, I no longer have the modem to test.

Kind regards,
Thomas Perrot

> > 
> > Rest looks good to me.
> > 
> > - Mani
> > 
> > > 02:00.0 Unassigned class [ff00]: Qualcomm Technologies, Inc
> > > Device 0308
> > >       Subsystem: Device 18d7:0301
> > > 
> > > Signed-off-by: Adam Xue <zxue@...tech.com>
> > > ---
> > >  drivers/bus/mhi/host/pci_generic.c | 4 ++++
> > >  1 file changed, 4 insertions(+)
> > > 
> > > diff --git a/drivers/bus/mhi/host/pci_generic.c
> > > b/drivers/bus/mhi/host/pci_generic.c
> > > index 03aa88795209..9bf8e7991745 100644
> > > --- a/drivers/bus/mhi/host/pci_generic.c
> > > +++ b/drivers/bus/mhi/host/pci_generic.c
> > > @@ -695,6 +695,7 @@ static const struct mhi_pci_dev_info
> > > mhi_sierra_em919x_info = {
> > >       .config = &modem_sierra_em919x_config,
> > >       .bar_num = MHI_PCI_DEFAULT_BAR_NUM,
> > >       .dma_data_width = 32,
> > > +     .mru_default = 32768,
> 
> I cannot test this now, I need to look for the EM9191 that I know I
> have somewhere, but this default MRU update feels right. Hopefully
> Thomas can verify.
> 

-- 
Thomas Perrot, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com


Download attachment "signature.asc" of type "application/pgp-signature" (660 bytes)