Pauli Nieminen [Tue, 18 Aug 2009 15:51:37 +0000 (18:51 +0300)]
libdrm_radeon: Fix loops so that compiler can optimize them.
GCC did war about optimization not possible because possible forever loop.
Signed-off-by: Pauli Nieminen <suokkos@gmail.com>
Pauli Nieminen [Fri, 7 Aug 2009 17:03:26 +0000 (20:03 +0300)]
libdrm/radeon: Update head of linked list not to point freed memory.
Signed-off-by: Pauli Nieminen <suokkos@gmail.com>
Ben Skeggs [Wed, 12 Aug 2009 04:21:00 +0000 (14:21 +1000)]
nouveau: support for copy-less pushbuf ioctl
Ben Skeggs [Tue, 18 Aug 2009 04:25:50 +0000 (14:25 +1000)]
nouveau: for the moment, assert if we exceed some reloc limits
Nasty, but nicer than silently not writing into the pushbuf
Dave Airlie [Mon, 17 Aug 2009 11:21:02 +0000 (21:21 +1000)]
radeon: fix bo wait at map time.
Christoph Brill [Sun, 16 Aug 2009 06:26:25 +0000 (08:26 +0200)]
Filter radeon pkgconfig file as do intel and nouveau
Dave Airlie [Sat, 15 Aug 2009 11:32:35 +0000 (21:32 +1000)]
radeon: fix GTT writing space check
Noticed by vehemens on irc.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Anssi Hannula [Fri, 24 Jul 2009 18:58:43 +0000 (21:58 +0300)]
link libdrm_intel with -lrt for new use of clock_gettime().
Signed-off-by: Eric Anholt <eric@anholt.net>
Dave Airlie [Sat, 1 Aug 2009 07:19:43 +0000 (17:19 +1000)]
libdrm_radeon: add tiling support
Dave Airlie [Sat, 1 Aug 2009 06:47:24 +0000 (16:47 +1000)]
radeon: don't map/unmap explicitly.
This caches the mapping and just use mapping as a sync point
Ben Skeggs [Mon, 27 Jul 2009 22:12:21 +0000 (08:12 +1000)]
nouveau: don't overwrite user-specified bo size needlessly
Ben Skeggs [Mon, 27 Jul 2009 21:46:20 +0000 (07:46 +1000)]
nouveau: user buffers need to be mappable
Ben Skeggs [Sun, 26 Jul 2009 21:23:09 +0000 (07:23 +1000)]
nouveau: drm api 0.0.15, update object header, remove fake bo support
Maarten Maathuis [Mon, 27 Jul 2009 17:24:13 +0000 (19:24 +0200)]
libdrm/nouveau: unmap before ufree means nvbo->sysmem != NULL, which inbalances cpu_prep/cpu_finish
- The bo was mapped with sysmem == NULL, so this means cpu prep is called.
- The bo was unmapped with sysmem != NULL, so this means cpu finish is not called.
- This can lead to a non-zero "cpu writers" count in ttm_bo.
Alex Deucher [Wed, 15 Jul 2009 18:51:38 +0000 (14:51 -0400)]
Add RADEON_INIT_R600_CP too
this rounds out the r6xx/r7xx stuff in radeon_drm.h
Alex Deucher [Wed, 15 Jul 2009 18:40:41 +0000 (14:40 -0400)]
Add R600_SCRATCH_REG_OFFSET to radeon_drm.h
This lets us libdrm from master for r6xx/r7xx 3d
Eric Anholt [Wed, 15 Jul 2009 17:04:44 +0000 (10:04 -0700)]
Bump to version 2.4.12 for release.
Eric Anholt [Fri, 10 Jul 2009 00:49:46 +0000 (17:49 -0700)]
intel: Free buffers in the BO cache that haven't been reused in a while.
The goal of the BO cache is to keep buffers on hand for fast continuous use,
as in every frame of a game or every batchbuffer of the X Server. Keeping
older buffers on hand not only doesn't serve this purpose, it may hurt
performance by resulting in disk cache getting kicked out, or even driving
the system to swap.
Bug #20766.
Dave Airlie [Thu, 9 Jul 2009 16:38:13 +0000 (02:38 +1000)]
configure: move AC_USE_SYSTEM_EXTENSIONS up higher
this was giving me a warning on automake 1.11
Ben Skeggs [Thu, 9 Jul 2009 01:41:47 +0000 (11:41 +1000)]
nouveau: fix pin for buffers created with bo_wrap
Ian Romanick [Mon, 6 Jul 2009 22:36:20 +0000 (15:36 -0700)]
s/AC_USE_SYSTEM_MACROS/AC_USE_SYSTEM_EXTENSIONS/
Not sure what I was thinking.
Pauli Nieminen [Mon, 6 Jul 2009 20:37:20 +0000 (23:37 +0300)]
libdrm: Make chown check for return value
If call was interrupted by signal we have to make call again.
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Ian Romanick [Mon, 6 Jul 2009 20:23:46 +0000 (13:23 -0700)]
libdrm: Set _XOPEN_SOURCE and _GNU_SOURCE
Several POSIX extensions are used in the libdrm code (e.g., mknod and ffs).
Set _XOPEN_SOURCE and _GNU_SOURCE to something reasonable to ensure that
prototypes for these functions are available. This is done in configure.ac
using AC_USE_SYSTEM_MACROS. This requires autoconf 2.60 or later. Eventually
the code should check for the existance of these defines and do something
reasonable if they are not available.
Inspired by a patch by Pauli Nieminen and suggestions from Julien Cristau.
Thanks.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Eric Anholt [Mon, 6 Jul 2009 18:55:28 +0000 (11:55 -0700)]
intel: Fix up math errors when allocating very large BOs.
The logbase2 would overflow and wrap the size around to 0, making the code
allocate a 4kb object instead. By simplifying the code to just walk the
14-entry bucket array comparing sizes instead of indexing on
ffs(1 << logbase2(size)), we avoid silly math errors and have code of
approximately the same speed.
Many thanks to Simon Farnsworth for debugging and providing a working patch.
Bug #27365.
Edward O'Callaghan [Thu, 2 Jul 2009 18:03:48 +0000 (11:03 -0700)]
intel: fix compiler warning about returning a void value.
bug #21999
Pauli Nieminen [Fri, 3 Jul 2009 23:18:51 +0000 (02:18 +0300)]
libdrm: Fix pci id numbers to unsigned so code won't mix signed/unsigned integers.
Ian Romanick [Mon, 6 Jul 2009 16:23:59 +0000 (09:23 -0700)]
strcasecmp is in strings.h.
Based on patch by Pauli Nieminen. Thanks.
Pauli Nieminen [Fri, 3 Jul 2009 23:18:49 +0000 (02:18 +0300)]
libdrm: Remove typedef for non-existing enum
Dave Airlie [Mon, 6 Jul 2009 03:34:24 +0000 (13:34 +1000)]
radeon: move cs space checking code to libdrm_radeon.
This ports a lot of the space checking code into a the common
library, so that the DDX and mesa can use it.
Jerome Glisse [Fri, 3 Jul 2009 13:03:03 +0000 (15:03 +0200)]
radeon: fix cs buffer realloc padding
We always realloc at least 0x1000 dwords (page on most system)
when growing the cs buffer this is to avoid having to realloc
at each cs_begin.
Dave Airlie [Fri, 3 Jul 2009 11:13:22 +0000 (21:13 +1000)]
radeon: fix realloc of packets.
This should use ndw not cdw, using cdw leads to realloc alignment going wrong
Dave Airlie [Thu, 2 Jul 2009 05:42:01 +0000 (15:42 +1000)]
radeon: add buffer naming function
Dave Airlie [Tue, 30 Jun 2009 02:19:28 +0000 (12:19 +1000)]
radeon: add support for write followed by read relocs.
the DDX does this and used to handle it internally
Dave Airlie [Mon, 22 Jun 2009 23:51:05 +0000 (09:51 +1000)]
remove some old ttm bits
Thomas Hellstrom [Mon, 22 Jun 2009 21:07:26 +0000 (23:07 +0200)]
drm: Strip old ttm.
Signed-off-by: Thomas Hellstrom <thellstrom-at-vmware-dot-com>
Jesse Barnes [Mon, 22 Jun 2009 19:27:49 +0000 (12:27 -0700)]
modetest: use gtt unmap for object
Match gtt map/unmap for correctness.
Dave Airlie [Wed, 17 Jun 2009 07:47:42 +0000 (17:47 +1000)]
libdrm/radeon: add initial libdrm_radeon
requires --enable-radeon-experimental-api for now
Ben Skeggs [Wed, 3 Jun 2009 03:54:43 +0000 (13:54 +1000)]
nouveau: 0.0.14 + extend bo interface to support subrange mapping
Normal map() should operate as before, and map_range()/map_flush() should
give correct results but lacking any performance difference from map().
Nothing exiting being done here yet, but the interface is a good start.
Ben Skeggs [Wed, 3 Jun 2009 22:59:53 +0000 (08:59 +1000)]
nouveau: fix bo mapping issue
Pekka Paalanen [Wed, 3 Jun 2009 19:26:43 +0000 (22:26 +0300)]
Fix build on kernels < 2.6.26
The commit
651e3dc6dd58a79c90db7513ee2fb28360a4560d, "drm: Fix
compilation on 2.6.30" broke all builds on kernels before 2.6.26.
Variadic functions cannot be inlined.
The variadic inline function is replaced with a variadic macro.
Also, the function dev_name() added by that same commit is never used,
so it is removed.
Signed-off-by: Pekka Paalanen <pq@iki.fi>
Ben Skeggs [Wed, 3 Jun 2009 00:24:30 +0000 (10:24 +1000)]
nouveau: fill in bo->tiled field for referenced buffers
Fixes the dri1 gallium driver if the front buffer happens to be non-linear.
Ben Skeggs [Tue, 2 Jun 2009 07:06:34 +0000 (17:06 +1000)]
nv50: magic fix for "3d busted without the binary driver first" issue
No idea what this reg does.. NVIDIA put 0x802 into it on some chips, the
criteria aren't too clear on when they do that however. Hopefully 0x800
will work everywhere...
Michael Buesch [Sun, 10 May 2009 22:21:14 +0000 (00:21 +0200)]
drm: Fix compilation on 2.6.30
This fixes DRM compilation of nouveau.ko on kernel 2.6.30.
Signed-off-by: Michael Buesch <mb@bu3sch.de>
Ben Skeggs [Tue, 26 May 2009 04:14:04 +0000 (14:14 +1000)]
nouveau: bump for 0.0.13
Danny Tholen [Wed, 6 May 2009 23:11:34 +0000 (00:11 +0100)]
nouveau: backlight support for ppc laptops
Patch allows the backlight to be manipulated under gnome on apple powerpc
based NV30 machines. It works fine on my powerbook, and should also work
for older NV17/NV18 machines.
Note that older powerpc specific tools (pbbuttonsd) have some problems with
this implementation (because the device is not yet there at the start time
of the daemon, and the code makes incorrect assumptions about the max
brightness values). However, IMHO these things can and should be addressed
in the daemon.
Some style/warning fixes applied by Stuart Bennett <stuart@freedesktop.org>
Eric Anholt [Tue, 19 May 2009 17:07:16 +0000 (10:07 -0700)]
intel: quiet a warning.
Eric Anholt [Mon, 18 May 2009 23:07:45 +0000 (16:07 -0700)]
intel: Only do BO caching up to 64MB objects.
This avoids making objects significantly bigger than they would be
otherwise, which would result in some failing at binding to the GTT.
Found from firefox hanging on:
http://upload.wikimedia.org/wikipedia/commons/b/b7/Singapore_port_panorama.jpg
due to a software fallback trying to do a GTT-mapped copy between two 73MB
BOs that were instead each 128MB, and failing because both couldn't fit
simultaneously.
The cost here is that we get no opportunity to cache these objects and
avoid the mapping. But since the objects are a significant percentage
of the aperture size, each mapped access is likely having to fault and rebind
the object most of the time anyway.
Bug #20152 (2/3)
Keith Packard [Thu, 14 May 2009 23:58:14 +0000 (16:58 -0700)]
libdrm/intel: Make get_pipe_from_crtc_id per-bufmgr. Return -1 on failure.
The convention is that all APIs are per-bufmgr, so make this one the same.
Then, have it return -1 on failure so that the application can know what's
going on and do something sensible.
Signed-off-by: Keith Packard <keithp@keithp.com>
Carl Worth [Wed, 29 Apr 2009 21:43:55 +0000 (14:43 -0700)]
Add new drm_intel_get_pipe_from_crtc_id function.
This wraps the new DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID ioctl,
allowing applications to discover the pipe number corresponding
to a given CRTC ID. This is necessary for doing pipe-specific
operations such as waiting for vblank on a given CRTC.
Keith Packard [Tue, 12 May 2009 22:33:28 +0000 (15:33 -0700)]
Bump to version 2.4.10 to mark addition of drm_intel_bo_disable_reuse
New API, new version.
Signed-off-by: Keith Packard <keithp@keithp.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Keith Packard [Mon, 11 May 2009 20:42:12 +0000 (13:42 -0700)]
libdrm/intel: add drm_intel_bo_disable_reuse api
Scanout buffers need to be freed through the kernel as it holds a reference
to them; exposing this API allows applications allocating scanout buffers to
flag them as not reusable.
Signed-off-by: Keith Packard <keithp@keithp.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Alan Coopersmith [Tue, 10 Jun 2008 00:03:28 +0000 (17:03 -0700)]
Use C99 versions of __FUNCTION__ & __volatile__ when not building with gcc
Alan Coopersmith [Sun, 10 May 2009 00:21:29 +0000 (17:21 -0700)]
Delete extra libdrm_lists.h line after libdrm_la_SOURCES in libdrm/Makefile.am
Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>
Stuart Bennett [Wed, 6 May 2009 22:56:51 +0000 (23:56 +0100)]
nouveau: backlight compat fix
Avoid failure to build on old kernels with CONFIG_BACKLIGHT_CLASS_DEVICE set
Danny Tholen [Wed, 6 May 2009 22:41:09 +0000 (23:41 +0100)]
nouveau: fix ppc for swab changes in Linux 2.6.29
Keith Packard [Sat, 2 May 2009 01:47:04 +0000 (18:47 -0700)]
libdrm/intel: assert that clients are using bo refcounting correctly
Add assertions to drm_intel_gem_bo_reference,
drm_intel_gem_bo_reference_locked and drm_intel_gem_bo_unreference_locked
that the object has not been freed (refcount > 0). Mistakes in refcounting
lead to attempts to insert a bo into a free list more than once which causes
application failure as empty free lists are dereferenced as buffer objects.
Signed-off-by: Keith Packard <keithp@keithp.com>
Jesse Barnes [Wed, 22 Apr 2009 00:13:16 +0000 (17:13 -0700)]
intel: NULL fake bo block when freeing in evict_all
Fixes assertion failures on later use of the object.
Dave Airlie [Wed, 22 Apr 2009 10:25:40 +0000 (20:25 +1000)]
libdrm/mode: align subpixel results
Ben Skeggs [Thu, 16 Apr 2009 23:29:19 +0000 (09:29 +1000)]
nouveau: write posting got lost somewhere, bring it back
Ben Skeggs [Wed, 15 Apr 2009 02:44:21 +0000 (12:44 +1000)]
nouveau: store bo handle in public struct in bo_ref_handle
Jesse Barnes [Fri, 10 Apr 2009 22:34:24 +0000 (15:34 -0700)]
Bump version to 2.4.9
Who needs 2.4.8 anyway?
Jesse Barnes [Fri, 10 Apr 2009 22:24:41 +0000 (15:24 -0700)]
Bump version to 2.4.8
Jesse Barnes [Fri, 10 Apr 2009 22:24:10 +0000 (15:24 -0700)]
Revert "libdrm: speed up connector & mode fetching"
This reverts commit
cd5c66c659168cbe2e3229ebf8be79f764ed0ee1. It broke too
many kernel assumptions about the double ioctl (connector status, mode
fetching, etc.)
Kristian Høgsberg [Thu, 9 Apr 2009 19:19:10 +0000 (15:19 -0400)]
Bump version to 2.4.7
Kristian Høgsberg [Thu, 9 Apr 2009 19:17:51 +0000 (15:17 -0400)]
test: Avoid recursive dependency in makefile
Kristian Høgsberg [Thu, 9 Apr 2009 19:02:40 +0000 (15:02 -0400)]
test: Makefile.am grammar nazi
Dave Airlie [Mon, 6 Apr 2009 23:26:32 +0000 (09:26 +1000)]
drm: fix test makefile
no idea if this is correct but it works so meh
Kristian Høgsberg [Mon, 6 Apr 2009 21:22:10 +0000 (17:22 -0400)]
Skip tests that require root
This lets us do make distcheck as non-root.
Kristian Høgsberg [Mon, 6 Apr 2009 21:18:17 +0000 (17:18 -0400)]
XFAIL auth and lock test cases
They're writing to the read end of a pipe and failing.
Kristian Høgsberg [Mon, 6 Apr 2009 21:13:01 +0000 (17:13 -0400)]
Use libudev in test case to only run gem tests for intel devices.
Robert Noland [Tue, 31 Mar 2009 18:33:18 +0000 (13:33 -0500)]
Move drm_vblank_cleanup() after lastclose.
This may prevent a possible panic on shutdown.
Jesse Barnes [Mon, 30 Mar 2009 21:35:30 +0000 (14:35 -0700)]
libdrm: speed up connector & mode fetching
This patch speeds up drmModeGetConnector by pre-allocating mode &
property info space before calling into the kernel. In many cases this
pre-allocation will be sufficient to hold the returned values (it's easy
enough to tweak if the common case becomes larger), which means we don't
have to make the second call, which saves a lot of time.
Acked-by: Jakob Bornecrantz <wallbraker@gmail.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Stuart Bennett [Mon, 30 Mar 2009 19:38:49 +0000 (20:38 +0100)]
nouveau: nForce 1/2 IGP memory sanity check from DDX
Jesse Barnes [Mon, 30 Mar 2009 18:25:09 +0000 (11:25 -0700)]
libdrm: bump version to 2.4.6
This version includes GTT unmap support for the Intel bufmgr.
Jesse Barnes [Thu, 26 Mar 2009 23:43:00 +0000 (16:43 -0700)]
libdrm/intel: support GTT maps correctly
libdrm has some support for GTT mapping already, but there are bugs
with it (no surprise since it hasn't been used much).
In fixing 20803, I found that sharing bo_gem->virtual was a bad idea,
since a previously mapped object might not end up getting GTT mapped,
leading to corruption. So this patch splits the fields according to
use, taking care to unmap both at free time (but preserving the map
caching).
There's still a risk we might run out of mappings (there's a sysctl
tunable for max number of mappings per process, defaulted to 64k or so
it looks like) but at least GTT maps will work with these changes (and
some others for fixing PAT breakage in the kernel).
Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Stuart Bennett [Tue, 24 Mar 2009 23:39:41 +0000 (23:39 +0000)]
nouveau: plug drm fifo ioremap leak (#14941)
Stuart Bennett [Tue, 24 Mar 2009 23:26:30 +0000 (23:26 +0000)]
nouveau: add linux compat defines for PCI config access, use them for nvidia IGPs
Stuart Bennett [Tue, 24 Mar 2009 16:42:36 +0000 (16:42 +0000)]
nouveau: use PFB_CSTATUS naming from ddx (reg introduced with nv10)
NV04 had a PFB_FIFO_DATA at the same address, which we don't use, so
remove it to reduce confusion
Ben Skeggs [Tue, 24 Mar 2009 22:10:12 +0000 (08:10 +1000)]
libdrm/nouveau: fix typo in nouveau_device_close()
Ben Skeggs [Fri, 20 Mar 2009 00:35:55 +0000 (10:35 +1000)]
libdrm/nouveau: ask the kernel for vram/gart aperture sizes
Ben Skeggs [Fri, 20 Mar 2009 00:32:47 +0000 (10:32 +1000)]
libdrm/nouveau: remove unneccesary null ptr checks
Ben Skeggs [Fri, 20 Mar 2009 00:24:40 +0000 (10:24 +1000)]
libdrm/nouveau: incr refcount on ref fence before decr on old fence
Maarten Maathuis [Thu, 19 Mar 2009 22:59:42 +0000 (23:59 +0100)]
libdrm/nouveau: unref fences when deleting bo
- This was causing a significant memory leak.
Ben Skeggs [Wed, 18 Mar 2009 00:25:22 +0000 (10:25 +1000)]
libdrm/nouveau: implement bo_handle_get for !mm_enabled
bo_handle_ref on !mm_enabled treats handle as an offset, make
bo_handle_get do the same rather than failing.
Robert Noland [Sun, 15 Mar 2009 18:52:58 +0000 (13:52 -0500)]
libdrm_nouveau: The handle that is passed to mmap needs to be drm_handle_t
drm_handle_t is defined to be a u32 on linux and a u64 on everything
else. This addresses an issue on FreeBSD amd64 where the map offsets
may be greater than 32bits. When the handle is cast to 32bit, mmap
cannot match the requested map and causes X to crash.
This should be a NOOP on linux since drm_handle_t is always 32bit.
Signed-off-by: Robert Noland <rnoland@2hip.net>
Robert Noland [Mon, 16 Mar 2009 06:07:15 +0000 (01:07 -0500)]
FreeBSD: Set up the nouveau build infrastructure
disabled by default until the rest of the patches are in.
Robert Noland [Mon, 16 Mar 2009 05:41:23 +0000 (00:41 -0500)]
FreeBSD: Add support for matching solely on vedor id.
This also adds that ability to set device name from VPD, but that
doesn't seem to be working...
Robert Noland [Mon, 16 Mar 2009 05:35:18 +0000 (00:35 -0500)]
FreeBSD: Improve the debug output of drm_mmap().
Robert Noland [Mon, 16 Mar 2009 05:30:28 +0000 (00:30 -0500)]
FreeBSD: Add list_for_each_prev() to our bank of compat functions.
Robert Noland [Mon, 16 Mar 2009 05:17:54 +0000 (00:17 -0500)]
FreeBSD: Don't set the PZERO flag to mtx_sleep.
We also don't support anything old enough to need tsleep.
Robert Noland [Mon, 16 Mar 2009 05:08:06 +0000 (00:08 -0500)]
FreeBSD: use flsl() instead of ffsl().
I noticed that we were computing drm_order differently than linux.
Robert Noland [Mon, 16 Mar 2009 05:07:31 +0000 (00:07 -0500)]
FreeBSD: Minor code cleanup.
Robert Noland [Mon, 16 Mar 2009 05:03:40 +0000 (00:03 -0500)]
FreeBSD: Increase MAX_PCI_RESOURCE
We can have more than 3 BARs to access.
Robert Noland [Mon, 16 Mar 2009 05:01:47 +0000 (00:01 -0500)]
FreeBSD: Cast map handles to vm_offset_t
This prevents some warnings with nouveau.
Robert Noland [Mon, 9 Mar 2009 18:37:57 +0000 (13:37 -0500)]
FreeBSD: Fix the printing of maps on amd64/i386 to be consistent
Robert Noland [Mon, 9 Mar 2009 18:34:26 +0000 (13:34 -0500)]
FreeBSD: Fix up the flags for bus_dmamem here as well.
Robert Noland [Mon, 9 Mar 2009 03:55:01 +0000 (22:55 -0500)]
FreeBSD: Fix up the flags to bus_dmamem_*
Allow it to sleep waiting for resources during the allocation stage.
Only use BUS_DMA_NOWAIT when loading the map.
vehemens [Mon, 9 Mar 2009 01:17:15 +0000 (18:17 -0700)]
FreeBSD use kdev for kernel device name
Signed-off-by: Robert Noland <rnoland@2hip.net>
Ben Skeggs [Thu, 5 Mar 2009 23:04:44 +0000 (09:04 +1000)]
drm/nouveau: make portion of vram as reserved for PRAMIN on all chipsets
NV04 was completely busted. Push buffers were getting allocated at the
end of VRAM, overwriting PRAMIN. So, it turns out PRAMIN is in VRAM on
all chips. Question answered!
Robert Noland [Thu, 5 Mar 2009 06:47:26 +0000 (00:47 -0600)]
FreeBSD: Rework DRM_[DEBUG,ERROR,INFO] macros a bit.