diff for duplicates of <1e15c452-f8be-85fa-e424-b7463cc703eb@intel.com> diff --git a/a/1.txt b/N1/1.txt index b1cffdd..5145b0b 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -5,8 +5,8 @@ Thank Yee-Li for discovering this problem. I would suggest we (intel) will process a patch to address this problem. Can I put you kindly as "Suggested-by"? > -> Neftin, Sasha <sasha.neftin at intel.com <mailto:sasha.neftin@intel.com>> -> ?2021?6?30??? ??2:13??? +> Neftin, Sasha <sasha.neftin@intel.com <mailto:sasha.neftin@intel.com>> +> 于2021年6月30日周三 下午2:13写道: > > On 6/29/2021 20:33, Yee Li wrote: > > @@ -26,7 +26,7 @@ will process a patch to address this problem. Can I put you kindly as > > Then: > if (lat_enc_d > max_ltr_enc_d) -> ? ? ? ? lat_enc = max_ltr_enc; +> lat_enc = max_ltr_enc; > what do you think? > > > @@ -36,106 +36,106 @@ will process a patch to address this problem. Can I put you kindly as > > Neftin, Sasha <sasha.neftin@intel.com > <mailto:sasha.neftin@intel.com> <mailto:sasha.neftin@intel.com > <mailto:sasha.neftin@intel.com>>> -> > ? 2021?6?29??? 22:49??? +> > 于 2021年6月29日周二 22:49写道: > > -> >? ? ?On 6/29/2021 11:21, YeeLi wrote: -> >? ? ?Yeeli, -> >? ? ? > In e1000e driver, a PCIe-like device, the max +> > On 6/29/2021 11:21, YeeLi wrote: +> > Yeeli, +> > > In e1000e driver, a PCIe-like device, the max > snoop/no-snoop latency -> >? ? ? > is the upper limit.So, directly compare the size of +> > > is the upper limit.So, directly compare the size of > lat_enc and -> >? ? ? > max_ltr_enc is incorrect. -> >? ? ? > -> >? ? ?why? -> >? ? ? >? ? ? In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns, +> > > max_ltr_enc is incorrect. +> > > +> > why? +> > > In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns, > correct. -> >? ? ? > -> >? ? ? >? ? ? In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns, +> > > +> > > In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns, > correct. -> >? ? ? > -> >? ? ? >? ? ? In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect. -> >? ? ? > -> >? ? ?Platform LTR encoded is 0x1003 - right. It is meant 1048576ns +> > > +> > > In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect. +> > > +> > Platform LTR encoded is 0x1003 - right. It is meant 1048576ns > x 3 = -> >? ? ?3145738ns. -> >? ? ?Now, -> >? ? ?for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct) -> >? ? ?for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct) -> >? ? ?for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?) -> >? ? ?18907136ns > 3145738ns, (latency encoded is great than +> > 3145738ns. +> > Now, +> > for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct) +> > for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct) +> > for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?) +> > 18907136ns > 3145738ns, (latency encoded is great than > maximum LTR -> >? ? ?encoded by platform) - so, there is no point to wait more +> > encoded by platform) - so, there is no point to wait more > than platform -> >? ? ?required, and lat_enc=max_ltr_enc. It is expected and we sent +> > required, and lat_enc=max_ltr_enc. It is expected and we sent > right -> >? ? ?value to the power management controller. -> >? ? ?What is the problem you try solve? +> > value to the power management controller. +> > What is the problem you try solve? > > -> >? ? ? > Decoded the lat_enc and max_ltr_enc before compare them is +> > > Decoded the lat_enc and max_ltr_enc before compare them is > necessary. -> >? ? ? > -> >? ? ? > Signed-off-by: YeeLi <seven.yi.lee@gmail.com +> > > +> > > Signed-off-by: YeeLi <seven.yi.lee@gmail.com > <mailto:seven.yi.lee@gmail.com> -> >? ? ?<mailto:seven.yi.lee at gmail.com <mailto:seven.yi.lee@gmail.com>>> -> >? ? ? > --- -> >? ? ? >? ?drivers/net/ethernet/intel/e1000e/ich8lan.c | 23 -> >? ? ?++++++++++++++++++++- -> >? ? ? >? ?1 file changed, 22 insertions(+), 1 deletion(-) -> >? ? ? > -> >? ? ? > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c -> >? ? ?b/drivers/net/ethernet/intel/e1000e/ich8lan.c -> >? ? ? > index 590ad110d383..3bff1b570b76 100644 -> >? ? ? > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c -> >? ? ? > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c -> >? ? ? > @@ -986,6 +986,27 @@ static s32 +> > <mailto:seven.yi.lee@gmail.com <mailto:seven.yi.lee@gmail.com>>> +> > > --- +> > > drivers/net/ethernet/intel/e1000e/ich8lan.c | 23 +> > ++++++++++++++++++++- +> > > 1 file changed, 22 insertions(+), 1 deletion(-) +> > > +> > > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c +> > b/drivers/net/ethernet/intel/e1000e/ich8lan.c +> > > index 590ad110d383..3bff1b570b76 100644 +> > > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c +> > > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c +> > > @@ -986,6 +986,27 @@ static s32 > e1000_k1_workaround_lpt_lp(struct -> >? ? ?e1000_hw *hw, bool link) -> >? ? ? >? ? ? ?return ret_val; -> >? ? ? >? ?} -> >? ? ? > -> >? ? ? > +static u32 convert_e1000e_ltr_scale(u32 val) -> >? ? ? > +{ -> >? ? ? > +? ? ?if (val > 5) -> >? ? ? > +? ? ? ? ? ? ?return 0; -> >? ? ? > + -> >? ? ? > +? ? ?return 1U << (5 * val); -> >? ? ? > +} -> >? ? ? > + -> >? ? ? > +static u64 decoded_ltr(u32 val) -> >? ? ? > +{ -> >? ? ? > +? ? ?u64 decoded_latency; -> >? ? ? > +? ? ?u32 value; -> >? ? ? > +? ? ?u32 scale; -> >? ? ? > + -> >? ? ? > +? ? ?value = val & 0x03FF; -> >? ? ? > +? ? ?scale = (val & 0x1C00) >> 10; -> >? ? ? > +? ? ?decoded_latency = value * +> > e1000_hw *hw, bool link) +> > > return ret_val; +> > > } +> > > +> > > +static u32 convert_e1000e_ltr_scale(u32 val) +> > > +{ +> > > + if (val > 5) +> > > + return 0; +> > > + +> > > + return 1U << (5 * val); +> > > +} +> > > + +> > > +static u64 decoded_ltr(u32 val) +> > > +{ +> > > + u64 decoded_latency; +> > > + u32 value; +> > > + u32 scale; +> > > + +> > > + value = val & 0x03FF; +> > > + scale = (val & 0x1C00) >> 10; +> > > + decoded_latency = value * > convert_e1000e_ltr_scale(scale); -> >? ? ? > + -> >? ? ? > +? ? ?return decoded_latency; -> >? ? ? > +} -> >? ? ? > + -> >? ? ? >? ?/** -> >? ? ? >? ? *? e1000_platform_pm_pch_lpt - Set platform power +> > > + +> > > + return decoded_latency; +> > > +} +> > > + +> > > /** +> > > * e1000_platform_pm_pch_lpt - Set platform power > management -> >? ? ?values -> >? ? ? >? ? *? @hw: pointer to the HW structure -> >? ? ? > @@ -1059,7 +1080,7 @@ static s32 +> > values +> > > * @hw: pointer to the HW structure +> > > @@ -1059,7 +1080,7 @@ static s32 > e1000_platform_pm_pch_lpt(struct -> >? ? ?e1000_hw *hw, bool link) -> >? ? ? >? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? E1000_PCI_LTR_CAP_LPT + 2, -> >? ? ?&max_nosnoop); -> >? ? ? >? ? ? ? ? ? ? ?max_ltr_enc = max_t(u16, max_snoop, +> > e1000_hw *hw, bool link) +> > > E1000_PCI_LTR_CAP_LPT + 2, +> > &max_nosnoop); +> > > max_ltr_enc = max_t(u16, max_snoop, > max_nosnoop); -> >? ? ? > -> >? ? ? > -? ? ? ? ? ? ?if (lat_enc > max_ltr_enc) -> >? ? ? > +? ? ? ? ? ? ?if (decoded_ltr(lat_enc) > +> > > +> > > - if (lat_enc > max_ltr_enc) +> > > + if (decoded_ltr(lat_enc) > > decoded_ltr(max_ltr_enc)) -> >? ? ? >? ? ? ? ? ? ? ? ? ? ? ?lat_enc = max_ltr_enc; -> >? ? ? >? ? ? ?} -> >? ? ? > -> >? ? ? > -> >? ? ?sasha +> > > lat_enc = max_ltr_enc; +> > > } +> > > +> > > +> > sasha > > > sasha diff --git a/a/content_digest b/N1/content_digest index 2a0dfd3..68f9ef5 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -4,9 +4,23 @@ "ref\00234b97a-f207-47b0-1545-582ee5282824@intel.com\0" "ref\0CALX8JfTOG7bXn7gP+4n1_3CfFFk6aL-RJDY03RCOeBSpUeTEPQ@mail.gmail.com\0" "From\0Neftin, Sasha <sasha.neftin@intel.com>\0" - "Subject\0[Intel-wired-lan] [PATCH] driver core: fix e1000e ltr bug\0" + "Subject\0Re: [Intel-wired-lan] [PATCH] driver core: fix e1000e ltr bug\0" "Date\0Thu, 1 Jul 2021 11:34:49 +0300\0" - "To\0intel-wired-lan@osuosl.org\0" + "To\0Yee Li <seven.yi.lee@gmail.com>\0" + "Cc\0jesse.brandeburg@intel.com" + anthony.l.nguyen@intel.com + davem@davemloft.net + kuba@kernel.org + intel-wired-lan@lists.osuosl.org + linux-kernel@vger.kernel.org + Ruinskiy + Dima <dima.ruinskiy@intel.com> + Edri + Michael <michael.edri@intel.com> + Efrati + Nir <nir.efrati@intel.com> + Neftin + " Sasha <sasha.neftin@intel.com>\0" "\00:1\0" "b\0" "On 6/30/2021 09:26, Yee Li wrote:\n" @@ -16,8 +30,8 @@ "will process a patch to address this problem. Can I put you kindly as \n" "\"Suggested-by\"?\n" "> \n" - "> Neftin, Sasha <sasha.neftin at intel.com <mailto:sasha.neftin@intel.com>> \n" - "> ?2021?6?30??? ??2:13???\n" + "> Neftin, Sasha <sasha.neftin@intel.com <mailto:sasha.neftin@intel.com>> \n" + "> \344\272\2162021\345\271\2646\346\234\21030\346\227\245\345\221\250\344\270\211 \344\270\213\345\215\2102:13\345\206\231\351\201\223\357\274\232\n" "> \n" "> On 6/29/2021 20:33, Yee Li wrote:\n" "> >\n" @@ -37,7 +51,7 @@ "> \n" "> Then:\n" "> if (lat_enc_d > max_ltr_enc_d)\n" - "> ? ? ? ? lat_enc = max_ltr_enc;\n" + "> \302\240 \302\240 \302\240 \302\240 lat_enc = max_ltr_enc;\n" "> what do you think?\n" "> \n" "> >\n" @@ -47,108 +61,108 @@ "> > Neftin, Sasha <sasha.neftin@intel.com\n" "> <mailto:sasha.neftin@intel.com> <mailto:sasha.neftin@intel.com\n" "> <mailto:sasha.neftin@intel.com>>>\n" - "> > ? 2021?6?29??? 22:49???\n" + "> > \344\272\216 2021\345\271\2646\346\234\21029\346\227\245\345\221\250\344\272\214 22:49\345\206\231\351\201\223\357\274\232\n" "> >\n" - "> >? ? ?On 6/29/2021 11:21, YeeLi wrote:\n" - "> >? ? ?Yeeli,\n" - "> >? ? ? > In e1000e driver, a PCIe-like device, the max\n" + "> >\302\240 \302\240 \302\240On 6/29/2021 11:21, YeeLi wrote:\n" + "> >\302\240 \302\240 \302\240Yeeli,\n" + "> >\302\240 \302\240 \302\240 > In e1000e driver, a PCIe-like device, the max\n" "> snoop/no-snoop latency\n" - "> >? ? ? > is the upper limit.So, directly compare the size of\n" + "> >\302\240 \302\240 \302\240 > is the upper limit.So, directly compare the size of\n" "> lat_enc and\n" - "> >? ? ? > max_ltr_enc is incorrect.\n" - "> >? ? ? >\n" - "> >? ? ?why?\n" - "> >? ? ? >? ? ? In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns,\n" + "> >\302\240 \302\240 \302\240 > max_ltr_enc is incorrect.\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240why?\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns,\n" "> correct.\n" - "> >? ? ? >\n" - "> >? ? ? >? ? ? In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns,\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns,\n" "> correct.\n" - "> >? ? ? >\n" - "> >? ? ? >? ? ? In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect.\n" - "> >? ? ? >\n" - "> >? ? ?Platform LTR encoded is 0x1003 - right. It is meant 1048576ns\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect.\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240Platform LTR encoded is 0x1003 - right. It is meant 1048576ns\n" "> x 3 =\n" - "> >? ? ?3145738ns.\n" - "> >? ? ?Now,\n" - "> >? ? ?for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct)\n" - "> >? ? ?for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct)\n" - "> >? ? ?for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?)\n" - "> >? ? ?18907136ns > 3145738ns, (latency encoded is great than\n" + "> >\302\240 \302\240 \302\2403145738ns.\n" + "> >\302\240 \302\240 \302\240Now,\n" + "> >\302\240 \302\240 \302\240for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct)\n" + "> >\302\240 \302\240 \302\240for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct)\n" + "> >\302\240 \302\240 \302\240for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?)\n" + "> >\302\240 \302\240 \302\24018907136ns > 3145738ns, (latency encoded is great than\n" "> maximum LTR\n" - "> >? ? ?encoded by platform) - so, there is no point to wait more\n" + "> >\302\240 \302\240 \302\240encoded by platform) - so, there is no point to wait more\n" "> than platform\n" - "> >? ? ?required, and lat_enc=max_ltr_enc. It is expected and we sent\n" + "> >\302\240 \302\240 \302\240required, and lat_enc=max_ltr_enc. It is expected and we sent\n" "> right\n" - "> >? ? ?value to the power management controller.\n" - "> >? ? ?What is the problem you try solve?\n" + "> >\302\240 \302\240 \302\240value to the power management controller.\n" + "> >\302\240 \302\240 \302\240What is the problem you try solve?\n" "> >\n" - "> >? ? ? > Decoded the lat_enc and max_ltr_enc before compare them is\n" + "> >\302\240 \302\240 \302\240 > Decoded the lat_enc and max_ltr_enc before compare them is\n" "> necessary.\n" - "> >? ? ? >\n" - "> >? ? ? > Signed-off-by: YeeLi <seven.yi.lee@gmail.com\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240 > Signed-off-by: YeeLi <seven.yi.lee@gmail.com\n" "> <mailto:seven.yi.lee@gmail.com>\n" - "> >? ? ?<mailto:seven.yi.lee at gmail.com <mailto:seven.yi.lee@gmail.com>>>\n" - "> >? ? ? > ---\n" - "> >? ? ? >? ?drivers/net/ethernet/intel/e1000e/ich8lan.c | 23\n" - "> >? ? ?++++++++++++++++++++-\n" - "> >? ? ? >? ?1 file changed, 22 insertions(+), 1 deletion(-)\n" - "> >? ? ? >\n" - "> >? ? ? > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" - "> >? ? ?b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" - "> >? ? ? > index 590ad110d383..3bff1b570b76 100644\n" - "> >? ? ? > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" - "> >? ? ? > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" - "> >? ? ? > @@ -986,6 +986,27 @@ static s32\n" + "> >\302\240 \302\240 \302\240<mailto:seven.yi.lee@gmail.com <mailto:seven.yi.lee@gmail.com>>>\n" + "> >\302\240 \302\240 \302\240 > ---\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240drivers/net/ethernet/intel/e1000e/ich8lan.c | 23\n" + "> >\302\240 \302\240 \302\240++++++++++++++++++++-\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\2401 file changed, 22 insertions(+), 1 deletion(-)\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240 > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" + "> >\302\240 \302\240 \302\240b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" + "> >\302\240 \302\240 \302\240 > index 590ad110d383..3bff1b570b76 100644\n" + "> >\302\240 \302\240 \302\240 > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" + "> >\302\240 \302\240 \302\240 > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n" + "> >\302\240 \302\240 \302\240 > @@ -986,6 +986,27 @@ static s32\n" "> e1000_k1_workaround_lpt_lp(struct\n" - "> >? ? ?e1000_hw *hw, bool link)\n" - "> >? ? ? >? ? ? ?return ret_val;\n" - "> >? ? ? >? ?}\n" - "> >? ? ? >\n" - "> >? ? ? > +static u32 convert_e1000e_ltr_scale(u32 val)\n" - "> >? ? ? > +{\n" - "> >? ? ? > +? ? ?if (val > 5)\n" - "> >? ? ? > +? ? ? ? ? ? ?return 0;\n" - "> >? ? ? > +\n" - "> >? ? ? > +? ? ?return 1U << (5 * val);\n" - "> >? ? ? > +}\n" - "> >? ? ? > +\n" - "> >? ? ? > +static u64 decoded_ltr(u32 val)\n" - "> >? ? ? > +{\n" - "> >? ? ? > +? ? ?u64 decoded_latency;\n" - "> >? ? ? > +? ? ?u32 value;\n" - "> >? ? ? > +? ? ?u32 scale;\n" - "> >? ? ? > +\n" - "> >? ? ? > +? ? ?value = val & 0x03FF;\n" - "> >? ? ? > +? ? ?scale = (val & 0x1C00) >> 10;\n" - "> >? ? ? > +? ? ?decoded_latency = value *\n" + "> >\302\240 \302\240 \302\240e1000_hw *hw, bool link)\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240return ret_val;\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240}\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240 > +static u32 convert_e1000e_ltr_scale(u32 val)\n" + "> >\302\240 \302\240 \302\240 > +{\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240if (val > 5)\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240return 0;\n" + "> >\302\240 \302\240 \302\240 > +\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240return 1U << (5 * val);\n" + "> >\302\240 \302\240 \302\240 > +}\n" + "> >\302\240 \302\240 \302\240 > +\n" + "> >\302\240 \302\240 \302\240 > +static u64 decoded_ltr(u32 val)\n" + "> >\302\240 \302\240 \302\240 > +{\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240u64 decoded_latency;\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240u32 value;\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240u32 scale;\n" + "> >\302\240 \302\240 \302\240 > +\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240value = val & 0x03FF;\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240scale = (val & 0x1C00) >> 10;\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240decoded_latency = value *\n" "> convert_e1000e_ltr_scale(scale);\n" - "> >? ? ? > +\n" - "> >? ? ? > +? ? ?return decoded_latency;\n" - "> >? ? ? > +}\n" - "> >? ? ? > +\n" - "> >? ? ? >? ?/**\n" - "> >? ? ? >? ? *? e1000_platform_pm_pch_lpt - Set platform power\n" + "> >\302\240 \302\240 \302\240 > +\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240return decoded_latency;\n" + "> >\302\240 \302\240 \302\240 > +}\n" + "> >\302\240 \302\240 \302\240 > +\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240/**\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 *\302\240 e1000_platform_pm_pch_lpt - Set platform power\n" "> management\n" - "> >? ? ?values\n" - "> >? ? ? >? ? *? @hw: pointer to the HW structure\n" - "> >? ? ? > @@ -1059,7 +1080,7 @@ static s32\n" + "> >\302\240 \302\240 \302\240values\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 *\302\240 @hw: pointer to the HW structure\n" + "> >\302\240 \302\240 \302\240 > @@ -1059,7 +1080,7 @@ static s32\n" "> e1000_platform_pm_pch_lpt(struct\n" - "> >? ? ?e1000_hw *hw, bool link)\n" - "> >? ? ? >? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? E1000_PCI_LTR_CAP_LPT + 2,\n" - "> >? ? ?&max_nosnoop);\n" - "> >? ? ? >? ? ? ? ? ? ? ?max_ltr_enc = max_t(u16, max_snoop,\n" + "> >\302\240 \302\240 \302\240e1000_hw *hw, bool link)\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 E1000_PCI_LTR_CAP_LPT + 2,\n" + "> >\302\240 \302\240 \302\240&max_nosnoop);\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240max_ltr_enc = max_t(u16, max_snoop,\n" "> max_nosnoop);\n" - "> >? ? ? >\n" - "> >? ? ? > -? ? ? ? ? ? ?if (lat_enc > max_ltr_enc)\n" - "> >? ? ? > +? ? ? ? ? ? ?if (decoded_ltr(lat_enc) >\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240 > -\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (lat_enc > max_ltr_enc)\n" + "> >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (decoded_ltr(lat_enc) >\n" "> decoded_ltr(max_ltr_enc))\n" - "> >? ? ? >? ? ? ? ? ? ? ? ? ? ? ?lat_enc = max_ltr_enc;\n" - "> >? ? ? >? ? ? ?}\n" - "> >? ? ? >\n" - "> >? ? ? >\n" - "> >? ? ?sasha\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240lat_enc = max_ltr_enc;\n" + "> >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240}\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240 >\n" + "> >\302\240 \302\240 \302\240sasha\n" "> >\n" "> \n" sasha -e88c2d18f97db331b94f5353112722c2d1032e7a0477322dc88766346dee2d50 +8df0fe7f893162223c28d690a0c599c0d2801c87ac0a5a31472cb85750ccd773
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.