From: "Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: [PATCH 0/2] ALSA: memalloc: Fix for Xen PV and non-IOMMU systems
Date: Wed, 25 Jan 2023 16:33:45 +0100 [thread overview]
Message-ID: <Y9FL2SOln564i6cE@mail-itl> (raw)
In-Reply-To: <87o7qmtyy8.wl-tiwai@suse.de>
[-- Attachment #1: Type: text/plain, Size: 980 bytes --]
On Wed, Jan 25, 2023 at 04:29:19PM +0100, Takashi Iwai wrote:
> On Tue, 24 Jan 2023 10:27:42 +0100,
> Takashi Iwai wrote:
> >
> > Hi,
> >
> > this is a patch series to address the recent regression on Xen PV (and
> > possibly non-IOMMU) systems about the SG-buffer memory allocation.
> > We switched to use dma_alloc_noncontiguous() as hoped it handling
> > everything right, but it turned out that this doesn't work always.
> > So this is one step back, use the explicit SG-buffer with
> > dma_alloc_coherent() calls, but in a bit more optimized ways, and also
> > applying only for those systems.
>
> It seems that the second patch causes a problem; at least I see casual
> Oopses on my system after using the patch. Let's scratch.
>
> I'll resubmit the fix. Marek, could you try that later and report
> back if it still works and doesn't break things again?
Sure, just cc me.
--
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
prev parent reply other threads:[~2023-01-25 15:34 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-24 9:27 [PATCH 0/2] ALSA: memalloc: Fix for Xen PV and non-IOMMU systems Takashi Iwai
2023-01-24 9:27 ` [PATCH 1/2] ALSA: memalloc: Explicit SG-allocations " Takashi Iwai
2023-01-24 9:27 ` [PATCH 2/2] ALSA: memalloc: Use coherent DMA allocation for fallback again Takashi Iwai
2023-01-25 15:29 ` [PATCH 0/2] ALSA: memalloc: Fix for Xen PV and non-IOMMU systems Takashi Iwai
2023-01-25 15:33 ` Marek Marczykowski-Górecki [this message]
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=Y9FL2SOln564i6cE@mail-itl \
--to=marmarek@invisiblethingslab.com \
--cc=alsa-devel@alsa-project.org \
--cc=tiwai@suse.de \
/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.