[Mesa-dev] [PATCH 02/11] vl: add entry point

Emil Velikov emil.l.velikov at gmail.com
Thu Jul 14 17:06:43 UTC 2016

On 14 July 2016 at 16:06, Zhang, Boyuan <Boyuan.Zhang at amd.com> wrote:
> For example, in patch 5/11 when " VaCreateContext", we used to CALLOC_STRUCT for "pps" and "sps" whenever we see video format is H.264. This is fine for decode ONLY case. Now, since we added H.264 encoding, "pps" and "sps" shouldn't be allocated. So we need to use the entry_point to determine this is H.264 decode or H.264 encode. We can use config to determine the entrypoint since config_id is passed to us for VaCreateContext call. However, for VaDestoyContext call, only context_id is passed to us. So we need to know the entrypoint in order to not free the pps/sps for encoding case.
This is a perfect example (one might call it too verbose, but that
shouldn't be an issue) of a good information to add in the commit
message :-)
If a question is coming from a person familiar with the code this is a
dead giveaway that the commit message is subpar.

I would kindly ask that you check through the readings suggested
earlier [1] [2]. It seems like you did not have the time/chance.


[1] http://who-t.blogspot.co.uk/2009/12/on-commit-messages.html
[2] http://chris.beams.io/posts/git-commit/

More information about the mesa-dev mailing list