Fix some -Wformat errors in scanpci

Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

libpciaccess: close mtrr fd on pci_cleanup

Since the fd is not closed, calling pci_system_init and
pci_system_cleanup more than 1024 times results in "too many files open"
error.

Signed-off-by: Nithin Nayak Sujir <nsujir@broadcom.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

Solaris: Give better error on realloc failure

commit a18460b385ae03 converted from a fixed maximum number of devices
to dynamically growing the list via realloc, but didn't update the
error message shown on failure.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Dirk Wallenstein <halsmit@t-online.de>
Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

linux sysfs: Fix read-write access in map_legacy

O_RDONLY | O_WRONLY != O_RDWR

><

Reported-by: Javier Pello <javier.pello@urjc.es>
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

scanpci: Build fix for systems without <err.h>

https://bugs.freedesktop.org/show_bug.cgi?id=31133

Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

configure.ac: Bump version to 0.12.901

Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

Update library version to reflect new API

Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

linux: Implement map_legacy

Signed-off-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

Add map_legacy interface

This allows platforms to hand back mmaps of the low 1M (ISA) address
space on a per-domain basis.

Signed-off-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

Strip trailing whitespace

Performed with: find * -type f | xargs perl -i -p -e 's{[ \t]+$}{}'
git diff -w & git diff -b show no diffs from this change

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

linux: Fix a crash in populate_devices

If scandir returns -1, the 'devices' array won't be initialized, and
attempting to free() it will crash.

Signed-off-by: Adam Jackson <ajax@redhat.com>

linux: Only set errno after an error

errno is only valid after an error, and was being filled with a garbage
value upon eof.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

Solaris support for multiple PCI segments (domains)

1. Removed hardcoded maximum size of 256 PCI devices, which is
too small for large systems.  The number of devices is dynamically
resized as needed.

2. pci_device_solx_devfs_probe() no longer walks the device tree
from the very top ("/") but instead starts at the nexus which
owns the bus.  Performance optimization for systems with multiple
bus nodes (including systems with just one segment/domain).

3. Added support for multiple domains/segments.  Code tested
on kernels with and without multiple segment support so it should
be safe to integrate independent of the kernel version.

Signed-off-by: John Martin <John.M.Martin@Oracle.COM>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>

config: comment, minor upgrade, quote and layout configure.ac

Group statements per section as per Autoconf standard layout
Quote statements where appropriate.
Autoconf recommends not using dnl instead of # for comments

Use AC_CONFIG_FILES to replace the deprecated AC_OUTPUT with parameters.
Add AC_CONFIG_SRCDIR([Makefile.am])
Update X.Org util-macros to version 1.8

This helps automated maintenance and release activities.
Details can be found in http://wiki.x.org/wiki/NewModuleGuidelines

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

libpciaccess 0.12.1

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

config: replace deprecated AC_HELP_STRING with AS_HELP_STRING

This silences an Automake warning.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: remove AC_PROG_CC as it overrides AC_PROG_C_C99

XORG_STRICT_OPTION from XORG_DEFAULT_OPTIONS calls
AC_PROG_C_C99. This sets gcc with -std=gnu99.
If AC_PROG_CC macro is called afterwards, it resets CC to gcc.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

config: replace deprecated AM_CONFIG_HEADER with AC_CONFIG_HEADERS

Regroup AC statements under the Autoconf initialization section.
Regroup AM statements under the Automake initialization section.
Add missing AC_CONFIG_SRCDIR
Replace obsolete argument to AC_OUTPUT with AC_CONFIG_FILES

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Fix detection of multi-function cards

Bit 7 of the pci device header type is an indicator of multi-function
devices, so needs to be masked when examining the bridge type.
Thanks go to Chih-Pin Wu for finding the bug and its fix.

https://bugs.launchpad.net/ubuntu/+source/libpciaccess/+bug/681207

Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Bryce Harrington <bryce@canonical.com>

Revert "Don't try to use bogus bridge data"

This reverts commit 2bda5b733bb12854760750c08138db95e77aea0c.

It actually conflicts with 9ba94caf57e3a8c3e9c6f3f5f068f4a7a7b3ff9d
which came earlier, but I didn't catch it probably because I did a blind
rebase before pushing.

When reading bridge bus info, we may need to read out the current bridge
info if the PCI bridge private is uninitialized, so remove the check at
the top that would always short circuit the case where the priv was
still 0.

References:
https://bugs.launchpad.net/ubuntu/+source/libpciaccess/+bug/681207.

Reported-by: Bryce Harrington <bryce@canonical.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

Sun's copyrights belong to Oracle now

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

config: use LDADD to obtain appropriate dependencies

In generated scanpci/Makefile, this line is added:
scanpci_DEPENDENCIES = $(top_builddir)/src/libpciaccess.la

If the content of the library changes, the tool will rebuild.
When scanpci was in the same directory as the library, the .la file was used.
This will also ensure the "installed" pciaccess library is not used.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

include: add a Makefile to install and distribute header file

Makefiles are simpler when they only handle what is in their
directory.

Reviewed-by: Rémi Cardona <remi@gentoo.org>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: replace INCLUDES, which is deprecated, with AM_CPPFLAGS

Fixes an Automake warning.

Reviewed-by: Rémi Cardona <remi@gentoo.org>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: provide a makefile for scanpci man pages

Reviewed-by: Rémi Cardona <remi@gentoo.org>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

scanpci.c: remove offending whitespace as reported by git diff

Reviewed-by: Rémi Cardona <remi@gentoo.org>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: provide a makefile for scanpci dev tool

No function changes.

Note that automake generates a wrapper script if libpciaccess
is not installed at configuration time, which is the most common case.

If it is installed at configuration time, you get an executable.

The makefile build order ensures the lib is build before scanpci.

Reviewed-by: Rémi Cardona <remi@gentoo.org>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: upgrade AC_DEFINE_DIR to AX_DEFINE_DIR

Use the latest version now hosted by GNU.
The renaming protects Autoconf namespace.
There is no functional change in the macro.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

libpciaccess 0.12.0 - bump version for release

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 <thellstrom@vmware.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>

pciaccess: fix use after free.

Using the pointer passed to realloc earlier isn't a recipe for success.

Signed-off-by: Dave Airlie <airlied@redhat.com>

Let libtool handle library flags for scanpci on Solaris

Stop hardcoding -ldevinfo since configure/libtool already know to add it
for static linking, and ELF dependencies handle it for dynamic linking

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

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 <tiago.vignatti@nokia.com>

vgaarb: decode should send new information to the kernel

This was introduced in e360c11c.

Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com>

Delay allocation of agp_info so we don't leak it on prior errors

   Memory leak of pointer 'agp_info'
        at line 119 of src/common_capability.c in function 'pci_fill_capabilities_generic'.
          'agp_info' allocated at line 107 with calloc(1, 12).
          'agp_info' leaks when err != 0 at line 118.
        at line 124 of src/common_capability.c in function 'pci_fill_capabilities_generic'.
          'agp_info' allocated at line 107 with calloc(1, 12).
          'agp_info' leaks when err != 0 at line 123.

[ This bug was found by the Parfait bug checking tool.
  For more information see http://research.sun.com/projects/parfait ]

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

config: update AC_PREREQ statement to 2.60

Unrelated to the previous patches, the new value simply reflects
the reality that the minimum level for autoconf to configure
all x.org modules is 2.60 dated June 2006.

ftp://ftp.gnu.org/gnu/autoconf/autoconf-2.60.tar.gz

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: remove the pkgconfig pc.in file from EXTRA_DIST

Automake always includes it in the tarball.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

libpciaccess x86 backend

This adds support on x86 for OSes that do not have a PCI interface,
tinkering with I/O ports, and makes use of it on GNU/Hurd.

Signed-off-by: Julien Cristau <jcristau@debian.org>

COPYING: add Red Hat's copyright notice

Update Sun license notices to current X.Org standard form

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

libpciaccess 0.11.0

Signed-off-by: Adam Jackson <ajax@redhat.com>

Fix I/O handle array allocator to work for devices past the first

Signed-off-by: Adam Jackson <ajax@redhat.com>

Fix pci_device_open_io() to actually return something useful

Remember, code review just means we're all inept.

Signed-off-by: Adam Jackson <ajax@redhat.com>

I/O port access routines

Acked-by: Tiago Vignatti <tiago.vignatti@nokia.com>
Acked-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Adam Jackson <ajax@redhat.com>

Add pci_device_get_parent_bridge()

Copied from linuxPci.c in the X server.

Acked-by: Tiago Vignatti <tiago.vignatti@nokia.com>
Signed-off-by: Adam Jackson <ajax@redhat.com>

Makefile.am: add ChangeLog and INSTALL on MAINTAINERCLEANFILES

Now that the INSTALL file is generated.
Allows running make maintainer-clean.

INSTALL, NEWS, README or AUTHORS files are missing/incorrect #24206

Add missing INSTALL file. Use standard GNU file on building tarball
README may have been updated
Remove AUTHORS file as it is empty and no content available yet.
Remove NEWS file as it is empty and no content available yet.

Deploy the new XORG_DEFAULT_OPTIONS #24242

This macro aggregate a number of existing macros that sets commmon
X.Org components configuration options. It shields the configuration file from
future changes.

Makefile.am: ChangeLog not required: EXTRA_DIST or *CLEANFILES #24432

ChangeLog filename is known to Automake and requires no further
coding in the makefile.

.gitignore: use common defaults with custom section # 24239

Using common defaults will reduce errors and maintenance.
Only the very small or inexistent custom section need periodic maintenance
when the structure of the component changes. Do not edit defaults.

shutup stupid gcc warning, strtok_r ignores the first parameter

libpciaccess 0.10.9

vgaarb: fix newbie typo

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

Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com>

vgaarb: check for fd before close it

Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com>

Free return value from scandir instead of leaking it.

Scandir is defined to return pointers to malloc'd storage, so make
sure that memory is nicely freed.

Signed-off-by: Keith Packard <keithp@keithp.com>

vgaarb: check pci_sys exists before initing vga arb

pciaccess: bump to 0.10.8

pciaccess: fix vga arb stubs

libpciaccess: bump to 0.10.7 for vga arb interface addition

Merge branch 'vga'

vgaarb: update stub interface

Revert "vgaarb: delete stub functions"

This reverts commit b7fd4d06890d801aae93fe1a1af7ab110fd388fd.

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

This reverts commit ef8aca5da46155e9ea0d1227215921ad9ce59150.

Merge remote branch 'origin/vga-arbiter'

vgaarb: fix pci_device_vgaarb_get_info() return value.

Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com>

vgaarb: fix indentation and remove trailing white spaces

Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com>

vgaarb: delete stub functions

Such stub functions is already with API outdated. Moreover, if some operating
system doesn't have arbiter's implementation then the app can set some
autoconf tricks like "ifdef HAVE_PCI_DEVICE_VGAARB_INIT".

Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com>

Silence compiler warning about pci_id_file_open definition

src/common_device_name.c:59: warning: function declaration isn't a prototype
src/common_device_name.c: In function 'pci_id_file_open':
src/common_device_name.c:59: warning: old-style function definition

configure.ac: fix help text for the linux-rom-fallback option

LINUX_ROM defaults to no, fix the help text to reflect that

Signed-off-by: Julien Cristau <jcristau@debian.org>

Merge branch 'master' into vga-arbiter

pciaccess: make linux rom reading fallback optional

On kernels which have ROM files, the boot VGA rom file
will contain the 0xc0000 file anyways. If another card
is missing a rom file there is no need for this.

This is fallback is disabled by default, on the premise
that distro shipping this new a pciaccess will be running
on kernels with working rom support which is been upstream
for a long time

Merge branch 'master' into vga-arbiter

pciaccess: make linux rom reading fallback optional

On kernels which have ROM files, the boot VGA rom file
will contain the 0xc0000 file anyways. If another card
is missing a rom file there is no need for this.

This is fallback is disabled by default, on the premise
that distro shipping this new a pciaccess will be running
on kernels with working rom support which is been upstream
for a long time

pciaccess: make linux rom reading fallback optional

On kernels which have ROM files, the boot VGA rom file
will contain the 0xc0000 file anyways. If another card
is missing a rom file there is no need for this.

This is fallback is disabled by default, on the premise
that distro shipping this new a pciaccess will be running
on kernels with working rom support which is been upstream
for a long time

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 allow a udev posting tool to return to the
state of what happened before it ran.

vgaarb: add support for targetting default device

vgaarb: clean up debug output

Merge remote branch 'main/master' into vga-arbiter

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

add support for finding if something has a kernel driver

Don't try to use bogus bridge data

If, for whatever reason, we weren't able to read bridge data (therefore
leaving bridge.pci or bridge.pcmcia empty), we shouldn't try to look at
the bus data inside the pci/pcmcia struct.

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 <airlied@redhat.com>

Comment typo.

Reorganize the code.

Fix typo in vgaarb stubs.

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/

Don't try to write boot_vga since that fails with EACCES

pciaccess: provide a method to detect if a device is boot VGA

When the linux kernel exposes this information, we can use this interface
in the X server to detect whether the kernel believes the device we
are looking at is the boot VGA device.

Signed-off-by: Dave Airlie <airlied@redhat.com>

Version bump: 0.10.6

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Resync COPYING file with code copyright notices

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Fix unused variable warnings in Solaris code

solx_devfs.c: In function `find_target_node':
solx_devfs.c:672: warning: unused variable `prop'
solx_devfs.c:673: warning: unused variable `i'

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Fix $CWARNFLAGS addition

Original addition to configure.ac had typo (missing N) - moving to
Makefile.am allows easier override at build time

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Move included m4 to a subdir so we can more easily update from upstream

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

openbsd_pci.c: use the correct size when mapping the legacy vga rom.

fixes errors (and probably bugs) on intel hardware (at the least).

Signed-off-by: Matthieu Herrb <matthieu.herrb@laas.fr>

Sun bug 6811468: pci_device_solx_devfs_probe accesses freed memory

di_fini() is being called in pci_device_solx_devfs_probe()
The di_fini (3DEVINFO) man page says "All  handles associated with this
snapshot become invalid after the  call to di_fini()".  But after that,
eight lines down, the subroutine was calling di_prop_lookup_ints with
a handle args.node which was stored from walking the device tree, and
then using the pointers that returned even further down.

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>