[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/4] xen/x86: Remove unused forward declaration in asm-x86/irq.h



On 14/01/2020 10:15, Jan Beulich wrote:
On 14.01.2020 11:05, Julien Grall wrote:
On 14/01/2020 09:31, Jan Beulich wrote:
On 13.01.2020 22:33, Julien Grall wrote:
From: Julien Grall <jgrall@xxxxxxxxxx>

None of the prototypes within the header asm-x86/irq.h actually requires
the forward declaration of "struct pirq". So remove it.

No functional changes intended.

Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

It is generally nice to identify if this was missed cleanup (the
need indeed went away in 4.12), or if such has never really been
needed.

Yes it is nice to have but this is a best effort basis for cleanup. They
are not fixes and therefore not going to be backported. So I don't feel
the need to browse more than 15 years worth of history and check whether
a cleanup were missed.

15 years? It took me less than a minute (a single grep) to figure
the version this became unnecessary in. And I wouldn't ask for
such on a pretty simple patch like this one if I anticipated a
lot of effort to be needed.

My comment is generic to cleanup... As I said, this is a best effort basis. Maybe I could have done it here, but I didn't feel the need to do it.


What matter for cleanup is the current context and whether they make
sense now.

I disagree. History often helps understand whether something was done
in a certain way without an obvious (from current state of things)
reason.

Anyway, I would be happy to add a word in the commit message if you
point me to the commit removing the need.

Me having told you the version it disappeared in would have made this
very low effort to you.
That's pretty much withdrawing knowledge you may have. I don't think this is suitable mindset for a collaborative project. The more you have more knowledge than me on the x86/pirq part.

Anyway, thank you for thinking I put little effort on this patch... That's a pretty great way to encourage people...

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.