All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Muench <davemuench@gmail.com>
To: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
Cc: xen-devel List <xen-devel@lists.xensource.com>
Subject: Re: [ivtv-devel] Problems loading ivtv in Xen - DMA issues?
Date: Fri, 15 Jul 2005 08:43:29 -0400	[thread overview]
Message-ID: <bc0c36d30507150543713394eb@mail.gmail.com> (raw)
In-Reply-To: <a7cabfda232da909dbfb7ab385493621@cl.cam.ac.uk>

On 7/13/05, Keir Fraser <Keir.Fraser@cl.cam.ac.uk> wrote:
> 
> I believe the code as checked in is strictly correct, but maybe we can
> relax things a little. What happens if you leave the direction tests in
> dma_map_single and dma_unmap_single, but remove them from
> dma_sync_single_for_cpu and dma_sync_single_for_device?

That works fine.

> If the above works, leave the direction tests in the new modified form;
> otherwise revert to the previous working state (ie. direction tests
> removed from all four functions). Either way, another thing that is
> interesting to try is the following:
> (1) At the start of dma_sync_single_for_cpu add:
>   if (direction == DMA_TO_DEVICE)
>      dma_sync_single_for_device(dev, dma_handle, size, direction);
> (2) At the start of dma_sync_single_for_device add:
>   if (direction == DMA_FROM_DEVICE)
>      dma_sync_single_for_cpu(dev, dma_handle, size, direction);
> 
> This will make the sync functions behave as they do in the ARM
> architecture, which also uses bounce buffers but has at least been
> looked over by the core kernel DMA guys.

That doesn't work however:

saa7115: decoder disable output
saa7115: decoder enable output
ivtv: DEC: REG_ENCSG1LEN wait failed
ivtv: DMA Registers State: xfer: 0x00000002, state: 0x00000001
dec_addr: 0x00000000 enc_addr: 0x46760060 control: 0x00000003
ivtv: DMA DEC Buffers:
 0x00000000:0x00000000:0x00000000
ivtv: DMA ENC Buffers:
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
ivtv: ENC: REG_DMAXFER wait failed
ivtv: DMA Registers State: xfer: 0x00000002, state: 0x00000001
dec_addr: 0x00000000 enc_addr: 0x46760060 control: 0x00000003
ivtv: DMA DEC Buffers:
 0x00000000:0x00000000:0x00000000
ivtv: DMA ENC Buffers:
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
ivtv: ENC: User stopped capture.
ivtv: ENC: REG_DMASTATUS2 wait failed
ivtv: DMA Registers State: xfer: 0x00000002, state: 0x00000001
dec_addr: 0x00000000 enc_addr: 0x46760060 control: 0x00000003
ivtv: DMA DEC Buffers:
 0x00000000:0x00000000:0x00000000
ivtv: DMA ENC Buffers:
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000
 0x00000000:0x00000000:0x00000000

I'm going to take that out and run with the conditionals in the map
functions only, and see if that holds up under longer testing.

Dave

-- 
David Muench - davemuench@gmail.com
Jabber ID: dave@jabber.wasteland.org

  reply	other threads:[~2005-07-15 12:43 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <bc0c36d305070607205e82ea14@mail.gmail.com>
     [not found] ` <1120750950.1556.24.camel@mud>
2005-07-07 16:07   ` [ivtv-devel] Problems loading ivtv in Xen - DMA issues? David Muench
2005-07-07 17:22     ` Keir Fraser
2005-07-07 17:34       ` David Muench
2005-07-08  4:01         ` Andrew May
2005-07-08 13:05         ` Keir Fraser
2005-07-08 15:38           ` David Muench
2005-07-08 16:50             ` Keir Fraser
2005-07-11 10:10             ` Keir Fraser
2005-07-11 13:08               ` David Muench
2005-07-11 13:20                 ` Keir Fraser
2005-07-13 19:51               ` David Muench
2005-07-13 22:06                 ` Keir Fraser
2005-07-15 12:43                   ` David Muench [this message]
2005-07-15 12:50                     ` Keir Fraser
2005-07-15 12:57                       ` David Muench

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=bc0c36d30507150543713394eb@mail.gmail.com \
    --to=davemuench@gmail.com \
    --cc=Keir.Fraser@cl.cam.ac.uk \
    --cc=xen-devel@lists.xensource.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.