LINUX.ORG.RU
ФорумTalks

Любителям поругать качество кода systemd

 , ,


0

2

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/...

Всем срочно на openbsd!111

Для Ъ:

/*
 * Send GARP packet to network peers after migrations.
 * After Quick Migration, the network is not immediately operational in the
 * current context when receiving RNDIS_STATUS_MEDIA_CONNECT event. So, add
 * another netif_notify_peers() into a delayed work, otherwise GARP packet
 * will not be sent after quick migration, and cause network disconnection.
 * Also, we update the carrier status here.
 */
static void netvsc_link_change(struct work_struct *w)
{
	struct net_device_context *ndev_ctx;
	struct net_device *net;
	struct netvsc_device *net_device;
	struct rndis_device *rdev;
	bool notify, refresh = false;
	char *argv[] = { "/etc/init.d/network", "restart", NULL };
	char *envp[] = { "HOME=/", "PATH=/sbin:/usr/sbin:/bin:/usr/bin", NULL };

Еще из той же оперы:

// /kernel/reboot.c
char poweroff_cmd[POWEROFF_CMD_PATH_LEN] = "/sbin/poweroff";
static const char reboot_cmd[] = "/sbin/reboot";
// /drivers/staging/rtl8192e/rtl8192e/rtl_dm.c
static char *ac_dc_script = "/etc/acpi/wireless-rtl-ac-dc-power.sh";
	char *argv[] = {ac_dc_script, DRV_NAME, NULL};
	static char *envp[] = {"HOME=/",
			"TERM=linux",
			"PATH=/usr/bin:/bin",
			 NULL};

А вы тут про жалкие goto в коде

★★★★★

Последнее исправление: derlafff (всего исправлений: 2)

Лол, это правда в ядре что ли?

Copyright (c) 2009, Microsoft Corporation.

Еще раз лол.

vazgen05 ★★★
()

netvsc_link_change
hyperv

Ненужно.

reboot.c

А тут что не так? Разве эти пути не диктуются каким-то стандартом?

Deleted
()
Ответ на: комментарий от Deleted

А тут что не так? Разве эти пути не диктуются каким-то стандартом?

Разве не ядро занимается куищще? Зачем escape в user-mode?

beastie ★★★★★
()

Не хватает тега «конструктивное нытьё».

Вообще, вместо того, чтобы на ЛОР это вбрасывать, лучше Линуса носом в код ткни, может он его как-то проворонил. Ну и, это, если знаешь, как сделать лучше, сделай.

r3lgar ★★★★★
()
Ответ на: комментарий от beastie

Зачем escape в user-mode?

Лёня в бога заигрался.

r3lgar ★★★★★
()

А в чём трабл?

Сабж. Мимослоу.

Northsoft ★★
()

так это же для hyperv

ну и буй на него вообще.

fornlr ★★★★★
()

А в чем проблема? Этот код используется только opal'ом.

kirk_johnson ★☆
()

Оно на сях разве? Я думал на php или python.

Lavos ★★★★★
()
Ответ на: комментарий от Deleted

А тут что не так?

Ну вот мне кажется, что ядро вообще не должно зависеть от userspace.

Разве эти пути не диктуются каким-то стандартом?

Размеры гайки диктуются ГОСТ 5915-70. Это сильно тебя утешит, когда она попадется тебе в супе?

t184256 ★★★★★
()
Ответ на: комментарий от mystery

Этот неговнокод используется для инициации штатного шатдауна из любого контекста ядра, каковой шатдаун, внезапно, обрабатывается системой инициализации. И это действо, так же внезапно, вызывается через юзерспйс, в котором живет инит, который должен аккуратно зашатдаунить все сервисы.

А некоторым говноэкспертам следует читать код и комментарии, прежде чем выдавать икспёрдные аценки

no-dashi ★★★★★
()
Последнее исправление: no-dashi (всего исправлений: 1)
Ответ на: комментарий от goingUp

А проблемы отдельно взятого драйвера. Примерно того же уровня, что загрузка фирмвари конкретной и не меньшей версии

no-dashi ★★★★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.