linux-kernel - Re: [PATCH v29 01/33] xhci: support setting interrupt moderation IMOD for secondary interrupters

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <87wmi02qcj.wl-tiwai@suse.de>
Date: Tue, 22 Oct 2024 15:56:44 +0200
From: Takashi Iwai <tiwai@...e.de>
To: Wesley Cheng <quic_wcheng@...cinc.com>
Cc: Greg KH <gregkh@...uxfoundation.org>,
	srinivas.kandagatla@...aro.org,
	mathias.nyman@...el.com,
	perex@...ex.cz,
	conor+dt@...nel.org,
	dmitry.torokhov@...il.com,
	corbet@....net,
	lgirdwood@...il.com,
	tiwai@...e.com,
	krzk+dt@...nel.org,
	pierre-louis.bossart@...ux.intel.com,
	Thinh.Nguyen@...opsys.com,
	broonie@...nel.org,
	bgoswami@...cinc.com,
	robh@...nel.org,
	linux-kernel@...r.kernel.org,
	devicetree@...r.kernel.org,
	linux-sound@...r.kernel.org,
	linux-input@...r.kernel.org,
	linux-usb@...r.kernel.org,
	linux-arm-msm@...r.kernel.org,
	linux-doc@...r.kernel.org,
	alsa-devel@...a-project.org,
	Mathias Nyman <mathias.nyman@...ux.intel.com>
Subject: Re: [PATCH v29 01/33] xhci: support setting interrupt moderation IMOD for secondary interrupters

On Fri, 18 Oct 2024 07:52:35 +0200,
Greg KH wrote:
> 
> On Thu, Oct 17, 2024 at 05:07:12PM -0700, Wesley Cheng wrote:
> > Hi Greg,
> > 
> > On 10/16/2024 11:40 PM, Greg KH wrote:
> > > On Tue, Oct 15, 2024 at 02:28:43PM -0700, Wesley Cheng wrote:
> > >> From: Mathias Nyman <mathias.nyman@...ux.intel.com>
> > >>
> > >> Allow creators of xHCI secondary interrupters to specify the interrupt
> > >> moderation interval value in nanoseconds when creating the interrupter.
> > >>
> > >> If not sure what value to use then use the xhci driver default
> > >> xhci->imod_interval
> > >>
> > >> Suggested-by: Wesley Cheng <quic_wcheng@...cinc.com>
> > >> Signed-off-by: Mathias Nyman <mathias.nyman@...ux.intel.com>
> > >> Link: https://lore.kernel.org/r/20240905143300.1959279-13-mathias.nyman@linux.intel.com
> > >> Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> > >> ---
> > >>  drivers/usb/host/xhci-mem.c | 8 +++++++-
> > >>  drivers/usb/host/xhci.c     | 4 ++--
> > >>  drivers/usb/host/xhci.h     | 5 ++++-
> > >>  3 files changed, 13 insertions(+), 4 deletions(-)
> > > This is already in 6.12-rc1, which makes me confused as to what tree you
> > > made this series against.
> > 
> > Sorry, I didn't fetch the latest changes from usb-next.
> 
> It wasn't even usb-next, it was 6.12-rc1, so I don't know what tree you
> based this on :(
> 
> > In this case, should I rebase and resbumit?
> 
> As the series can't be applied as-is, probably.  But I think you might
> want to collect some acks from the sound people and xhci developers, as
> I can't do anything with this until they look at the changes.

Honestly speaking, I couldn't follow fully the discussions about the
fundamental design -- IIRC, Pierre and others had concerns to the way
to manage the offload device via kcontrols.  Did we get consensus?

I believe that's the biggest obstacle in the audio side, i.e. what's
visible to users.  The kernel internals can be corrected at any time
later.


thanks,

Takashi