]> git.proxmox.com Git - mirror_spl-debian.git/log
mirror_spl-debian.git
14 years agoPrep for 0.4.7 tag, updated META and ChangeLog.
Brian Behlendorf [Sat, 21 Nov 2009 00:52:29 +0000 (16:52 -0800)]
Prep for 0.4.7 tag, updated META and ChangeLog.

14 years agoEnsure *.order and *.markers build products are removed by distclean rule.
Brian Behlendorf [Sat, 21 Nov 2009 00:01:00 +0000 (16:01 -0800)]
Ensure *.order and *.markers build products are removed by distclean rule.

14 years agoEnsure spl_config.h is include in spl-generic.c
Brian Behlendorf [Sun, 15 Nov 2009 23:04:33 +0000 (15:04 -0800)]
Ensure spl_config.h is include in spl-generic.c

14 years agoAlways use the generic mutex_destroy().
Brian Behlendorf [Sun, 15 Nov 2009 23:04:02 +0000 (15:04 -0800)]
Always use the generic mutex_destroy().

14 years agoAdd mutex_enter_nested() as wrapper for mutex_lock_nested().
Brian Behlendorf [Sun, 15 Nov 2009 22:27:15 +0000 (14:27 -0800)]
Add mutex_enter_nested() as wrapper for mutex_lock_nested().

This symbol can be used by GPL modules which use the SPL to handle
cases where a call path takes a two different locks by the same
name.  This is needed to avoid a false positive in the lock checker.

14 years agoLinux 2.6.31 kmem cache alignment fixes and cleanup.
Brian Behlendorf [Fri, 13 Nov 2009 19:12:43 +0000 (11:12 -0800)]
Linux 2.6.31 kmem cache alignment fixes and cleanup.

The big fix here is the removal of kmalloc() in kv_alloc().  It used
to be true in previous kernels that kmallocs over PAGE_SIZE would
always be pages aligned.  This is no longer true atleast in 2.6.31
there are no longer any alignment expectations.  Since kv_alloc()
requires the resulting address to be page align we no only either
directly allocate pages in the KMC_KMEM case, or directly call
__vmalloc() both of which will always return a page aligned address.
Additionally, to avoid wasting memory size is always a power of two.

As for cleanup several helper functions were introduced to calculate
the aligned sizes of various data structures.  This helps ensure no
case is accidentally missed where the alignment needs to be taken in
to account.  The helpers now use P2ROUNDUP_TYPE instead of P2ROUNDUP
which is safer since the type will be explict and we no longer count
on the compiler to auto promote types hopefully as we expected.

Always wnforce minimum (SPL_KMEM_CACHE_ALIGN) and maximum (PAGE_SIZE)
alignment restrictions at cache creation time.

Use SPL_KMEM_CACHE_ALIGN in splat alignment test.

14 years agoRemove __GFP_NOFAIL in kmem and retry internally.
Brian Behlendorf [Thu, 12 Nov 2009 23:11:24 +0000 (15:11 -0800)]
Remove __GFP_NOFAIL in kmem and retry internally.

As of 2.6.31 it's clear __GFP_NOFAIL should no longer be used and it
may disappear from the kernel at any time.  To handle this I have simply
added *_nofail wrappers in the kmem implementation which perform the
retry for non-atomic allocations.

From linux-2.6.31 mm/page_alloc.c:1166
/*
 * __GFP_NOFAIL is not to be used in new code.
 *
 * All __GFP_NOFAIL callers should be fixed so that they
 * properly detect and handle allocation failures.
 *
 * We most definitely don't want callers attempting to
 * allocate greater than order-1 page units with
 * __GFP_NOFAIL.
 */
WARN_ON_ONCE(order > 1);

14 years agoLinux 2.6.31 Compatibility Updates
Brian Behlendorf [Tue, 10 Nov 2009 22:06:57 +0000 (14:06 -0800)]
Linux 2.6.31 Compatibility Updates

SPL_AC_2ARGS_SET_FS_PWD macro updated to explicitly include
linux/fs_struct.h which was dropped from linux/sched.h.

min_wmark_pages, low_wmark_pages, high_wmark_pages macros
introduced in newer kernels.  For older kernels mm_compat.h
was introduced to define them as needed as direct mappings
to per zone min_pages, low_pages, max_pages.

14 years agoPrep for 0.4.6 tag, updated META and ChangeLog.
Brian Behlendorf [Mon, 2 Nov 2009 18:24:12 +0000 (10:24 -0800)]
Prep for 0.4.6 tag, updated META and ChangeLog.

14 years agoAutoconf --enable-debug-* cleanup
Brian Behlendorf [Fri, 30 Oct 2009 20:58:51 +0000 (13:58 -0700)]
Autoconf --enable-debug-* cleanup

Cleanup the --enable-debug-* configure options, this has been pending
for quite some time and I am glad I finally got to it.  To summerize:

1) All SPL_AC_DEBUG_* macros were updated to be a more autoconf
friendly.  This mainly involved shift to the GNU approved usage of
AC_ARG_ENABLE and ensuring AS_IF is used rather than directly using
an if [ test ] construct.

2) --enable-debug-kmem=yes by default.  This simply enabled keeping
a running tally of total memory allocated and freed and reporting a
memory leak if there was one at module unload.  Additionally, it
ensure /proc/spl/kmem/slab will exist by default which is handy.
The overhead is low for this and it should not impact performance.

3) --enable-debug-kmem-tracking=no by default.  This option was added
to provide a configure option to enable to detailed memory allocation
tracking.  This support was always there but you had to know where to
turn it on.  By default this support is disabled because it is known
to badly hurt performence, however it is invaluable when chasing a
memory leak.

4) --enable-debug-kstat removed.  After further reflection I can't see
why you would ever really want to turn this support off.  It is now
always on which had the nice side effect of simplifying the proc handling
code in spl-proc.c.  We can now always assume the top level directory
will be there.

5) --enable-debug-callb removed.  This never really did anything, it was
put in provisionally because it might have been needed.  It turns out
it was not so I am just removing it to prevent confusion.

14 years agoAdd autoconf checks for atomic64_cmpxchg + atomic64_xchg
Brian Behlendorf [Fri, 30 Oct 2009 20:53:17 +0000 (13:53 -0700)]
Add autoconf checks for atomic64_cmpxchg + atomic64_xchg

These functions didn't exist for all archs prior to 2.6.24.  This
patch addes an autoconf test to detect this and add them when needed.
The autoconf check is needed instead of just an #ifndef because in
the most modern kernels atomic64_{cmp}xchg are implemented as in
inline function and not a #define.

14 years agoUse Linux atomic primitives by default.
Brian Behlendorf [Fri, 30 Oct 2009 17:55:25 +0000 (10:55 -0700)]
Use Linux atomic primitives by default.

Previously Solaris style atomic primitives were implemented simply by
wrapping the desired operation in a global spinlock.  This was easy to
implement at the time when I wasn't 100% sure I could safely layer the
Solaris atomic primatives on the Linux counterparts.  It however was
likely not good for performance.

After more investigation however it does appear the Solaris primitives
can be layered on Linux's fairly safely.  The Linux atomic_t type really
just wraps a long so we can simply cast the Solaris unsigned value to
either a atomic_t or atomic64_t.  The only lingering problem for both
implementations is that Solaris provides no atomic read function.  This
means reading a 64-bit value on a 32-bit arch can (and will) result in
word breaking.  I was very concerned about this initially, but upon
further reflection it is a limitation of the Solaris API.  So really
we are just being bug-for-bug compatible here.

With this change the default implementation is layered on top of Linux
atomic types.  However, because we're assuming a lot about the internal
implementation of those types I've made it easy to fall-back to the
generic approach.  Simply build with --enable-atomic_spinlocks if
issues are encountered with the new implementation.

14 years agoI should not have removed these, they are important.
Brian Behlendorf [Tue, 27 Oct 2009 23:17:06 +0000 (16:17 -0700)]
I should not have removed these, they are important.

14 years agoRebase cmn_err on vcmn_err and don't warn about missing \n
Brian Behlendorf [Tue, 27 Oct 2009 22:54:45 +0000 (15:54 -0700)]
Rebase cmn_err on vcmn_err and don't warn about missing \n

The cmn_err/vcmn_err functions are layered on top of the debug
system which usually expects a newline at the end.  However, there
really doesn't need to be a newline there and there in fact should
not be for the CE_CONT case so let's just drop the warning.

Also we make a half-hearted attempt to handle a leading ! which
means only send it to the syslog not the console.  In this case
we just send to the the debug logs and not the console.

14 years agoRemove usage of the __id_u macro for portability.
Brian Behlendorf [Mon, 5 Oct 2009 19:51:58 +0000 (12:51 -0700)]
Remove usage of the __id_u macro for portability.

This macro was removed from the default RPM macro file.  Interestly,
some of the arch specific macro's add it back it based on your distro
but it should not be counted on.  However, __id still exists and its
command line args have historically been fairly stable so we will
directly use %{__id} -un to get the user name.

14 years agoUse kobject_set_name() for increased portability.
Brian Behlendorf [Fri, 2 Oct 2009 23:21:59 +0000 (16:21 -0700)]
Use kobject_set_name() for increased portability.

As of 2.6.25 kobj->k_name was replaced with kobj->name.  Some distros
such as RHEL5 (2.6.18) add a patch to prevent this from being a problem
but other older distros such as SLES10 (2.6.16) have not.  To avoid
the whole issue I'm updating the code to use kobject_set_name() which
does what I want and has existed all the way back to 2.6.11.

14 years agoSet cwd to '/' for the process executing insmod.
Brian Behlendorf [Thu, 1 Oct 2009 23:06:15 +0000 (16:06 -0700)]
Set cwd to '/' for the process executing insmod.

Ricardo has pointed out that under Solaris the cwd is set to '/'
during module load, while under Linux it is set to the callers cwd.
To handle this cleanly I've reworked the module *_init()/_exit()
macros so they call a *_setup()/_cleanup() function when any SPL
dependent module is loaded or unloaded.  This gives us a chance to
perform any needed modification of the process, in this case changing
the cwd.  It also handily provides a way to avoid creating wrapper
init()/exit() functions because the Solaris and Linux prototypes
differ slightly.  All dependent modules should now call the spl
helper macros spl_module_{init,exit}() instead of the native linux
versions.

Unfortunately, it appears that under Linux there has been no consistent
API in the kernel to set the cwd in a module.  Because of this I have
had to add more autoconf magic than I'd like.  However, what I have
done is correct and has been tested on RHEL5, SLES11, FC11, and CHAOS
kernels.

In addition, I have change the rootdir type from a 'void *' to the
correct 'vnode_t *' type.  And I've set rootdir to a non-NULL value.

14 years agoExpand SEM() outside init_rwsem and directly call __init_rwsem().
Brian Behlendorf [Tue, 29 Sep 2009 10:19:09 +0000 (03:19 -0700)]
Expand SEM() outside init_rwsem and directly call __init_rwsem().

We need to directly call __init_rwsem() or the name gets expanded
to SEM(lock-name).  This is safe and correct for the support arches
x86/x86_64/ppc/ppc64.

14 years agoReimplement mutexs for Linux lock profiling/analysis
Brian Behlendorf [Fri, 25 Sep 2009 21:47:01 +0000 (14:47 -0700)]
Reimplement mutexs for Linux lock profiling/analysis

For a generic explanation of why mutexs needed to be reimplemented
to work with the kernel lock profiling see commits:
  e811949a57044d60d12953c5c3b808a79a7d36ef and
  d28db80fd0fd4fd63aec09037c44408e51a222d6

The specific changes made to the mutex implemetation are as follows.
The Linux mutex structure is now directly embedded in the kmutex_t.
This allows a kmutex_t to be directly case to a mutex struct and
passed directly to the Linux primative.

Just like with the rwlocks it is critical that these functions be
implemented as '#defines to ensure the location information is
preserved.  The preprocessor can then do a direct replacement of
the Solaris primative with the linux primative.

Just as with the rwlocks we need to track the lock owner.  Here
things get a little more interesting because depending on your
kernel version, and how you've built your kernel Linux may already
do this for you.  If your running a 2.6.29 or newer kernel on a
SMP system the lock owner will be tracked.  This was added to Linux
to support adaptive mutexs, more on that shortly.  Alternately, your
kernel might track the lock owner if you've set CONFIG_DEBUG_MUTEXES
in the kernel build.  If neither of the above things is true for
your kernel the kmutex_t type will include and track the lock owner
to ensure correct behavior.  This is all handled by a new autoconf
check called SPL_AC_MUTEX_OWNER.

Concerning adaptive mutexs these are a very recent development and
they did not make it in to either the latest FC11 of SLES11 kernels.
Ideally, I'd love to see this kernel change appear in one of these
distros because it does help performance.  From Linux kernel commit:
  0d66bf6d3514b35eb6897629059443132992dbd7
  "Testing with Ingo's test-mutex application...
  gave a 345% boost for VFS scalability on my testbox"
However, if you don't want to backport this change yourself you
can still simply export the task_curr() symbol.  The kmutex_t
implementation will use this symbol when it's available to
provide it's own adaptive mutexs.

Finally, DEBUG_MUTEX support was removed including the proc handlers.
This was done because now that we are cleanly integrated with the
kernel profiling all this information and much much more is available
in debug kernel builds.  This code was now redundant.

Update mutexs validated on:
    - SLES10   (ppc64)
    - SLES11   (x86_64)
    - CHAOS4.2 (x86_64)
    - RHEL5.3  (x86_64)
    - RHEL6    (x86_64)
    - FC11     (x86_64)

14 years agoUpdate rwlocks to track owner to ensure correct semantics
Brian Behlendorf [Fri, 25 Sep 2009 21:14:35 +0000 (14:14 -0700)]
Update rwlocks to track owner to ensure correct semantics

The behavior of RW_*_HELD was updated because it was not quite right.
It is not sufficient to return non-zero when the lock is help, we must
only do this when the current task in the holder.

This means we need to track the lock owner which is not something
tracked in a Linux semaphore.  After some experimentation the
solution I settled on was to embed the Linux semaphore at the start
of a larger krwlock_t structure which includes the owner field.
This maintains good performance and allows us to cleanly intergrate
with the kernel lock analysis tools.  My reasons:

1) By placing the Linux semaphore at the start of krwlock_t we can
then simply cast krwlock_t to a rw_semaphore and pass that on to
the linux kernel.  This allows us to use '#defines so the preprocessor
can do direct replacement of the Solaris primative with the linux
equivilant.  This is important because it then maintains the location
information for each rw_* call point.

2) Additionally, by adding the owner to krwlock_t we can keep this
needed extra information adjacent to the lock itself.  This removes
the need for a fancy lookup to get the owner which is optimal for
performance.  We can also leverage the existing spin lock in the
semaphore to ensure owner is updated correctly.

3) All helper functions which do not need to strictly be implemented
as a define to preserve location information can be done as a static
inline function.

4) Adding the owner to krwlock_t allows us to remove all memory
allocations done during lock initialization.  This is good for all
the obvious reasons, we do give up the ability to specific the lock
name.  The Linux profiling tools will stringify the lock name used
in the code via the preprocessor and use that.

Update rwlocks validated on:
- SLES10   (ppc64)
- SLES11   (x86_64)
- CHAOS4.2 (x86_64)
- RHEL5.3  (x86_64)
- RHEL6    (x86_64)
- FC11     (x86_64)

14 years agoReimplement rwlocks for Linux lock profiling/analysis.
Brian Behlendorf [Fri, 18 Sep 2009 23:09:47 +0000 (16:09 -0700)]
Reimplement rwlocks for Linux lock profiling/analysis.

It turns out that the previous rwlock implementation worked well but
did not integrate properly with the upstream kernel lock profiling/
analysis tools.  This is a major problem since it would be awfully
nice to be able to use the automatic lock checker and profiler.

The problem is that the upstream lock tools use the pre-processor
to create a lock class for each uniquely named locked.  Since the
rwsem was embedded in a wrapper structure the name was always the
same.  The effect was that we only ended up with one lock class for
the entire SPL which caused the lock dependency checker to flag
nearly everything as a possible deadlock.

The solution was to directly map a krwlock to a Linux rwsem using
a typedef there by eliminating the wrapper structure.  This was not
done initially because the rwsem implementation is specific to the arch.
To fully implement the Solaris krwlock API using only the provided rwsem
API is not possible.  It can only be done by directly accessing some of
the internal data member of the rwsem structure.

For example, the Linux API provides a different function for dropping
a reader vs writer lock.  Whereas the Solaris API uses the same function
and the caller does not pass in what type of lock it is.  This means to
properly drop the lock we need to determine if the lock is currently a
reader or writer lock.  Then we need to call the proper Linux API function.
Unfortunately, there is no provided API for this so we must extracted this
information directly from arch specific lock implementation.  This is
all do able, and what I did, but it does complicate things considerably.

The good news is that in addition to the profiling benefits of this
change.  We may see performance improvements due to slightly reduced
overhead when creating rwlocks and manipulating them.

The only function I was forced to sacrafice was rw_owner() because this
information is simply not stored anywhere in the rwsem.  Luckily this
appears not to be a commonly used function on Solaris, and it is my
understanding it is mainly used for debugging anyway.

In addition to the core rwlock changes, extensive updates were made to
the rwlock regression tests.  Each class of test was extended to provide
more API coverage and to be more rigerous in checking for misbehavior.

This is a pretty significant change and with that in mind I have been
careful to validate it on several platforms before committing.  The full
SPLAT regression test suite was run numberous times on all of the following
platforms.  This includes various kernels ranging from 2.6.16 to 2.6.29.

- SLES10   (ppc64)
- SLES11   (x86_64)
- CHAOS4.2 (x86_64)
- RHEL5.3  (x86_64)
- RHEL6    (x86_64)
- FC11     (x86_64)

14 years agoVarious spec file tweaks to handle rpm building of several distros.
Brian Behlendorf [Fri, 14 Aug 2009 21:09:16 +0000 (14:09 -0700)]
Various spec file tweaks to handle rpm building of several distros.
Supported and tested distros now include SLES10, SLES11, Chaos 4.x,
RHEL5, and Fedora 11.  This update was mainly to address rebuildable
kernel module rpms, and correct rpm dependencies for each distro.

14 years agoExplicit check for requires_* rpm defines
Brian Behlendorf [Thu, 13 Aug 2009 22:02:34 +0000 (15:02 -0700)]
Explicit check for requires_* rpm defines
Due to different distros and/or versions of rpm mishandling the shorthand
syntax simply use the longer version which get interpreted correctly.

14 years agoTag spl-0.4.5.
Brian Behlendorf [Tue, 4 Aug 2009 19:22:33 +0000 (12:22 -0700)]
Tag spl-0.4.5.
Update the ChangeLog with a summary of the changes since the last release
and update the META file to reflect the new version number.

14 years agoRequired missing symbols for FC11 kernels (2.6.29.4-167.fc11.x86_64)
Brian Behlendorf [Fri, 31 Jul 2009 19:44:34 +0000 (12:44 -0700)]
Required missing symbols for FC11 kernels (2.6.29.4-167.fc11.x86_64)

14 years agoDisable stack overflow checking by default.
Brian Behlendorf [Thu, 30 Jul 2009 20:52:11 +0000 (13:52 -0700)]
Disable stack overflow checking by default.
The run time stack overflow checking is being disabled by default
because it is not safe for use with 2.6.29 and latter kernels.  These
kernels do now have their own stack overflow checking so this support
has become redundant anyway.  It can be re-enabled for older kernels or
arches without stack overflow checking by redefining CHECK_STACK().

14 years agoUpdate global_page_state() support for 2.6.29 kernels.
Brian Behlendorf [Tue, 28 Jul 2009 22:06:42 +0000 (15:06 -0700)]
Update global_page_state() support for 2.6.29 kernels.
Basically everything we need to monitor the global memory state of
the system is now cleanly available via global_page_state().  The
problem is that this interface is still fairly recent, and there
has been one change in the page state enum which we need to handle.
These changes basically boil down to the following:
- If global_page_state() is available we should use it.  Several
  autoconf checks have been added to detect the correct enum names.
- If global_page_state() is not available check to see if
  get_zone_counts() symbol is available and use that.
- If the get_zone_counts() symbol is not exported we have no choice
  be to dynamically aquire it at load time.  This is an absolute
  last resort for old kernel which we don't want to patch to
  cleanly export the symbol.

14 years agoRemove get/put_task_struct as they are not available for SLES11
Brian Behlendorf [Tue, 28 Jul 2009 22:04:21 +0000 (15:04 -0700)]
Remove get/put_task_struct as they are not available for SLES11
This interface is going away, and it's not as if most callers actually
use crhold/crfree when working with credentials.  So it'll be okay
they we're not taking a reference on the task structure the odds of
it going away while working with a credential and pretty small.

14 years agoAdd basic credential support and splat tests.
Brian Behlendorf [Tue, 28 Jul 2009 00:18:59 +0000 (17:18 -0700)]
Add basic credential support and splat tests.
The previous credential implementation simply provided the needed types and
a couple of dummy functions needed.  This update correctly ties the basic
Solaris credential API in to one of two Linux kernel APIs.

Prior to 2.6.29 the linux kernel embeded all credentials in the task
structure.  For these kernels, we pass around the entire task struct as if
it were the credential, then we use the helper functions to extract the
credential related bits.

As of 2.6.29 a new credential type was added which we can and do fairly
cleanly layer on top of.  Once again the helper functions nicely hide
the implementation details from all callers.

Three tests were added to the splat test framework to verify basic
correctness.  They should be extended as needed when need credential
functions are added.

14 years agoRemove LINUXINCLUDE from autoconf wrapper, breaks 2.6.28+ kernels.
Brian Behlendorf [Mon, 27 Jul 2009 16:52:01 +0000 (09:52 -0700)]
Remove LINUXINCLUDE from autoconf wrapper, breaks 2.6.28+ kernels.
Modern kernel build systems at least post 2.6.16 will set this properly
so we should not.  In fact post 2.6.28 the include headers have moved
under arch so the guess we make here is completely wrong.  Letting
the kernel build system set this ensure it will be correct.

14 years agoPositive Solaris ioctl return codes need to be negated for use by libc
Brian Behlendorf [Thu, 23 Jul 2009 23:14:52 +0000 (16:14 -0700)]
Positive Solaris ioctl return codes need to be negated for use by libc

14 years agoAllow kmem or vmem based slab for slab_lock and slab_overcommit tests.
Brian Behlendorf [Thu, 23 Jul 2009 20:50:53 +0000 (13:50 -0700)]
Allow kmem or vmem based slab for slab_lock and slab_overcommit tests.
The slab_overcommit test case could hang on a system with fragmented
memory because it was creating a kmem based slab with 256K objects.
To avoid this I've removed the KMC_KMEM flag which allows the slab
to decide if it should be kmem or vmem backed based on the object
side.  The slab_lock test shares this code and will also be effected.
But the point of these two tests is to stress cache locking and
memory overcommit, the type of slab is not critical.  In fact, allowing
the slab to do the default smart thing is preferable.

14 years agoThe HAVE_PATH_IN_NAMEIDATA compat macros should have been used here.
Brian Behlendorf [Wed, 22 Jul 2009 21:28:19 +0000 (14:28 -0700)]
The HAVE_PATH_IN_NAMEIDATA compat macros should have been used here.

14 years agoCheck arch/default/ path when detecting kernel objects on SLES
Brian Behlendorf [Wed, 22 Jul 2009 13:59:28 +0000 (06:59 -0700)]
Check arch/default/ path when detecting kernel objects on SLES
We still preferentially use arch/arch looking for a native version
but if that fails it is acceptable to use default.

14 years agoRegister a basic compat ioctl handler (32 vs 64 bit compat)
Brian Behlendorf [Tue, 21 Jul 2009 17:13:58 +0000 (10:13 -0700)]
Register a basic compat ioctl handler (32 vs 64 bit compat)
Simply pass the ioctl on to the normal handler.  If the ioctl
helper macros are used correctly this should be safe as they
will handle the packing/unpacking of the data encoded in the
ioctl command.  And actually, if the caller does not use the
IO* macros at all, and just passes small values, it will probably
be OK as well.  We only get in to trouble if they try and use
the upper 32-bits.  Endianness is not really a concern here, we
we are pretty much assumed they user and kernel will match.

14 years agoFixed NULL dereference by tcd_for_each() when the kmalloc() call in module/spl/spl...
Ricardo M. Correia [Tue, 14 Jul 2009 22:24:59 +0000 (15:24 -0700)]
Fixed NULL dereference by tcd_for_each() when the kmalloc() call in module/spl/spl-debug.c:1163 returns NULL.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
14 years agoPrevent integer overflow after ~164 days of uptime.
Ricardo M. Correia [Tue, 14 Jul 2009 22:23:25 +0000 (15:23 -0700)]
Prevent integer overflow after ~164 days of uptime.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
14 years agoAdd a little paranoia here to ensure endianess is set correctly.
Brian Behlendorf [Tue, 14 Jul 2009 21:28:04 +0000 (14:28 -0700)]
Add a little paranoia here to ensure endianess is set correctly.

14 years agoAdd basic groupmember() function, not sup groups.
Brian Behlendorf [Fri, 10 Jul 2009 17:58:06 +0000 (10:58 -0700)]
Add basic groupmember() function, not sup groups.

14 years agoAdd ddi_copyin/ddi_copyout support for fake kernel originated ioctls.
Brian Behlendorf [Fri, 10 Jul 2009 17:56:32 +0000 (10:56 -0700)]
Add ddi_copyin/ddi_copyout support for fake kernel originated ioctls.

14 years agoDefine ACE_ALL_PERMS for use by ACLs
Brian Behlendorf [Thu, 9 Jul 2009 22:00:25 +0000 (15:00 -0700)]
Define ACE_ALL_PERMS for use by ACLs

14 years agoDefine FKIOCTL which is used on Solaris to mark an in-kernel ioctl.
Brian Behlendorf [Thu, 9 Jul 2009 21:59:41 +0000 (14:59 -0700)]
Define FKIOCTL which is used on Solaris to mark an in-kernel ioctl.

14 years agoAdd ASSERTV macro to simplify removing variables (the V in ASSERTV)
Brian Behlendorf [Thu, 9 Jul 2009 19:15:23 +0000 (12:15 -0700)]
Add ASSERTV macro to simplify removing variables (the V in ASSERTV)
which are only used in ASSERT().

14 years agoAdd basic support for TASKQ_THREADS_CPU_PCT taskq flag which is
Brian Behlendorf [Thu, 9 Jul 2009 17:07:52 +0000 (10:07 -0700)]
Add basic support for TASKQ_THREADS_CPU_PCT taskq flag which is
used to scale the number of threads based on the number of online
CPUs.  As CPUs are added/removed we should rescale the thread
count appropriately, but currently this is only done at create.

14 years agoUpdate ChangeLog
Brian Behlendorf [Thu, 2 Jul 2009 21:19:11 +0000 (14:19 -0700)]
Update ChangeLog

14 years agoCleanly handle --with-linux=NONE option when used to generate source
Brian Behlendorf [Thu, 2 Jul 2009 17:47:28 +0000 (10:47 -0700)]
Cleanly handle --with-linux=NONE option when used to generate source
rpms.  These should not be fatal because we actually don't need them
until we build the source rpm.  When doing mock builds this is
important because these dependent rpms will only be installed if
they are specificed in the source rpms spec file.

14 years agoSimplify rpm build rules, added config/rpm.am.
Brian Behlendorf [Wed, 1 Jul 2009 21:37:44 +0000 (14:37 -0700)]
Simplify rpm build rules, added config/rpm.am.
Distro friendly changes such that the kernel modules are packaged seperately.

14 years agoAdd spl.release to spl-devel to simply dependent package version check.
Brian Behlendorf [Mon, 29 Jun 2009 23:41:21 +0000 (16:41 -0700)]
Add spl.release to spl-devel to simply dependent package version check.

14 years agoInstall spl-devel products in /usr/src/spl-SPL_VERSION/LINUX_VERSION/
Brian Behlendorf [Fri, 26 Jun 2009 23:30:44 +0000 (16:30 -0700)]
Install spl-devel products in /usr/src/spl-SPL_VERSION/LINUX_VERSION/
Remove the spl symlink, it's just confusing

14 years agoUse do_div on older kernel where do_div64 doesn't exist.
Brian Behlendorf [Fri, 26 Jun 2009 20:10:52 +0000 (13:10 -0700)]
Use do_div on older kernel where do_div64 doesn't exist.

14 years agoAdditional tuning to get the BuildRequires right for all cases.
Brian Behlendorf [Fri, 26 Jun 2009 19:43:27 +0000 (12:43 -0700)]
Additional tuning to get the BuildRequires right for all cases.
pl.spec~

14 years agoSimplify the kernel depenency logic
Brian Behlendorf [Fri, 26 Jun 2009 18:37:06 +0000 (11:37 -0700)]
Simplify the kernel depenency logic

14 years agoSpec file update, for some reason the following shorthand syntax
Brian Behlendorf [Fri, 26 Jun 2009 17:34:40 +0000 (10:34 -0700)]
Spec file update, for some reason the following shorthand syntax
was failing so it was replaced with the longer %if version.

%{!?foo: %define foo bar}

changed to

%if %{undefined foo}
 %define foo bar
%endif

14 years agoSRPM build farm / mock itergration
Brian Behlendorf [Fri, 26 Jun 2009 16:40:14 +0000 (09:40 -0700)]
SRPM build farm / mock itergration

14 years agoBuild farm integration to ensure BuildRequires are correct
Brian Behlendorf [Thu, 25 Jun 2009 23:11:13 +0000 (16:11 -0700)]
Build farm integration to ensure BuildRequires are correct

14 years agoPackaging Fixes
Brian Behlendorf [Thu, 25 Jun 2009 22:31:53 +0000 (15:31 -0700)]
Packaging Fixes
- Kernel modules should be built using the LINUX_OBJ Makefiles and
  not the LINUX Makefiles to ensure the proper install paths are used.
- Install modules in to addon/spl/
- Ensure no additional kernel module build products are packaged.
- Simplified spl.spec.in which supports RHEL, CHAOS, SLES, FEDORA.

14 years agoUpdate ChangeLog with a high level summary of the changes from
Brian Behlendorf [Mon, 22 Jun 2009 22:31:40 +0000 (15:31 -0700)]
Update ChangeLog with a high level summary of the changes from
0.4.3 to 0.4.4 prior to tagging.  Full details can be found in
the git commit history.

14 years agoPackaging improvements for RHEL and SLES (part 2)
Brian Behlendorf [Tue, 16 Jun 2009 18:34:28 +0000 (11:34 -0700)]
Packaging improvements for RHEL and SLES (part 2)
- Allow checking for exported symbols in both Module.symvers
  and Module.symvers.  My stock SLES kernel ships an objects
  directory with Module.symvers, yet produces a Module.symvers
  in the local build directory.

14 years agoPackaging improvements for RHEL and SLES
Brian Behlendorf [Tue, 16 Jun 2009 17:44:59 +0000 (10:44 -0700)]
Packaging improvements for RHEL and SLES
- Properly honor --prefix in build system and rpm spec file.
- Add '--define require_kdir' to spec file to support building
  rpms against kernel sources installed in non-default locations.
- Add '--define require_kobj' to spec file to support building
  rpms against kernel object installed in non-default locations.
- Stop suppressing errors in autogen.sh script.
- Improved logic to detect missing kernel objects when they are
  not located with the source.  This is the common case for SLES
  as well as in-tree chaos kernel builds and is done to simply
  support for multiple arches.
- Moved spl-devel build products to /usr/src/spl-<version>, a
  spl symlink is created to reference the last installed version.

14 years agoSLES10 Fixes (part 9)
Brian Behlendorf [Thu, 21 May 2009 17:56:11 +0000 (10:56 -0700)]
SLES10 Fixes (part 9)
- Proper ioctl() 32/64-bit binary compatibility.  We need to ensure the
  ioctl data itself is always packed the same for 32/64-bit binaries.
  Additionally, the correct thing to do is encode this size in bytes
  as part of the command using _IOC_SIZE().
- Minor formatting changes to respect the 80 character limit.
- Move all SPLAT_SUBSYSTEM_* defines in to splat-ctl.h.
- Increase SPLAT_SUBSYSTEM_UNKNOWN because we were getting close
  to accidentally using it for a real registered subsystem.

14 years agoSLES10 Fixes (part 8)
Brian Behlendorf [Wed, 20 May 2009 23:33:08 +0000 (16:33 -0700)]
SLES10 Fixes (part 8)
- Add compat_ioctl() handler, by default 64-bit SLES systems build 32-bit
  ELF binaries.  For the 32-bit binaries to pass ioctl information to a
  64-bit kernel a compatibility handler needs to be registered.  In our
  case no additional conversions are needed to convert 32-bit ioctl()
  commands to 64-bit commands so we can just call the default handler.

14 years agoSLES10 Fixes (part 7)
Brian Behlendorf [Wed, 20 May 2009 22:30:13 +0000 (15:30 -0700)]
SLES10 Fixes (part 7)
- Initial SLES testing uncovered a long standing bug in the debug
  tracing.  The tcd_for_each() macro expected a NULL to terminate
  the trace_data[i] array but this was only ever true due to luck.
  All trace_data[] iterators are now properly capped by TCD_TYPE_MAX.
- SPLAT_MAJOR 229 conflicted with a 'hvc' device on my SLES system.
  Since this was always an arbitrary choice I picked something else.
- The HAVE_PGDAT_LIST case should set pgdat_list_addr to the value stored
  at the address of the memory location returned by kallsyms_lookup_name().

14 years agoSLES10 Fixes (part 6)
Brian Behlendorf [Wed, 20 May 2009 21:23:13 +0000 (14:23 -0700)]
SLES10 Fixes (part 6)
- Prior to 2.6.17 there were no *_pgdat helper functions in mm/mmzone.c.
  Instead for_each_zone() operated directly on pgdat_list which may or
  may not have been exported depending on how your kernel was compiled.
  Now new configure checks determine if you have the helpers or not, and
  if the needed symbols are exported.  If they are not exported then they
  are dynamically aquired at runtime by kallsyms_lookup_name().

14 years agoPowerpc Fixes (part 1):
Brian Behlendorf [Wed, 20 May 2009 19:23:24 +0000 (12:23 -0700)]
Powerpc Fixes (part 1):
- Enable builds for powerpc ISA type.
- Add DIV_ROUND_UP and roundup macros if unavailable.
- Cast 64-bit values for %lld format string to (long long) to
  quiet compile warning.

14 years agoSLES10 Fixes (part 5):
Brian Behlendorf [Wed, 20 May 2009 18:54:40 +0000 (11:54 -0700)]
SLES10 Fixes (part 5):
- Fix incorrect mapping for spl_device_create()->class_device_create()
  which is the prefered API for 2.6.13 to 2.6.17 based kernels.

14 years agoSLES10 Fixes (part 4):
Brian Behlendorf [Wed, 20 May 2009 18:31:55 +0000 (11:31 -0700)]
SLES10 Fixes (part 4):
- Configure check for SLES specific API change to vfs_unlink()
  and vfs_rename() which added a 'struct vfsmount *' argument.
  This was for something called the linux-security-module, but
  it appears that it was never adopted upstream.

14 years agoSLES10 Fixes (part 3):
Brian Behlendorf [Wed, 20 May 2009 17:56:13 +0000 (10:56 -0700)]
SLES10 Fixes (part 3):
- Configure check for mutex_lock_nested().  This function was introduced
  as part of the mutex validator in 2.6.18, but if it's unavailable then
  it's safe to fallback to a plain mutex_lock().

14 years agoSLES10 Fixes (part 2):
Brian Behlendorf [Wed, 20 May 2009 17:08:37 +0000 (10:08 -0700)]
SLES10 Fixes (part 2):
- Configure check, the div64_64() function was renamed to
  div64_u64() as of 2.6.26.
- Configure check, the global_page_state() fuction was introduced
  in 2.6.18 kernels.  The earlier 2.6.16 based SLES10 must not try
  and use it, thankfully get_zone_counts() is still available.
- To simplify debugging poison all symbols aquired dynamically
  using spl_kallsyms_lookup_name() with SYMBOL_POISON.
- Add console messages when the user mode helpers fail.
- spl_kmem_init_globals() use bit shifts instead of division.
- When the monotonic clock is unavailable __gethrtime() must perform
  the HZ division as an 'unsigned long long' because the SPL only
  implements __udivdi3(), and not __divdi3() for 'long long' division
  on 32-bit arches.

14 years agoSLES10 Fixes (part 1):
Brian Behlendorf [Tue, 19 May 2009 18:42:39 +0000 (11:42 -0700)]
SLES10 Fixes (part 1):
- Exclude -obj when detecting installed kernel source.
- Detect -obj directory for out of tree kernel builds.
- Allow kernel build system to set CC to ensure -m64 is set properly.
  This is an issue on 64-bit SLES systems which by default always
  build 32-bit binaries (unlike RHEL/Fedora which default to 64-bit)

15 years agoPrep for spl-0.4.3 tag.
Brian Behlendorf [Fri, 20 Mar 2009 21:48:30 +0000 (14:48 -0700)]
Prep for spl-0.4.3 tag.

15 years agoAdd list_move_tail() function.
Brian Behlendorf [Fri, 20 Mar 2009 04:40:07 +0000 (21:40 -0700)]
Add list_move_tail() function.

15 years agoRemove useless EOL white space padding from `splat -l` command.
Brian Behlendorf [Wed, 18 Mar 2009 18:56:42 +0000 (11:56 -0700)]
Remove useless EOL white space padding from `splat -l` command.

15 years agoFix vmem leak in kmem_cache_test (missing splat_kmem_cache_test_kcp_free())
Brian Behlendorf [Wed, 18 Mar 2009 18:56:00 +0000 (11:56 -0700)]
Fix vmem leak in kmem_cache_test (missing splat_kmem_cache_test_kcp_free())

15 years agoAllow spl_config.h to be included by dependant packages
Brian Behlendorf [Tue, 17 Mar 2009 21:55:59 +0000 (14:55 -0700)]
Allow spl_config.h to be included by dependant packages

We need dependent packages to be able to include spl_config.h so they
can leverage the configure checks the SPL has done.  This is important
because several of the spl headers need the results of these checks to
work properly.  Unfortunately, the autoheader build product is always
private to a particular build and defined certain common things.
(PACKAGE, VERSION, etc).  This prevents other packages which also use
autoheader from being include because the definitions conflict.  To
avoid this problem the SPL build system leverage AH_BOTTOM to include
a spl_unconfig.h at the botton of the autoheader build product.  This
custom include undefs all known shared symbols to prevent the confict.
This does however mean that those definition are also not availble
to the SPL package either.  The SPL package therefore uses the
equivilant SPL_META_* definitions.

15 years agoFC10/i686 Compatibility Update (2.6.27.19-170.2.35.fc10.i686)
Brian Behlendorf [Tue, 17 Mar 2009 19:16:31 +0000 (12:16 -0700)]
FC10/i686 Compatibility Update (2.6.27.19-170.2.35.fc10.i686)

In the interests of portability I have added a FC10/i686 box to
my list of development platforms.  The hope is this will allow me
to keep current with upstream kernel API changes, and at the same
time ensure I don't accidentally break x86 support.  This patch
resolves all remaining issues observed under that environment.

1) SPL_AC_ZONE_STAT_ITEM_FIA autoconf check added.  As of 2.6.21
the kernel added a clean API for modules to get the global count
for free, inactive, and active pages.  The SPL attempts to detect
if this API is available and directly map spl_global_page_state()
to global_page_state().  If the full API is not available then
spl_global_page_state() is implemented as a thin layer to get
these values via get_zone_counts() if that symbol is available.

2) New kmem:vmem_size regression test added to validate correct
vmem_size() functionality.  The test case acquires the current
global vmem state, allocates from the vmem region, then verifies
the allocation is correctly reflected in the vmem_size() stats.

3) Change splat_kmem_cache_thread_test() to always use KMC_KMEM
based memory.  On x86 systems with limited virtual address space
failures resulted due to exhaustig the address space.  The tests
really need to problem exhausting all memory on the system thus
we need to use the physical address space.

4) Change kmem:slab_lock to cap it's memory usage at availrmem
instead of using the native linux nr_free_pages().  This provides
additional test coverage of the SPL Linux VM integration.

5) Change kmem:slab_overcommit to perform allocation of 256K
instead of 1M.  On x86 based systems it is not possible to create
a kmem backed slab with entires of that size.  To compensate for
this the number of allocations performed in increased by 4x.

6) Additional autoconf documentation for proposed upstream API
changes to make additional symbols available to modules.

7) Console error messages added when spl_kallsyms_lookup_name()
fails to locate an expected symbol.  This causes the module to fail
to load and we need to know exactly which symbol was not available.

15 years agoFix taskq_wait() not waiting bug
Brian Behlendorf [Sun, 15 Mar 2009 22:13:49 +0000 (15:13 -0700)]
Fix taskq_wait() not waiting bug

I'm very surprised this has not surfaced until now.  But the taskq_wait()
implementation work only wait successfully the first time it was called.
Subsequent usage of taskq_wait() on the taskq would not wait.

The issue was caused by tq->tq_lowest_id being set to MAX_INT after the
first wait completed.  This caused subsequent waits which check that the
waiting id is less than the lowest taskq id to always succeed.  The fix
is to ensure that tq->tq_lowest_id is never set larger than tq->tq_next.id.

Additional fixes which were added to this patch include:
1) Fix a race by placing the taskq_wait_check() in the tq->tq_lock spinlock.
2) taskq_wait() should wait for the largest outstanding id.
3) Multiple spelling corrections.
4) Added taskq wait regression test to validate correct behavior.

15 years agoMutex tests updated to use task queues instead of work queues.
Brian Behlendorf [Sun, 15 Mar 2009 22:05:38 +0000 (15:05 -0700)]
Mutex tests updated to use task queues instead of work queues.

Mainly for portability reasons I have rebased the mutex tests on Solaris
taskqs instead of linux work queues.  The linux workqueue API changed post
2.6.18 kernels and using task queues avoids having to conditionally detect
which workqueue API to use.

Additionally, this is basically free additional testing for the task queues.
Much to my surprise after updating these test cases they did expose a long
standing bug in the taskq_wait() implementation.  This patch does not
address that issue but the followup patch does.

15 years agoAdded SPL_AC_5ARGS_DEVICE_CREATE autoconf configure check
Brian Behlendorf [Fri, 13 Mar 2009 20:38:43 +0000 (13:38 -0700)]
Added SPL_AC_5ARGS_DEVICE_CREATE autoconf configure check

As of 2.6.27 kernels the device_create() API changed to include
a private data argument.  This check detects which version of
device_create() function the kernel has and properly defines
spl_device_create() to use the correct prototype.

15 years agoFix off-by-1 truncation of hw_serial when converting from integer to string, when...
Ricardo M. Correia [Thu, 12 Mar 2009 21:23:34 +0000 (21:23 +0000)]
Fix off-by-1 truncation of hw_serial when converting from integer to string, when writing to /proc/sys/kernel/spl/spl_hostid.
Fixes hostid mismatch which leads to assertion failure when the hostid/hw_serial is a 10-character decimal number:

$ zpool status
  pool: lustre
 state: ONLINE
lt-zpool: zpool_main.c:3176: status_callback: Assertion `reason == ZPOOL_STATUS_OK' failed.
zsh: 5262 abort      zpool status

15 years agoMinor bug fix in XDR code introduced in last minute change before landing.
Ricardo M. Correia [Wed, 11 Mar 2009 05:39:03 +0000 (05:39 +0000)]
Minor bug fix in XDR code introduced in last minute change before landing.

1) Removed xdr_bytesrec typedef which has no consumers.  If we re-add
   it should also probably be xdr_bytesrec_t.

15 years agoAdd XDR implementation
Ricardo M. Correia [Tue, 10 Mar 2009 19:51:45 +0000 (19:51 +0000)]
Add XDR implementation

Added proper XDR implementation (Lustre bug 17662), needed for on-disk
compatibility between platforms of different endianness.

15 years agoBuild system cleanup
Brian Behlendorf [Tue, 10 Mar 2009 17:57:52 +0000 (10:57 -0700)]
Build system cleanup

1) Undefine non-unique entries in spl_config.h
2) Minor Makefile cleanup
3) Don't use includedir for proper kernel header install

15 years agoBuild System Default Kernel
Brian Behlendorf [Mon, 9 Mar 2009 23:50:37 +0000 (16:50 -0700)]
Build System Default Kernel

Update the method used for determining which kernel to build against
when not specified.  Previous 'uname -r' was used but this makes the
assumption that the running kernel is the one you want to use, this is
often not the case.  It is better to examine the usual kernel-devel
install locations and select one of the installed kernels.

15 years agoBuild system and packaging (RPM support)
Brian Behlendorf [Thu, 5 Mar 2009 17:08:07 +0000 (09:08 -0800)]
Build system and packaging (RPM support)

An update to the build system to properly support all commonly
used Makefile targets these include:

  make all        # Build everything
  make install    # Install everything
  make clean   # Clean up build products
  make distclean  # Clean up everything
  make dist       # Create package tarball
  make srpm       # Create package source RPM
  make rpm        # Create package binary RPMs
  make tags       # Create ctags and etags for everything

Extra care was taken to ensure that the source RPMs are fully
rebuildable against Fedora/RHEL/Chaos kernels.  To build binary
RPMs from the source RPM for your system simply run:

  rpmbuild --rebuild spl-x.y.z-1.src.rpm

This will produce two binary RPMs with correct 'requires'
dependencies for your kernel.  One will contain all spl modules
and support utilities, the other is a devel package for compiling
additional kernel modules which are dependant on the spl.

  spl-x.y.z-1_<kernel version>.x86_64.rpm
  spl-devel-x.y.2-1_<kernel version>.x86_64.rpm

15 years agoXXX: Temporarily disable vmem_size().
Ricardo M. Correia [Thu, 26 Feb 2009 18:05:39 +0000 (18:05 +0000)]
XXX: Temporarily disable vmem_size().

15 years agoMerge branch 'kallsyms'
Brian Behlendorf [Wed, 4 Mar 2009 18:19:41 +0000 (10:19 -0800)]
Merge branch 'kallsyms'

15 years agoLinux VM Integration Cleanup
Brian Behlendorf [Wed, 25 Feb 2009 21:20:40 +0000 (13:20 -0800)]
Linux VM Integration Cleanup

Remove all instances of functions being reimplemented in the SPL.
When the prototypes are available in the linux headers but the
function address itself is not exported use kallsyms_lookup_name()
to find the address.  The function name itself can them become a
define which calls a function pointer.  This is preferable to
reimplementing the function in the SPL because it ensures we get
the correct version of the function for the running kernel.  This
is actually pretty safe because the prototype is defined in the
headers so we know we are calling the function properly.

This patch also includes a rhel5 kernel patch we exports the needed
symbols so we don't need to use kallsyms_lookup_name().  There are
autoconf checks to detect if the symbol is exported and if so to
use it directly.  We should add patches for stock upstream kernels
as needed if for no other reason than so we can easily track which
additional symbols we needed exported.  Those patches can also be
used by anyone willing to rebuild their kernel, but this should
not be a requirement.  The rhel5 version of the export-symbols
patch has been applied to the chaos kernel.

Additional fixes:
1) Implement vmem_size() function using get_vmalloc_info()
2) SPL_CHECK_SYMBOL_EXPORT macro updated to use $LINUX_OBJ instead
   of $LINUX because Module.symvers is a build product.  When
   $LINUX_OBJ != $LINUX we will not properly detect exported symbols.
3) SPL_LINUX_COMPILE_IFELSE macro updated to add include2 and
   $LINUX/include search paths to allow proper compilation when
   the kernel target build directory is not the source directory.

15 years agoChanged ptob()/btop() mult/div into bit shifts.
Ricardo M. Correia [Wed, 25 Feb 2009 21:06:15 +0000 (21:06 +0000)]
Changed ptob()/btop() mult/div into bit shifts.
Added necessary include for PAGE_SHIFT.

15 years agoAdded btop() and moved ptob() to include/sys/param.h.
Ricardo M. Correia [Sun, 22 Feb 2009 03:05:40 +0000 (03:05 +0000)]
Added btop() and moved ptob() to include/sys/param.h.

15 years agoChanged z_compress_level() and z_uncompress() prototypes to match the ones in Solaris.
Ricardo M. Correia [Sun, 22 Feb 2009 03:35:51 +0000 (03:35 +0000)]
Changed z_compress_level() and z_uncompress() prototypes to match the ones in Solaris.
Fixes compilation warning.

15 years agoMatching kmem_free() fix for use after free case.
Brian Behlendorf [Thu, 19 Feb 2009 20:28:10 +0000 (12:28 -0800)]
Matching kmem_free() fix for use after free case.

See commit bb01879ebe5ba5d8e2c2177d3b1ce0272c17ce07 for a full
description.  This issue should have been addressed in the same
commit but it slipped my mind.

15 years agoAdd zone_get_hostid() function
Brian Behlendorf [Thu, 19 Feb 2009 19:26:17 +0000 (11:26 -0800)]
Add zone_get_hostid() function

Minimal support added for the zone_get_hostid() function.  Only
global zones are supported therefore this function must be called
with a NULL argumment.  Additionally, I've added the HW_HOSTID_LEN
define and updated all instances where a hard coded magic value
of 11 was used; "A good riddance of bad rubbish!"

15 years agoCoverity 9657: Resource Leak
Brian Behlendorf [Wed, 18 Feb 2009 18:16:26 +0000 (10:16 -0800)]
Coverity 9657: Resource Leak

Accidentally leaked list item li in error path.  The fix is to
adjust this error path to ensure the allocated list item which
has not yet been added to the list gets freed.  To do this we
simply add a new goto label slightly earlier to use the existing
cleanup logic and minimize the number of unique return points.

15 years agoCoverity 9656: Forward NULL
Brian Behlendorf [Wed, 18 Feb 2009 18:09:01 +0000 (10:09 -0800)]
Coverity 9656: Forward NULL

This was a false positive the callpath being walked is impossible
because the splat_kmem_cache_test_kcp_alloc() function will ensure
kcp->kcp_kcd[0] is initialized to NULL.  However, there is no harm
is making this explicit for the test case so I'm adding a line to
clearly set it to correct the analysis.

15 years agoCoverity 9649, 9650, 9651: Uninit
Brian Behlendorf [Wed, 18 Feb 2009 17:48:07 +0000 (09:48 -0800)]
Coverity 9649, 9650, 9651: Uninit

This check was originally added to detect double initializations
of mutex types (which it did find).  Unfortunately, Coverity is
right that there is a very small chance we could trigger the
assertion by accident because an uninitialized stack variable
happens to contain the mutex magic.  This is particularly unlikely
since we do poison the mutexs when destroyed but still possible.
Therefore I'm simply removing the assertion.

15 years agoCoverity 9654, 9654: Use After Free
Brian Behlendorf [Wed, 18 Feb 2009 00:41:08 +0000 (16:41 -0800)]
Coverity 9654, 9654: Use After Free

Because vmem_free() was implemented as a macro using the ','
operator to evaluate both arguments and we performed the free
before evaluating size we would deference the free'd pointer.
To resolve the problem we just invert the ordering and evaluate
size first just as if it was evaluated by the caller when being
passed to this function.  This ensure that if the caller is
doing something reckless like performing an assignment as
part of the size argument we still perform it and it simply
doesn't get removed by the macro.  Oh course nobody should
be doing this sort of thing, but just in case.

15 years agoCoverity 9652, 9653: No Effect
Brian Behlendorf [Wed, 18 Feb 2009 00:30:58 +0000 (16:30 -0800)]
Coverity 9652, 9653: No Effect

Removed 2 ASSERT()s which had no effect because by definition
size_t is always an unsigned type thus is always >= 0.

15 years agoCoverity 9641: Buffer Size
Brian Behlendorf [Wed, 18 Feb 2009 00:24:26 +0000 (16:24 -0800)]
Coverity 9641: Buffer Size

When SPLAT_TEST_INIT() initialized SPLAT_KMEM_TEST11_NAME the short
short test name overran the static length buffer of SPLAT_NAME_SIZE.
This was fixed by increasing the buffer length from 16 to 20 bytes.

15 years agokmem slab magazine ageing deadlock
Brian Behlendorf [Tue, 17 Feb 2009 23:52:18 +0000 (15:52 -0800)]
kmem slab magazine ageing deadlock

- The previous magazine ageing sceme relied on the on_each_cpu()
  function to call spl_magazine_age() on each cpu.  It turns out
  this could deadlock with do_flush_tlb_all() which also relies
  on the IPI based on_each_cpu().  To avoid this problem a per-
  magazine delayed work item is created and indepentantly
  scheduled to the correct cpu removing the need for on_each_cpu().
- Additionally two unused fields were removed from the type
  spl_kmem_cache_t, they were hold overs from previous cleanup.
    - struct work_struct work
    - struct timer_list timer

15 years agokmem slab fixes
Brian Behlendorf [Fri, 13 Feb 2009 18:28:55 +0000 (10:28 -0800)]
kmem slab fixes

- spl_slab_reclaim() 'continue' changed back to 'break' from commit
  37db7d8cf9936e6d2851a4329c11efcd9f61305c.  The original was correct,
  I have added a comment to ensure this does not happen again.
- spl_slab_reclaim() further optimized by moving the destructor call
  in spl_slab_free() outside the skc->skc_lock.  This minimizes the
  length of time the spin lock is held, allows the destructors to
  be invoked concurrently for different objects, and as a bonus makes
  it safe (although unwise) to sleep in the destructors.