From: Julian Scheel <julian@jusst.de>
To: linux-media@vger.kernel.org
Subject: New USB-Statistics API
Date: Tue, 08 Dec 2009 10:16:36 +0100 [thread overview]
Message-ID: <4B1E1974.6000207@jusst.de> (raw)
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
next reply other threads:[~2009-12-08 9:16 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-08 9:16 Julian Scheel [this message]
2009-12-08 12:24 ` New DVB(!!)-Statistics API Julian Scheel
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=4B1E1974.6000207@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