Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Sat, 2 Feb 2019 15:05:43 +0530
From:   Vinod Koul <vkoul@kernel.org>
To:     Federico Vaga <federico.vaga@cern.ch>
Cc:     dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-doc@vger.kernel.org
Subject: Re: DMA Engine Documentation: TX Descriptor and Submission
Message-ID: <20190202093226.GC4296@vkoul-mobl>
References: <1655406.8MLMLfCWyH@pcbe13614>
 <20190201041750.GM4635@vkoul-mobl>
 <77C59695-001C-46B3-9C41-C4C7A763D001@cern.ch>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <77C59695-001C-46B3-9C41-C4C7A763D001@cern.ch>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 01-02-19, 09:59, Federico Vaga wrote:
> 
> 
> On February 1, 2019 4:17:50 AM UTC, Vinod Koul <vkoul@kernel.org> wrote:
> >On 28-01-19, 09:47, Federico Vaga wrote:
> >> Hi,
> >> 
> >> I have a new question concerning documentation.
> >> 
> >>
> >https://www.kernel.org/doc/html/latest/driver-api/dmaengine/client.html
> >> 
> >> >From this document it is not really clear, at least to me, if
> >clients can 
> >> consider valid the `struct dma_async_tx_descriptor` after submission
> >to the 
> >> DMA engine.
> >
> >Nope they can't and should not touch the descriptor after submission.
> >The client get cookie and that is supposed to be used
> 
> Good, thanks
> 
> 
> >> 
> >> Clients get a TX descriptor from a DMA engine using things like 
> >> `dmaengine_prep_*`. These calls - may - allocate new descriptors and
> >return 
> >> them to the caller; this may include other structures which are not
> >visible to 
> >> clients. So, if my understanding is correct, this means that it's the
> >DMA 
> >> engine that, on TX completion, releases any TX descriptor allocated
> >by 
> >> `dmaengine_prep_*`. This implies that the pointer that the client is
> >using 
> >> must be considered invalid right after `dmaengine_submit()`.
> >> If what I understood by reading the documentation and the code is
> >correct, 
> >> then I think that this should be mentioned in the Documentation.
> >> If I'm wrong, please tell me where :)
> >
> >And what exactly are you trying to do here..?
> 

Please wrap your replies within 80chars

> What if I answer: "the right thing"? Joking. I just expressed my
> understanding of something which is not documented properly (my
> opinion). I wanted to be sure that the logic, the reasons behind are
> the ones that I understood. I will propose a patch to the
> documentation, unless you want to do it.  -- Inviato dal mio
> dispositivo Android con K-9 Mail. Perdonate la brevit?.

Sure please feel free to send a patch :)

Mostly people ask these questions based on a driver they are working
off, and it makes sense to fix the assumptions by asking what they are
trying to achieve...

-- 
~Vinod