From: Mathias Nyman <mathias.nyman@linux.intel.com>
To: Wesley Cheng <wesley.cheng@oss.qualcomm.com>,
Mathias Nyman <mathias.nyman@intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
Michal Pecio <michal.pecio@gmail.com>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-sound@vger.kernel.org, 胡连勤 <hulianqin@vivo.com>
Subject: Re: [PATCH v4 0/5] Add larger page size support for USB audio offload path
Date: Fri, 2 Oct 2026 01:33:52 +0300 [thread overview]
Message-ID: <92d80a55-4422-4659-8f2d-e28693d6d9d7@linux.intel.com> (raw)
In-Reply-To: <d7ce8b23-64c0-43dd-a8c8-268224b9462c@oss.qualcomm.com>
On 10/1/26 01:23, Wesley Cheng wrote:
>
> Hi Mathias,
>
> Just checking to see if there was anything that we needed to account for in this series? I think there was the discussion that came up here:
> https://lore.kernel.org/all/TYUPR06MB6217000B59003EDF233D7246D2B22@TYUPR06MB6217.apcprd06.prod.outlook.com/
Patch v4 1/5: Looks good, is useful even as a standalone patch. Can be taken immediately.
Patch v4 2/5: Looks good, not useful as standalone patch, can be taken with rest of series.
Patch v4 3/5: I need to revisit this and look at new dma pool lifetime vs xHC accessing those DMA addresses.
PAtch v4 Rest: ALSA, no opinion.
>
> But, I think your suggestion about adding the sideband check in the xhci_setup_device() would mitigate that problem. Esp, if we don't want xHCI core to really know about sideband ops.
That would be the bandaid in:
https://lore.kernel.org/all/c6c9890f-46d8-42f1-a7e7-8a5d551fa22c@linux.intel.com/
I can turn that into a proper patch.
Lianqin also gave some preliminary positive feedback, but I'm not sure if the code was
ever properly tested.
Thanks
Mathias
next prev parent reply other threads:[~2026-10-01 22:33 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-10 1:50 [PATCH v4 0/5] Add larger page size support for USB audio offload path Wesley Cheng
2026-09-10 1:50 ` [PATCH v4 1/5] usb: xhci: sideband: fix ring sg table for sub-page TRB segments Wesley Cheng
2026-09-10 1:50 ` [PATCH v4 2/5] usb: xhci: Allow to specify a different segment pool for allocations Wesley Cheng
2026-09-10 1:50 ` [PATCH v4 3/5] usb: xhci: sideband: allocate sideband ring segments from a dedicated pool Wesley Cheng
2026-10-05 11:03 ` Mathias Nyman
2026-10-13 1:01 ` Wesley Cheng
2026-10-13 9:11 ` Michal Pecio
2026-09-10 1:51 ` [PATCH v4 4/5] ALSA: usb-audio: qcom: tag sideband endpoints before ring allocation Wesley Cheng
2026-09-10 1:51 ` [PATCH v4 5/5] ALSA: usb-audio: qcom: fix xfer ring IOMMU unmap on 16K+ page kernels Wesley Cheng
2026-09-30 22:23 ` [PATCH v4 0/5] Add larger page size support for USB audio offload path Wesley Cheng
2026-10-01 22:33 ` Mathias Nyman [this message]
2026-10-02 1:42 ` Wesley Cheng
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=92d80a55-4422-4659-8f2d-e28693d6d9d7@linux.intel.com \
--to=mathias.nyman@linux.intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=hulianqin@vivo.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mathias.nyman@intel.com \
--cc=michal.pecio@gmail.com \
--cc=perex@perex.cz \
--cc=tiwai@suse.com \
--cc=wesley.cheng@oss.qualcomm.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.