[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH] include/public/io/ring.h: Remove xen_mb, xen_rmb, xen_wmb macros
- To: 'Juergen Gross' <jgross@xxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>
- From: Paul Durrant <Paul.Durrant@xxxxxxxxxx>
- Date: Fri, 5 Jul 2019 12:13:39 +0000
- Accept-language: en-GB, en-US
- Authentication-results: esa4.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=Paul.Durrant@xxxxxxxxxx; spf=Pass smtp.mailfrom=Paul.Durrant@xxxxxxxxxx; spf=None smtp.helo=postmaster@xxxxxxxxxxxxxxx
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, KonradRzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
- Delivery-date: Fri, 05 Jul 2019 12:13:48 +0000
- Ironport-sdr: GjCGS3b93QFcmFPt5soHo8dfIQlHFHVmb2bAdcQMrWJS75Ps47oFpvifFAF6eG5JCpBtVGBZ4V ngMtspRWq3dTH/0vOLp6GIvd+qa9fLIBpKsmN55Tvhp8a255wre1En2oFy6eERGqyCCgN6WaXO U3NbExeaNk0CXYLi5C4lFRtUutbnsnyjIiGHsBdMPmfE9SgoLbwt2u6PeT2QrKWtpDYJZd6L2f kO1/T4Si61cWpuO/N/h2WfPk18ZOpSk3AOrUVbGr52cDM22E450y48YoCoJzW0oo7cPKP7KoeO WCM=
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Thread-index: AQHVMntmcjAjDu/MbEW4KaEw4V64e6a6eY2AgAAlqmCAAOmPAIAAKFiAgAA/XlA=
- Thread-topic: [Xen-devel] [PATCH] include/public/io/ring.h: Remove xen_mb, xen_rmb, xen_wmb macros
> -----Original Message-----
[snip]
>
> I'm completely on Jan's side here.
>
> What would be possible perhaps is to split ring.h into two headers: a
> new one with the pure ring definitions and ring.h #include-ing
> xen-compat.h and the new header and #define-ing the xen*mb() macros.
>
> Not sure this would be worth it, though.
>
Ok. Probably not worth it, as you say. If we don't feel comfortable remove old
cruft like these then projects importing the headers will just have to hack it
or live with importing xen-compat too. Anthony already submitted a patch doing
the former for QEMU.
Paul
>
> Juergen
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel
- References:
- [Xen-devel] [PATCH] include/public/io/ring.h: Remove xen_mb, xen_rmb, xen_wmb macros
- Re: [Xen-devel] [PATCH] include/public/io/ring.h: Remove xen_mb, xen_rmb, xen_wmb macros
- Re: [Xen-devel] [PATCH] include/public/io/ring.h: Remove xen_mb, xen_rmb, xen_wmb macros
- Re: [Xen-devel] [PATCH] include/public/io/ring.h: Remove xen_mb, xen_rmb, xen_wmb macros
- Re: [Xen-devel] [PATCH] include/public/io/ring.h: Remove xen_mb, xen_rmb, xen_wmb macros
|
