NEWS

Merge pull request #1452 from poettering/journal-vacuum

A variety of journal vacuuming improvements, plus an nspawn fix

Merge pull request #1462 from lnykryn/analyze-manpage

man: LEVEL in systemd-analyze set-log level is not optional

Merge pull request #1406 from blaskovic/journal-remote-typo

journal-remote: typo in log_error when no sources are specified

[tomegun: this was a pun, but let's not do that]

Merge pull request #1448 from bengal/lldp-export-v5

LLDP: add API to export neighbors list (v5)

Merge pull request #1459 from ssahani/bridge1

networkd: add bridge properties

Merge pull request #1372 from jemk/prefsrc

networkd: add support to configure preferred source of static routes

man: LEVEL in systemd-analyze set-log level is not optional

rhbz#1268336

Merge pull request #1461 from Danielmachon/patch-1

Update NEWS

Update NEWS

Fixed spelling error: seriban -> serbian

NEWS update

man: add man for bridge params

networkd: add bridge properties

ForwardDelaySec:   forward delay
HelloTimeSec:      hello time
MaxAgeSec:         maximum message age

for more information see
http://www.tldp.org/HOWTO/BRIDGE-STP-HOWTO/set-up-the-bridge.html

In kernel
br_dev_newlink: does not have the this functionality to set while
creation.
br_changelink: after creation we can change the parameters.

we need to first create then set it the parameters.

Introduce new callback post_create .This should
set the properties after the creation.

sd-rtnl: introduce new API to set the NL header flags

By default we set as NLM_F_CREATE | NLM_F_EXCL in
sd_rtnl_message_new_link
But incase of bridge we need to set NLM_F_REQUEST | NLM_F_ACK.

If NLM_F_EXCL is set then we are unable to set the parameters. As bridge
supports setting properties after creation not during creation.

sd-rtnl: sd-netlink: add support for bridge NL parametrs

Rename rtnl_link_info_data_bridge_types to
rtnl_link_bridge_management_types
as they are of nested types of IFLA_AF_SPEC.

missing.h : add bridge params

Merge pull request #1457 from martinpitt/master

keymap: Add Thinkpad Yoga 12 (2015)

Merge pull request #1454 from kostich/master

po, catalog: added Serbian translation.

keymap: Add Thinkpad Yoga 12 (2015)

Fixes #1440

po, catalog: added Serbian translation.

core: don't unset reload result, unless we begin a start or reload operation

Much like the result of the service itself we should not reset the
reload result unless we actually start from the beginning, so that
clients can query it at any time.

Specifically, let's reset the result states only when we begin with a
start operation (for both the main result, and the reload result), when
we begin with a reload operation (only for the load result), or when the
use explicitly asks for that via "systemctl reset-failed".

This is a more generic fix for #1447.

Fixes #1447.

Merge pull request #1450 from evverx/man-journal-fields

man: systemd.journal-fields: add info about _TRANSPORT=audit

man: include numeric prefixes in example file names for .link, .netdev, .network files

In order to avoid confusion with the default files we ship, let's use a
low prefix number for all examples.

Fixes #1409.

man: fix journalctl --syslog-identifier= documentation

We don't actually accept patterns, hence don't claim so. This is a
fix-up for 730836403aee5f5bb998e6e3622ea7068fce0699.

Fixes #1256.

man: document that the automatic journal limits are capped to 4G by default

Fixes #1441.

nspawn: fix --image= when nspawn is run as service

nspawn needs access to /dev/loop to implement --image=, hence grant that
in the service file.

Fixes #1446.

man: systemd.journal-fields: add info about _TRANSPORT=audit

journal: rework vacuuming logic

Implement a maximum limit on number of journal files to keep around.
Enforcing a limit is useful on this since our performance when viewing
pays a heavy penalty for each journal file to interleve. This setting is
turned on now by default, and set to 100.

Also, actully implement what 348ced909724a1331b85d57aede80a102a00e428
promised: use whatever we find on disk at startup as lower bound on how
much disk space we can use. That commit introduced some provisions to
implement this, but actually never did.

This also adds "journalctl --vacuum-files=" to vacuum files on disk by
their number explicitly.

journal: use automatic clenup for ACL types

journal: improve some messages

Indicate that we are ignoring errors, when we ignore them, and log that
at LOG_WARNING level.

Use the right error code for the log message.

journal: simplify things by using the LESS_BY() macro

journal: don't affect atime of journal files when vacuuming

Let's try to use O_NOATIME if we can when vacuuming old journal files,
if we have the permissions for it, so that vacuuming doesn't count as
proper journal read access.

journal: prefer stack allocation

journal: make journal_file_close() return NULL

The way it is customary everywhere else in our sources.

journal: port over to fd_getcrtime_at()

Let's use fd_getcrtime_at(), since that *at() family of calls is how we
read the rest of the file metadata, too.

lldp: add reception tests

Add some tests to simulate the reception of LLDP frames and to verify
the correctness of the data in the MIB.

lldp: fix parsing of TLV length

tlv_packet_read_bytes() and tlv_packet_read_string() returned the
wrong length when called after other functions which modify the offset
in the container.

In other words, if the TLV data length is X and we do a
tlv_packet_read_u8(), a subsequent tlv_packet_read_bytes() should
return a length of (X - 1).

lldp: move lldp_receive_packet() to lldp-internal.c

In order to implement tests for the LLDP state machine, we need to
mock lldp_network_bind_raw_socket(). Move the other function
lldp_receive_packet() to another file so that we can replace the first
function with a custom one and keep the second one.

lldp: check return value of lldp_tlv_packet_exit_container()

lldp: add support for organizationally specific TLVs

LLDP TLVs of type 127 are used to carry organizationally specific
information and include additional fields to specify the OUI and
subtype.

Add support for parsing such fields and functions to access the most
common IEEE 802.1 specific TLVs.

lldp: factor out common code in lldp-tlv.c

lldp: add sd_lldp_tlv_packet_get_destination_type()

It can be useful to know the destination address of a LLDP frame
because it determines the scope of propagation of the frame and thus
this information be used to know whether the neighbor is connected to
the same physical link.

See clause 7.1 of IEEE Std 802.1AB-2009.

lldp: add public function to export LLDP TLV packets

Add a public function to get a list of current LLDP neighbours' TLV
packets. The function populates an array of pointers to the opaque
type sd_lldp_packet and returns the number of elements found. Callers
must take care of freeing the array and decreasing the refcount of
elements when done.

lldp: move TLV related functions to lldp-tlv.c

Move some public functions from lldp-internal.c to lldp-tlv.c, as now
they are not internal functions anymore.

lldp: export opaque TLV type and accessor functions

Export struct tlv_packet as a public opaque sd_lldp_packet type and
make its accessor functions public.

lldp: change order of arguments of lldp_read_*() functions

These functions are going to be exported, swap the 'data' and 'length'
arguments so that their signature is consistent with the rest of the
code.

lldp: add reference counter to struct tlv_packet

Add a reference counter to the tlv_packet structure so that it can be
shared between multiple users and properly free'd when no longer in
use.

util: rework fgetxattrat_fake() to use O_PATH

That way, we don't ever open the file, thus leave the atime untouched,
and this works even when unprivileged.

Merge pull request #1439 from poettering/CODING_STYLE

Coding style

logind: remove warn_melody() logic

The internal speaker is usually not available on modern latops that
support suspend, and even if it is available in the hardware, most
distributions turned support for it off in the kernel. And even if it is
enabled, it's probably still a bad idea to make use of it for the
suspend-failures. If anything a proper sound should be played.

Long story short, let's remove support of this anachronism.

update CODING_STYLE with various additions

Merge pull request #1428 from franciozzy/tagenhance

Enhance matching of TAG keys

Merge pull request #1438 from poettering/rfkill-rework

An rfkill rework among other things

Merge pull request #1430 from evverx/driver-syslog

journald: add syslog fields for driver messages

update NEWS

rfkill: rework and make it listen on /dev/rfkill

With this rework we introduce systemd-rfkill.service as singleton that
is activated via systemd-rfkill.socket that listens on /dev/rfkill. That
way, we get notified each time a new rfkill device shows up or changes
state, in which case we restore and save its current setting to disk.

This is nicer than the previous logic, as this means we save/restore
state even of rfkill devices that are around only intermittently, and
save/restore the state even if the system is shutdown abruptly instead
of cleanly.

This implements what I suggested in #1019 and obsoletes it.

core: add new setting Writable= to ListenSpecial= socket units

Writable= is a new boolean setting. If ture, then ListenSpecial= will
open the specified path in O_RDWR mode, rather than just O_RDONLY.

This is useful for implementing services like rfkill, where /dev/rfkill
is more useful when opened in write mode, if we want to not only save
but also restore its state.

systemctl: fix how we decode types and states

There was a bad memory access among other smaller issues.

man: minor wording improvements to USB FunctionFS documentation

fix typo in NEWS

Merge pull request #1434 from reverendhomer/patch-1

systemctl: fix memory leak in systemctl_parse_argv

Merge pull request #1433 from michich/log-errno

tree-wide: add missing errno arguments to log_*_errno()

udev: fix format string after conversion to log_*_errno()

Use %m where previously %s was used together with strerrno().

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

sd-device: fix format strings after conversion to log_*_errno()

Use %m where previously %s was used together with strerrno().

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

systemctl: fix memory leak in systemctl_parse_argv

This commit fixes Coverity #1325228

tree-wide: add missing errno arguments to log_*_errno()

A few of the recent conversions to log_*_errno() were missing the errno
value arguments.

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

Merge pull request #1429 from poettering/dhcp-hostname

DHCP and journal remoting fix, as well as NEWS update

Merge pull request #1427 from dbuch/master

sd-pppoe: use strna() instead of strempty()

Merge pull request #1426 from poettering/log-syntax

logging fixes and more

journald: add syslog fields for driver messages

journal: make sure to set MHD_USE_PIPE_FOR_SHUTDOWN for libmicrohttpd servers

This makes sure libmicrohttpd won't call shutdown() on our listening
sockets, which make sure socket activation and re-activation will work
cleanly.

See:

https://github.com/systemd/systemd/pull/1286
https://lists.gnu.org/archive/html/libmicrohttpd/2015-09/msg00014.html

Fixes #1286

dhcp: make sure we can deal with a single trailing NUL byte in the hostname

Fixes #1337

update NEWS

sd-pppoe: use strna() instead of strempty()

In cases where we use for instance. log_debug() with a statement + string
it is better to output (Service-name: N/A) instead of (Service-name: )

Support OP_NOMATCH for TAG key

The TAG key can be used in rules for event matching. At the moment, it
does not support inequality tests. This patch enhances the key test to
validate the rule if it does not contain a given TAG (by TAG!="value").

Signed-off-by: Felipe Franciosi <felipe@paradoxo.org>

Merge pull request #1425 from kaysievers/wip

gpt-auto-generator: check fstab for /boot entries

Merge pull request #1422 from keszybz/sd-daemon-mq-badf

sd-daemon: fix return value for sd_is_mq

Merge pull request #1419 from keszybz/shell-completion

Shell completion tweaks

core: make sure we expose the empty string when we detect no virtualization

On the bus, stick to exposing the empty string as "no virtualization",
instead of none.

Fixes #1423

tree-wide: remove a number of invocations of strerror() and replace by %m

Let's clean up our tree a bit, and reduce invocations of the
thread-unsafe strerror() by replacing it with printf()'s %m specifier.

basic: split out cpu set specific APIs into cpu-set-util.[ch]

conf-parser: make use of DEFINE_PARSER macro to parse mode_t

Let's unify more code.

Also, rework the macro to accept a trailing semicolon, to make the code
prettier.

util: there cannot be trailing garbage when parsing cpu sets

extract_first() already skips trailing whitespace, hence no reason to
explicitly check for it.

util: rename parse_cpu_set() to parse_cpu_set_and_warn()

It's pretty untypical for our parsing functions to log on their own.
Clarify in the name that this one does.

log: move log_invalid_utf8() to log.h

Also, make sure it follows the same scheme as log_syntax() does in its
behaviour.

tree-wide: clean up log_syntax() usage

- Rely everywhere that we use abs() on the error code passed in anyway,
  thus don't need to explicitly negate what we pass in

- Never attach synthetic error number information to log messages. Only
  log about errors we *receive* with the error number we got there,
  don't log any synthetic error, that don#t even propagate, but just eat
  up.

- Be more careful with attaching exactly the error we get, instead of
  errno or unrelated errors randomly.

- Fix one occasion where the error number and line number got swapped.

- Make sure we never tape over OOM issues, or inability to resolve
  specifiers

Merge pull request #1424 from evverx/journalctl-rotate

journalctl: add --rotate option

gpt-auto-generator: check fstab for /boot entries

We need to prevent the creation of the gpt automount unit, which will not
get overridden by the fstab mount unit.

https://github.com/systemd/systemd/issues/1378

man: journalctl: add info about --rotate

shell-completion: journalctl: add --rotate

journalctl: add --rotate option

shortcut for `systemctl kill --kill-who main --signal SIGUSR2 systemd-journald`

sd-daemon: fix sd_is_mq for non-mq fds

mq_getattr returns -1/EBADF for file descriptors which are not mq.
But we should return 0 in this case.

We first check that fd is a valid fd, so we can assume that if
mq_getattr returns EBADF, it is simply a non-mq fd. There is a slight
race, but there doesn't seem to be a nice way to fix it.

Merge pull request #1421 from keszybz/formatting-fixes

logind: avoid a dot before parenthesis

Merge pull request #1420 from keszybz/doc-updates

Doc updates

systemctl: do not set variable to be immediately overwritten

NEWS: typos

man: update description of exit.target

exit.target is now used for both system and user sessions,
so remove "on user service manager exit". Also reword that
paragraph: services will be killed before the manager exits,
even if they do not conflict with shutdown target, but we
recommend that they conflict with shutdown target so that
systemd schedules them to be stopped immediately when starting
to exit.

In the first paragraph, containers should be mentioned last,
and the more general systems first.

logind: avoid a dot before parenthesis

systemd-logind[27]: System is rebooting. (Applied kernel updates.)

is changed to

systemd-logind[27]: System is rebooting (Applied kernel updates).

Users should not add a dot in the sentence in --message, i.e. the correct usage is now:
$ systemctl reboot --message "Applied kernel updates"

man: add link to kernel docs for net_cls

man: use "=" when referring to configuration file settings

This convention is almost universal in systemd man pages, and makes
it easier to visually parse the docs.

Also fix some markup along the way.