Solaris: probe improvement

Remove pcitool dependency in probing phase. Use the data
collected from devinfo tree instead in creating pci file
system.

Signed-off-by: Henry Zhao <henzhao@glory3.(none)>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Solaris: Add domain support for sparc platform

As a result the code of finding nexus node for a device in sparc is
simplified and made the same as x86.

Signed-off-by: Henry Zhao <henry.zhao@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

autogen.sh: Implement GNOME Build API

http://people.gnome.org/~walters/docs/build-api.txt

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

configure: Remove AM_MAINTAINER_MODE

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

PCI: set correct mask value when matching for bridges.

The mask must not be zero otherwise the matching condition will never
be true: ((val & mask) == set).

Signed-off-by: Egbert Eich <eich@freedesktop.org>
Reviewed-by: Guillem Jover <guillem@hadrons.org>

scanpci: print meaningful info on BASEROM

Signed-off-by: Henry Zhao <henry.zhao@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

libpciaccess: sparc rom read fix

Get rom address and size from device's "assigned-addresses" property
if they exist.

Signed-off-by: Henry Zhao <henry.zhao@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

libpciaccess: implementation of boot_vga in Solaris

Signed-off-by: Henry Zhao <henry.zhao@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

libpciaccess: fix memory usage errors

This patch fixes memory usage errors that only occur on large SPARC
machines with more than 256 PCI devices. In this case, memory is being
used after it has been freed by calls to both free() and realloc().
This error was introduced by a previous patch:
2011-03-30 Solaris support for multiple PCI segments (domains)

Signed-off-by: Mark Logan <mark.logan@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Implement legacy io & map for x86 backend

Add the legacy io and map methods for the x86 backend, using ioperm,
in/out, and the existing mmap method.

Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

Solaris: refactor pci_device_solx_devfs_map_range to reduce code duplication

The sparc & x86 cases were doing essentially the same things with
different paths, so make the path setup be inside the platform
specific #ifdefs, make the open, mmap, & error handling common code.

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

Solaris: Implement map_legacy & legacy_io functions

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jay Cotton <jay.cotton@oracle.com>

Bump to 0.13.1

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

Fix pci_io_handle allocation routines.

The previous version of new_io_handle()  would invalidate all previous
allocations when realloc moves the base pointer of the ios array.
Since I cannot figure out where this array is useful, just get rid of
it, providing sound stable memory allocation.

Fixes vgaHWSaveFonts() in 1.12 xserver on OpenBSD, but other
sub-systems relaying on pci_io could be affected too.

Signed-off-by: Matthieu Herrb <matthieu.herrb@laas.fr>
Tested-by: Tormod Volden <debian.tormod@gmail.com>

Fix openbsd_pci.c:pci_device_vgaarb_set_target();

A return (0) got lost, add it back.

Signed-off-by: Matthieu Herrb <matthieu.herrb@laas.fr>
Reviewed-by: Mark Kettenis <mark.kettenis@xs4all.nl>

linux: Fix out[bwl] macros on non-I/O architectures

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

Merge branch 'master' of ssh://git.freedesktop.org/git/xorg/lib/libpciaccess

configure.ac: Bump to 0.13

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

Use O_CLOEXEC for internal file descriptors

Well, almost all of them.  zlib doesn't support "e" in the mode string
in gzopen() though it will silently accept and ignore it, and Solaris appears
not to support "e" in the mode string at all.

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

linux: Don't use /dev/port

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

Use correct type for pci_id file accessor functions

This fixes the compiler warning of using the wrong type for gzgets() and
gzclose() as they want a gzFile argument, not a pointer to gzFile. The
abstraction layer pci_id_file should just abstract the full type.

Signed-off-by: Mathias Krause <mathias.krause@secunet.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

solx_devfs.c: fix gcc warnings about casting away const when reading data

solx_devfs.c: In function `pci_device_solx_devfs_write':
solx_devfs.c:1085: warning: cast discards qualifiers from pointer target type
solx_devfs.c:1089: warning: cast discards qualifiers from pointer target type
solx_devfs.c:1093: warning: cast discards qualifiers from pointer target type
solx_devfs.c:1097: warning: cast discards qualifiers from pointer target type

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

Close pci.ids file when bailing out because realloc failed

Error: File Leak
   Leaked File f
      at line 272 of src/common_device_name.c in function 'populate_vendor'.
        f initialized at line 204 with fopen("/usr/share/hwdata/pci.ids", "r").

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

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

Solaris: reorder functions to remove need for lots of static prototypes

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

OpenBSD: Implement map_legacy and legacy_io

Signed-off-by: Mark Kettenis <kettenis@openbsd.org>
Reviewed-by: Matthieu Herrb <matthieu.herrb@laas.fr>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Tested-by: Matthieu Herrb <matthieu.herrb@laas.fr>

delete_io_handle: fix deletion of last handle

When num_ios goes from 1 to 0, a realloc(ios, 0); call is made.
This is equivalent to free(ios) and NULL is returned.

However, the previous logic in the code incorrectly discards this NULL
return value. When we next call new_io_handle(), realloc(ios, X) is
called with "ios" pointing to freed memory. This causes glibc to abort.

Correct this logic to detect the 1-to-0 case and handle it correctly.
Other cases are unchanged; there is still value in checking the
return value from realloc() as it also returns NULL on error.

Signed-off-by: Daniel Drake <dsd@laptop.org>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Add VGA Arbiter support for OpenBSD.

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

OpenBSD: Indicate that devices need access to legacy VGA resources.

Makes sure xserver disables DRI if the VGA arbiter is in use.

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

Solaris: improve support for sparc platform

(1) added prom property retrieval on sparc
(2) added multiple domain support on sparc
(3) use kernel device as mapping device
(4) performance improvements by removing redundant ioctl

Signed-off-by: Henry Zhao <henry.zhao@oracle.com>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

configure.ac: Bump to 0.12.902

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

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