| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20090730054204.GA11291@verge.net.au> Date: Thu, 30 Jul 2009 15:42:10 +1000 From: Simon Horman <horms@...ge.net.au> To: Ryo Tsuruta <ryov@...inux.co.jp> Cc: agk@...hat.com, dm-devel@...hat.com, linux-kernel@...r.kernel.org, containers@...ts.linux-foundation.org, virtualization@...ts.linux-foundation.org, xen-devel@...ts.xensource.com Subject: Re: [PATCH] dm-ioband-v1.12.2: I/O bandwidth controller On Thu, Jul 30, 2009 at 11:54:54AM +0900, Ryo Tsuruta wrote: > Hi Alasdair, > > This is dm-ioband v1.12.2 release that is a minor update to fit the > current dm-devel tree. > > Could you please replace dm-add-ioband.patch in your tree with this > patch? And I would appreciate it if you would let me know your > thoughts about merging dm-ioband to upstream. > > Thanks, > Ryo Tsuruta > > About dm-ioband: > dm-ioband is an I/O bandwidth controller implemented as a > device-mapper driver and can control bandwidth on per partition, per > user, per process, per virtual machine (such as KVM or Xen) basis. > Please visit our website for further information. > http://sourceforge.net/apps/trac/ioband/ > > Signed-off-by: Ryo Tsuruta <ryov@...inux.co.jp> > Signed-off-by: Hirokazu Takahashi <taka@...inux.co.jp> Hi Tsuruta-san, have you run sparse over this code? There seem to be a few interesting warnings generated (and some at the top noise too). $ make C=1 CF="-D__CHECK_ENDIAN__" [snip] include/trace/events/dm-ioband.h:9:1: warning: symbol 'ftrace_raw_output_ioband_hold_urgent_bio' was not declared. Should it be static? include/trace/events/dm-ioband.h:44:1: warning: symbol 'ftrace_raw_output_ioband_hold_bio' was not declared. Should it be static? include/trace/events/dm-ioband.h:79:1: warning: symbol 'ftrace_raw_output_ioband_make_pback_list' was not declared. Should it be static? include/trace/events/dm-ioband.h:114:1: warning: symbol 'ftrace_raw_output_ioband_make_issue_list' was not declared. Should it be static? include/trace/events/dm-ioband.h:149:1: warning: symbol 'ftrace_raw_output_ioband_release_urgent_bios' was not declared. Should it be static? include/trace/events/dm-ioband.h:180:1: warning: symbol 'ftrace_raw_output_ioband_make_request' was not declared. Should it be static? include/trace/events/dm-ioband.h:210:1: warning: symbol 'ftrace_raw_output_ioband_pushback_bio' was not declared. Should it be static? include/trace/events/dm-ioband.h:9:1: warning: symbol 'ftrace_define_fields_ioband_hold_urgent_bio' was not declared. Should it be static? include/trace/events/dm-ioband.h:44:1: warning: symbol 'ftrace_define_fields_ioband_hold_bio' was not declared. Should it be static? include/trace/events/dm-ioband.h:79:1: warning: symbol 'ftrace_define_fields_ioband_make_pback_list' was not declared. Should it be static? include/trace/events/dm-ioband.h:114:1: warning: symbol 'ftrace_define_fields_ioband_make_issue_list' was not declared. Should it be static? include/trace/events/dm-ioband.h:149:1: warning: symbol 'ftrace_define_fields_ioband_release_urgent_bios' was not declared. Should it be static? include/trace/events/dm-ioband.h:180:1: warning: symbol 'ftrace_define_fields_ioband_make_request' was not declared. Should it be static? include/trace/events/dm-ioband.h:210:1: warning: symbol 'ftrace_define_fields_ioband_pushback_bio' was not declared. Should it be static? drivers/md/dm-ioband-ctl.c:178:2: warning: context problem in 'suspend_ioband_device': '_spin_unlock_irqrestore' expected different context drivers/md/dm-ioband-ctl.c:178:2: context 'lock': wanted >= 1, got 0 drivers/md/dm-ioband-ctl.c:600:3: warning: context imbalance in 'prevent_burst_bios': unexpected unlock drivers/md/dm-ioband-ctl.c:600:3: context '<noident>': wanted 0, got -1 drivers/md/dm-ioband-ctl.c:816:3: warning: context imbalance in 'ioband_map': __context__ statement expected different context drivers/md/dm-ioband-ctl.c:816:3: context '<noident>': wanted >= 0, got -1 CC [M] drivers/md/dm-ioband-ctl.o CHECK drivers/md/dm-ioband-policy.c CC [M] drivers/md/dm-ioband-policy.o CHECK drivers/md/dm-ioband-rangebw.c drivers/md/dm-ioband-rangebw.c:592:6: warning: symbol 'range_bw_timeover' was not declared. Should it be static? drivers/md/dm-ioband-rangebw.c:287:4: warning: context imbalance in 'range_bw_queue_full': __context__ statement expected different context drivers/md/dm-ioband-rangebw.c:287:4: context '<noident>': wanted >= 0, got -1 CC [M] drivers/md/dm-ioband-rangebw.o drivers/md/dm-ioband-rangebw.c: In function 'range_bw_prepare_token': drivers/md/dm-ioband-rangebw.c:210: warning: 'io_mode' may be used uninitialized in this function -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists