my subsequent patches are dependant on the patches by holger from may 5th. for personal testing and for information, i packed them together. i tested them and they work well.
Mit freundlichen Grüßen,
.-. /'v'\ (/ ) ------------------------------------------------------------------"-"- |_| i.A. Andreas Eversberg Administrator Betrieb und Entstörung
Versatel Nord GmbH
Nordstr. 2 D-24937 Flensburg
Fon: +49-461-9099749 | Fax: +49-461-909960749 andreas.eversberg@versatel.de@versatel.de | www.versatel.de
Sitz der Gesellschaft: Flensburg, Registergericht: Flensburg, HRB 3395 FL Geschäftsführer: Soeren Wendler, Dr. Hai Cheng, Dr. Max Padberg, Peter Schindler, Joachim Bellinghoven
On Tuesday 09 June 2009 10:14:31 Andreas.Eversberg wrote:
my subsequent patches are dependant on the patches by holger from may 5th. for personal testing and for information, i packed them together. i tested them and they work well.
Cool. I will take this as an ack to land them... There is one change you made and it looks like this one:
@@ -941,7 +956,6 @@ if (!subscr) { DEBUGP(DRR, "<- Can't find any subscriber for this ID\n"); /* FIXME: request id? close channel? */ - return -EINVAL; } DEBUGP(DRR, "<- Channel was requested by %s\n", subscr->name ? subscr->name : subscr->imsi);
We have two cases in there that will return -EINVAL. Maybe we should replace both with an error signal?
z.
-
Andreas.Eversberg -
Holger Freyther