external/PRIVATE/libpciaccess: Klocwork Fixes

BZ: 119061

KW fixes for libpciaccess module

Change-Id: I334290fc1de5fbcfe56721feb27663e1a1518cd4
Signed-off-by: arsharma <[email protected]

external/PRIVATE/libpciaccess: Klocwork Fixes

BZ: 119061

KW fixes for libpciaccess module

Change-Id: I334290fc1de5fbcfe56721feb27663e1a1518cd4
Signed-off-by: arsharma <[email protected]>
Reviewed-on: http://android.intel.com:8080/115795
Reviewed-by: Satyanantha, Rama Gopal M <[email protected]>
Reviewed-by: Lister, Ian <[email protected]>
Reviewed-by: S, Deepak <[email protected]>
Reviewed-by: Barbalho, Rafael <[email protected]>
Reviewed-by: Reese, Armin C <[email protected]>
Reviewed-by: Stimson, Dale B <[email protected]>
Reviewed-by: Goel, Akash <[email protected]>
Tested-by: Stimson, Dale B <[email protected]>
Reviewed-by: buildbot <[email protected]>
Tested-by: buildbot <[email protected]>

show more ...

Don't include config.h on Android

Android doesn't use autotools, so the header is never generated.

A previous commit removed some of the includes; this commit hopefully
removes the rest.

Signed-of

Don't include config.h on Android

Android doesn't use autotools, so the header is never generated.

A previous commit removed some of the includes; this commit hopefully
removes the rest.

Signed-off-by: Chad Versace <[email protected]>

show more ...

Fix long standing MTRR bug.

Add an include that makes the file common_vgaarb.c see the same
struct pci_system as the file linux_sysfs.c.

Without this fix, on a MTRR system, the vgaarb_fd member
wou

Fix long standing MTRR bug.

Add an include that makes the file common_vgaarb.c see the same
struct pci_system as the file linux_sysfs.c.

Without this fix, on a MTRR system, the vgaarb_fd member
would overwrite the mtrr_fd member and cause the MTRR_IOC_ADD_ENTRY call
to be issued to the incorrect device causing the infamous
"Inappropriate ioctl for device (25)" error.

This error would cause MTRR setup to fail on all systems relying on it
and is severe enough to warrant a new release.

Signed-off-by: Thomas Hellstrom <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>

show more ...

vgaarb: read back vga count when setting new decoding

Decode kernel operation can also change vga refcount, so we need to read back
the information when calling it.

Signed-off-by: Tiago Vignatti <t

vgaarb: read back vga count when setting new decoding

Decode kernel operation can also change vga refcount, so we need to read back
the information when calling it.

Signed-off-by: Tiago Vignatti <[email protected]>

show more ...

vgaarb: decode should send new information to the kernel

This was introduced in e360c11c.

Signed-off-by: Tiago Vignatti <[email protected]>

shutup stupid gcc warning, strtok_r ignores the first parameter

vgaarb: fix newbie typo

Eh, we don't need to check for the fd anyway. Kudos jcristau for reporting.

Signed-off-by: Tiago Vignatti <[email protected]>

vgaarb: check for fd before close it

Signed-off-by: Tiago Vignatti <[email protected]>

vgaarb: check pci_sys exists before initing vga arb

Revert "vgaarb: fix pci_device_vgaarb_get_info() return value."

This reverts commit ef8aca5da46155e9ea0d1227215921ad9ce59150.

vgaarb: fix pci_device_vgaarb_get_info() return value.

Signed-off-by: Tiago Vignatti <[email protected]>

vgaarb: fix indentation and remove trailing white spaces

Signed-off-by: Tiago Vignatti <[email protected]>

vgaarb: add vga count + resource accessor

vgaarb: more api changes + cleanup

add a comma for ease of parsing,
use hex on the interface in both directions instead of hex on one
and decimal on the other

vgaarb: change API to target taking a device + lock/unlock not taking one

working on the target device - a target device of NULL
reselects the target that existed when we started.

this is mainly to

vgaarb: change API to target taking a device + lock/unlock not taking one

working on the target device - a target device of NULL
reselects the target that existed when we started.

this is mainly to allow a udev posting tool to return to the
state of what happened before it ran.

show more ...

vgaarb: add support for targetting default device

vgaarb: clean up debug output

vgaarb: fixup api for new count reporting + decodes interface was wrong.

decodes is a information setting interface so it takes new decodes
and sends them to the kernel

vgaarb: fixup api and decode rsrc.

This fixes up the API and stores the vga arb fd in the sys_pci structure,
instead of hiding it in a random dev struct.

It also reads back after setting the target

vgaarb: fixup api and decode rsrc.

This fixes up the API and stores the vga arb fd in the sys_pci structure,
instead of hiding it in a random dev struct.

It also reads back after setting the target and works out the decodes.

Signed-off-by: Dave Airlie <[email protected]>

show more ...

Comment typo.

Reorganize the code.

Delete vga_arb_rec.

vgaarb: a bunch of clean ups to fit in libpciaccess style.

vgaarb: make it compile.

Import libvgaaccess (VGA Arbiter) implementation from C3SL repository:
http://git.c3sl.ufpr.br/