| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
Some debug logging, such as memory allocation failure, uses similar
log messageing in several places. Add the file line number to
the debug log to help differentiate where notable errors occured.
Change-Id: I2151e21533716ea0badbb6303b713e644b544e6d
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
|
|
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
Avoid calling close() on file descriptors that have not been intialized,
(or initialized to 0) and descriptors that are known to be invalid.
Closing file descriptors due to handle failing initialization is handled
in the intialization code. In the lease-manager, file descriptors can be
invalid during normal operation, so validity is tested at shutdown.
Bug-AGL: SPEC-3862
Change-Id: I798273195cba297c14c6b97d50c7614164fda7df
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
|
|
If a client application crashes while owning a lease,
the lease will automatically be revoked and all references
freed to make it available for another (or the restarted
original) client to claim and continue display. This will
cause the screen to blank while the lease is unowned.
Add an option to revoke the lease, but keep the reference to
the lease descriptor, so that the screen does not blank while
unowned.
This only has effect when a client exits without calling the
`dlm_release_lease` function. Otherwise the lease is revoked
and the descriptor dereferenced, just as before.
Bug-AGL: SPEC-3862
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
Change-Id: I3fc2c8fde4118b6e298ba31807fd5fa4766a97dd
|
|
Collect all protocol related functionality in the same
place instead of spreading it around across different
modules.
Bug-AGL: SPEC-3862
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
Change-Id: I169a200401e4feaa289d53240a7f5467d6a88bd3
|
|
Explicitly request / release leases instead of implicitly
by opening and closing the connection. This will allow the
lease manager to take different action when a client
shuts down gracefully vs when it crashes, holding a lease.
Bug-AGL: SPEC-3862
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
Change-Id: Ibc68bee855ce18e56eb6f57e5ad1743248320013
|
|
After a lease is tranisitined to a new client, the reference to
the previous client's lease fd is no longer needed and should
be released.
Close the old fd after the new client has either updated the
display or its lease is revoked.
Bug-AGL: SPEC-3816
Change-Id: I9612913e2960dce94bcfc6a35c0105a5670a453d
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
|
|
When enabled allow the ownership of a lease to transition from
one client to another.
If a request is made for a lease that is already allocated to
a client, revoke the lease from the old client and issue a new
lease for the same resources to the new client.
This allows the ownership of the leased resources to be transferred
without the display being closed and the screen blanking.
During the tranistion, hold a reference to the old clients lease fd so
that the last frame presented by the old client will remain on screen until
the new client presents its first frame.
Currenly there is no notification or authentication mechanism
implemeted for the transition. Any client can request a transition
from any other. The only notification that a transfer has happened
is that the previous client's DRM API calls will suddenly fail with
-ENOENT, since all resources will have been removed.
It is up to the client to sensibly handle this condition.
Bug-AGL: SPEC-3816
Change-Id: Iaff87e275f909f3b6ef448df39364d6fe62007b1
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
|
|
Allow multiple clients to issue lease requests
on a server at the same time. This is
necessary to be able to grant or deny leases,
not just on a first-come-first-served basis.
Future patches will add extra contitions,
such as command-line options and lease configuration
settings to determine when and how lease requests should
be granted.
This update changes the behaviour of the lease-server interface
so that it reports every client connection request, instead of
when a server has accepted a request, so update the test suite to
reflect this.
Bug-AGL: SPEC-3816
Change-Id: I48cc392dd62a8c06ea74178bc52c627032817203
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
|
|
libcheck tests normally run each test in a separate process
so the server objects will automatically be destroyed after
each test. If the test suite is run with CK_FORK=no, for
example when debugging the test with gdb, failing to
destory the server object will leave the test sockets
from the previous test open, which will stop the next
test from creating its own sockets.
Bug-AGL: SPEC-3816
Change-Id: I42a0976c392bdc19bb60acc78178aac8ee3e14ce
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
|
|
The initial version implements the basic functionality
of the client/server communication and lease management.
For now, one lease is created per valid connector (dependent
on CRTC availablity).
Bug-AGL: SPEC-3729
Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
Change-Id: I2b37a892742cc22bdc53a5172c8ad3d8a7bb5e66
|
|
|
Damian Hobson-Garcia
Jan-Simon Möller