October 2 2017 12:47 PM, "Daniel P. Berrange" <[email protected]> wrote:
>> This will change the type of `DomainSerial.Source` from
>> `*DomainChardevSource` to a new `*DomainSerialSource`.
>> 
>> This is done to add support for networked serial ports and
>> keep the original DomainChardevSource unchanged.
> 
> I'm not sure I see the point of this. Any chardev backed device type
> (parallel, serial, vmchannel) should be allowed to use network
> backends - there's no restriction that says its only for serial
> ports.
> 

I was not aware that all chardev backed devices should be able to use network 
backend.
Does this mean I can simple extend the DomainChardevSource, add the required 
options
and there will be no naming conflict?

If so, this would mean there is no need for a breaking change.
If you would be so kind to confirm the above, I will send in a new patch with 
the 
additional fields.


Kind regards,
Jeroen Simonetti

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to