Linux USB
 help / color / mirror / Atom feed
From: Oliver Neukum <oneukum@suse.com>
To: "Hiago De Franco" <hiagofranco@gmail.com>,
	oneukum@suse.com, "Maciej Żenczykowski" <maze@google.com>
Cc: davem@davemloft.net, edumazet@google.com,
	hiago.franco@toradex.com, kuba@kernel.org,
	linux-usb@vger.kernel.org, pabeni@redhat.com
Subject: Re: Question regarding CDC NCM and VNC performance issue
Date: Thu, 7 Dec 2023 10:41:51 +0100	[thread overview]
Message-ID: <c2ee0ecf-993e-4736-b005-588fa6ef51a3@suse.com> (raw)
In-Reply-To: <20231206162353.53767-1-hiagofranco@gmail.com>



On 06.12.23 17:23, Hiago De Franco wrote:
> From: Hiago De Franco <hiago.franco@toradex.com>
> 

Good morning,

> Hi Oliver,
> 
> On 06.12.23 13:21, Oliver Neukum wrote:
>> Before we proceed, however, your patch raises a fundamental question:
>> Are we talking about gadget or host mode here?
>> Could you please describe your setup in detail?
> 
> Sure, I'm running a Apalis iMX6QD 512MB, which is basically an ARM 32-bits
> device with NXP iMX6QD. This device is running Linux with wayland, weston
> and the neatvnc server, so I can access my app using a vnc client on my
> host PC (also running Linux, Debian 12).

OK, you are using Linux on _both_ sides. Interesting, far from obvious, though.
(Putting Maciej into CC)  
> Finally, I open a VNC client on my host PC and connect to 192.168.11.1,
> where the issue begins with it being very slow/frozen. After applying the
> patch on the device (Apalis iMX6), now VNC runs smoothly.

For debugging purposes your first patch modifies the result of
USB_CDC_GET_NTB_PARAMETERS. Thereby both the host and the gadget change
their behavior. Hence we cannot tell which side has a problem with its
buffering at the old buffer size.

Can you tell us which side in your test case produces many small packets?

Furthermore, for testing purposes, could you decrease TX_TIMEOUT_NSECS in
f_ncm.c by an order of magnitude.

	Regards
		Oliver


  reply	other threads:[~2023-12-07  9:41 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-04 18:37 Question regarding CDC NCM and VNC performance issue Hiago De Franco
2023-12-05  9:50 ` Oliver Neukum
2023-12-05 12:22   ` Hiago De Franco
2023-12-05 12:38     ` Oliver Neukum
2023-12-05 12:55       ` Hiago De Franco
2023-12-05 20:14       ` Hiago De Franco
2023-12-06 12:21         ` Oliver Neukum
2023-12-06 16:23           ` Hiago De Franco
2023-12-07  9:41             ` Oliver Neukum [this message]
2023-12-07 10:03               ` Francesco Dolcini
2023-12-07 11:07                 ` Oliver Neukum
2023-12-07 11:38                   ` Maciej Żenczykowski
2023-12-07 18:57                     ` Hiago De Franco
2023-12-07 19:37                       ` Maciej Żenczykowski
2023-12-11 20:28                         ` Hiago De Franco
2023-12-11 20:44                           ` Maciej Żenczykowski
2023-12-12  9:48                             ` Oliver Neukum
2023-12-12 20:32                               ` Hiago De Franco
2023-12-12 21:21                                 ` Oliver Neukum
2023-12-13  1:11                                   ` Maciej Żenczykowski
2023-12-15 21:20                                     ` Hiago De Franco
2023-12-15 21:44                                       ` Maciej Żenczykowski
2023-12-18 14:34                                         ` Oliver Neukum
2023-12-18 20:59                                           ` Hiago De Franco
2023-12-18 23:00                                             ` Maciej Żenczykowski
2023-12-19  8:45                                               ` Oliver Neukum
2023-12-19 12:19                                                 ` Maciej Żenczykowski
2023-12-19 13:24                                                   ` Oliver Neukum
2023-12-19 16:25                                                     ` Maciej Żenczykowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c2ee0ecf-993e-4736-b005-588fa6ef51a3@suse.com \
    --to=oneukum@suse.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=hiago.franco@toradex.com \
    --cc=hiagofranco@gmail.com \
    --cc=kuba@kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=maze@google.com \
    --cc=pabeni@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox