Age | Commit message (Collapse) | Author | Files | Lines |
|
This drop adds socket-id for first console. This was left out before
but with the new design this parameter is required and enforced by
obmc-console code. It is decided to set "socket-id = console0"
initially because that is the current path used by bmcweb but in future
you can change this value to whatever is appropriate.
The console information is now available through the DBUS interface.
Each console server register it's object path on DBUS. The leaf node of
the object path is used to create a console path. For example in the
following console0 the leaf node "console0" is used to make up the
console path and the GUI will use console path "/console/console0" to
connect to the console.
busctl tree xyz.openbmc_project.Console.console0
`-/xyz
`-/xyz/openbmc_project
`-/xyz/openbmc_project/console
`-/xyz/openbmc_project/console/console0
Here is a discord thread where new design was discussed.
https://discord.com/channels/775381525260664832/1083551792094249051
Tested:
Tested on the rainier system.
Related commits:
1) phosphor-dbus-interface: https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/61486
2) obmc-console: https://gerrit.openbmc.org/c/openbmc/obmc-console/+/62496
3) bmcweb: https://gerrit.openbmc.org/c/openbmc/bmcweb/+/62525
Change-Id: I5fd06f3aac69872a3e5032d592a9b59711795cfe
Signed-off-by: Ninad Palsule <ninadpalsule@us.ibm.com>
|
|
IntelCPUSensors supports communication via peci-legacy.
This communication should be done via specific middleware libpeci.
Add the corresponding dependency to make it work.
Change-Id: Ibc381a7406d33f074031b0abcc2aa88164b7c1d1
Signed-off-by: Oleksandr Shulzhenko <oleksandr.shulzhenko.viktorovych@intel.com>
|
|
Upstream obmc-console now supports the meson build system. I consider
the autotools build system deprecated and intend to remove it.
Change-Id: Ic8eea195bee0b3bea1cb9418593596ce1aaceeae
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Andrew Jeffery (21):
config: Remove use of %m sscanf format specifier
obmc-console: Convert build to meson
console-server: tty_find_device(): Assign rc on error
test: test-config-parse-logsize: Don't print size uninitialised
obmc-console: Add clang-format configuration
obmc-console: Fix bugprone-implicit-widening-of-multiplication-result
obmc-console: Fix bugprone-macro-parentheses
console-server: Validate values and fix narrowing conversions
obmc-console: Fix bugprone-narrowing-conversions
obmc-console: Extract definition of _GNU_SOURCE
obmc-console: Fix bugprone-reserved-identifier
obmc-console: Address bugprone-sizeof-expression
obmc-console: Fix cert-dcl03-c,misc-static-assert
obmc-console: Fix readability-braces-around-statements
obmc-console: Fix readability-else-after-return
obmc-console: Fix readability-inconsistent-declaration-parameter-name
obmc-console: Fix readability-isolate-declaration
obmc-console: Fix ringbuffer-test-utils analysis
obmc-console: Add clang-tidy configuration
test: test-config-parse-logsize: Fix invalid size_t format specifier
meson: Install obmc-console-server in $sbindir for consistency
Ninad Palsule (1):
Changes for new pre-defined usergroup hostconsole
Change-Id: I5f226c4b8177a385e80a2fd211e2ff4709043a62
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
The new pre-defined usergroup named "hostconsole" is added to
differentiate access between host console and manager console.
The only users allowed to interact with host console are part of the
"hostconsole" group.
This is a fixed is the github issue:
https://github.com/openbmc/phosphor-user-manager/issues/15
In commit https://gerrit.openbmc.org/c/openbmc/bmcweb/+/50835 ssh was
mapped to both ManagerConsole and HostConsole. The split is discussed
in the commit https://gerrit.openbmc.org/c/openbmc/bmcweb/+/50835?tab=comments
Note: The changes are spread across multiple repositories listed under
"Related commits:"
The openbmc changes are as follows:
- Removed a dependency on dropbear.default file. Added a new environment
file dropbear.env for obmc-console. If we want to add port specific
configuration then we can add dropbear.%i.env file.
- The DROPBEAR_EXTRA_ARGS variable updated to include "-G hostconsole"
flag.
- New update script added to add new hostconsole group and also add all
users part of the priv-admin group to this new group.
- Similarly changes are made to add new group during install time and
add root user in this group.
Tested:
Loaded on system and qemu eumulator. Made sure that the only user
can ssh to host console are member of hostconsole group.
Related commits:
docs: https://gerrit.openbmc.org/c/openbmc/docs/+/60968
phosphor-user-manager: https://gerrit.openbmc.org/c/openbmc/phosphor-user-manager/+/61583
openbmc: https://gerrit.openbmc.org/c/openbmc/openbmc/+/61582
obmc-console: https://gerrit.openbmc.org/c/openbmc/obmc-console/+/61581
bmcweb: https://gerrit.openbmc.org/c/openbmc/bmcweb/+/61580
Change-Id: Icced48da188fb76828bf4ff5c705d6f1300ae3e7
Signed-off-by: Ninad Palsule <ninadpalsule@us.ibm.com>
|
|
usb0 is a virtual USB Ethernet which is configured to static IP
192.168.0.10 in /etc/systemd/network/00-bmc-usb0.network without DHCP
parameter with expecting that DHCP=no.
However, IPMI and Redfish report IP source as DHCP, not static.
This commit explicitly specifies in network configuration as DHCP=no so
that IPMI and Redfish report correctly.
Tested:
1. Check address source for usb0 via IPMI
# ipmitool lan print 3 | grep "IP Address Source"
IP Address Source : Static Address
2. Check address source for usb0 via Redfish
Signed-off-by: Thang Q. Nguyen <thang@os.amperecomputing.com>
Change-Id: Id14bfc289da6d36552bd37448ec1e4c30ff32a0a
|
|
MichalX Szopinski (1):
Remove hardcoded chassis from Fan store
Change-Id: I7838c431638aa2796b06a045ca4866473bd919d1
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: I875b9669c6fb005ec1a334866ad0a6e3b76ac05f
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Ed Tanous (2):
Remove pointers vector
Remove duplicate std::move
Change-Id: I6df92d44c9b3ac1676a86355a7bdad878f7ea18c
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Matt Spinler (1):
log_services: Fix isContentTypeAllowed checks
Change-Id: Ice3d95ba7615e0884bfd83834464479756826cfb
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Andrew Jeffery (5):
ringbuffer: Implement min() as an inline function
obmc-console: Mark unused parameters as such
console-server: Drop trailing semi-colon from console_handler_register()
obmc-console: Clean up invalid sign comparisons
socket-handler: Fix void pointer arithmetic
Change-Id: I102d29d021091724e5316d38dbf03edf40e377da
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
jinuthomas (1):
Additional change to support ee1004 driver
Change-Id: I6b673d92cc92f2479b25b7cabfbcbbc551ec9c1d
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Konstantin Aladyshev (1):
meson: Disable 'verify-signature' option by default
Change-Id: If0a1be07f8c9849589785958c1128f093f11b38c
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Andrew Jeffery (8):
tests: platform: TEST(GetNumericEffecterValue, testGoodEncodeResponse) UB
utils: Fix integer promotion UB in dec2bcd{16,32} encoders
msgbuf: Handle pointer-overflow sanitization
tests: platform: Fix TEST(GetStateSensorReadings, testBadDecodeResponse)
platform: Fix LE encoding of present_reading
platform: Test invalid heartbeat conditions after assignment
platform: pldm_msgbuf for decode_set_state_effecter_states_req()
platform: pldm_msgbuf for decode_get_pdr_req()
Change-Id: I0abf1aff2407d4d8a33e82d59931f3ac8e861177
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: Id5e8a08d58dae56e470a1feb11adf9c8c474618f
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (2):
meson: remove deprecated get_pkgconfig_variable
build: fix missing nlohmann_json dependency
Change-Id: Ia1b29a3ab78ab6e2e81f837091a4c684c81ae541
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Konstantin Aladyshev (1):
Deassert thresholds when sensor is unavailable
Patrick Rudolph (2):
PSUSensor: Add more labels
PSUSensor: Add new PMBUS chip TDA38640
Change-Id: I44487143a735a75b239ebf629f9eb8d8ffb8e89d
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: Ida741da44fcc607e935d18cb4fe320c83ca4a1b8
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I35a9ffed937919467dec62a1fbff729f4a51366c
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5fd106fcc9de9895bc0d79f265e5c0ab06ee624a
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I30885f8c2577c3c2aa333a61210e6738fbf016c7
|
|
All traffic to/from tray are via gbmcbr. We need to allow the incoming
traffic that establish a tcp connection to allow bmc client traffic like
netboot downloading. This add a rule for that.
Change-Id: I2f3afeea6320b20d7e0f740b102b2f227799032d
Signed-off-by: Yuxiao Zhang <yuxiaozhang@google.com>
|
|
Commit is intended to enable import of trace
parser modules from hostboot repo. It aims
to pullin modules which parse trace section
of eLog committed by XGPE, PGPE and QME.
Change-Id: I067af7b9594df13dab7184e568d0d008d4e47c65
Signed-off-by: Prem Shanker Jha <premjha2@in.ibm.com>
|
|
Cleaning up the OWNERS files to reduce the amount of "does not resolve
to any account" errors that pop up consistently in the gerrit logs.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I4cb94805515d9456668ff8dfc07c4f60ca343c07
|
|
Cleaning up the OWNERS files to reduce the amount of "does not resolve
to any account" errors that pop up consistently in the gerrit logs.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I4fb7ff1946b21ab95d4d5bb47461fed6a7f7d07f
|
|
Cleaning up the OWNERS files to reduce the amount of "does not resolve
to any account" errors that pop up consistently in the gerrit logs.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I0855112d10982a9c08f8a9102e3045a513e91e72
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I554299542d5a5c1125ab4a9a152a28937052a128
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iced76041c1207dc9ab28a83fd849ba3cd0a07bb5
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iab07f5079bad00f3cb821f60329e5ad88e45ef4b
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: I3fef8071d4e6ffc168a9c1cfb66bbf7255a79305
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: I572063b65d96de765fc85dde3fb33775578629d1
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: Ifbc87ad693f5a580379bb5aa456dee88d0fdc53b
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
The $gpiopresent expansion needs to be unquoted, because gpioget needs
to take the gpiochip and line number (both of which gpiofind produces)
as separate arguments.
Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: Iaa5943cf247e523a9c901325ce2290a4f537a50f
|
|
npcm8xx-tip-fw already specifies per-IGPS_MACHINE configuration in
https://github.com/openbmc/openbmc/blob/4b8284784129e050c556b372e7116f884a168e8c/meta-nuvoton/recipes-bsp/images/npcm8xx-tip-fw_0.5.6.0.4.5.bb#L3
Change-Id: I635dfb6c2ed769d0f098e6ab2443dceaca79b789
Signed-off-by: Anthony <anthonyhkf@google.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I1f668e1e40c937e016c60af13705ea60936ed739
|
|
Ed Tanous (2):
Fix clang-tidy issue in aggregation
Copy privilegeset into websocket rule
Change-Id: Ic6a27189affbd9956f6ff5975a79c168839ce238
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Priyanga Ramasamy (1):
Remove adding parent's presence for child FRUs
Change-Id: I27db957fcd96820a5cc4e14b0906e5c1ca227ae3
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: I29772cd83e574fd3d0805a530f1c0b79c93e7ef8
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
This commit fixes shelllcheck source in ampere_uartmux_ctrl.sh
to point to the exact source path.
Signed-off-by: Chau Ly <chaul@amperecomputing.com>
Change-Id: I38d92582bb7783b18ff19fb8ae5e5abe0c1e3d44
|
|
This commit refactors obmc-console setup for mtjade
Tested:
1. Successfully connects to host consoles via ssh sol
$ ssh root@${bmc} -p 220*
2. Successfully connects host consoles via ipmi sol
$ ipmitool -I lanplus -H ${bmc} -U root -P 0penBmc \
-C 17 sol activate
3. All the host console logs are in /var/log/
Signed-off-by: Chau Ly <chaul@amperecomputing.com>
Change-Id: Ie01361139644f5836646d1d98934cda52828ec70
|
|
This commit refactors obmc-console setup for mtmitchell
Tested:
1. Successfully connects to host consoles via ssh sol
$ ssh root@${bmc} -p 220*
2. Successfully connects host consoles via ipmi sol
$ ipmitool -I lanplus -H ${bmc} -U root -P 0penBmc \
-C 17 sol activate
3. All the host console logs are in /var/log/
Signed-off-by: Chau Ly <chaul@amperecomputing.com>
Change-Id: Ia1e6faaa4757e8f478352fd03ed977beeac15200
|
|
Patrick Williams (2):
sdbus++: regenerate meson
meson: remove deprecated get_pkgconfig_variable
Change-Id: I4a516284510196f96c80164769775a21232e4d0f
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Zane Shelley (1):
Rename chip data attention types to be more descriptive
Change-Id: Ia75eaa2d2a4360b09097e7e16e1f91e3c8242c67
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
meson: remove deprecated get_pkgconfig_variable
Change-Id: I4056e23c1de1edd9edc54bc046534e956d89ee64
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
ArchanaKakani (1):
Populate correct terminusId
Pavithra Barithaya (1):
PLDM : fix the invalid effecter and sensor PDRs
Change-Id: I3ca1924025e435fafd8d0c9e9d25aa3a489e73cf
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Add the power-supply configuration files for the acbel-fsg032 power
supplies used in the Bonnell systems in order to get telemetry data
added to D-Bus via phosphor-hwmon.
Bonnell power supplies on bus 3 at 0x5a and 0x5b
Change-Id: I97a1e1ba57ee1b2da6462b6921f121683498fe87
Signed-off-by: Lakshmi Yadlapati <lakshmiy@us.ibm.com>
|
|
Enable Bonnell power supply driver acbel-fsg032
Change-Id: I271682b659565d5eb679ce624fc2fcc6b4b5c8c0
Signed-off-by: Lakshmi Yadlapati <lakshmiy@us.ibm.com>
|
|
Many scripts in meta-ampere had `shellcheck source=/dev/null`, which
isn't as useful as putting the real sourced script. Fix up all of
these in the meta-layer to point to the right script, which as a
side-effect allow a few global-disables to be removed.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Icb43dc76a19edbdada8b4caa6ced57ff07f42d27
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iaefc691917edf1342854fa4ea0950283a7d175c8
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I74d66a1a5334967c5049a8d94a736c130d4c94da
|