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

[Jürgen Groß <jgross@xxxxxxxx>]

On 09/10/2014 06:25 PM, David Vrabel wrote:
> 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.
Couldn't you modify the current macro users to avoid the sparse errors?
E.g. by explicitly specifying the second argument as "NULL" when the
default is desired?

Removing the "+ 0" from evaluation of "drvname" in the macro definition
would enforce specifying "drvname".

Juergen

> 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

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