Obsolete
/
libipc-old
Archived
3
0
Fork 0

SwitchDB: in/out should be fine.

master
Philippe Pittoli 2023-01-08 20:06:22 +01:00
parent 0cf2e5ef1f
commit 506bd21d57
3 changed files with 120 additions and 148 deletions

View File

@ -28,11 +28,8 @@ pub const CBEvent = struct {
// For IO callbacks (switching). // For IO callbacks (switching).
pub const Type = enum { pub const Type = enum {
NO_ERROR, // No error. A message was generated. NO_ERROR, // No error. A message was generated.
ERROR, // Generic error.
FD_CLOSING, // The fd is closing. FD_CLOSING, // The fd is closing.
FD_ERROR, // Generic error.
PARSING_ERROR, // The message was read but with errors.
IGNORE, // The message should be ignored (protocol specific). IGNORE, // The message should be ignored (protocol specific).
}; };
t: CBEvent.Type,
}; };

View File

@ -19,7 +19,7 @@ const Switch = @import("./switch.zig").Switch;
const print_eq = @import("./util.zig").print_eq; const print_eq = @import("./util.zig").print_eq;
const Messages = @import("./message.zig").Messages; const Messages = @import("./message.zig").Messages;
const Switches = @import("./switch.zig").Switches; const SwitchDB = @import("./switch.zig").SwitchDB;
const Connections = @import("./connection.zig").Connections; const Connections = @import("./connection.zig").Connections;
pub const PollFD = std.ArrayList(std.os.pollfd); pub const PollFD = std.ArrayList(std.os.pollfd);
@ -39,7 +39,7 @@ pub const Context = struct {
pollfd: PollFD, // .fd (fd_t) + .events (i16) + .revents (i16) pollfd: PollFD, // .fd (fd_t) + .events (i16) + .revents (i16)
tx: Messages, // Messages to send, once their fd is available. tx: Messages, // Messages to send, once their fd is available.
switchdb: Switches, // Relations between fd. switchdb: SwitchDB, // Relations between fd.
timer: ?i32 = null, // No timer by default (no TIMER event). timer: ?i32 = null, // No timer by default (no TIMER event).
@ -62,7 +62,7 @@ pub const Context = struct {
, .connections = Connections.init(allocator) , .connections = Connections.init(allocator)
, .pollfd = PollFD.init(allocator) , .pollfd = PollFD.init(allocator)
, .tx = Messages.init(allocator) , .tx = Messages.init(allocator)
, .switchdb = Switches.init(allocator) , .switchdb = SwitchDB.init(allocator)
, .allocator = allocator , .allocator = allocator
}; };
} }
@ -208,16 +208,6 @@ pub const Context = struct {
return try self.connect_service (service_name); return try self.connect_service (service_name);
} }
// Connection to a service, but with switched with the client fd.
// pub fn connection_switched(self: *Self
// , path: [] const u8
// , clientfd: i32) !i32 {
// // print("connection switched from {} to path {s}\n", .{clientfd, path});
// var newfd = try self.connect_ (Connection.Type.SWITCHED, path);
// // TODO: record switch.
// return newfd;
// }
pub fn accept_new_client(self: *Self, event: *Event, server_index: usize) !void { pub fn accept_new_client(self: *Self, event: *Event, server_index: usize) !void {
// net.StreamServer // net.StreamServer
var serverfd = self.pollfd.items[server_index].fd; var serverfd = self.pollfd.items[server_index].fd;
@ -370,7 +360,7 @@ pub const Context = struct {
} }
// SWITCHED = send message to the right dest (or drop the switch) // SWITCHED = send message to the right dest (or drop the switch)
else if (self.connections.items[i].t == .SWITCHED) { else if (self.connections.items[i].t == .SWITCHED) {
self.swichdb.handle_event_read (&current_event, i, fd.fd); current_event = self.swichdb.handle_event_read (i, fd.fd);
if (current_event.t == .SWITCH_RX) { if (current_event.t == .SWITCH_RX) {
self.schedule(current_event.message.?); self.schedule(current_event.message.?);
} }
@ -404,21 +394,25 @@ pub const Context = struct {
if(fd.revents & std.os.linux.POLL.OUT > 0) { if(fd.revents & std.os.linux.POLL.OUT > 0) {
fd.events &= ~ @as(i16, std.os.linux.POLL.OUT); fd.events &= ~ @as(i16, std.os.linux.POLL.OUT);
var index: usize = undefined;
for (self.tx.items) |m, index_| {
if (m.fd == self.pollfd.items[i].fd) {
index = index_;
break;
}
}
var m = self.tx.swapRemove(index);
// SWITCHED = write message for its switch buddy (callbacks) // SWITCHED = write message for its switch buddy (callbacks)
if (self.connections.items[i].t == .SWITCHED) { if (self.connections.items[i].t == .SWITCHED) {
return self.swichdb.handle_event_write (&current_event, i, fd.fd); current_event = self.swichdb.handle_event_write (i, fd.fd, m);
// TODO: remove the message from the tx array.
// Message inner memory is already freed.
return current_event;
} }
else { else {
// otherwise = write message for the msg.fd // otherwise = write message for the msg.fd
var index: usize = undefined;
for (self.tx.items) |m, index_| {
if (m.fd == self.pollfd.items[i].fd) {
index = index_;
break;
}
}
var m = self.tx.swapRemove(index);
try self.write (m); try self.write (m);
m.deinit(); m.deinit();
return Event.init(Event.Type.TX, i, fd.fd, null); return Event.init(Event.Type.TX, i, fd.fd, null);

View File

@ -15,7 +15,9 @@ const print = std.debug.print;
const Event = ipc.Event; const Event = ipc.Event;
pub const Switches = struct { /// SwitchDB
pub const SwitchDB = struct {
const Self = @This(); const Self = @This();
db: std.AutoArrayHashMap(i32, ManagedConnection), db: std.AutoArrayHashMap(i32, ManagedConnection),
@ -38,23 +40,23 @@ pub const Switches = struct {
// Read message from a switched fd. // Read message from a switched fd.
pub fn read (self: *Self, fd: i32) !?Message { pub fn read (self: *Self, fd: i32) !?Message {
// TODO: assert there is an entry with this fd as a key.
var managedconnection = self.db.get(fd); var managedconnection = self.db.get(fd);
var message: Message = undefined; var message: Message = undefined;
var r: CBEventType = managedconnection.in(fd, &message); var r: CBEventType = managedconnection.?.in(fd, &message);
switch (r) { switch (r) {
// The message should be ignored (protocol specific). // The message should be ignored (protocol specific).
.IGNORE => { return null; }, CBEventType.IGNORE => { return null; },
// No error. A message was generated. // No error. A message was generated.
.NO_ERROR => { CBEventType.NO_ERROR => {
message.fd = managedconnection.dest; message.fd = managedconnection.?.dest;
return message; return message;
}, },
.FD_CLOSING => { return error.closeFD; }, CBEventType.FD_CLOSING => { return error.closeFD; },
// Generic error, or the message was read but with errors. // Generic error, or the message was read but with errors.
.FD_ERROR CBEventType.ERROR => {
.PARSING_ERROR => {
return error.generic; return error.generic;
}, },
} }
@ -64,21 +66,20 @@ pub const Switches = struct {
// Write a message to a switched fd. // Write a message to a switched fd.
pub fn write (self: *Self, message: Message) !void { pub fn write (self: *Self, message: Message) !void {
// TODO: assert there is an entry with this fd as a key.
var managedconnection = self.db.get(message.fd); var managedconnection = self.db.get(message.fd);
var r = managedconnection.out(managedconnection.dest, message); var r = managedconnection.?.out(managedconnection.?.dest, &message);
switch (r) { switch (r) {
// The message should be ignored (protocol specific). // The message should be ignored (protocol specific).
// No error. A message was generated. // No error. A message was generated.
.NO_ERROR => { CBEventType.NO_ERROR => {
return; return;
}, },
.FD_CLOSING => { return error.closeFD; }, CBEventType.FD_CLOSING => { return error.closeFD; },
// Generic error, or the message was read but with errors. // Generic error, or the message was read but with errors.
.IGNORE CBEventType.IGNORE,
.FD_ERROR CBEventType.ERROR => {
.PARSING_ERROR => {
return error.generic; return error.generic;
}, },
} }
@ -86,44 +87,38 @@ pub const Switches = struct {
unreachable; unreachable;
} }
pub fn handle_event_read (self: *Self, event: *Event, index: usize, fd: i32) void { pub fn handle_event_read (self: *Self, index: usize, fd: i32) Event {
var message: ?Message = null; var message: ?Message = null;
message = self.read (fd) catch |err| switch(err) { message = self.read (fd) catch |err| switch(err) {
error.closeFD => { error.closeFD => {
event.* = Event.init(Event.Type.DISCONNECTION, index, fd, null); return Event.init(Event.Type.DISCONNECTION, index, fd, null);
return;
}, },
error.generic => { error.generic => {
event.* = Event.init(Event.Type.ERROR, index, fd, null); return Event.init(Event.Type.ERROR, index, fd, null);
return;
}, },
}; };
event.* = Event.init(Event.Type.SWITCH_RX, index, fd, message); return Event.init(Event.Type.SWITCH_RX, index, fd, message);
return;
} }
pub fn handle_event_write (self: *Self, event: *Event, index: usize, message: Message) void { pub fn handle_event_write (self: *Self, index: usize, message: Message) Event {
defer message.deinit(); defer message.deinit();
var fd = message.fd; var fd = message.fd;
self.write(message) catch |err| switch(err) { self.write(message) catch |err| switch(err) {
error.closeFD => { error.closeFD => {
event.* = Event.init(Event.Type.DISCONNECTION, index, fd, null); return Event.init(Event.Type.DISCONNECTION, index, fd, null);
return;
}, },
error.generic => { error.generic => {
event.* = Event.init(Event.Type.ERROR, index, fd, null); return Event.init(Event.Type.ERROR, index, fd, null);
return;
}, },
}; };
event.* = Event.init(Event.Type.SWITCH_TX, index, fd, null); return Event.init(Event.Type.SWITCH_TX, index, fd, null);
return;
} }
}; };
pub const ManagedConnection = struct { const ManagedConnection = struct {
dest : i32, dest : i32,
in : *const fn (origin: i32, m: *Message) CBEventType = default_in, in : *const fn (origin: i32, m: *Message) CBEventType = default_in,
out : *const fn (origin: i32, m: *Message) CBEventType = default_out, out : *const fn (origin: i32, m: *const Message) CBEventType = default_out,
}; };
test "creation and display" { test "creation and display" {
@ -132,7 +127,7 @@ test "creation and display" {
defer _ = gpa.deinit(); defer _ = gpa.deinit();
const allocator = gpa.allocator(); const allocator = gpa.allocator();
var switchdb = Switches.init(allocator); var switchdb = SwitchDB.init(allocator);
defer switchdb.deinit(); defer switchdb.deinit();
try switchdb.db.put(5, ManagedConnection {.dest = 6}); try switchdb.db.put(5, ManagedConnection {.dest = 6});
@ -141,25 +136,76 @@ test "creation and display" {
try print_eq("{ (5,6)(6,5) }", .{switchdb}); try print_eq("{ (5,6)(6,5) }", .{switchdb});
} }
// test "read" { fn successful_in (_: i32, m: *Message) CBEventType {
// const config = .{.safety = true}; m.* = Message.init(8, std.heap.c_allocator, "coucou") catch {
// var gpa = std.heap.GeneralPurposeAllocator(config){}; return CBEventType.ERROR;
// defer _ = gpa.deinit(); };
// const allocator = gpa.allocator(); return CBEventType.NO_ERROR;
// }
// var switchdb = Switches.init(allocator);
// defer switchdb.deinit();
//
// }
// For IO callbacks (switching). fn successful_out (_: i32, _: *const Message) CBEventType {
// pub const Type = enum { return CBEventType.NO_ERROR;
// NO_ERROR, // No error. A message was generated. }
// FD_CLOSING, // The fd is closing.
// FD_ERROR, // Generic error. test "successful exchanges" {
// PARSING_ERROR, // The message was read but with errors. const config = .{.safety = true};
// IGNORE, // The message should be ignored (protocol specific). var gpa = std.heap.GeneralPurposeAllocator(config){};
// }; defer _ = gpa.deinit();
const allocator = gpa.allocator();
var switchdb = SwitchDB.init(allocator);
defer switchdb.deinit();
try switchdb.db.put(5, ManagedConnection {.dest = 6, .in = successful_in, .out = successful_out});
try switchdb.db.put(6, ManagedConnection {.dest = 5, .in = successful_in, .out = successful_out});
// should return a new message (hardcoded: fd 8, payload "coucou")
var event_1: Event = switchdb.handle_event_read (1, 5);
if (event_1.m) |m| { m.deinit(); }
else { return error.NoMessage; }
// should return a new message (hardcoded: fd 8, payload "coucou")
var event_2: Event = switchdb.handle_event_read (1, 6);
if (event_2.m) |m| { m.deinit(); }
else { return error.NoMessage; }
var message = try Message.init(6, allocator, "coucou");
var event_3 = switchdb.handle_event_write (5, message);
if (event_3.m) |_| { return error.ShouldNotCarryMessage; }
}
fn unsuccessful_in (_: i32, _: *Message) CBEventType {
return CBEventType.ERROR;
}
fn unsuccessful_out (_: i32, _: *const Message) CBEventType {
return CBEventType.ERROR;
}
test "unsuccessful exchanges" {
const config = .{.safety = true};
var gpa = std.heap.GeneralPurposeAllocator(config){};
defer _ = gpa.deinit();
const allocator = gpa.allocator();
var switchdb = SwitchDB.init(allocator);
defer switchdb.deinit();
try switchdb.db.put(5, ManagedConnection {.dest = 6, .in = unsuccessful_in, .out = unsuccessful_out});
try switchdb.db.put(6, ManagedConnection {.dest = 5, .in = unsuccessful_in, .out = unsuccessful_out});
// should return a new message (hardcoded: fd 8, payload "coucou")
var event_1: Event = switchdb.handle_event_read (1, 5);
if (event_1.m) |_| { return error.ShouldNotCarryMessage; }
// should return a new message (hardcoded: fd 8, payload "coucou")
var event_2: Event = switchdb.handle_event_read (1, 6);
if (event_2.m) |_| { return error.ShouldNotCarryMessage; }
var message = try Message.init(6, allocator, "coucou");
var event_3 = switchdb.handle_event_write (5, message);
if (event_3.m) |_| { return error.ShouldNotCarryMessage; }
}
fn default_in (origin: i32, m: *Message) CBEventType { fn default_in (origin: i32, m: *Message) CBEventType {
print ("receiving a message originated from {}\n", .{origin}); print ("receiving a message originated from {}\n", .{origin});
@ -168,21 +214,20 @@ fn default_in (origin: i32, m: *Message) CBEventType {
// This may be kinda hacky, idk. // This may be kinda hacky, idk.
var stream: net.Stream = .{ .handle = origin }; var stream: net.Stream = .{ .handle = origin };
packet_size = stream.read(buffer[0..]) catch return CBEventType.FD_ERROR; packet_size = stream.read(buffer[0..]) catch return CBEventType.ERROR;
// Let's handle this as a disconnection. // Let's handle this as a disconnection.
if (packet_size <= 4) { if (packet_size <= 4) {
return CBEventType.FD_CLOSING; return CBEventType.FD_CLOSING;
} }
// TODO: handle memory errors.
m.* = Message.read(origin, buffer[0..], std.heap.c_allocator) m.* = Message.read(origin, buffer[0..], std.heap.c_allocator)
catch return CBEventType.FD_ERROR; catch return CBEventType.ERROR;
return CBEventType.NO_ERROR; return CBEventType.NO_ERROR;
} }
fn default_out (origin: i32, m: *Message) CBEventType { fn default_out (origin: i32, m: *const Message) CBEventType {
print ("sending a message originated from {}\n", .{origin}); print ("sending a message originated from {}\n", .{origin});
// Message contains the fd, no need to search for // Message contains the fd, no need to search for
// the right structure to copy, let's just recreate // the right structure to copy, let's just recreate
@ -194,72 +239,8 @@ fn default_out (origin: i32, m: *Message) CBEventType {
var writer = fbs.writer(); var writer = fbs.writer();
// returning basic errors, no details. // returning basic errors, no details.
_ = m.write(writer) catch return CBEventType.FD_ERROR; _ = m.write(writer) catch return CBEventType.ERROR;
_ = stream.write (fbs.getWritten()) catch return CBEventType.FD_ERROR; _ = stream.write (fbs.getWritten()) catch return CBEventType.ERROR;
return CBEventType.NO_ERROR; return CBEventType.NO_ERROR;
} }
// // TODO: actual switching.
// pub const Switch = struct {
// origin : i32,
// destination : i32,
//
// orig_in : *const fn (origin: i32, m: *Message) CBEventType,
// orig_out : *const fn (origin: i32, m: *Message) CBEventType,
// dest_in : *const fn (origin: i32, m: *Message) CBEventType,
// dest_out : *const fn (origin: i32, m: *Message) CBEventType,
//
// const Self = @This();
//
// pub fn init(origin: i32, destination: i32) Self {
// return Self {
// .origin = origin
// , .destination = destination
// , .orig_in = default_in
// , .orig_out = default_out
// , .dest_in = default_in
// , .dest_out = default_out
// };
// }
//
// pub fn set_callbacks(self: *Self, fd: i32,
// in : *const fn (origin: i32, m: *Message) CBEventType,
// out : *const fn (origin: i32, m: *Message) CBEventType) void {
//
// if (fd == self.origin) {
// self.orig_in = in;
// self.orig_out = out;
// }
// else {
// self.dest_in = in;
// self.dest_out = out;
// }
// }
//
// pub fn format(self: Self, comptime _: []const u8, _: fmt.FormatOptions, out_stream: anytype) !void {
// try fmt.format(out_stream
// , "switch {} <-> {}"
// , .{ self.origin, self.destination} );
// }
// };
// test "Switch - creation and display" {
// const config = .{.safety = true};
// var gpa = std.heap.GeneralPurposeAllocator(config){};
// defer _ = gpa.deinit();
// const allocator = gpa.allocator();
//
// var switchdb = Switches.init(allocator);
// defer switchdb.deinit();
//
// var first = Switch.init(3,8); // origin destination
// var second = Switch.init(2,4); // origin destination
// try switchdb.append(first);
// try switchdb.append(second);
//
// try print_eq("switch 3 <-> 8", first);
// try print_eq("switch 2 <-> 4", second);
//
// try testing.expect(2 == switchdb.items.len);
// }