Bug 9000

Summary: Некорректно парсит параметры и грузит процессор на 100%
Product: Sisyphus Reporter: stalker <stalker>
Component: mtrAssignee: Arseny Maslennikov <arseny>
Status: CLOSED NOTABUG QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: arseny, ender, evg, ldv, mike, vvk
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
dmesg на 2.6.20 c acpi=noirq none

Description stalker 2006-02-01 00:09:37 MSK 
mtr некорерктно парсит параметры.
 -p и --psize действуют по разному, + -p - в зависиомости от порядка

mtr -p 512 -c 2 -r 127.0.0.1
HOST: f37.local                  Loss%   Snt   Last   Avg  Best  Wrst StDev

 mtr -p 100 -c 2 -r 127.0.0.1
HOST: f37.local                  Loss%   Snt   Last   Avg  Best  Wrst StDev

 mtr --psize 100 -c 2 -r 127.0.0.1
HOST: f37.local                  Loss%   Snt   Last   Avg  Best  Wrst StDev
  1. localhost.localdomain         0.0%     2    0.2   0.2   0.2   0.2   0.0

mtr -c 2 -r 127.0.0.1 -p 100
1 127.0.0.1 0 1 1 0 0 0
1 localhost 0 1 1 0 0 0
1 localhost 0 2 2 0 0 0

mtr -c 2 -r 127.0.0.1 --psize 100
HOST: f37.local                  Loss%   Snt   Last   Avg  Best  Wrst StDev
  1. localhost.localdomain         0.0%     2    0.2   0.2   0.2   0.2   0.0

mtr -c 2 -r --psize 100 127.0.0.1
HOST: f37.local                  Loss%   Snt   Last   Avg  Best  Wrst StDev
  1. localhost.localdomain         0.0%     2    0.2   0.2   0.2   0.2   0.0

stalker]$  mtr -c 2 -r --psize 100 127.0.0.1
HOST: f37.local                  Loss%   Snt   Last   Avg  Best  Wrst StDev
  1. localhost.localdomain         0.0%     2    0.2   0.2   0.2   0.2   0.0

time mtr -n -c 10 -r -p 400 127.0.0.1
1.72user 7.69system 0:09.74elapsed 96%CPU (0avgtext+0avgdata 0maxresident)k
0inputs+0outputs (0major+618minor)pagefaults 0swaps
Comment 1 Michael Shigorin 2006-02-01 15:11:21 MSK 
С этим лучше в upstream сходить, поскольку works for me (не пользуюсь этой
опцией, времени на раскопки/общение/исправление сейчас скорее не получается).

Разницу в работе эквивалентных, судя по manpage, -p и --psize -- подтверждаю.
Comment 2 Michael Shigorin 2006-12-13 17:45:18 MSK 
Для 0.72-alt2 это подтверждается, но просьба по возможности озадачиться общением
с апстримом самому -- мне mtr хватает за глаза без аргументов...
Comment 3 stalker 2007-03-20 00:00:35 MSK 
Created attachment 1868 [details]
dmesg на 2.6.20 c acpi=noirq
Comment 4 Dmitry V. Levin 2007-03-24 18:27:00 MSK 
В mtr теперь всё наоборот:
-p синоним --split,
-s синоним --psize.
В коде написано: changed 'p' to 's' to match ping option.