public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Julian Scheel <julian@jusst.de>
To: linux-media@vger.kernel.org
Subject: Re: New DVB(!!)-Statistics API
Date: Tue, 08 Dec 2009 13:24:39 +0100	[thread overview]
Message-ID: <4B1E4587.3070304@jusst.de> (raw)
In-Reply-To: <4B1E1974.6000207@jusst.de>

Am 08.12.09 10:16, schrieb Julian Scheel:
> Hello together,
>
> after the last thread which asked about signal statistics details 
> degenerated into a discussion about the technical possibilites for 
> implementing an entirely new API, which lead to nothing so far, I 
> wanted to open a new thread to bring this forward. Maybe some more 
> people can give their votes for the different options
>
> Actually Manu did propose a new API for fetching enhanced statistics. 
> It uses new IOCTL to directly fetch the statistical data in one go 
> from the frontend. This propose was so far rejected by Mauro who wants 
> to use the S2API get/set calls instead.
>
> Now to give everyone a quick overview about the advantages and 
> disadvantages of both approaches I will try to summarize it up:
>
> 1st approach: Introduce new IOCTL
>
> Pros:
> - Allows a quick fetch of the entire dataset, which ensures that:
>  1. all values are fetched in one go (as long as possible) from the 
> frontend, so that they can be treated as one united and be valued in 
> conjunction
>  2. the requested values arrive the caller in an almost constant 
> timeframe, as the ioctl is directly executed by the driver
> - It does not interfere with the existing statistics API, which has to 
> be kept alive as it is in use for a long time already. (unifying it's 
> data is not the approach of this new API)
>
> Cons:
> - Forces the application developers to interact with two APIs. The 
> S2API for tuning on the one hand and the Statistics API for reading 
> signal values on the other hand.
>
> 2nd approach: Set up S2API calls for reading statistics
>
> Pros:
> - Continous unification of the linuxtv API, allowing all calls to be 
> made through one API. -> easy for application developers
>
> Cons:
> - Due to the key/value pairs used for S2API getters the statistical 
> values can't be read as a unique block, so we loose the guarantee, 
> that all of the values can be treatend as one unit expressing the 
> signals state at a concrete time.
> - Due to the general architecture of the S2API the delay between 
> requesting and receiving the actual data could become too big to allow 
> realtime interpretation of the data (as it is needed for applications 
> like satellite finders, etc.)
>
> I hope that this summary is technically correct in all points, if not 
> I'd be thankful for objective corrections. I am not going to 
> articulate my personal opinion in this mail, but I will do it in 
> another mail in reply to this one, so that this mail can be seen as a 
> neutral summary of the current situation.
>
> So now it's everyones turn to think about the options and give an 
> opinion. In the end I am quite sure that all of us would have great 
> benefits of an improved statistics API.
>
> Cheers,
> Julian
> -- 
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

The above mail of course has nothing to do with USB-Statistics. I must 
have been slightly confused when typing the message title! Sorry for that!

  reply	other threads:[~2009-12-08 12:24 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-08  9:16 New USB-Statistics API Julian Scheel
2009-12-08 12:24 ` Julian Scheel [this message]
2009-12-08 13:22 ` New DVB-Statistics API Mauro Carvalho Chehab
2009-12-08 21:46   ` Manu Abraham
2009-12-08 23:43     ` Mauro Carvalho Chehab
2009-12-09 11:42       ` Manu Abraham
2009-12-09 13:02         ` Mauro Carvalho Chehab
2009-12-09 22:12           ` Julian Scheel
2009-12-09 23:38             ` Mauro Carvalho Chehab
2010-02-03  9:49   ` New DVB-Statistics API - please vote Mauro Carvalho Chehab
2010-02-03 10:40     ` Hans Verkuil
2010-02-03 12:37       ` Manu Abraham

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=4B1E4587.3070304@jusst.de \
    --to=julian@jusst.de \
    --cc=linux-media@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox