Re: [Xen-devel] [PATCHv2] xen: remove DEFINE_XENBUS_DRIVER() macro

[David Vrabel <david.vrabel@xxxxxxxxxx>]

On 10/09/14 17:00, Konrad Rzeszutek Wilk wrote:
> On Wed, Sep 10, 2014 at 01:07:49PM +0100, David Vrabel wrote:
>> The DEFINE_XENBUS_DRIVER() macro looks a bit weird and causes sparse
>> errors.
> 
> .. but it is also useful for downstream distros to bolt on Xen patches.
> 
> Is this urgent? Could it wait until Novell/SuSE has switched over
> to using pvops and then this can go in?

If the macro didn't cause sparse errors, I could be persuaded to wait.

I do not think we should avoid fixing bugs or improving the readability
or maintainability of the code to help out someone still using
non-upstream Xen support.

In general, the cost of maintaining non-upstream forks should not be
paid for by upstream users/developers.

David

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel