[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH 04/19] xen/mce: remove unused x86_mcinfo_add()
>>> On 20.02.17 at 02:52, <haozhong.zhang@xxxxxxxxx> wrote: > On 02/17/17 02:55 -0700, Jan Beulich wrote: >> >>> On 17.02.17 at 07:39, <haozhong.zhang@xxxxxxxxx> wrote: >> > Signed-off-by: Haozhong Zhang <haozhong.zhang@xxxxxxxxx> >> >> Not sure here, I can't imagine the function being here without >> reason, even if the plans perhaps never got carried out. Could >> you please identify in the commit message here whether there >> ever were callers (and if so, when the last one went away) or >> what commit introduced the function without any users? >> > > The commit that removed the last uses of x86_mcinfo_add() is > > commit 9d13fd9fd320a7740c6446c048ff6a2990095966 > Author: Keir Fraser <keir.fraser@xxxxxxxxxx> > Date: Mon Jun 7 15:46:48 2010 +0100 > > x86 mce: Change the method to get the extended MCA information. > > Several changes to get the extended MCA information: > a) Use the x86_mcinfo_reserve in mcinfo_extended to reserve buffer > from > mc_info, instead of using the stack > b) For intel's extended MSR, we don't need write them one > by one as the MSR are continous > c) We don't need enum mca_extinfo, since we can consider > the extended MSR as either per bank, or global. Currently > we add a hook in global data collection, and didn't call > register intel_get_extended_msrs as callback. Later that > hook can be replaced by cleaner way > > Signed-off-by: Jiang, Yunhong <yunhong.jiang@xxxxxxxx> > > Specially, it's a) in that commit that intended to directly update the > buffer reserved by x86_mcinfo_reserve(), instead of updating another > buffer and using x86_mcinfo_add() to copy it to the buffer reserved by > x86_mcinfo_reserve(). It looks that x86_mcinfo_add() was deprecated by > that commit, so it's reasonable to remove it. Okay, so please add a reference to that commit to your patch description. Jan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |