Re: [PATCH 2/7] xen: sched_rt: implement the .free_pdata hook

[Dario Faggioli <dario.faggioli@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1244 bytes --]

On Mon, 2015-03-16 at 14:23 -0400, Meng Xu wrote:
> 2015-03-16 13:10 GMT-04:00 Dario Faggioli <dario.faggioli@citrix.com>:

> > We discussed about this issue before, here:
> > http://lists.xen.org/archives/html/xen-devel/2014-09/msg02765.html
> >
> > But, as far as it looks, the code we ended up merging still had the
> > issue (i.e., it did not have the hook implemented)!
> >
> > I was working a bit on/off (for family reasons) in that period, so I
> > can't really tell how that happened. :-/
> > Not a big deal... this code is marked as experimental for a reason, but
> > it probably make sense to double check that anything else similar
> > happened, can you do that?
>
> Sure! I will review that problem, do the double check and let you know
> the result!
> 
I've re-tested that scenario myself, and verified it's working before
sending this series. :-)

What I was suggesting was for you to double check that everything that
you have in your local dev repo(s)/branch(es) (either from that time or
more recent ones) is actually upstream.

In fact, looking at the referenced thread, it looks like this issue
shouldn't have been present in the code we have checked-in, and yet,
there it was.

Regards,
Dario

[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

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