X-Git-Url: https://git.madduck.net/etc/awesome.git/blobdiff_plain/912bd26ede8901c98fd8d7a35f8493ee448109ad..c9ed5d4f802701348ee3b1039c1bbb18a91526a1:/widgets/net.lua?ds=inline diff --git a/widgets/net.lua b/widgets/net.lua index 983da41..ee2cfa7 100644 --- a/widgets/net.lua +++ b/widgets/net.lua @@ -8,8 +8,6 @@ --]] local helpers = require("lain.helpers") - -local notify_fg = require("beautiful").fg_focus local naughty = require("naughty") local wibox = require("wibox") @@ -21,93 +19,126 @@ local setmetatable = setmetatable -- Network infos -- lain.widgets.net -local net = {} -function net.get_device() - local ws = helpers.read_pipe("ip link show | cut -d' ' -f2,9") - ws = ws:match("%w+: UP") or ws:match("ppp%w+: UNKNOWN") - if ws ~= nil then - return ws:match("(%w+):") - else - return "network off" +local function worker(args) + local net = { last_t = 0, last_r = 0, devices = {} } + + function net.get_first_device() + local ws = helpers.read_pipe("ip link show | cut -d' ' -f2,9") + ws = ws:match("%w+: UP") or ws:match("ppp%w+: UNKNOWN") + if ws then return { ws:match("(%w+):") } + else return {} end end -end -local function worker(args) - local args = args or {} - local timeout = args.timeout or 1 - local units = args.units or 1024 --kb - local notify = args.notify or "on" - local screen = args.screen or 1 + local args = args or {} + local timeout = args.timeout or 2 + local units = args.units or 1024 --kb + local notify = args.notify or "on" + local screen = args.screen or 1 local settings = args.settings or function() end - - iface = args.iface or net.get_device() + local iface = args.iface or net.get_first_device() net.widget = wibox.widget.textbox('') - helpers.set_map(iface, true) - helpers.set_map("net_t", 0) - helpers.set_map("net_r", 0) + -- Compatibility with old API where iface was a string corresponding to 1 interface + if type(iface) == "string" then + iftable = {iface} + else + iftable = iface + end + + -- Mark all devices as initially online/active + for i, dev in ipairs(iftable) do + helpers.set_map(dev, true) + end function update() + -- This check is required to ensure we keep looking for one device if + -- none is found by net.get_first_device() at startup (i.e. iftable = {}) + if next(iftable) == nil then + iftable = net.get_first_device() + end + + -- These are the totals over all specified interfaces net_now = { - sent = "0.0", - received = "0.0" + -- New api - Current state of requested devices + devices = {}, + -- Bytes since last iteration + sent = 0, + received = 0 } - if iface == "" or string.match(iface, "network off") - then - iface = net.get_device() - end + -- Total bytes transfered + local total_t = 0 + local total_r = 0 - local now_t = helpers.first_line('/sys/class/net/' .. iface .. - '/statistics/tx_bytes') or 0 - local now_r = helpers.first_line('/sys/class/net/' .. iface .. - '/statistics/rx_bytes') or 0 + for i, dev in ipairs(iftable) do + local dev_now = {} + local dev_before = net.devices[dev] or { last_t = 0, last_r = 0 } - if now_t ~= helpers.get_map("net_t") - or now_r ~= helpers.get_map("net_r") then - net_now.carrier = helpers.first_line('/sys/class/net/' .. iface .. - '/carrier') or "0" - net_now.state = helpers.first_line('/sys/class/net/' .. iface .. - '/operstate') or "down" + dev_now.carrier = helpers.first_line(string.format('/sys/class/net/%s/carrier', dev)) or '0' + dev_now.state = helpers.first_line(string.format('/sys/class/net/%s/operstate', dev)) or 'down' - net_now.sent = (now_t - net.last_t) / timeout / units - net_now.sent = string.gsub(string.format('%.1f', net_now.sent), ",", ".") + local now_t = tonumber(helpers.first_line(string.format('/sys/class/net/%s/statistics/tx_bytes', dev)) or 0) + local now_r = tonumber(helpers.first_line(string.format('/sys/class/net/%s/statistics/rx_bytes', dev)) or 0) - net_now.received = (now_r - net.last_r) / timeout / units - net_now.received = string.gsub(string.format('%.1f', net_now.received), ",", ".") + dev_now.sent = (now_t - dev_before.last_t) / timeout / units + dev_now.received = (now_r - dev_before.last_r) / timeout / units - widget = net.widget - settings() + net_now.sent = net_now.sent + dev_now.sent + net_now.received = net_now.received + dev_now.received - helpers.set_map("net_t", now_t) - helpers.set_map("net_r", now_r) - end + dev_now.sent = string.gsub(string.format('%.1f', dev_now.sent), ',', '.') + dev_now.received = string.gsub(string.format('%.1f', dev_now.received), ',', '.') + + dev_now.last_t = now_t + dev_now.last_r = now_r - if net_now.carrier ~= "1" and notify == "on" - then - if helpers.get_map(iface) - then + -- This will become dev_before in the next update/iteration + net.devices[dev] = dev_now + + total_t = total_t + now_t + total_r = total_r + now_r + + -- Notify only once when connection is loss + if string.match(dev_now.carrier, "0") and notify == "on" and helpers.get_map(dev) then naughty.notify({ - title = iface, + title = dev, text = "no carrier", - timeout = 7, - position = "top_left", icon = helpers.icons_dir .. "no_net.png", - fg = notify_fg or "#FFFFFF", screen = screen }) - helpers.set_map(iface, false) + helpers.set_map(dev, false) + elseif string.match(dev_now.carrier, "1") then + helpers.set_map(dev, true) end - else - helpers.set_map(iface, true) + + -- Old api compatibility + net_now.carrier = dev_now.carrier + net_now.state = dev_now.state + -- And new api + net_now.devices[dev] = dev_now + -- With the new api new_now.sent and net_now.received will be the + -- totals across all specified devices + + end + + if total_t ~= net.last_t or total_r ~= net.last_r then + -- Convert to a string to round the digits after the float point + net_now.sent = string.gsub(string.format('%.1f', net_now.sent), ',', '.') + net_now.received = string.gsub(string.format('%.1f', net_now.received), ',', '.') + + widget = net.widget + settings() + + net.last_t = total_t + net.last_r = total_r end end - helpers.newtimer(iface, timeout, update, false) + helpers.newtimer(iface, timeout, update) - return net.widget + return setmetatable(net, { __index = net.widget }) end -return setmetatable(net, { __call = function(_, ...) return worker(...) end }) +return setmetatable({}, { __call = function(_, ...) return worker(...) end })