Thanks Prithu,
The patch looks fine. Raghu, can you commit it?
Regards,
Amar
2008/6/24 <prithu@xxxxxxxxxxxxx>:
> Hi,
> Here is a patch for moderation.
>
> PATCH
>
>
> -----------------------------------------------------------------------------*
> local directory is at
> gluster@xxxxxxxxxx/glusterfs--mainline--3.0--patch-198
> * comparing to gluster@xxxxxxxxxx/glusterfs--mainline--3.0--patch-198
> M xlators/performance/write-behind/src/write-behind.c
>
> * modified files
>
> --- orig/xlators/performance/write-behind/src/write-behind.c
> +++ mod/xlators/performance/write-behind/src/write-behind.c
> @@ -1316,11 +1316,17 @@
> return -1;
> }
> }
> + if (conf->window_size == 0){
> + gf_log (this->name,
> + GF_LOG_ERROR,
> + "WARNING: window-size not specified -- Setting it tobe equal to
> the
> aggregate-size -- please check the spec-file if this is not intended");
> + conf->window_size = conf->aggregate_size;
> + }
>
> if (conf->window_size < conf->aggregate_size) {
> gf_log (this->name,
> GF_LOG_ERROR,
> - "FATAL: window-size (%d) is less than aggregate-size (%d)",
> conf->window_size, conf->aggregate_size);
> + "FATAL: The aggregate-size (%d) cannot be more than the
> window-size
> (%d)", conf->aggregate_size, conf->window_size);
> FREE (conf);
> return -1;
> }
> --------------------------------------------------------------------------
>
> Prithu
>
>
>
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel@xxxxxxxxxx
> http://lists.nongnu.org/mailman/listinfo/gluster-devel
>
--
Amar Tumballi
Gluster/GlusterFS Hacker
[bulde on #gluster/irc.gnu.org]
http://www.zresearch.com - Commoditizing Super Storage!
