Re: [PATCH 0/2] pkt_sched: sch_qfq: efficiency and codestyle improvements
From: Paolo Valente Date: Wed, 10 Jul 2013 15:46:07 +0200 > Hi, > this patchset tries to address the two issues raised by Dave in > http://marc.info/?l=linux-kernel=136254542019113=2 > > The first of these issues has been also raised by > David Laight: > http://marc.info/?l=linux-kernel=136256446624557=2 > >> Although two topics for possibly resolving later: >> >> 1) That 1ULL bit mask fix is quite expensive on 32-bit, it would >> probably be cheaper to test for that case using a helper function >> that nops out on 64-bit. Although this is not so important. >> > I have tried to find a simple solution, based on the properties of the mask. > It is in the first patch. > >> 2) That static inline forward declaration is ugly, better to remove >> the inline tag (let the compiler handle it) or move the function >> above all the call sites. >> > Done by the second patch. Looks good, thanks for doing this, both applied. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2] pkt_sched: sch_qfq: efficiency and codestyle improvements
From: Paolo Valente paolo.vale...@unimore.it Date: Wed, 10 Jul 2013 15:46:07 +0200 Hi, this patchset tries to address the two issues raised by Dave in http://marc.info/?l=linux-kernelm=136254542019113w=2 The first of these issues has been also raised by David Laight: http://marc.info/?l=linux-kernelm=136256446624557w=2 Although two topics for possibly resolving later: 1) That 1ULL bit mask fix is quite expensive on 32-bit, it would probably be cheaper to test for that case using a helper function that nops out on 64-bit. Although this is not so important. I have tried to find a simple solution, based on the properties of the mask. It is in the first patch. 2) That static inline forward declaration is ugly, better to remove the inline tag (let the compiler handle it) or move the function above all the call sites. Done by the second patch. Looks good, thanks for doing this, both applied. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 0/2] pkt_sched: sch_qfq: efficiency and codestyle improvements
Hi, this patchset tries to address the two issues raised by Dave in http://marc.info/?l=linux-kernel=136254542019113=2 The first of these issues has been also raised by David Laight: http://marc.info/?l=linux-kernel=136256446624557=2 > Although two topics for possibly resolving later: > > 1) That 1ULL bit mask fix is quite expensive on 32-bit, it would > probably be cheaper to test for that case using a helper function > that nops out on 64-bit. Although this is not so important. > I have tried to find a simple solution, based on the properties of the mask. It is in the first patch. > 2) That static inline forward declaration is ugly, better to remove > the inline tag (let the compiler handle it) or move the function > above all the call sites. > Done by the second patch. Paolo Valente (2): pkt_sched: sch_qfq: improve efficiency of make_eligible pkt_sched: sch_qfq: remove forward declaration of qfq_update_agg_ts net/sched/sch_qfq.c | 127 ++-- 1 file changed, 63 insertions(+), 64 deletions(-) -- 1.8.1.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 0/2] pkt_sched: sch_qfq: efficiency and codestyle improvements
Hi, this patchset tries to address the two issues raised by Dave in http://marc.info/?l=linux-kernelm=136254542019113w=2 The first of these issues has been also raised by David Laight: http://marc.info/?l=linux-kernelm=136256446624557w=2 Although two topics for possibly resolving later: 1) That 1ULL bit mask fix is quite expensive on 32-bit, it would probably be cheaper to test for that case using a helper function that nops out on 64-bit. Although this is not so important. I have tried to find a simple solution, based on the properties of the mask. It is in the first patch. 2) That static inline forward declaration is ugly, better to remove the inline tag (let the compiler handle it) or move the function above all the call sites. Done by the second patch. Paolo Valente (2): pkt_sched: sch_qfq: improve efficiency of make_eligible pkt_sched: sch_qfq: remove forward declaration of qfq_update_agg_ts net/sched/sch_qfq.c | 127 ++-- 1 file changed, 63 insertions(+), 64 deletions(-) -- 1.8.1.2 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/