Author: imp
Date: Thu Feb 21 07:19:50 2013
New Revision: 247089
URL: http://svnweb.freebsd.org/changeset/base/247089
Log:
Remove incorrect comment about splsoftclock.
Modified:
head/sys/dev/si/si.c
Modified: head/sys/dev/si/si.c
===
Author: imp
Date: Thu Feb 21 07:16:40 2013
New Revision: 247088
URL: http://svnweb.freebsd.org/changeset/base/247088
Log:
Locking for todr got pushed down into inittodr and the client
libraries it calls (although some might not be doing it right). We are
serialized right now by giant as well
On Feb 20, 2013, at 11:49 PM, Alexey Dokuchaev wrote:
> On Thu, Feb 21, 2013 at 06:38:24AM +, Warner Losh wrote:
>> New Revision: 247086
>> URL: http://svnweb.freebsd.org/changeset/base/247086
>>
>> Log:
>> Correct comment about use of pmtimer, and the real reason it isn't
>> used or desir
On Thu, Feb 21, 2013 at 06:38:24AM +, Warner Losh wrote:
> New Revision: 247086
> URL: http://svnweb.freebsd.org/changeset/base/247086
>
> Log:
> Correct comment about use of pmtimer, and the real reason it isn't
> used or desirable for amd64.
Thanks!
> + * pmtimer is used only with the
Author: adrian
Date: Thu Feb 21 06:38:49 2013
New Revision: 247087
URL: http://svnweb.freebsd.org/changeset/base/247087
Log:
Add an option to allow the minimum number of delimiters to be tweaked.
This is primarily for debugging purposes.
Tested:
* AR5416, STA mode
Modified:
hea
Author: imp
Date: Thu Feb 21 06:38:24 2013
New Revision: 247086
URL: http://svnweb.freebsd.org/changeset/base/247086
Log:
Correct comment about use of pmtimer, and the real reason it isn't
used or desirable for amd64.
Modified:
head/sys/x86/isa/clock.c
Modified: head/sys/x86/isa/clock.c
==
Author: adrian
Date: Thu Feb 21 06:18:40 2013
New Revision: 247085
URL: http://svnweb.freebsd.org/changeset/base/247085
Log:
Add a new option to limit the maximum size of aggregates.
The default is to limit them to what the hardware is capable of.
Add sysctl twiddles for both the non-RTS
Author: kib
Date: Thu Feb 21 06:12:26 2013
New Revision: 247084
URL: http://svnweb.freebsd.org/changeset/base/247084
Log:
MFC r246476:
Document the ERESTART translation to EINTR for devfs nodes.
Modified:
stable/8/lib/libc/sys/open.2
Directory Properties:
stable/8/lib/libc/ (props chang
Author: kib
Date: Thu Feb 21 06:10:36 2013
New Revision: 247083
URL: http://svnweb.freebsd.org/changeset/base/247083
Log:
MFC r246472:
Stop translating the ERESTART error from the open(2) into EINTR.
Posix requires that open(2) is restartable for SA_RESTART.
Modified:
stable/8/sys/fs/devf
Author: kib
Date: Thu Feb 21 06:00:33 2013
New Revision: 247082
URL: http://svnweb.freebsd.org/changeset/base/247082
Log:
MFC r246486:
Document P_PPTRACE.
Modified:
stable/9/bin/ps/ps.1
Directory Properties:
stable/9/bin/ps/ (props changed)
Modified: stable/9/bin/ps/ps.1
==
Author: kib
Date: Thu Feb 21 05:59:16 2013
New Revision: 247081
URL: http://svnweb.freebsd.org/changeset/base/247081
Log:
MFC r246485:
Document the detail of interaction between vfork and PT_TRACEME.
Modified:
stable/9/lib/libc/sys/ptrace.2
Directory Properties:
stable/9/lib/libc/ (prop
Author: kib
Date: Thu Feb 21 05:57:51 2013
New Revision: 247080
URL: http://svnweb.freebsd.org/changeset/base/247080
Log:
MFC r246476:
Document the ERESTART translation to EINTR for devfs nodes.
Modified:
stable/9/lib/libc/sys/open.2
Directory Properties:
stable/9/lib/libc/ (props chang
Author: kib
Date: Thu Feb 21 05:55:52 2013
New Revision: 247079
URL: http://svnweb.freebsd.org/changeset/base/247079
Log:
MFC r246802:
Print slightly more useful information on the 'bad pte' panic.
Modified:
stable/9/sys/amd64/amd64/pmap.c
Directory Properties:
stable/9/sys/ (props chan
Author: kib
Date: Thu Feb 21 05:53:42 2013
New Revision: 247078
URL: http://svnweb.freebsd.org/changeset/base/247078
Log:
MFC r246801:
Assert that user address is never qremoved.
Modified:
stable/9/sys/amd64/amd64/pmap.c
Directory Properties:
stable/9/sys/ (props changed)
Modified: sta
Author: kib
Date: Thu Feb 21 05:47:52 2013
New Revision: 247077
URL: http://svnweb.freebsd.org/changeset/base/247077
Log:
MFC r246484:
Allow ptrace(2) operation on the child created by vfork(2), if the
debugger is not the parent.
Modified:
stable/9/sys/kern/kern_exec.c
stable/9/sys/kern
Author: kib
Date: Thu Feb 21 05:38:11 2013
New Revision: 247076
URL: http://svnweb.freebsd.org/changeset/base/247076
Log:
MFC r246472:
Stop translating the ERESTART error from the open(2) into EINTR.
Posix requires that open(2) is restartable for SA_RESTART.
Modified:
stable/9/sys/fs/devf
Author: adrian
Date: Thu Feb 21 02:52:13 2013
New Revision: 247073
URL: http://svnweb.freebsd.org/changeset/base/247073
Log:
Remove this unneeded printf(), sorry!
Modified:
head/sys/dev/ath/ath_hal/ar5416/ar5416_attach.c
Modified: head/sys/dev/ath/ath_hal/ar5416/ar5416_attach.c
=
Author: imp
Date: Thu Feb 21 02:43:44 2013
New Revision: 247072
URL: http://svnweb.freebsd.org/changeset/base/247072
Log:
The request queue is already locked, so we don't need the splsofclock/splx
here to note future work.
Modified:
head/sys/fs/nfsclient/nfs_clstate.c
Modified: head/sys/fs
Author: jamie
Date: Thu Feb 21 02:41:37 2013
New Revision: 247071
URL: http://svnweb.freebsd.org/changeset/base/247071
Log:
Don't worry if a module is already loaded when looking for a fstype to mount
(possible in a race condition).
Reviewed by: kib
MFC after:1 week
Modified:
he
Author: imp
Date: Thu Feb 21 02:40:20 2013
New Revision: 247070
URL: http://svnweb.freebsd.org/changeset/base/247070
Log:
Most other giant locked storage drivers in the tree don't use
splsoftclock to note the need for future locking, so remove it from
here.
Modified:
head/sys/pci/ncr.c
M
Author: imp
Date: Thu Feb 21 02:34:04 2013
New Revision: 247069
URL: http://svnweb.freebsd.org/changeset/base/247069
Log:
The other giant locked storage drivers have removed splbio(), for the
most part, so remove it here too. Anybody locking this driver will need
far more than locks where sp
Author: imp
Date: Thu Feb 21 00:40:08 2013
New Revision: 247068
URL: http://svnweb.freebsd.org/changeset/base/247068
Log:
Fix broken usage of splhigh() by removing it.
Modified:
head/sys/x86/isa/atrtc.c
Modified: head/sys/x86/isa/atrtc.c
==
Author: imp
Date: Thu Feb 21 00:36:12 2013
New Revision: 247067
URL: http://svnweb.freebsd.org/changeset/base/247067
Log:
Kill now-bogus splhigh() -- it is a nop
Modified:
head/sys/pci/ncr.c
Modified: head/sys/pci/ncr.c
Author: imp
Date: Thu Feb 21 00:27:51 2013
New Revision: 247066
URL: http://svnweb.freebsd.org/changeset/base/247066
Log:
Replace splhigh() with critical_enter()/leave() to ensure we write the
config mode unlock sequence quickly enough. This likely isn't too critical,
since splhigh() has bee
Author: imp
Date: Thu Feb 21 00:26:31 2013
New Revision: 247065
URL: http://svnweb.freebsd.org/changeset/base/247065
Log:
No longer need splhigh() since locking was done, delete it and
comments about it.
Modified:
head/sys/dev/ppc/ppc_isa.c
Modified: head/sys/dev/ppc/ppc_isa.c
Author: jfv
Date: Thu Feb 21 00:25:45 2013
New Revision: 247064
URL: http://svnweb.freebsd.org/changeset/base/247064
Log:
Refresh on the shared code for the E1000 drivers.
- bear with me, there are lots of white space changes, I would not
do them, but I am a mere consumer of this stuff
Author: jilles
Date: Wed Feb 20 23:26:14 2013
New Revision: 247063
URL: http://svnweb.freebsd.org/changeset/base/247063
Log:
MFC r92,r230095: Show errno messages in cd.
Added:
stable/8/tools/regression/bin/sh/builtins/cd8.0
- copied unchanged from r230095,
head/tools/regression/bin/
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 2013-02-20 17:20:50 -0500, Warner Losh wrote:
> Author: imp Date: Wed Feb 20 22:20:49 2013 New Revision: 247059
> URL: http://svnweb.freebsd.org/changeset/base/247059
>
> Log: Remove the unused spl functions: spl0, splsoftcam,
> splsofttty, splsof
Author: np
Date: Wed Feb 20 23:15:40 2013
New Revision: 247062
URL: http://svnweb.freebsd.org/changeset/base/247062
Log:
cxgbe(4): Assume that CSUM_TSO in the transmit path implies CSUM_IP and
CSUM_TCP too. They are all set explicitly by the kernel usually.
While here, fix an unrelated b
Author: pjd
Date: Wed Feb 20 22:59:53 2013
New Revision: 247061
URL: http://svnweb.freebsd.org/changeset/base/247061
Log:
When porting XTS-related code from OpenBSD I forgot to update copyright (only
OpenBSD was credited in one of two commits). Fix it.
Reported by: Theo de Raadt
Revie
Author: bapt
Date: Wed Feb 20 22:51:42 2013
New Revision: 247060
URL: http://svnweb.freebsd.org/changeset/base/247060
Log:
Do not use deprecated functions from libarchive
Modified:
head/usr.sbin/pkg/pkg.c
Modified: head/usr.sbin/pkg/pkg.c
=
Author: imp
Date: Wed Feb 20 22:20:49 2013
New Revision: 247059
URL: http://svnweb.freebsd.org/changeset/base/247059
Log:
Remove the unused spl functions: spl0, splsoftcam, splsofttty,
splsofttq and splstatclock.
Other used spl functions to follow.
Modified:
head/sys/sys/systm.h
Modif
Author: jilles
Date: Wed Feb 20 21:57:01 2013
New Revision: 247058
URL: http://svnweb.freebsd.org/changeset/base/247058
Log:
MFC r246371: sh: Do not test for digit_contig in mksyntax.
ISO/IEC 9899:1999 (E) 5.2.1p3 guarantees that the values of the characters
0123456789 are contiguous.
Author: jilles
Date: Wed Feb 20 21:49:52 2013
New Revision: 247057
URL: http://svnweb.freebsd.org/changeset/base/247057
Log:
MFC r246507: sh: Fix a comment.
Modified:
stable/9/bin/sh/parser.h
Directory Properties:
stable/9/bin/sh/ (props changed)
Modified: stable/9/bin/sh/parser.h
==
Author: des
Date: Wed Feb 20 21:16:50 2013
New Revision: 247056
URL: http://svnweb.freebsd.org/changeset/base/247056
Log:
revert 247035
Modified:
head/sys/dev/ixgbe/ixgbe_phy.c
Modified: head/sys/dev/ixgbe/ixgbe_phy.c
==
Jack Vogel writes:
> My attitude? And what part of courtesy were you missing, did I make
> comments about you or your behavior. At this point I'd say you owe me
> an apology as well as the uncommit.
I'll revert the commit, but you won't get an apology. You and / or
Intel clearly don't understand
My attitude? And what part of courtesy were you missing, did I make comments
about you or your behavior. At this point I'd say you owe me an apology as
well
as the uncommit.
Jack
On Wed, Feb 20, 2013 at 11:07 AM, Dag-Erling Smørgrav wrote:
> Juli Mallett writes:
> > Please don't. Many others
Author: pfg
Date: Wed Feb 20 20:56:07 2013
New Revision: 247055
URL: http://svnweb.freebsd.org/changeset/base/247055
Log:
MFC r246347, r246348, r246349, r246350, r246351, r246352:
ext2fs: Miscellaneous cleanups and fixes.
Use EXT2_LINK_MAX instead of LINK_MAX.
Use nitems().
Corre
Author: emaste
Date: Wed Feb 20 20:42:56 2013
New Revision: 247054
URL: http://svnweb.freebsd.org/changeset/base/247054
Log:
Fix parallel build race with DEBUG_FLAGS
Reported by: Jan Beich on freebsd-current
Tested by: markj
Modified:
head/share/mk/bsd.lib.mk
Modified: head/share/mk/b
Author: pfg
Date: Wed Feb 20 20:12:17 2013
New Revision: 247053
URL: http://svnweb.freebsd.org/changeset/base/247053
Log:
MFC r239359:
Remove unused member of struct indir (in_exists) from UFS and EXT2 code.
Discussed with: mjg
Modified:
stable/9/sys/fs/ext2fs/ext2_b
On Wednesday, February 20, 2013 1:42:00 pm John-Mark Gurney wrote:
> John Baldwin wrote this message on Wed, Feb 20, 2013 at 08:09 -0500:
> > On Tuesday, February 19, 2013 4:35:17 pm John-Mark Gurney wrote:
> > > Author: jmg
> > > Date: Tue Feb 19 21:35:17 2013
> > > New Revision: 247012
> > > URL:
Author: brooks
Date: Wed Feb 20 19:32:31 2013
New Revision: 247052
URL: http://svnweb.freebsd.org/changeset/base/247052
Log:
Support hardlinks in manifest files by the same logic as the treewalk
code.
Reviewed by: marcel
Sponsored by: DARPA, AFRL
Modified:
head/usr.sbin/makefs/makef
Juli Mallett writes:
> Please don't. Many others have accepted/respected Jack's
> maintainership by letting him keep control of the in-tree driver as
> much as is possible, particularly outside of ixgbe.c.
Jack claims to have a hard lock on the driver. While not unheard of, it
is somewhat unusu
Jack Vogel writes:
> No, I'm not confused, I am the owner of the driver, and keep the
> FreeBSD code and the Intel code in sync, any changes to the code
> should be run by me first.
Where is this documented?
DES
--
Dag-Erling Smørgrav - d...@des.no
__
On Wed, Feb 20, 2013 at 10:57 AM, Dag-Erling Smørgrav wrote:
> Jack Vogel writes:
>> OK, this change must be backed out. This was not run past me, and this
>> is a shared code file, that means its code that we license in both
>> GPL, BSD, and closed source licensing, and thus we CANNOT accept
>>
Author: joel (doc committer)
Date: Wed Feb 20 19:05:13 2013
New Revision: 247051
URL: http://svnweb.freebsd.org/changeset/base/247051
Log:
Sort sections.
Modified:
head/lib/libc/stdlib/qsort.3
Modified: head/lib/libc/stdlib/qsort.3
No, I'm not confused, I am the owner of the driver, and keep the FreeBSD
code
and the Intel code in sync, any changes to the code should be run by me
first.
Jack
On Wed, Feb 20, 2013 at 10:57 AM, Dag-Erling Smørgrav wrote:
> Jack Vogel writes:
> > OK, this change must be backed out. This was
Jack Vogel writes:
> OK, this change must be backed out. This was not run past me, and this
> is a shared code file, that means its code that we license in both
> GPL, BSD, and closed source licensing, and thus we CANNOT accept
> changes without special handling. Further, I do not author this code
John Baldwin wrote this message on Wed, Feb 20, 2013 at 08:09 -0500:
> On Tuesday, February 19, 2013 4:35:17 pm John-Mark Gurney wrote:
> > Author: jmg
> > Date: Tue Feb 19 21:35:17 2013
> > New Revision: 247012
> > URL: http://svnweb.freebsd.org/changeset/base/247012
> >
> > Log:
> > add suppor
Author: keramida (doc committer)
Date: Wed Feb 20 18:31:55 2013
New Revision: 247050
URL: http://svnweb.freebsd.org/changeset/base/247050
Log:
Various improvements to the qsort(3) usage example:
- Remove unused #include.
- Do not cast away const.
- Use the canonical idiom to compare two
Author: gibbs
Date: Wed Feb 20 17:55:17 2013
New Revision: 247049
URL: http://svnweb.freebsd.org/changeset/base/247049
Log:
Avoid panic when tearing down the DTrace pid provider for a
process that has crashed.
sys/cddl/contrib/opensolaris/uts/common/dtrace/fasttrap.c:
In fasttrap_
OK, this change must be backed out. This was not run past me, and this is a
shared code
file, that means its code that we license in both GPL, BSD, and closed
source licensing,
and thus we CANNOT accept changes without special handling. Further, I do
not author
this code, its done by another team i
Author: gibbs
Date: Wed Feb 20 17:46:38 2013
New Revision: 247048
URL: http://svnweb.freebsd.org/changeset/base/247048
Log:
Orphaned processes that are being traced are killed by the
kernel. Properly restore, continue, and detach from processes
being DTraced when DTrace exits with an error
Author: kib
Date: Wed Feb 20 17:39:52 2013
New Revision: 247047
URL: http://svnweb.freebsd.org/changeset/base/247047
Log:
Convert machine/elf.h, machine/frame.h, machine/sigframe.h,
machine/signal.h and machine/ucontext.h into common x86 includes,
copying from amd64 and merging with i386.
Author: alc
Date: Wed Feb 20 16:48:52 2013
New Revision: 247046
URL: http://svnweb.freebsd.org/changeset/base/247046
Log:
Initialize vm_max_kernel_address on non-FDT platforms. (This should have
been included in r246926.)
The second parameter to pmap_bootstrap() is redundant. Eliminate
Author: kientzle
Date: Wed Feb 20 16:32:38 2013
New Revision: 247045
URL: http://svnweb.freebsd.org/changeset/base/247045
Log:
Fix "fdt addr" to accept literal addresses rather than va offsets.
When initializing the fdt, query U-Boot as well.
With this change, it is now feasible to have U
On Tuesday, February 19, 2013 4:35:17 pm John-Mark Gurney wrote:
> Author: jmg
> Date: Tue Feb 19 21:35:17 2013
> New Revision: 247012
> URL: http://svnweb.freebsd.org/changeset/base/247012
>
> Log:
> add support for AES and PCLMULQDQ instructions to binutils...
>
> Thanks to Mike Belopuhov
Author: pluknet
Date: Wed Feb 20 15:44:40 2013
New Revision: 247044
URL: http://svnweb.freebsd.org/changeset/base/247044
Log:
ip_savecontrol() style fixes. No functional changes.
- fix indentation
- put the operator at the end of the line for long statements
- remove spaces between the typ
Author: brooks
Date: Wed Feb 20 15:28:40 2013
New Revision: 247043
URL: http://svnweb.freebsd.org/changeset/base/247043
Log:
Allow '.' components in manifest paths. They are always the first
component of mtree -C and install -M output and are easily skipped.
Reviewed by: marcel
Sponso
Author: brooks
Date: Wed Feb 20 15:25:40 2013
New Revision: 247042
URL: http://svnweb.freebsd.org/changeset/base/247042
Log:
Fix the -N option in manifest mode by using pwcache(3). This also
speeds up image creation appreciably.
Reviewed by: marcel
Sponsored by: DARPA, AFRL
Modified:
Author: brooks
Date: Wed Feb 20 15:18:42 2013
New Revision: 247041
URL: http://svnweb.freebsd.org/changeset/base/247041
Log:
Add a -D flag that causes duplicate entries in an mtree manifest to be
treated as warnings rather than errors.
Reviewed by: marcel
Sponsored by: DARPA, AFRL
Mod
Author: melifaro
Date: Wed Feb 20 14:19:09 2013
New Revision: 247037
URL: http://svnweb.freebsd.org/changeset/base/247037
Log:
Fix several new & old style issues.
Pointed by: ae, pluknet, zont
MFC with: r247036
Modified:
head/usr.bin/systat/ifcmds.c
head/usr.bin/systat/ifstat.c
Author: melifaro
Date: Wed Feb 20 13:47:05 2013
New Revision: 247036
URL: http://svnweb.freebsd.org/changeset/base/247036
Log:
Add interface name filtering via 'match' cmd.
Add 'pps' cmd for switching beetween interface packets/bytes statistics.
Submitted by: vsevolod
MFC after:2 we
Author: des
Date: Wed Feb 20 12:59:21 2013
New Revision: 247035
URL: http://svnweb.freebsd.org/changeset/base/247035
Log:
Reduce excessive nesting.
Modified:
head/sys/dev/ixgbe/ixgbe_phy.c
Modified: head/sys/dev/ixgbe/ixgbe_phy.c
==
Author: pluknet
Date: Wed Feb 20 12:40:26 2013
New Revision: 247034
URL: http://svnweb.freebsd.org/changeset/base/247034
Log:
Check if the -sec option is given without an argument.
PR: bin/170413
Submitted by: Andrey Simonenko
MFC after:1 week
Modified:
head/usr.sbin/m
Author: adrian
Date: Wed Feb 20 12:14:49 2013
New Revision: 247033
URL: http://svnweb.freebsd.org/changeset/base/247033
Log:
Configure larger TX FIFO default and maximum level values.
This has reduced the number of TX delimiter and data underruns when
doing large UDP transfers (>100mbit).
On Tue, 29 Jan 2013, Andrey Zonov wrote:
On 1/28/13 6:51 PM, Bruce Evans wrote:
On Mon, 28 Jan 2013, Andrey Zonov wrote:
Log:
- Show page faults requiring I/O on vmstat display.
No space is available there for showing it.
Yep, you're right.
+mvprintw(VMSTATROW, VMSTATCOL + 9, "iof
Author: adrian
Date: Wed Feb 20 11:24:11 2013
New Revision: 247030
URL: http://svnweb.freebsd.org/changeset/base/247030
Log:
If any of the TX queues have underrun reporting enabled, enable
HAL_INT_TXURN in the interrupt mask register.
This should now allow for TXURN interrupts to be poste
Author: adrian
Date: Wed Feb 20 11:22:44 2013
New Revision: 247029
URL: http://svnweb.freebsd.org/changeset/base/247029
Log:
A couple of quick tidyups:
* Delete this debugging print - I used it when debugging the initial
TX descriptor chaining code. It now works, so let's toss it.
Author: adrian
Date: Wed Feb 20 11:20:51 2013
New Revision: 247028
URL: http://svnweb.freebsd.org/changeset/base/247028
Log:
Enable TX FIFO underrun interrupts. This allows the TX FIFO threshold
adjustment code to now run.
Tested:
* AR5416, STA
TODO:
* Much more thorough t
Author: adrian
Date: Wed Feb 20 11:17:29 2013
New Revision: 247027
URL: http://svnweb.freebsd.org/changeset/base/247027
Log:
oops, tab!
Modified:
head/sys/dev/ath/if_ath.c
Modified: head/sys/dev/ath/if_ath.c
==
--- h
Author: adrian
Date: Wed Feb 20 11:17:03 2013
New Revision: 247026
URL: http://svnweb.freebsd.org/changeset/base/247026
Log:
Post interrupts in the ath alq trace.
Modified:
head/sys/dev/ath/if_ath.c
Modified: head/sys/dev/ath/if_ath.c
=
Author: adrian
Date: Wed Feb 20 11:14:55 2013
New Revision: 247025
URL: http://svnweb.freebsd.org/changeset/base/247025
Log:
CFG_ERR, DATA_UNDERRUN and DELIM_UNDERRUN are all flags, rather than
part of ts_status. Thus:
* make sure we decode them from ts_flags, rather than ts_status;
* m
On 2013-02-20 10:49, Stefan Farfeleder wrote:
>On Wed, Feb 20, 2013 at 09:32:43AM +, David Chisnall wrote:
>>On 20 Feb 2013, at 08:25, m...@freebsd.org wrote:
>>> These should be declared const int *. And the cast shouldn't be
>>> needed in C, since void * can be assigned to any other pointer
On Wed, Feb 20, 2013 at 09:32:43AM +, David Chisnall wrote:
> On 20 Feb 2013, at 08:25, m...@freebsd.org wrote:
>
> > These should be declared const int *. And the cast shouldn't be
> > needed in C, since void * can be assigned to any other pointer type.
>
> In fact, the entire function body
On 2013-02-20 09:32, David Chisnall wrote:
> On 20 Feb 2013, at 08:25, m...@freebsd.org wrote:
> > These should be declared const int *. And the cast shouldn't be
> > needed in C, since void * can be assigned to any other pointer type.
>
> In fact, the entire function body can be replaced with:
>
On 20 Feb 2013, at 08:25, m...@freebsd.org wrote:
> These should be declared const int *. And the cast shouldn't be
> needed in C, since void * can be assigned to any other pointer type.
In fact, the entire function body can be replaced with:
return (*(int*)p1 - *(int*)p2);
qsort doesn't req
On Tue, Feb 19, 2013 at 3:57 PM, Giorgos Keramidas wrote:
> Author: keramida (doc committer)
> Date: Tue Feb 19 23:57:39 2013
> New Revision: 247014
> URL: http://svnweb.freebsd.org/changeset/base/247014
>
> Log:
> Add a sample program that shows how a custom comparison function and
> qsort(3)
78 matches
Mail list logo
