Hi Michael 2011/12/5 Michael Krufky <mkrufky@xxxxxxxxxxx>: > On Mon, Dec 5, 2011 at 9:28 AM, HoP <jpetrous@xxxxxxxxx> wrote: >> Hi >> >> 2011/12/5 Florian Fainelli <f.fainelli@xxxxxxxxx>: >>> Hello, >>> >>> >>> On 12/03/11 01:37, HoP wrote: >>>> >>>> Hi Alan. >>>> >>>> 2011/12/3 Alan Cox<alan@xxxxxxxxxxxxxxxxxxx>: >>>>> >>>>> On Thu, 1 Dec 2011 15:58:41 +0100 >>>>> HoP<jpetrous@xxxxxxxxx> wrote: >>>>> >>>>>> Hi, >>>>>> >>>>>> let me ask you some details of your interesting idea (how to >>>>>> achieve the same functionality as with vtunerc driver): >>>>>> >>>>>> [...] >>>>>> >>>>>>> The driver, as proposed, is not really a driver, as it doesn't support >>>>>>> any >>>>>>> hardware. The kernel driver would be used to just copy data from one >>>>>>> userspace >>>>>> >>>>>> >>>>>> Please stop learning me what can be called driver and what nope. >>>>>> Your definition is nonsense and I don't want to follow you on it. >>>>> >>>>> >>>>> You can stick your fingers in your ears and shout all you like but given >>>>> Mauro is the maintainer I'd suggest you work with him rather than making >>>>> it painful. One of the failures we routinely exclude code from the kernel >>>>> for is best described as "user interface of contributor" >>>> >>>> >>>> You may be not read all my mails but I really tried to be very positive in >>>> them. >>>> I wanted to focus on my Subject, but Mauro has, sometimes, the demand >>>> to focus on insignificant things (like if the code is driver or not). At >>>> least >>>> it is my feeling from all those disscussions with him. >>>> >>>>> >>>>> It's a loopback that adds a performance hit. The right way to do this is >>>>> in userspace with the userspace infrastructure. At that point you can >>>>> handle all the corner cases properly, integrate things like service >>>>> discovery into your model and so on - stuff you'll never get to work that >>>>> well with kernel loopback hackery. >>>>> >>>>>> Can you show me, how then can be reused most important part >>>>>> of dvb-core subsystem like tuning and demuxing? Or do you want me >>>>>> to invent wheels and to recode everything in the library? Of course >>>>> >>>>> >>>>> You could certainly build a library from the same code. That might well >>>>> be a good thing for all kinds of 'soft' DV applications. At that point >>>>> the discussion to have is the best way to make that code sharable between >>>>> a userspace library and the kernel and buildable for both. >>>>> >>>>>> I can be wrong, I'm no big kernel hacker. So please show me the >>>>>> way for it. BTW, even if you can find the way, then data copying >>>>>> from userspace to the kernel and back is also necessery. I really >>>>>> don't see any advantage of you solution. >>>>> >>>>> >>>>> In a properly built media subsystem you shouldn't need any copies beyond >>>>> those that naturally occur as part of a processing pass and are therefore >>>>> free. >>>> >>>> >>>> I may describe project goal, in few sentences: We have small box, running >>>> embedded linux with 2 satellite tuners on input and ethernet. Nothing >>>> more. >>>> We have designed the box for live sat TV/Radio reception, distributing >>>> them >>>> down to the network. One of the mode of working is "vtuner", what allows >>>> reuse those tuners remotely on linux desktop. The kernel part is very >>>> simple >>>> code exposing kernel's dvb-core to the userspace. Userspace client/server >>>> tools do all resource discovery and connection management. It works >>>> nicely and guys with vdr who is using it are rather satisfied with it. >>>> So, the main >>>> goal of vtuner code is to fully virtualize remote DVB adapter. To any >>>> linux dvb api >>>> compatible applications and tools. The vtuner kernel code seems to be >>>> the simplest and straightforward way to achieve it. >>> >>> >>> The company I work for also has something like this. We can attach a DVB >>> tuner to either the Gateway or the STB we provide and use it indifferently , >>> except that we have the following architecture: >>> >>> - a DVB daemon controlling the physical DVB tuner and exposing methods for >>> tuning/scanning/zapping >>> - a web server and web services for accessing the DVB daemon methods >>> - a RTSP streamer with associated methods for controlling streaming >>> >>> The software running on both devices is the same (one compiled for ARM, the >>> other for x86). >>> >>> I do not see any problem with this solution, people wanting to get the >>> stream can still get the RTSP stream directly joining the multicast group, >>> which is fortunately OS agnostic at the same time. >> >> Nice. Are your solution open-sourced? If so, can you give me URL? >> >>>> I still think the code is very similar to NBD (Network block device) what >>>> sits >>>> in the kernel and is using silently. I guess NBD also do data copying >>>> from/to user space. Is there something what I overlooked? >>>> >>>> Can you show me the way (hint please) I can initiate TCP connection >>>> from within kernel space? If I can do it, then the big disadvantage >>>> of data passing to and from kernel can be removed. >>> >>> >>> Don't do this in kernel-space (remember the mechanism/policy split). >> >> Yes, that can be the issue, I understand that. >> >> You know - I'm a bit confused. Somebody are pointing on double >> data copying (userspace networked daemon -> kernel -> application) >> issue and another one warn me to not start network connection >> from the kernel. But if it works for NFS or CIFS then it should not >> be so weaky, isn't it? >> >>>> I must say that the box is primary focused to the DLNA/UpnP world, so >>>> vtuner feature is something like interesting addon only. But I was myself >>>> very nice surprised how good it behaves on real installations and that >>>> was reason I decided to try to get it included in kernel. I see that >>>> in present there is no willingness for code acceptation, so I will >>>> continue >>>> out of the kernel tree. >>>> >>>> Anyway, if I can find the way how to start TCP connection from the kernel >>>> part, I understand it can boost throughput very nicely up. >>> >>> >>> And here is a new hack. >> >> I'm really tired from all those "hack, crap, pigback ..." wordings. >> >> What exactly vtuner aproach does so hackish (other then exposing >> DVB internals, what is every time made if virtualization support is developing)? >> >> The code itself no need to patch any line of vanilla kernel source, it even >> doesn't change any processing of the rest of kernel, it is very simple >> driver/code/whatever. >> >> I can understand that some developers don't like to get dvb-core opened, >> I don't agree with them, but I don't really need to fight with them. >> Its theirs opinion. >> But even I try to see what is so hackish in vtuner implementation, I don't catch >> anything. Simplicity? I thought that simplicity is a big advanatge (simple code, >> easy to analyze). What else? > > Honza, > > I just want to reiterate my *positive* opinion. (you already heard my > negative, I don't need to repeat it) ... I am very excited to hear > that you're working on this virtual tuner driver, and I don't care > what policy says what -- I, personally, want to play with this for my > own uses. > > Let's stop arguing, and just write code! That's what makes us happy > - lets be happy :-) > > I haven't had a chance yet to look at your full sources, but I > reiterate how excited I am that this virtual dvb adapter exists now. > Regardless of whether the community (or I) agrees as to its kernel > inclusion, nothing stops you from continuing your work, and publishing > tools that use this virtual adapter. > > If, in the end, the community wants this merged into the kernel, then > the community will have spoken and things will go the way that they > should. > > Meanwhile, you don't need to be included in the vanilla kernel if you > want to be used. People have gcc toolchains and they can build the > module themselves if they want it. > > Let's not worry about politics or kernel inclusion -- let's just work > on our projects. The work speaks for itself best. > > I hope you take these words as positive encouragement, regardless of > what I think, personally, about merging to the kernel -- that is an > entirely separate issue. > I really appreciate your email. Believe me that Mauro's NACK is not reason for me to stop work on vtuner. Fortunatelly the code is very compact and not need for any kernel patching so doing out-of-tree development is possible and works very well. I hope for one day when the code gets accepted somehow. In the meantime I'm going to place some notes about development progress time to time here. Next time w/o any word regarding code inclusion :-) Regards Honza -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
