Commit Graph

60667 Commits

Author SHA1 Message Date
Axel Dörfler
20a31c45b9 netstat: Added family, protocol, and state filter options
* Uses the same option names as net-tools's netstat.

Change-Id: I3363a091dfa1bcf09065f77d3fdc9c9bf27cbcaf
Reviewed-on: https://review.haiku-os.org/c/haiku/+/1701
Reviewed-by: Adrien Destugues <pulkomandy@gmail.com>
2019-08-12 22:09:34 +00:00
Augustin Cavalier
96e64e6e14 Application Kit: Add B_CLONEABLE_AREA permission to cloned server areas.
The Media Kit needs this for overlays.
2019-08-11 15:27:51 -04:00
Augustin Cavalier
c8e11499cc XHCI: Force an event ring poll if a first timeout occurs on commands.
FreeBSD does something similar here. May help with some of the
stranger event ring lockups.
2019-08-10 23:45:15 -04:00
Augustin Cavalier
8bfe3d25f4 XHCI: Rename ERST_EHB to ERDP_BUSY for clarity's sake.
No functional change intended.
2019-08-10 23:02:20 -04:00
Augustin Cavalier
a614ac8101 XHCI: Remove unused and unnecessary data structure definitions. 2019-08-10 22:48:01 -04:00
Augustin Cavalier
da73ed96dd kernel/vm: Enable area-cloning protection for userland areas, too.
We allow teams to clone areas within themselves, but I'm not sure
exactly what use that has. The kernel can of course clone anything
it wants to, still.

Hopefully this will prove substantially less disruptive than the
reverse change last year, as the preceding commits are likely the
only major consumers of this API, rather than a variety of drivers
that need to be individually tested.
2019-08-10 19:49:24 -04:00
Augustin Cavalier
126d639908 libroot: Set B_CLONEABLE_AREA under BeOS ABI.
As you can see, we already did the same for B_EXECUTE_AREA.
2019-08-10 19:47:56 -04:00
Augustin Cavalier
329866d1d0 kernel/vm: Set CLONEABLE_AREA before cloning areas for transfer. 2019-08-10 19:45:57 -04:00
Augustin Cavalier
40b0fbbb5d runtime_loader: Set B_CLONEABLE_AREA on the debug area.
The debug_server uses it for symbol lookup.
2019-08-10 17:55:47 -04:00
Augustin Cavalier
964cab4615 media: Set B_CLONEABLE_AREA where applicable.
Cloning cloned buffers seems especially strange, but
apparently it is actually done. The buffer management
code really needs to be cleaned up...
2019-08-10 17:55:27 -04:00
Augustin Cavalier
23121bac3b app_server: Set B_CLONEABLE_AREA where applicable.
This is all that is needed to boot a minimum image with userland
area cloning protections enabled (media_server is not included
in such builds.)
2019-08-10 16:13:28 -04:00
Augustin Cavalier
8a0c9d52c6 OS: Rename B_USER_CLONEABLE_AREA to B_CLONEABLE_AREA.
It now lives in OS.h. The idea is that this will now be
accessible to userland applications, so userland memory
is protected from access by other processes, just as
kernel memory is.

No functional change (the constants are still the same,
though I've changed some to use shifts to make clear
which bits are allocated are which are unused.)
2019-08-10 15:51:41 -04:00
Augustin Cavalier
e4d335404f package_infos: Set system_package flag for x86_64. 2019-08-10 14:36:25 -04:00
Augustin Cavalier
3198a0af7f package_infos: "haiku" no longer provides compress/tput. 2019-08-10 14:36:13 -04:00
PulkoMandy
d5cd4a9d51 openfirmware: adjust for 64bit
Sparcv9 runs Openboot in 64 bit mode, which means the cell size is
64bit. Use intptr_t where appropriate to make the open firmware calls
work.

Beware, some values are still 32bit, this matters for example for
of_getprop, if you get 32bits into a 64bit variables it will be in the
MSB of it (big endian only weakness...) and confuse things. See for
example in console.cpp, where the input and output handles are retrieved
as 32bit values. It seems wise to check the expected size when using
of_getprop in these cases, instead of just checking for errors.

Change-Id: Ie72ebc4afe7c6d7602a47478f0bfb6b8247004b8
Reviewed-on: https://review.haiku-os.org/c/1369
Reviewed-by: waddlesplash <waddlesplash@gmail.com>
2019-08-10 17:59:04 +00:00
Adrien Destugues
bb6e4f5d94 Boot menu: clarify variable usage
The video mode hook gets passed a menu and an item, but is actually
interested in the submenu pointed by the item and the items inside that.

Don't reuse the passed parameters and instead declare new local
variables.

Change-Id: Id17adde552fac2c248afa78380ff299fb21bc968
Reviewed-on: https://review.haiku-os.org/c/1693
Reviewed-by: waddlesplash <waddlesplash@gmail.com>
2019-08-10 17:58:09 +00:00
Adrien Destugues
95901dc145 Bluetooth replicant: fix memory leak.
Change-Id: I4587fc9c0d981bafd38e0f517f6ef60a8cae7fec
Reviewed-on: https://review.haiku-os.org/c/1696
Reviewed-by: Jérôme Duval <jerome.duval@gmail.com>
2019-08-10 17:57:28 +00:00
CodeforEvolution
6a739773fc CodyCam: Style Fixes, Make Resource File Verbose
No functional change intended, added some copyrights
in the process.

Change-Id: Ic6ebe276bb939c425d65aff1b3b07c7c2e27a33a
Reviewed-on: https://review.haiku-os.org/c/1698
Reviewed-by: waddlesplash <waddlesplash@gmail.com>
2019-08-10 17:57:03 +00:00
Ryan Leavengood
f5953f6155 Package Kit: Purposely ignore an error getting package cache
Fixes #15230.

The comment pretty much explains it. I think needing this means this code could
be redesigned, but I don't know enough yet to make that fix.

It also feels like this whole job system and Command pattern is overly
complicated when good old functions would be fine, certainly for pkgman. But
maybe this is used more heavily in HaikuDepot.
2019-08-09 23:13:52 -04:00
Augustin Cavalier
6a23120503 kernel/fs: Fix another instance of free() being called on a FD.
Fixes the second panic in #15213.
2019-08-08 23:34:40 -04:00
Ryan Leavengood
b711002d34 HaikuDepot: Set package state when loading single package
Without this, even installed packages still get an "Install" button.

Fixes #14821.

This was implemented by adding BPackageRoster::IsPackageActive. I decided to
have this take a location since GetActivePackages also did, but as noted in my
TODO comment, I think this is awkward.

It would also be nice to show the user they have a different version of a
particular package, but that would require some changes to IsPackageActive.

Change-Id: Iab0d35eb6b671a17711b0214b15164d296927e5a
Reviewed-on: https://review.haiku-os.org/c/1694
Reviewed-by: Stephan Aßmus <superstippi@gmx.de>
2019-08-08 07:09:35 +00:00
Augustin Cavalier
925c3a133b kernel/thread: Check the proper field to determine if the thread is blocked.
Fixes #15234.
2019-08-07 23:52:03 -04:00
Alexander von Gluck IV
b18e88a384 preferences/network: Add modem icon for dial-up 2019-08-07 09:44:03 -05:00
Alexander von Gluck IV
e27cfa1b9f preferences/network: Ungroup VPN and Dialup
* After a bunch of trials, the groupings for VPN and Dial-up
  connections look too busy.  By putting the "connections"
  at the same level as the interfaces things get a bit clearer
  and a little more modern.
* Lots of work to do still for "adding / removing" connections.
2019-08-07 08:18:17 -05:00
Alexander von Gluck IV
7424792a9f preferences/network: Fix status labels for stateful connections 2019-08-07 08:13:59 -05:00
Alexander von Gluck IV
f932d9902f preferences/network: Add dedicated VPN icon 2019-08-06 19:32:18 -05:00
Alexander von Gluck IV
68378c326e network_settings/vpn: Make add-on more generic. 2019-08-06 18:07:10 -05:00
Adrien Destugues
87bdc2b02b fix dumping semaphores by name
There was no check for strtoul success or failure (it returns 0 on
failure but that is also a valid conversion result). Detect if endptr
has been advanced instead (meaning there were some parsed characters in
strtoul so the argument at least starts with a number)

Change-Id: Ieefbd57a250ddcdb9362094389151c2a432e4c73
Reviewed-on: https://review.haiku-os.org/c/1683
Reviewed-by: Rene Gollent <rene@gollent.com>
2019-08-06 19:20:39 +00:00
Adrien Destugues
58d16d9fe2 rt2860: fix off-by-two access
To be upstreamed to FreeBSD?

Change-Id: Ia4389f4964429f80025003f97792b1ecd1097193
Reviewed-on: https://review.haiku-os.org/c/1681
Reviewed-by: Rene Gollent <rene@gollent.com>
2019-08-06 19:20:39 +00:00
Adrien Destugues
639bb82701 AboutSystem: remove NetBSD credits
We don't ship ftp anymore, and our tput comes from ncurses now.

Change-Id: I80e99b72f5bb16147a234d6d72a277a5035b6e00
Reviewed-on: https://review.haiku-os.org/c/1680
Reviewed-by: Rene Gollent <rene@gollent.com>
2019-08-06 19:20:39 +00:00
Adrien Destugues
ec2b89264c oce_if: fix oout of bounds access
Change-Id: Id052d9e7714c0251e39384671da9a0f76f6c13c4
Reviewed-on: https://review.haiku-os.org/c/1682
Reviewed-by: Rene Gollent <rene@gollent.com>
2019-08-06 03:18:36 +00:00
Augustin Cavalier
af0be8dbc5 kernel/condition_variable: Clean up comments and reduce needless unlocks.
This "race prevention" code does not seem to be really hit at all
in practice, at least from testing, so no need to do a full
unlock/lock universally for it.

I'm still not sure why the previous fixes here removed 80% of
the performance benefits of the original change; I need to
investigate that more.
2019-08-05 22:58:28 -04:00
Augustin Cavalier
2c588b031f kernel: Properly separate and handle THREAD_BLOCK_TYPE_USER.
Consider this scenario:
 * A userland thread puts its ID into some structure so that it
   can be woken up later, sets its wait_status to initiate the
   begin of the wait, and then calls _user_block_thread.
 * A second thread finishes whatever task the first thread
   intended to wait for, reads the ID almost immediately
   after it was written, and calls _user_unblock_thread.
 * _user_unblock_thread was called so soon that the first
   thread is not yet blocked on the _user_block_thread block,
   but is instead blocked on e.g. the thread's main mutex.
 * The first thread's thread_block() call returns B_OK.
   As in this example it was inside mutex_lock, it thinks
   that it now owns the mutex.
 * But it doesn't own the mutex, and so (until yesterday)
   all sorts of mayhem and then a random crash occurs, or
   (after yesterday) an assert-failure is tripped that
   the thread does not own the mutex it expected to.

The above scenario is not a hypothetical, but is in fact the
exact scenario behind the strange panics in #15211.

The solution is to only have _user_unblock_thread actually
unblock threads that were blocked by _user_block_thread,
so I've introduced a new BLOCK_TYPE to differentiate these.
While I'm at it, remove the BLOCK_TYPE_USER_BASE, which was
never used (and now never will be.) If we want to differentiate
different consumers of _user_block_thread for debugging
purposes, we should use the currently-unused "object"
argument to thread_block, instead of cluttering the
relatively-clean block type debugging code with special
types.

One final note: The race condition which was the case of
this bug does not, in fact, imply a deadlock on the part
of the rw_lock here. The wait_status is protected by the
thread's mutex, which is acquired by both _user_block_thread
and _user_unblock_thread, and so if _user_unblock_thread
succeeds faster than _user_block_thread can initiate
the block, it will just see that wait_status is already
<= 0 and return immediately.

Fixes #15211.
2019-08-05 22:31:02 -04:00
Pascal Abresch
64a4b621b8 added media and acpi keys to ps/2 driver
Change-Id: I92999325785dd81d2425cc06c70e9bedc26379ec
Reviewed-on: https://review.haiku-os.org/c/haiku/+/1675
Reviewed-by: Adrien Destugues <pulkomandy@gmail.com>
2019-08-05 21:50:34 +00:00
Alexander von Gluck IV
6c331fc7e1 kit/package: Error on short file read
* It's safe to assume that if the file is shorter than
  the provided header, things will go poorly.
* Avoids a random vauge ReadBuffer error.
* This doesn't fix #15230, but makes the issue clearer.

Change-Id: I3471e6de384a0c9be94049ad891c01be980f7846
Reviewed-on: https://review.haiku-os.org/c/1679
Reviewed-by: Adrien Destugues <pulkomandy@gmail.com>
2019-08-05 15:07:24 +00:00
Andrew Lindesay
02e836dc3c HaikuDepot: Barber Pole Idle Appearance
This change renders a striped background in the
'barber pole' in HaikuDepot application when it is
idle.  This makes the 'barber pole' easier to use
in situations where the space that the UI control
takes up should not be blank.  The logic for the
striped background is from the 'drivesetup'
application.

Change-Id: I87791c70b4d1a21d91e661433d6c940ca69ece87
Reviewed-on: https://review.haiku-os.org/c/1674
Reviewed-by: Adrien Destugues <pulkomandy@gmail.com>
2019-08-05 07:39:39 +00:00
Augustin Cavalier
719cfad7fd kernel/user_debugger: Don't try to cast an integer to a pointer.
Should fix the x86_64 build.
2019-08-04 20:12:53 -04:00
Augustin Cavalier
9dc67a1d96 kernel/condition_variable: Rework remove-from-condition logic again.
This avoids a few potential race conditions mmlr pointed out on
the mailing list. See inline comments.

Change-Id: I605523c1d2683c749751599c417a68a20c70edea
2019-08-04 20:10:15 -04:00
Augustin Cavalier
8dae411c1e kernel/locks: Tweak mutex_unlock KDEBUG semantics slightly.
Previously, both the "old" owner and the "new" lock owner set
the lock holder to the "new" lock holder. Now the old owner does
it before calling unblock(), and we check in the lock() functions
that we are indeed the new owner of the lock.

This may affect what the panic is for #15211 and its duplicates:
I have a suspicion that these threads are getting unblocked
when they have no business being unblocked, and this should
catch that condition.
2019-08-04 19:08:03 -04:00
Augustin Cavalier
24f7b647cd kernel/user_debugger: Initialize the condition variables.
Should (actually) fix #15223 and #15226. After things settle
down a bit, I'll refactor our ConditionVariable constructors
to detect this at compile time instead of creating obscure
KDLs...
2019-08-04 15:52:01 -04:00
Autocomitter
7739bbedf8 Update translations from Pootle 2019-08-04 08:05:05 +00:00
Augustin Cavalier
287428271d kernel/condition_variable: Acquire and release locks in the right order.
Should fix #15223.
2019-08-03 20:31:41 -04:00
Augustin Cavalier
2f1e2ae469 kernelland_emu: Adapt to new ConditionVariable API.
This could use some optimizations, but it's only used by
userlandfs for now.
2019-08-03 11:46:05 -04:00
Augustin Cavalier
37eda488be kernel/condition_variable: Granularize locking.
Before this commit, *all* ConditionVariable operations (yes, all;
even Wait, Notify, etc.) went through a single spinlock, that also
protected the sConditionVariableHash. This obviously does not scale
so well with core count, to say the least!

With this commit, we add spinlocks to each Variable and Entry.
This makes locking somewhat more complicated (and nuanced; see
inline comment), but the trade-off seems completely worth it:

(compile HaikuDepot in VMware, 2 cores)
before
real 1m20.219s
user 1m5.619s
sys  0m40.724s

after
real 1m12.667s
user 0m57.684s
sys  0m37.251s

The more cores there are, the more of an optimization this will
likely prove to be. But 10%-across-the-board is not bad to say
the least.

Change-Id: I1e40a997fff58a79e987d7cdcafa8f7358e1115a
2019-08-03 11:24:34 -04:00
Augustin Cavalier
489612d43f kernel/condition_variable: Reorder some methods.
This way the static functions (used only in KDL) are below
the actual notification methods, which are now all grouped
together in one place.

No functional change.
2019-08-03 11:12:06 -04:00
waddlesplash
6cb38c6319 low_resource_manager: Do not spin forever during critical low resource conditions.
Since this thread has a very high priority, this causes the whole system
to lock up, making recovery from such a critical condition even more
impossible. Instead use the Warning-level timeout instead (0.3 seconds),
and of course we will be notified via the semaphore if something occurs
we should know about.
2019-08-02 22:18:56 -04:00
Ryan Leavengood
47df2afbf8 ShowImage: Wrap to first image when going next on last image
I find this handy and in addition it makes the slide show cyclic again,
fixing #10386.

Going the other way might also be useful, but that requires some additions to
the Navigator code first.
2019-08-02 21:27:14 -04:00
Augustin Cavalier
677e901f80 kernel/module: Utilize RecursiveLocker. 2019-08-02 21:06:09 -04:00
Augustin Cavalier
afb345915d linkcatkeys: Avoid discarding potential error values.
Spotted by Clang and the [[nodiscard]] patch.
2019-08-02 21:06:09 -04:00
Augustin Cavalier
dfbf1c8a0a Package Kit: Avoid discarding potential error values.
Spotted by Clang and the [[nodiscard]] patch.
2019-08-02 21:06:09 -04:00