svn commit: r216016 - head/sys/sparc64/include

2010-11-28 Thread Max Khon
Author: fjoe
Date: Sun Nov 28 19:26:20 2010
New Revision: 216016
URL: http://svn.freebsd.org/changeset/base/216016

Log:
  Define VM_KMEM_SIZE_MAX on sparc64. Otherwise kernel built with
  DEBUG_MEMGUARD panics early in kmeminit() with the message
  "kmem_suballoc: bad status return of 1" because of zero "size" argument
  passed to kmem_suballoc() due to "vm_kmem_size_max" being zero.
  
  The problem also exists on ia64.

Modified:
  head/sys/sparc64/include/vmparam.h

Modified: head/sys/sparc64/include/vmparam.h
==
--- head/sys/sparc64/include/vmparam.h  Sun Nov 28 18:59:52 2010
(r216015)
+++ head/sys/sparc64/include/vmparam.h  Sun Nov 28 19:26:20 2010
(r216016)
@@ -237,6 +237,14 @@
 #endif
 
 /*
+ * Ceiling on amount of kmem_map kva space.
+ */
+#ifndef VM_KMEM_SIZE_MAX
+#defineVM_KMEM_SIZE_MAX((VM_MAX_KERNEL_ADDRESS - \
+VM_MIN_KERNEL_ADDRESS + 1) * 3 / 5)
+#endif
+
+/*
  * Initial pagein size of beginning of executable file.
  */
 #ifndefVM_INITIAL_PAGEIN
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r216016 - head/sys/sparc64/include

2010-11-29 Thread Max Khon
Marius,

On Mon, Nov 29, 2010 at 1:45 AM, Marius Strobl wrote:

On Sun, Nov 28, 2010 at 07:26:20PM +, Max Khon wrote:
> > Author: fjoe
> > Date: Sun Nov 28 19:26:20 2010
> > New Revision: 216016
> > URL: http://svn.freebsd.org/changeset/base/216016
> >
> > Log:
> >   Define VM_KMEM_SIZE_MAX on sparc64. Otherwise kernel built with
> >   DEBUG_MEMGUARD panics early in kmeminit() with the message
> >   "kmem_suballoc: bad status return of 1" because of zero "size" argument
> >   passed to kmem_suballoc() due to "vm_kmem_size_max" being zero.
> >
> >   The problem also exists on ia64.
> >
> > Modified:
> >   head/sys/sparc64/include/vmparam.h
> >
> > Modified: head/sys/sparc64/include/vmparam.h
> >
> ==
> > --- head/sys/sparc64/include/vmparam.hSun Nov 28 18:59:52 2010
>  (r216015)
> > +++ head/sys/sparc64/include/vmparam.hSun Nov 28 19:26:20 2010
>  (r216016)
> > @@ -237,6 +237,14 @@
> >  #endif
> >
> >  /*
> > + * Ceiling on amount of kmem_map kva space.
> > + */
> > +#ifndef VM_KMEM_SIZE_MAX
> > +#define  VM_KMEM_SIZE_MAX((VM_MAX_KERNEL_ADDRESS - \
> > +VM_MIN_KERNEL_ADDRESS + 1) * 3 / 5)
> > +#endif
> > +
> > +/*
> >   * Initial pagein size of beginning of executable file.
> >   */
> >  #ifndef  VM_INITIAL_PAGEIN
>
> How was that value determined?
>

I've just copied it from amd64 to be non-zero for now. Do you have a better
idea of what it should look like?

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r216080 - head/sys/sparc64/include

2010-11-30 Thread Max Khon
Author: fjoe
Date: Tue Nov 30 16:49:06 2010
New Revision: 216080
URL: http://svn.freebsd.org/changeset/base/216080

Log:
  Change VM_KMEM_SIZE_MAX   to be just (VM_MAX_KERNEL_ADDRESS - 
VM_MIN_KERNEL_ADDRESS)
  
  Suggested by: marius

Modified:
  head/sys/sparc64/include/vmparam.h

Modified: head/sys/sparc64/include/vmparam.h
==
--- head/sys/sparc64/include/vmparam.h  Tue Nov 30 16:46:15 2010
(r216079)
+++ head/sys/sparc64/include/vmparam.h  Tue Nov 30 16:49:06 2010
(r216080)
@@ -240,8 +240,7 @@
  * Ceiling on amount of kmem_map kva space.
  */
 #ifndef VM_KMEM_SIZE_MAX
-#defineVM_KMEM_SIZE_MAX((VM_MAX_KERNEL_ADDRESS - \
-VM_MIN_KERNEL_ADDRESS + 1) * 3 / 5)
+#defineVM_KMEM_SIZE_MAX(VM_MAX_KERNEL_ADDRESS - 
VM_MIN_KERNEL_ADDRESS)
 #endif
 
 /*
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r216016 - head/sys/sparc64/include

2010-11-30 Thread Max Khon
Marius,

On Tue, Nov 30, 2010 at 1:24 AM, Marius Strobl wrote:


> > > On Sun, Nov 28, 2010 at 07:26:20PM +0000, Max Khon wrote:
> > > > > Author: fjoe
> > > > > Date: Sun Nov 28 19:26:20 2010
> > > > > New Revision: 216016
> > > > > URL: http://svn.freebsd.org/changeset/base/216016
> > > > >
> > > > > Log:
> > > > >   Define VM_KMEM_SIZE_MAX on sparc64. Otherwise kernel built with
> > > > >   DEBUG_MEMGUARD panics early in kmeminit() with the message
> > > > >   "kmem_suballoc: bad status return of 1" because of zero "size"
> argument
> > > > >   passed to kmem_suballoc() due to "vm_kmem_size_max" being zero.
> > > > >
> > > > >   The problem also exists on ia64.
> > > > >
> > > > > Modified:
> > > > >   head/sys/sparc64/include/vmparam.h
> > > > >
> > > > > Modified: head/sys/sparc64/include/vmparam.h
> > > > >
> > > >
> ==
> > > > > --- head/sys/sparc64/include/vmparam.hSun Nov 28 18:59:52
> 2010
> > > >  (r216015)
> > > > > +++ head/sys/sparc64/include/vmparam.hSun Nov 28 19:26:20
> 2010
> > > >  (r216016)
> > > > > @@ -237,6 +237,14 @@
> > > > >  #endif
> > > > >
> > > > >  /*
> > > > > + * Ceiling on amount of kmem_map kva space.
> > > > > + */
> > > > > +#ifndef VM_KMEM_SIZE_MAX
> > > > > +#define  VM_KMEM_SIZE_MAX((VM_MAX_KERNEL_ADDRESS - \
> > > > > +VM_MIN_KERNEL_ADDRESS + 1) * 3 / 5)
> > > > > +#endif
> > > > > +
> > > > > +/*
> > > > >   * Initial pagein size of beginning of executable file.
> > > > >   */
> > > > >  #ifndef  VM_INITIAL_PAGEIN
> > > >
> > > > How was that value determined?
> > > >
> > >
> > > I've just copied it from amd64 to be non-zero for now. Do you have a
> better
> > > idea of what it should look like?
> >
> > Well, on sparc64 VM_MAX_KERNEL_ADDRESS already is dynamically adjusted
> > to the maximum appropriate for the specific CPU during the early cycles
> > of the kernel so I'd think one could just use VM_MAX_KERNEL_ADDRESS -
> > VM_MIN_KERNEL_ADDRESS for VM_KMEM_SIZE_MAX there, I'm not sure what
> > the intention of the ceiling provided by that macro actually is though
>

Ok, this sounds reasonable. I changed it to be just  (VM_MAX_KERNEL_ADDRESS
- VM_MIN_KERNEL_ADDRESS)

> In any case, the commit message of r180210 which changed the amd64
> > version to the current one talks about limiting the kmem map to 3.6GB
> > and while it also fails to explain where that value comes from it
> > looks rather amd64 specific and the formula used by the macro will
> > result in a different ceiling on sparc64 and thus inappropriate. I've
> > CC'ed alc@ who hopefully can shed some light on this.
> > Apart from this the actual bug here seems to be that memguard_fudge()
> > can't deal with a km_max being zero or that zero is passed to it as
> > kmeminit() allows for VM_KMEM_SIZE_MAX not being defined.
>

I thought about fixing memguard_fudge() too, but defining VM_KMEM_SIZE_MAX
seemed to be less intrusive change to me.

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r236742 - stable/9/usr.bin/make

2012-06-08 Thread Max Khon
Author: fjoe
Date: Fri Jun  8 10:03:37 2012
New Revision: 236742
URL: http://svn.freebsd.org/changeset/base/236742

Log:
  MFC: r231544
  
  Include target names in diagnostic output.
  
  Submitted by: Garrett Cooper

Modified:
  stable/9/usr.bin/make/job.c
Directory Properties:
  stable/9/   (props changed)
  stable/9/usr.bin/   (props changed)
  stable/9/usr.bin/make/   (props changed)

Modified: stable/9/usr.bin/make/job.c
==
--- stable/9/usr.bin/make/job.c Fri Jun  8 09:51:57 2012(r236741)
+++ stable/9/usr.bin/make/job.c Fri Jun  8 10:03:37 2012(r236742)
@@ -954,17 +954,19 @@ JobFinish(Job *job, int *status)
lastNode = job->node;
}
fprintf(out,
-   "*** Completed successfully\n");
+   "*** [%s] Completed successfully\n",
+   job->node->name);
}
} else {
if (usePipes && job->node != lastNode) {
MESSAGE(out, job->node);
lastNode = job->node;
}
-   fprintf(out, "*** Error code %d%s\n",
+   fprintf(out, "*** [%s] Error code %d%s\n",
+   job->node->name,
WEXITSTATUS(*status),
(job->flags & JOB_IGNERR) ?
-   "(ignored)" : "");
+   " (ignored)" : "");
 
if (job->flags & JOB_IGNERR) {
*status = 0;
@@ -1005,7 +1007,8 @@ JobFinish(Job *job, int *status)
MESSAGE(out, job->node);
lastNode = job->node;
}
-   fprintf(out, "*** Continued\n");
+   fprintf(out, "*** [%s] Continued\n",
+   job->node->name);
}
if (!(job->flags & JOB_CONTINUING)) {
DEBUGF(JOB, ("Warning: process %jd was 
not "
@@ -1029,7 +1032,8 @@ JobFinish(Job *job, int *status)
lastNode = job->node;
}
fprintf(out,
-   "*** Signal %d\n", WTERMSIG(*status));
+   "*** [%s] Signal %d\n", job->node->name,
+   WTERMSIG(*status));
fflush(out);
}
}
@@ -1056,7 +1060,8 @@ JobFinish(Job *job, int *status)
MESSAGE(out, job->node);
lastNode = job->node;
}
-   fprintf(out, "*** Stopped -- signal %d\n", WSTOPSIG(*status));
+   fprintf(out, "*** [%s] Stopped -- signal %d\n",
+   job->node->name, WSTOPSIG(*status));
job->flags |= JOB_RESUME;
TAILQ_INSERT_TAIL(&stoppedJobs, job, link);
fflush(out);
@@ -3042,13 +3047,15 @@ Compat_RunCommand(char *cmd, GNode *gn)
if (status == 0) {
return (0);
} else {
-   printf("*** Error code %d", status);
+   printf("*** [%s] Error code %d",
+   gn->name, status);
}
} else if (WIFSTOPPED(reason)) {
status = WSTOPSIG(reason);
} else {
status = WTERMSIG(reason);
-   printf("*** Signal %d", status);
+   printf("*** [%s] Signal %d",
+   gn->name, status);
}
   
if (ps.errCheck) {
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r237350 - head/sys/dev/puc

2012-06-20 Thread Max Khon
Author: fjoe
Date: Thu Jun 21 03:10:48 2012
New Revision: 237350
URL: http://svn.freebsd.org/changeset/base/237350

Log:
  Add support for the following Moxa PCIe multiport serial boards:
  - CP102E
  - CP102EL
  - CP132EL
  - CP114EL
  - CP118EL-A
  - CP168EL-A
  
  MFC after:1 week

Modified:
  head/sys/dev/puc/puc_cfg.h
  head/sys/dev/puc/pucdata.c

Modified: head/sys/dev/puc/puc_cfg.h
==
--- head/sys/dev/puc/puc_cfg.h  Thu Jun 21 02:48:49 2012(r237349)
+++ head/sys/dev/puc/puc_cfg.h  Thu Jun 21 03:10:48 2012(r237350)
@@ -79,7 +79,7 @@ struct puc_cfg {
int8_t  ports;
int8_t  rid;/* Rid of first port */
int8_t  d_rid;  /* Delta rid of next ports */
-   int8_t  d_ofs;  /* Delta offset of next ports */
+   int16_t d_ofs;  /* Delta offset of next ports */
puc_config_f*config_function;
 };
 

Modified: head/sys/dev/puc/pucdata.c
==
--- head/sys/dev/puc/pucdata.c  Thu Jun 21 02:48:49 2012(r237349)
+++ head/sys/dev/puc/pucdata.c  Thu Jun 21 03:10:48 2012(r237350)
@@ -507,6 +507,18 @@ const struct puc_cfg puc_pci_devices[] =
.config_function = puc_config_quatech
},
 
+   {   0x1393, 0x1024, 0x, 0,
+   "Moxa Technologies, Smartio CP-102E/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x14, 0, 0x200
+   },
+
+   {   0x1393, 0x1025, 0x, 0,
+   "Moxa Technologies, Smartio CP-102EL/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x14, 0, 0x200,
+   },
+
{   0x1393, 0x1040, 0x, 0,
"Moxa Technologies, Smartio C104H/PCI",
DEFAULT_RCLK * 8,
@@ -550,6 +562,25 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_4S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1144, 0x, 0,
+   "Moxa Technologies, Smartio CP-114EL/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_4S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
+   },
+
+   {   0x1393, 0x1182, 0x, 0,
+   "Moxa Technologies, Smartio CP-118EL-A/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_8S, 0x14, 0, 0x200,
+   },
+
+   {   0x1393, 0x1322, 0x, 0,
+   "Moxa Technologies, Smartio CP-132EL/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x14, 0, 0x200,
+   },
+
{   0x1393, 0x1680, 0x, 0,
"Moxa Technologies, C168H/PCI",
DEFAULT_RCLK * 8,
@@ -568,6 +599,12 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_8S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1683, 0x, 0,
+   "Moxa Technologies, Smartio CP-168EL-A/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_8S, 0x14, 0, 0x200,
+   },
+
{   0x13a8, 0x0152, 0x, 0,
"Exar XR17C/D152",
DEFAULT_RCLK * 8,
@@ -1127,9 +1164,7 @@ static int
 puc_config_moxa(struct puc_softc *sc, enum puc_cfg_cmd cmd, int port,
 intptr_t *res)
 {
-   const struct puc_cfg *cfg = sc->sc_cfg;
-   
-   if (cmd == PUC_CFG_GET_OFS && cfg->device == 0x1045) {
+   if (cmd == PUC_CFG_GET_OFS) {
*res = ((port == 3) ? 7 : port) * 0x200;
return 0;
}
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r237357 - head/sys/dev/puc

2012-06-20 Thread Max Khon
Author: fjoe
Date: Thu Jun 21 04:57:59 2012
New Revision: 237357
URL: http://svn.freebsd.org/changeset/base/237357

Log:
  Remove Moxa CP-132EL definition (RS422/485-only board).

Modified:
  head/sys/dev/puc/pucdata.c

Modified: head/sys/dev/puc/pucdata.c
==
--- head/sys/dev/puc/pucdata.c  Thu Jun 21 04:52:26 2012(r237356)
+++ head/sys/dev/puc/pucdata.c  Thu Jun 21 04:57:59 2012(r237357)
@@ -575,12 +575,6 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_8S, 0x14, 0, 0x200,
},
 
-   {   0x1393, 0x1322, 0x, 0,
-   "Moxa Technologies, Smartio CP-132EL/PCIe",
-   DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200,
-   },
-
{   0x1393, 0x1680, 0x, 0,
"Moxa Technologies, C168H/PCI",
DEFAULT_RCLK * 8,
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r237703 - stable/9/sys/dev/puc

2012-06-28 Thread Max Khon
Author: fjoe
Date: Thu Jun 28 11:15:48 2012
New Revision: 237703
URL: http://svn.freebsd.org/changeset/base/237703

Log:
  MFC: r237350, r237357
  
  Add support for the following Moxa PCIe multiport serial boards:
  - CP102E
  - CP102EL
  - CP114EL
  - CP118EL-A
  - CP168EL-A

Modified:
  stable/9/sys/dev/puc/puc_cfg.h
  stable/9/sys/dev/puc/pucdata.c
Directory Properties:
  stable/9/   (props changed)
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/puc/puc_cfg.h
==
--- stable/9/sys/dev/puc/puc_cfg.h  Thu Jun 28 10:40:57 2012
(r237702)
+++ stable/9/sys/dev/puc/puc_cfg.h  Thu Jun 28 11:15:48 2012
(r237703)
@@ -79,7 +79,7 @@ struct puc_cfg {
int8_t  ports;
int8_t  rid;/* Rid of first port */
int8_t  d_rid;  /* Delta rid of next ports */
-   int8_t  d_ofs;  /* Delta offset of next ports */
+   int16_t d_ofs;  /* Delta offset of next ports */
puc_config_f*config_function;
 };
 

Modified: stable/9/sys/dev/puc/pucdata.c
==
--- stable/9/sys/dev/puc/pucdata.c  Thu Jun 28 10:40:57 2012
(r237702)
+++ stable/9/sys/dev/puc/pucdata.c  Thu Jun 28 11:15:48 2012
(r237703)
@@ -507,6 +507,18 @@ const struct puc_cfg puc_pci_devices[] =
.config_function = puc_config_quatech
},
 
+   {   0x1393, 0x1024, 0x, 0,
+   "Moxa Technologies, Smartio CP-102E/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x14, 0, 0x200
+   },
+
+   {   0x1393, 0x1025, 0x, 0,
+   "Moxa Technologies, Smartio CP-102EL/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x14, 0, 0x200,
+   },
+
{   0x1393, 0x1040, 0x, 0,
"Moxa Technologies, Smartio C104H/PCI",
DEFAULT_RCLK * 8,
@@ -550,6 +562,19 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_4S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1144, 0x, 0,
+   "Moxa Technologies, Smartio CP-114EL/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_4S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
+   },
+
+   {   0x1393, 0x1182, 0x, 0,
+   "Moxa Technologies, Smartio CP-118EL-A/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_8S, 0x14, 0, 0x200,
+   },
+
{   0x1393, 0x1680, 0x, 0,
"Moxa Technologies, C168H/PCI",
DEFAULT_RCLK * 8,
@@ -568,6 +593,12 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_8S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1683, 0x, 0,
+   "Moxa Technologies, Smartio CP-168EL-A/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_8S, 0x14, 0, 0x200,
+   },
+
{   0x13a8, 0x0152, 0x, 0,
"Exar XR17C/D152",
DEFAULT_RCLK * 8,
@@ -1121,9 +1152,7 @@ static int
 puc_config_moxa(struct puc_softc *sc, enum puc_cfg_cmd cmd, int port,
 intptr_t *res)
 {
-   const struct puc_cfg *cfg = sc->sc_cfg;
-   
-   if (cmd == PUC_CFG_GET_OFS && cfg->device == 0x1045) {
+   if (cmd == PUC_CFG_GET_OFS) {
*res = ((port == 3) ? 7 : port) * 0x200;
return 0;
}
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r238775 - stable/8/sys/dev/puc

2012-07-25 Thread Max Khon
Author: fjoe
Date: Wed Jul 25 17:42:57 2012
New Revision: 238775
URL: http://svn.freebsd.org/changeset/base/238775

Log:
  MFC: r227457, r237350, r237357
  
  Add support for the following Moxa PCIe multiport serial boards:
  - CP102E
  - CP102EL
  - CP104EL-A
  - CP104JU
  - CP114EL
  - CP118EL-A
  - CP168EL-A

Modified:
  stable/8/sys/dev/puc/puc_cfg.h
  stable/8/sys/dev/puc/pucdata.c
Directory Properties:
  stable/8/   (props changed)
  stable/8/sys/   (props changed)
  stable/8/sys/dev/   (props changed)

Modified: stable/8/sys/dev/puc/puc_cfg.h
==
--- stable/8/sys/dev/puc/puc_cfg.h  Wed Jul 25 17:25:44 2012
(r238774)
+++ stable/8/sys/dev/puc/puc_cfg.h  Wed Jul 25 17:42:57 2012
(r238775)
@@ -79,7 +79,7 @@ struct puc_cfg {
int8_t  ports;
int8_t  rid;/* Rid of first port */
int8_t  d_rid;  /* Delta rid of next ports */
-   int8_t  d_ofs;  /* Delta offset of next ports */
+   int16_t d_ofs;  /* Delta offset of next ports */
puc_config_f*config_function;
 };
 

Modified: stable/8/sys/dev/puc/pucdata.c
==
--- stable/8/sys/dev/puc/pucdata.c  Wed Jul 25 17:25:44 2012
(r238774)
+++ stable/8/sys/dev/puc/pucdata.c  Wed Jul 25 17:42:57 2012
(r238775)
@@ -51,6 +51,7 @@ static puc_config_f puc_config_amc;
 static puc_config_f puc_config_diva;
 static puc_config_f puc_config_exar;
 static puc_config_f puc_config_icbook;
+static puc_config_f puc_config_moxa;
 static puc_config_f puc_config_oxford_pcie;
 static puc_config_f puc_config_quatech;
 static puc_config_f puc_config_syba;
@@ -506,6 +507,18 @@ const struct puc_cfg puc_pci_devices[] =
.config_function = puc_config_quatech
},
 
+   {   0x1393, 0x1024, 0x, 0,
+   "Moxa Technologies, Smartio CP-102E/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x14, 0, 0x200
+   },
+
+   {   0x1393, 0x1025, 0x, 0,
+   "Moxa Technologies, Smartio CP-102EL/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x14, 0, 0x200,
+   },
+
{   0x1393, 0x1040, 0x, 0,
"Moxa Technologies, Smartio C104H/PCI",
DEFAULT_RCLK * 8,
@@ -518,12 +531,25 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_4S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1042, 0x, 0,
+   "Moxa Technologies, Smartio CP-104JU/PCI",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_4S, 0x18, 0, 8,
+   },
+
{   0x1393, 0x1043, 0x, 0,
"Moxa Technologies, Smartio CP-104EL/PCIe",
DEFAULT_RCLK * 8,
PUC_PORT_4S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1045, 0x, 0,
+   "Moxa Technologies, Smartio CP-104EL-A/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_4S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
+   },
+
{   0x1393, 0x1120, 0x, 0,
"Moxa Technologies, CP-112UL",
DEFAULT_RCLK * 8,
@@ -536,6 +562,19 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_4S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1144, 0x, 0,
+   "Moxa Technologies, Smartio CP-114EL/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_4S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
+   },
+
+   {   0x1393, 0x1182, 0x, 0,
+   "Moxa Technologies, Smartio CP-118EL-A/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_8S, 0x14, 0, 0x200,
+   },
+
{   0x1393, 0x1680, 0x, 0,
"Moxa Technologies, C168H/PCI",
DEFAULT_RCLK * 8,
@@ -554,6 +593,12 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_8S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1683, 0x, 0,
+   "Moxa Technologies, Smartio CP-168EL-A/PCIe",
+   DEFAULT_RCLK * 8,
+   PUC_PORT_8S, 0x14, 0, 0x200,
+   },
+
{   0x13a8, 0x0152, 0x, 0,
"Exar XR17C/D152",
DEFAULT_RCLK * 8,
@@ -1104,6 +1149,17 @@ puc_config_icbook(struct puc_softc *sc, 
 }
 
 static int
+puc_config_moxa(struct puc_softc *sc, enum puc_cfg_cmd cmd, int port,
+intptr_t *res)
+{
+   if (cmd == PUC_CFG_GET_OFS) {
+   *res = ((port == 3) ? 7 : port) * 0x200;
+   return 0;
+   }
+   return (ENXIO);
+}
+
+static int
 puc_config_quatech(struct puc_softc *sc, enum puc_cfg_cmd cmd, int port,
 intptr_t *res)
 {
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227130 - head/sys/netgraph

2011-11-05 Thread Max Khon
Author: fjoe
Date: Sun Nov  6 05:20:27 2011
New Revision: 227130
URL: http://svn.freebsd.org/changeset/base/227130

Log:
  Constify "address" argument of ng_address_path().

Modified:
  head/sys/netgraph/netgraph.h
  head/sys/netgraph/ng_base.c

Modified: head/sys/netgraph/netgraph.h
==
--- head/sys/netgraph/netgraph.hSun Nov  6 03:18:50 2011
(r227129)
+++ head/sys/netgraph/netgraph.hSun Nov  6 05:20:27 2011
(r227130)
@@ -1135,7 +1135,7 @@ SYSCTL_DECL(_net_graph);
  */
 intng_address_ID(node_p here, item_p item, ng_ID_t ID, ng_ID_t retaddr);
 intng_address_hook(node_p here, item_p item, hook_p hook, ng_ID_t retaddr);
-intng_address_path(node_p here, item_p item, char *address, ng_ID_t raddr);
+intng_address_path(node_p here, item_p item, const char *address, ng_ID_t 
raddr);
 intng_bypass(hook_p hook1, hook_p hook2);
 hook_p ng_findhook(node_p node, const char *name);
 struct ng_type *ng_findtype(const char *type);

Modified: head/sys/netgraph/ng_base.c
==
--- head/sys/netgraph/ng_base.c Sun Nov  6 03:18:50 2011(r227129)
+++ head/sys/netgraph/ng_base.c Sun Nov  6 05:20:27 2011(r227130)
@@ -3515,7 +3515,7 @@ ng_address_hook(node_p here, item_p item
 }
 
 int
-ng_address_path(node_p here, item_p item, char *address, ng_ID_t retaddr)
+ng_address_path(node_p here, item_p item, const char *address, ng_ID_t retaddr)
 {
node_p  dest = NULL;
hook_p  hook = NULL;
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227131 - head/sys/netgraph

2011-11-05 Thread Max Khon
Author: fjoe
Date: Sun Nov  6 05:23:42 2011
New Revision: 227131
URL: http://svn.freebsd.org/changeset/base/227131

Log:
  Fix potential double mbuf free: M_PREPEND may free mbuf chain and return
  NULL but item will still have the reference ot the mbuf chain and will free
  it upon destruction.

Modified:
  head/sys/netgraph/ng_cisco.c

Modified: head/sys/netgraph/ng_cisco.c
==
--- head/sys/netgraph/ng_cisco.cSun Nov  6 05:20:27 2011
(r227130)
+++ head/sys/netgraph/ng_cisco.cSun Nov  6 05:23:42 2011
(r227131)
@@ -359,12 +359,13 @@ cisco_rcvdata(hook_p hook, item_p item)
 
/* OK so it came from a protocol, heading out. Prepend general data
   packet header. For now, IP,IPX only  */
-   m = NGI_M(item); /* still associated with item */
+   NGI_GET_M(item, m);
M_PREPEND(m, CISCO_HEADER_LEN, M_DONTWAIT);
if (!m) {
error = ENOBUFS;
goto out;
}
+   NGI_M(item) = m;
h = mtod(m, struct cisco_header *);
h->address = CISCO_UNICAST;
h->control = 0;
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227132 - head/sys/netgraph

2011-11-05 Thread Max Khon
Author: fjoe
Date: Sun Nov  6 05:24:54 2011
New Revision: 227132
URL: http://svn.freebsd.org/changeset/base/227132

Log:
  - Fix potential double mbuf free: M_PREPEND may free mbuf chain and return
  NULL but item will still have the reference ot the mbuf chain and will free
  it upon destruction.
  - Fix memory leak (unfree'd item on error path).

Modified:
  head/sys/netgraph/ng_atmllc.c

Modified: head/sys/netgraph/ng_atmllc.c
==
--- head/sys/netgraph/ng_atmllc.c   Sun Nov  6 05:23:42 2011
(r227131)
+++ head/sys/netgraph/ng_atmllc.c   Sun Nov  6 05:24:54 2011
(r227132)
@@ -153,7 +153,7 @@ ng_atmllc_rcvdata(hook_p hook, item_p it
int error;
 
priv = NG_NODE_PRIVATE(NG_HOOK_NODE(hook));
-   m = NGI_M(item);
+   NGI_GET_M(item, m);
outhook = NULL;
padding = 0;
 
@@ -170,6 +170,7 @@ ng_atmllc_rcvdata(hook_p hook, item_p it
if (m->m_len < sizeof(struct atmllc) + ETHER_HDR_LEN) {
m = m_pullup(m, sizeof(struct atmllc) + ETHER_HDR_LEN);
if (m == NULL) {
+   NG_FREE_ITEM(item);
return (ENOMEM);
}
}
@@ -236,6 +237,7 @@ ng_atmllc_rcvdata(hook_p hook, item_p it
}
 
if (outhook == NULL) {
+   NG_FREE_M(m);
NG_FREE_ITEM(item);
return (0);
}
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227151 - in head/sys: conf kern

2011-11-06 Thread Max Khon
Author: fjoe
Date: Sun Nov  6 08:10:41 2011
New Revision: 227151
URL: http://svn.freebsd.org/changeset/base/227151

Log:
  Add KLD_DEBUG option.

Modified:
  head/sys/conf/options
  head/sys/kern/kern_linker.c

Modified: head/sys/conf/options
==
--- head/sys/conf/options   Sun Nov  6 08:03:33 2011(r227150)
+++ head/sys/conf/options   Sun Nov  6 08:10:41 2011(r227151)
@@ -59,6 +59,7 @@ GDB
 KDBopt_global.h
 KDB_TRACE  opt_kdb.h
 KDB_UNATTENDED opt_kdb.h
+KLD_DEBUG  opt_kld.h
 SYSCTL_DEBUG   opt_sysctl.h
 
 # Miscellaneous options.

Modified: head/sys/kern/kern_linker.c
==
--- head/sys/kern/kern_linker.c Sun Nov  6 08:03:33 2011(r227150)
+++ head/sys/kern/kern_linker.c Sun Nov  6 08:10:41 2011(r227151)
@@ -28,6 +28,7 @@
 __FBSDID("$FreeBSD$");
 
 #include "opt_ddb.h"
+#include "opt_kld.h"
 #include "opt_hwpmc_hooks.h"
 
 #include 
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227205 - stable/8/sys/kern

2011-11-06 Thread Max Khon
Author: fjoe
Date: Sun Nov  6 09:27:40 2011
New Revision: 227205
URL: http://svn.freebsd.org/changeset/base/227205

Log:
  Fix the build with KLD_DEBUG defined.

Modified:
  stable/8/sys/kern/kern_linker.c

Modified: stable/8/sys/kern/kern_linker.c
==
--- stable/8/sys/kern/kern_linker.c Sun Nov  6 09:17:48 2011
(r227204)
+++ stable/8/sys/kern/kern_linker.c Sun Nov  6 09:27:40 2011
(r227205)
@@ -632,7 +632,7 @@ linker_file_unload(linker_file_t file, i
 */
if ((error = module_unload(mod)) != 0) {
KLD_DPF(FILE, ("linker_file_unload: module %s"
-   " failed unload\n", mod));
+   " failed unload\n", module_getname(mod)));
return (error);
}
MOD_XLOCK;
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227902 - head/usr.sbin/sade

2011-11-23 Thread Max Khon
Author: fjoe
Date: Wed Nov 23 19:06:30 2011
New Revision: 227902
URL: http://svn.freebsd.org/changeset/base/227902

Log:
  - Remove unused old libdialog includes
  - Remove unused functions

Modified:
  head/usr.sbin/sade/misc.c
  head/usr.sbin/sade/sade.h

Modified: head/usr.sbin/sade/misc.c
==
--- head/usr.sbin/sade/misc.c   Wed Nov 23 18:38:08 2011(r227901)
+++ head/usr.sbin/sade/misc.c   Wed Nov 23 19:06:30 2011(r227902)
@@ -226,51 +226,6 @@ root_bias(char *path)
 return tmp;
 }
 
-/*
- * These next routines are kind of specialized just for building item lists
- * for dialog_menu().
- */
-
-/* Add an item to an item list */
-dialogMenuItem *
-item_add(dialogMenuItem *list, char *prompt, char *title,
-int (*checked)(dialogMenuItem *self),
-int (*fire)(dialogMenuItem *self),
-void (*selected)(dialogMenuItem *self, int is_selected),
-void *data, int *aux, int *curr, int *max)
-{
-dialogMenuItem *d;
-
-if (*curr == *max) {
-   *max += 20;
-   list = (dialogMenuItem *)safe_realloc(list, sizeof(dialogMenuItem) * 
*max);
-}
-d = &list[(*curr)++];
-bzero(d, sizeof(*d));
-d->prompt = prompt ? strdup(prompt) : NULL;
-d->title = title ? strdup(title) : NULL;
-d->checked = checked;
-d->fire = fire;
-d->selected = selected;
-d->data = data;
-d->aux = (long)aux;
-return list;
-}
-
-/* Toss the items out */
-void
-items_free(dialogMenuItem *list, int *curr, int *max)
-{
-int i;
-
-for (i = 0; list[i].prompt; i++) {
-   safe_free(list[i].prompt);
-   safe_free(list[i].title);
-}
-safe_free(list);
-*curr = *max = 0;
-}
-
 int
 Mkdir(char *ipath)
 {
@@ -311,12 +266,6 @@ Mkdir(char *ipath)
 }
 
 int
-Mkdir_command(char *key, void *dir)
-{
-return (Mkdir((char*)dir));
-}
-
-int
 Mount(char *mountp, void *dev)
 {
 struct ufs_args ufsargs;
@@ -353,126 +302,6 @@ Mount(char *mountp, void *dev)
 }
 
 WINDOW *
-openLayoutDialog(char *helpfile, char *title, int x, int y, int width, int 
height)
-{
-WINDOW *win;
-static charhelp[FILENAME_MAX];
-
-/* We need a curses window */
-win = newwin(LINES, COLS, 0, 0);
-if (win) {
-   /* Say where our help comes from */
-   if (helpfile) {
-   use_helpline("Press F1 for more information on this screen.");
-   use_helpfile(systemHelpFile(helpfile, help));
-   }
-   /* Setup a nice screen for us to splat stuff onto */
-   draw_box(win, y, x, height, width, dialog_attr, border_attr);
-   wattrset(win, dialog_attr);
-   mvwaddstr(win, y, x + (COLS - strlen(title)) / 2, title);
-}
-return win;
-}
-
-ComposeObj *
-initLayoutDialog(WINDOW *win, Layout *layout, int x, int y, int *max)
-{
-ComposeObj *obj = NULL, *first;
-int n;
-
-/* Loop over the layout list, create the objects, and add them
-   onto the chain of objects that dialog uses for traversal*/
-
-n = 0;
-while (layout[n].help != NULL) {
-   int t = TYPE_OF_OBJ(layout[n].type);
-
-   switch (t) {
-   case STRINGOBJ:
-   layout[n].obj = NewStringObj(win, layout[n].prompt, layout[n].var,
-layout[n].y + y, layout[n].x + x, 
layout[n].len, layout[n].maxlen);
-   ((StringObj *)layout[n].obj)->attr_mask = 
ATTR_OF_OBJ(layout[n].type);
-   break;
-   
-   case BUTTONOBJ:
-   layout[n].obj = NewButtonObj(win, layout[n].prompt, layout[n].var, 
layout[n].y + y, layout[n].x + x);
-   break;
-   
-   default:
-   msgFatal("Don't support this object yet!");
-   }
-   AddObj(&obj, t, (void *) layout[n].obj);
-   n++;
-}
-*max = n - 1;
-/* Find the first object in the list */
-for (first = obj; first->prev; first = first->prev);
-return first;
-}
-
-int
-layoutDialogLoop(WINDOW *win, Layout *layout, ComposeObj **obj, int *n, int 
max, int *cbutton, int *cancel)
-{
-char help_line[80];
-int ret, i, len = strlen(layout[*n].help);
-
-/* Display the help line at the bottom of the screen */
-for (i = 0; i < 79; i++)
-   help_line[i] = (i < len) ? layout[*n].help[i] : ' ';
-help_line[i] = '\0';
-use_helpline(help_line);
-display_helpline(win, LINES - 1, COLS - 1);
-wrefresh(win);
-   
-/* Ask for libdialog to do its stuff */
-ret = PollObj(obj);
-/* Handle special case stuff that libdialog misses. Sigh */
-switch (ret) {
-case SEL_ESC:  /* Bail out */
-   *cancel = TRUE;
-   return FALSE;
- 
-   /* This doesn't work for list dialogs. Oh well. Perhaps
-  should special case the move from the OK button ``up''
-  to make it go to the interface list, but then it gets
-  awkward for the user to go back and correct screw up's
-  in the per-inte

svn commit: r227903 - head/usr.sbin/sade

2011-11-23 Thread Max Khon
Author: fjoe
Date: Wed Nov 23 19:11:03 2011
New Revision: 227903
URL: http://svn.freebsd.org/changeset/base/227903

Log:
  Remove unused functions.

Modified:
  head/usr.sbin/sade/misc.c
  head/usr.sbin/sade/sade.h

Modified: head/usr.sbin/sade/misc.c
==
--- head/usr.sbin/sade/misc.c   Wed Nov 23 19:06:30 2011(r227902)
+++ head/usr.sbin/sade/misc.c   Wed Nov 23 19:11:03 2011(r227903)
@@ -56,27 +56,6 @@ file_readable(char *fname)
 return FALSE;
 }
 
-/* Quick check to see if a file is executable */
-Boolean
-file_executable(char *fname)
-{
-if (!access(fname, X_OK))
-   return TRUE;
-return FALSE;
-}
-
-/* Concatenate two strings into static storage */
-char *
-string_concat(char *one, char *two)
-{
-static char tmp[FILENAME_MAX];
-
-/* Yes, we're deliberately cavalier about not checking for overflow */
-strcpy(tmp, one);
-strcat(tmp, two);
-return tmp;
-}
-
 /* sane strncpy() function */
 char *
 sstrncpy(char *dst, const char *src, int size)
@@ -85,19 +64,6 @@ sstrncpy(char *dst, const char *src, int
 return strncpy(dst, src, size);
 }
 
-/* Concatenate three strings into static storage */
-char *
-string_concat3(char *one, char *two, char *three)
-{
-static char tmp[FILENAME_MAX];
-
-/* Yes, we're deliberately cavalier about not checking for overflow */
-strcpy(tmp, one);
-strcat(tmp, two);
-strcat(tmp, three);
-return tmp;
-}
-
 /* Clip the whitespace off the end of a string */
 char *
 string_prune(char *str)
@@ -118,29 +84,6 @@ string_skipwhite(char *str)
 return str;
 }
 
-/* copy optionally and allow second arg to be null */
-char *
-string_copy(char *s1, char *s2)
-{
-if (!s1)
-   return NULL;
-if (!s2)
-   s1[0] = '\0';
-else
-   strcpy(s1, s2);
-return s1;
-}
-
-/* convert an integer to a string, using a static buffer */
-char *
-itoa(int value)
-{
-static char buf[13];
-
-snprintf(buf, 12, "%d", value);
-return buf;
-}
-
 Boolean
 directory_exists(const char *dirname)
 {
@@ -159,22 +102,6 @@ directory_exists(const char *dirname)
 return (TRUE);
 }
 
-char *
-pathBaseName(const char *path)
-{
-char *pt;
-char *ret = (char *)path;
-
-pt = strrchr(path,(int)'/');
-
-if (pt != 0)   /* if there is a slash */
-{
-   ret = ++pt; /* start the file after it */
-}
-
-return(ret);
-}
-
 /* A free guaranteed to take NULL ptrs */
 void
 safe_free(void *ptr)
@@ -198,34 +125,6 @@ safe_malloc(size_t size)
 return ptr;
 }
 
-/* A realloc that checks errors */
-void *
-safe_realloc(void *orig, size_t size)
-{
-void *ptr;
-
-if (size <= 0)
-   msgFatal("Invalid realloc size of %ld!", (long)size);
-ptr = reallocf(orig, size);
-if (!ptr)
-   msgFatal("Out of memory!");
-return ptr;
-}
-
-/* Create a path biased from the VAR_INSTALL_ROOT variable (if not /) */
-char *
-root_bias(char *path)
-{
-static char tmp[FILENAME_MAX];
-char *cp = variable_get(VAR_INSTALL_ROOT);
-
-if (!strcmp(cp, "/"))
-   return path;
-strcpy(tmp, variable_get(VAR_INSTALL_ROOT));
-strcat(tmp, path);
-return tmp;
-}
-
 int
 Mkdir(char *ipath)
 {

Modified: head/usr.sbin/sade/sade.h
==
--- head/usr.sbin/sade/sade.h   Wed Nov 23 19:06:30 2011(r227902)
+++ head/usr.sbin/sade/sade.h   Wed Nov 23 19:11:03 2011(r227903)
@@ -387,19 +387,11 @@ extern intdiskLabelCommit(dialogMenuIte
 
 /* misc.c */
 extern Boolean file_readable(char *fname);
-extern Boolean file_executable(char *fname);
 extern Boolean directory_exists(const char *dirname);
-extern char*root_bias(char *path);
-extern char*itoa(int value);
-extern char*string_concat(char *p1, char *p2);
-extern char*string_concat3(char *p1, char *p2, char *p3);
 extern char*string_prune(char *str);
 extern char*string_skipwhite(char *str);
-extern char*string_copy(char *s1, char *s2);
-extern char*pathBaseName(const char *path);
 extern voidsafe_free(void *ptr);
 extern void*safe_malloc(size_t size);
-extern void*safe_realloc(void *orig, size_t size);
 extern int Mkdir(char *);
 extern int Mount(char *, void *data);
 
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227928 - head/usr.sbin/sade

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 08:56:24 2011
New Revision: 227928
URL: http://svn.freebsd.org/changeset/base/227928

Log:
  Remove unused code.

Deleted:
  head/usr.sbin/sade/config.c
Modified:
  head/usr.sbin/sade/Makefile
  head/usr.sbin/sade/sade.h

Modified: head/usr.sbin/sade/Makefile
==
--- head/usr.sbin/sade/Makefile Thu Nov 24 07:39:01 2011(r227927)
+++ head/usr.sbin/sade/Makefile Thu Nov 24 08:56:24 2011(r227928)
@@ -6,7 +6,7 @@ _wizard=wizard.c
 
 PROG=  sade
 MAN=   sade.8
-SRCS=  command.c config.c devices.c \
+SRCS=  command.c devices.c \
disks.c dispatch.c dmenu.c \
globals.c install.c \
label.c main.c menus.c misc.c \

Modified: head/usr.sbin/sade/sade.h
==
--- head/usr.sbin/sade/sade.h   Thu Nov 24 07:39:01 2011(r227927)
+++ head/usr.sbin/sade/sade.h   Thu Nov 24 08:56:24 2011(r227928)
@@ -301,13 +301,6 @@ extern voidcommand_execute(void);
 extern voidcommand_shell_add(char *key, const char *fmt, ...) 
__printflike(2, 3);
 extern voidcommand_func_add(char *key, commandFunc func, void *data);
 
-/* config.c */
-extern voidconfigEnvironmentRC_conf(void);
-extern voidconfigRC_conf(void);
-extern int configFstab(dialogMenuItem *self);
-extern int configRC(dialogMenuItem *self);
-extern int configWriteRC_conf(dialogMenuItem *self);
-
 /* devices.c */
 extern DMenu   *deviceCreateMenu(DMenu *menu, DeviceType type, int 
(*hook)(dialogMenuItem *d),
  int (*check)(dialogMenuItem *d));
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227929 - head/usr.sbin/sade

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 10:08:22 2011
New Revision: 227929
URL: http://svn.freebsd.org/changeset/base/227929

Log:
  Remove unused code.

Deleted:
  head/usr.sbin/sade/dispatch.c
Modified:
  head/usr.sbin/sade/Makefile

Modified: head/usr.sbin/sade/Makefile
==
--- head/usr.sbin/sade/Makefile Thu Nov 24 08:56:24 2011(r227928)
+++ head/usr.sbin/sade/Makefile Thu Nov 24 10:08:22 2011(r227929)
@@ -7,7 +7,7 @@ _wizard=wizard.c
 PROG=  sade
 MAN=   sade.8
 SRCS=  command.c devices.c \
-   disks.c dispatch.c dmenu.c \
+   disks.c dmenu.c \
globals.c install.c \
label.c main.c menus.c misc.c \
msg.c system.c termcap.c \
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227934 - head/usr.sbin/tzsetup

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 13:24:06 2011
New Revision: 227934
URL: http://svn.freebsd.org/changeset/base/227934

Log:
  Port to new libdialog.

Modified:
  head/usr.sbin/tzsetup/Makefile
  head/usr.sbin/tzsetup/tzsetup.c

Modified: head/usr.sbin/tzsetup/Makefile
==
--- head/usr.sbin/tzsetup/Makefile  Thu Nov 24 13:18:58 2011
(r227933)
+++ head/usr.sbin/tzsetup/Makefile  Thu Nov 24 13:24:06 2011
(r227934)
@@ -3,11 +3,11 @@
 PROG=  tzsetup
 MAN=   tzsetup.8
 
-CFLAGS+= -I${.CURDIR}/../../gnu/lib/libodialog -I.
+CFLAGS+= -I${.CURDIR}/../../contrib/dialog -I.
 
 WARNS?=3
 
-DPADD= ${LIBODIALOG} ${LIBNCURSES}
-LDADD= -lodialog -lncurses
+DPADD= ${LIBDIALOG} ${LIBNCURSESW} ${LIBM}
+LDADD= -ldialog -lncursesw -lm
 
 .include 

Modified: head/usr.sbin/tzsetup/tzsetup.c
==
--- head/usr.sbin/tzsetup/tzsetup.c Thu Nov 24 13:18:58 2011
(r227933)
+++ head/usr.sbin/tzsetup/tzsetup.c Thu Nov 24 13:24:06 2011
(r227934)
@@ -35,7 +35,6 @@
 #include 
 __FBSDID("$FreeBSD$");
 
-#include 
 #include 
 #include 
 #include 
@@ -49,6 +48,8 @@ __FBSDID("$FreeBSD$");
 #include 
 #include 
 
+#include 
+
 #define_PATH_ZONETAB   "/usr/share/zoneinfo/zone.tab"
 #define_PATH_ISO3166   "/usr/share/misc/iso3166"
 #define_PATH_ZONEINFO  "/usr/share/zoneinfo"
@@ -56,8 +57,92 @@ __FBSDID("$FreeBSD$");
 #define_PATH_DB"/var/db/zoneinfo"
 #define_PATH_WALL_CMOS_CLOCK   "/etc/wall_cmos_clock"
 
+/* special return codes for `fire' actions */
+#define DITEM_FAILURE   1
+
+/* flags - returned in upper 16 bits of return status */
+#define DITEM_LEAVE_MENU(1 << 16)
+#define DITEM_RECREATE  (1 << 18)
+
+/* for use in describing more exotic behaviors */
+typedef struct dialogMenuItem {
+   char *prompt;
+   char *title;
+   int (*fire)(struct dialogMenuItem *self);
+   void *data;
+} dialogMenuItem;
+
+static int
+xdialog_menu(const char *title, const char *cprompt, int height, int width,
+int menu_height, int item_no, dialogMenuItem *ditems)
+{
+   int i, result, choice;
+   DIALOG_LISTITEM *listitems;
+   DIALOG_VARS save_vars;
+
+   dlg_save_vars(&save_vars);
+
+   /* initialize list items */
+   listitems = dlg_calloc(DIALOG_LISTITEM, item_no);
+   assert_ptr(listitems, "xdialog_menu");
+   for (i = 0; i < item_no; i++) {
+   listitems[i].name = ditems[i].prompt;
+   listitems[i].text = ditems[i].title;
+   }
+
+   /* calculate width */
+   if (width < 0) {
+   int tag_x = 0;
+
+   for (i = 0; i < item_no; i++) {
+   int j, l;
+
+   l = strlen(listitems[i].name);
+   for (j = 0; j < item_no; j++) {
+   int k = strlen(listitems[j].text);
+   tag_x = MAX(tag_x, l + k + 2);
+   }
+   }
+   width = MAX(dlg_count_columns(cprompt), title != NULL ? 
dlg_count_columns(title) : 0);
+   width = MAX(width, tag_x + 4) + 4;
+   }
+   width = MAX(width, 24);
+   if (width > COLS)
+   width = COLS;
+
+again:
+   dialog_vars.default_item = ditems[choice].prompt;
+   result = dlg_menu(title, cprompt, height, width,
+   menu_height, item_no, listitems, &choice, NULL);
+   switch (result) {
+   case DLG_EXIT_ESC:
+   result = -1;
+   break;
+   case DLG_EXIT_OK:
+   if (ditems[choice].fire != NULL) {
+   int status;
+
+   status = ditems[choice].fire(ditems + choice);
+   if (status & DITEM_RECREATE) {
+   dlg_clear();
+   goto again;
+   }
+   }
+   result = 0;
+   break;
+   case DLG_EXIT_CANCEL:
+   default:
+   result = 1;
+   break;
+   }
+
+   free(listitems);
+   dlg_restore_vars(&save_vars);
+   return result;
+}
+
 static charpath_zonetab[MAXPATHLEN], path_iso3166[MAXPATHLEN],
-   path_zoneinfo[MAXPATHLEN], path_localtime[MAXPATHLEN], 
+   path_zoneinfo[MAXPATHLEN], path_localtime[MAXPATHLEN],
path_db[MAXPATHLEN], path_wall_cmos_clock[MAXPATHLEN];
 
 static int reallydoit = 1;
@@ -77,8 +162,6 @@ static int   set_zone_utc(void);
 struct continent {
dialogMenuItem *menu;
int nitems;
-   int ch;
-   int sc;
 };
 
 static struct continentafrica, america, antarctica, arctic, asia, 
atlantic;
@@ -98,7 +181,7 @@ static struct continent_names {
{ "Eu

svn commit: r227935 - head/tools

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 14:16:01 2011
New Revision: 227935
URL: http://svn.freebsd.org/changeset/base/227935

Log:
  Utility script to build specific parts of the source tree on all arches

Added:
  head/tools/tinder.sh   (contents, props changed)

Added: head/tools/tinder.sh
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/tinder.shThu Nov 24 14:16:01 2011(r227935)
@@ -0,0 +1,58 @@
+#!/bin/sh
+#
+# Copyright (c) 2011 Max Khon, The FreeBSD Project
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions
+# are met:
+# 1. Redistributions of source code must retain the above copyright
+#notice, this list of conditions and the following disclaimer.
+# 2. Redistributions in binary form must reproduce the above copyright
+#notice, this list of conditions and the following disclaimer in the
+#documentation and/or other materials provided with the distribution.
+#
+# THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+# ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+# IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+# ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+# FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+# DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+# OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+# HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+# LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+# OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+# SUCH DAMAGE.
+#
+# $FreeBSD$
+#
+
+#
+# Utility script to build specific parts of the source tree on all arches
+#
+# Example:
+#
+# cd /usr/src
+# make toolchains  # build toolchain for all arches
+# sh tools/tinder.sh gnu/lib/libdialog usr.sbin/sade NO_CLEAN=yes
+#  # build libdialog and sade for all architectures
+#  # without making clean
+#
+
+if [ $# -eq 0 ]; then
+   echo 1>&2 "Usage: `basename $0` [MAKEVAR=value...] path..."
+   exit 1
+fi
+
+# MAKE_ARGS is intentionally not reset to allow caller to specify additional 
MAKE_ARGS
+SUBDIR=
+for i in "$@"; do
+   case "$i" in
+   *=*)
+   MAKE_ARGS="$MAKE_ARGS $i"
+   ;;
+   *)
+   SUBDIR="$SUBDIR $i"
+   esac
+done
+make tinderbox UNIVERSE_TARGET="_cleanobj _obj _depend everything" $MAKE_ARGS 
SUBDIR_OVERRIDE="$SUBDIR"
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227936 - head/gnu/usr.bin/dialog

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 14:17:04 2011
New Revision: 227936
URL: http://svn.freebsd.org/changeset/base/227936

Log:
  Use ${VAR} instead of $(VAR) consistently.

Modified:
  head/gnu/usr.bin/dialog/Makefile

Modified: head/gnu/usr.bin/dialog/Makefile
==
--- head/gnu/usr.bin/dialog/MakefileThu Nov 24 14:16:01 2011
(r227935)
+++ head/gnu/usr.bin/dialog/MakefileThu Nov 24 14:17:04 2011
(r227936)
@@ -3,7 +3,7 @@
 DIALOG=${.CURDIR}/../../../contrib/dialog
 PROG=   dialog
 
-DPADD= $(LIBDIALOG) $(LIBNCURSES) $(LIBM)
+DPADD= ${LIBDIALOG} ${LIBNCURSES} ${LIBM}
 LDADD= -ldialog -lncursesw -lm
 CFLAGS+= -I${.CURDIR} -I${DIALOG}
 .PATH: ${DIALOG}
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227937 - head/tools

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 14:22:05 2011
New Revision: 227937
URL: http://svn.freebsd.org/changeset/base/227937

Log:
  Add missing ;;

Modified:
  head/tools/tinder.sh

Modified: head/tools/tinder.sh
==
--- head/tools/tinder.shThu Nov 24 14:17:04 2011(r227936)
+++ head/tools/tinder.shThu Nov 24 14:22:05 2011(r227937)
@@ -53,6 +53,7 @@ for i in "$@"; do
;;
*)
SUBDIR="$SUBDIR $i"
+   ;;
esac
 done
 make tinderbox UNIVERSE_TARGET="_cleanobj _obj _depend everything" $MAKE_ARGS 
SUBDIR_OVERRIDE="$SUBDIR"
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227939 - head/usr.sbin/sade

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 15:47:01 2011
New Revision: 227939
URL: http://svn.freebsd.org/changeset/base/227939

Log:
  Disk selection dialog is now a normal menu, not a checkboxed menu:
  checkboxed menu is confusing and also can not be implemented
  in new libdialog.

Modified:
  head/usr.sbin/sade/devices.c
  head/usr.sbin/sade/disks.c
  head/usr.sbin/sade/install.c
  head/usr.sbin/sade/label.c
  head/usr.sbin/sade/menus.c
  head/usr.sbin/sade/sade.h

Modified: head/usr.sbin/sade/devices.c
==
--- head/usr.sbin/sade/devices.cThu Nov 24 15:12:57 2011
(r227938)
+++ head/usr.sbin/sade/devices.cThu Nov 24 15:47:01 2011
(r227939)
@@ -122,7 +122,7 @@ deviceTry(struct _devname dev, char *try
 
 /* Register a new device in the devices array */
 Device *
-deviceRegister(char *name, char *desc, char *devname, DeviceType type, Boolean 
enabled,
+deviceRegister(char *name, char *desc, char *devname, DeviceType type,
   Boolean (*init)(Device *), FILE * (*get)(Device *, char *, 
Boolean),
   void (*shutdown)(Device *), void *private)
 {
@@ -135,7 +135,6 @@ deviceRegister(char *name, char *desc, c
newdev->description = desc;
newdev->devname = devname;
newdev->type = type;
-   newdev->enabled = enabled;
newdev->init = init ? init : dummyInit;
newdev->get = get ? get : dummyGet;
newdev->shutdown = shutdown ? shutdown : dummyShutdown;
@@ -223,7 +222,7 @@ deviceGetAll(void)
continue;
}
 
-   deviceRegister(names[i], names[i], d->name, DEVICE_TYPE_DISK, FALSE,
+   deviceRegister(names[i], names[i], d->name, DEVICE_TYPE_DISK,
   dummyInit, dummyGet, dummyShutdown, d);
if (isDebug())
msgDebug("Found a disk device named %s\n", names[i]);
@@ -237,7 +236,7 @@ deviceGetAll(void)
 
/* Got one! */
snprintf(devname, sizeof devname, "/dev/%s", c1->name);
-   dev = deviceRegister(c1->name, c1->name, strdup(devname), 
DEVICE_TYPE_DOS, TRUE,
+   dev = deviceRegister(c1->name, c1->name, strdup(devname), 
DEVICE_TYPE_DOS,
mediaInitDOS, mediaGetDOS, mediaShutdownDOS, NULL);
dev->private = c1;
if (isDebug())
@@ -314,7 +313,7 @@ deviceCount(Device **devs)
  * menu is cloned.
  */
 DMenu *
-deviceCreateMenu(DMenu *menu, DeviceType type, int (*hook)(dialogMenuItem *d), 
int (*check)(dialogMenuItem *d))
+deviceCreateMenu(DMenu *menu, DeviceType type, int (*hook)(dialogMenuItem *d))
 {
 Device **devs;
 int numdevs;
@@ -338,7 +337,6 @@ deviceCreateMenu(DMenu *menu, DeviceType
if (j == numDevs)
tmp->items[i].title = "";
tmp->items[i].fire = hook;
-   tmp->items[i].checked = check;
 }
 tmp->items[i].title = NULL;
 return tmp;

Modified: head/usr.sbin/sade/disks.c
==
--- head/usr.sbin/sade/disks.c  Thu Nov 24 15:12:57 2011(r227938)
+++ head/usr.sbin/sade/disks.c  Thu Nov 24 15:47:01 2011(r227939)
@@ -305,25 +305,6 @@ getBootMgr(char *dname, u_char **bootCod
 #endif
 #endif /* WITH_SLICES */
 
-int
-diskGetSelectCount(Device ***devs)
-{
-int i, cnt, enabled;
-char *cp;
-Device **dp;
-
-cp = variable_get(VAR_DISK);
-dp = *devs = deviceFind(cp, DEVICE_TYPE_DISK);
-cnt = deviceCount(dp);
-if (!cnt)
-   return -1;
-for (i = 0, enabled = 0; i < cnt; i++) {
-   if (dp[i]->enabled)
-   ++enabled;
-}
-return enabled;
-}
-
 #ifdef WITH_SLICES
 void
 diskPartition(Device *dev)
@@ -641,7 +622,7 @@ diskPartition(Device *dev)
Set_Boot_Mgr(d, mbrContents, mbrSize);
 #endif
 
-   if (DITEM_STATUS(diskPartitionWrite(NULL)) != DITEM_SUCCESS)
+   if (DITEM_STATUS(diskPartitionWrite(dev)) != DITEM_SUCCESS)
msgConfirm("Disk partition write returned an error 
status!");
else
msgConfirm("Wrote FDISK partition information out 
successfully.");
@@ -770,7 +751,7 @@ bootalloc(char *name, size_t *size)
 }
 #endif /* !__ia64__ */
 
-#ifdef WITH_SLICES 
+#ifdef WITH_SLICES
 static int
 partitionHook(dialogMenuItem *selected)
 {
@@ -781,58 +762,28 @@ partitionHook(dialogMenuItem *selected)
msgConfirm("Unable to find disk %s!", selected->prompt);
return DITEM_FAILURE;
 }
-/* Toggle enabled status? */
-if (!devs[0]->enabled) {
-   devs[0]->enabled = TRUE;
-   diskPartition(devs[0]);
-}
-else
-   devs[0]->enabled = FALSE;
+diskPartition(devs[0]);
 return DITEM_SUCCESS;
 }
 
-static int
-partitionCheck(dialogMenuItem *selected)
-{
-Device **devs = NULL;
-
-devs = deviceFind(selected->prompt, DEVICE_TYPE_DISK);
-if 

svn commit: r227944 - head/usr.sbin/sade

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 18:37:16 2011
New Revision: 227944
URL: http://svn.freebsd.org/changeset/base/227944

Log:
  Port to new libdialog.

Modified:
  head/usr.sbin/sade/Makefile
  head/usr.sbin/sade/devices.c
  head/usr.sbin/sade/disks.c
  head/usr.sbin/sade/dmenu.c
  head/usr.sbin/sade/install.c
  head/usr.sbin/sade/label.c
  head/usr.sbin/sade/main.c
  head/usr.sbin/sade/menus.c
  head/usr.sbin/sade/misc.c
  head/usr.sbin/sade/msg.c
  head/usr.sbin/sade/sade.h
  head/usr.sbin/sade/system.c

Modified: head/usr.sbin/sade/Makefile
==
--- head/usr.sbin/sade/Makefile Thu Nov 24 17:58:14 2011(r227943)
+++ head/usr.sbin/sade/Makefile Thu Nov 24 18:37:16 2011(r227944)
@@ -19,9 +19,10 @@ WARNS?=  3
 .if ${MACHINE} == "pc98"
 CFLAGS+= -DPC98
 .endif
-CFLAGS+= -I${.CURDIR}/../../gnu/lib/libodialog -I.
+CFLAGS+= -I${.CURDIR}/../../contrib/dialog -I.
+DEBUG_FLAGS= -O0 -g
 
-DPADD= ${LIBODIALOG} ${LIBNCURSES} ${LIBUTIL} ${LIBDISK}
-LDADD= -lodialog -lncurses -lutil -ldisk
+DPADD= ${LIBDIALOG} ${LIBNCURSESW} ${LIBM} ${LIBUTIL} ${LIBDISK}
+LDADD= -ldialog -lncursesw -lm -lutil -ldisk
 
 .include 

Modified: head/usr.sbin/sade/devices.c
==
--- head/usr.sbin/sade/devices.cThu Nov 24 17:58:14 2011
(r227943)
+++ head/usr.sbin/sade/devices.cThu Nov 24 18:37:16 2011
(r227944)
@@ -29,7 +29,6 @@
  *
  */
 
-#include "sade.h"
 #include 
 #include 
 #include 
@@ -40,6 +39,8 @@
 #include 
 #include 
 
+#include "sade.h"
+
 /* how much to bias minor number for a given /dev/s slice */
 #define SLICE_DELTA(0x1)
 
@@ -247,7 +248,7 @@ deviceGetAll(void)
}
free(names);
 }
-dialog_clear_norefresh();
+dlg_clear();
 }
 
 /* Rescan all devices, after closing previous set - convenience function */

Modified: head/usr.sbin/sade/disks.c
==
--- head/usr.sbin/sade/disks.c  Thu Nov 24 17:58:14 2011(r227943)
+++ head/usr.sbin/sade/disks.c  Thu Nov 24 18:37:16 2011(r227944)
@@ -113,7 +113,7 @@ check_geometry(Disk *d)
 if (d->bios_cyl > 65536 || d->bios_hd > 256 || d->bios_sect >= 64)
 #endif
 {
-   dialog_clear_norefresh();
+   dlg_clear();
sg = msgYesNo("WARNING:  It is safe to use a geometry of %lu/%lu/%lu 
for %s on\n"
  "computers with modern BIOS versions.  If this disk is to 
be used\n"
  "on an old machine it is recommended that it does not 
have more\n"
@@ -223,21 +223,21 @@ getBootMgr(char *dname, u_char **bootipl
 char *cp;
 int i = 0;
 
+dlg_clr_result();
 cp = variable_get(VAR_BOOTMGR);
 if (!cp) {
/* Figure out what kind of IPL the user wants */
sprintf(str, "Install Boot Manager for drive %s?", dname);
MenuIPLType.title = str;
-   i = dmenuOpenSimple(&MenuIPLType, FALSE);
+   i = dmenuOpen(&MenuIPLType);
 } else {
if (!strncmp(cp, "boot", 4))
-   BootMgr = 0;
+dlg_add_result(MenuIPLType.items[0].prompt);
else
-   BootMgr = 1;
+dlg_add_result(MenuIPLType.items[1].prompt);
 }
 if (cp || i) {
-   switch (BootMgr) {
-   case 0:
+if (!strcmp(dialog_vars.input_result, MenuIPLType.items[0].prompt)) {
if (!boot0) boot0 = bootalloc("boot0", &boot0_size);
*bootipl = boot0;
*bootipl_size = boot0_size;
@@ -245,10 +245,7 @@ getBootMgr(char *dname, u_char **bootipl
*bootmenu = boot05;
*bootmenu_size = boot05_size;
return;
-   case 1:
-   default:
-   break;
-   }
+}
 }
 *bootipl = NULL;
 *bootipl_size = 0;
@@ -266,36 +263,33 @@ getBootMgr(char *dname, u_char **bootCod
 char *cp;
 int i = 0;
 
+dlg_clr_result();
 cp = variable_get(VAR_BOOTMGR);
 if (!cp) {
/* Figure out what kind of MBR the user wants */
sprintf(str, "Install Boot Manager for drive %s?", dname);
MenuMBRType.title = str;
-   i = dmenuOpenSimple(&MenuMBRType, FALSE);
+   i = dmenuOpen(&MenuMBRType);
 }
 else {
+   if (!strcmp(cp, "standard"))
+dlg_add_result(MenuMBRType.items[0].prompt);
if (!strncmp(cp, "boot", 4))
-   BootMgr = 0;
-   else if (!strcmp(cp, "standard"))
-   BootMgr = 1;
+dlg_add_result(MenuMBRType.items[1].prompt);
else
-   BootMgr = 2;
+dlg_add_result(MenuMBRType.items[2].prompt);
 }
 if (cp || i) {
-   switch (BootMgr) {
-   case 0:
-   if (!boot0) boot0 = bootalloc("boot0", &boot0_size);
-   *bootCode = boot0;
-   *bootCodeSize = boot0_size;
-   return;
-   case 1:
+if (!strcmp(dialog_vars.input_result, MenuMBRType.items[0].

svn commit: r227947 - head/usr.sbin/tzsetup

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 18:45:23 2011
New Revision: 227947
URL: http://svn.freebsd.org/changeset/base/227947

Log:
  calloc +1 DIALOG_LISTITEM to prevent possible wild pointer access
  in dlg_default_listitem().

Modified:
  head/usr.sbin/tzsetup/tzsetup.c

Modified: head/usr.sbin/tzsetup/tzsetup.c
==
--- head/usr.sbin/tzsetup/tzsetup.c Thu Nov 24 18:44:14 2011
(r227946)
+++ head/usr.sbin/tzsetup/tzsetup.c Thu Nov 24 18:45:23 2011
(r227947)
@@ -76,14 +76,14 @@ static int
 xdialog_menu(const char *title, const char *cprompt, int height, int width,
 int menu_height, int item_no, dialogMenuItem *ditems)
 {
-   int i, result, choice;
+   int i, result, choice = 0;
DIALOG_LISTITEM *listitems;
DIALOG_VARS save_vars;
 
dlg_save_vars(&save_vars);
 
/* initialize list items */
-   listitems = dlg_calloc(DIALOG_LISTITEM, item_no);
+   listitems = dlg_calloc(DIALOG_LISTITEM, item_no + 1);
assert_ptr(listitems, "xdialog_menu");
for (i = 0; i < item_no; i++) {
listitems[i].name = ditems[i].prompt;
@@ -111,7 +111,7 @@ xdialog_menu(const char *title, const ch
width = COLS;
 
 again:
-   dialog_vars.default_item = ditems[choice].prompt;
+   dialog_vars.default_item = listitems[choice].name;
result = dlg_menu(title, cprompt, height, width,
menu_height, item_no, listitems, &choice, NULL);
switch (result) {
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227951 - in head: . gnu/lib share/mk

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 20:31:06 2011
New Revision: 227951
URL: http://svn.freebsd.org/changeset/base/227951

Log:
  libodialog: disconnect from the build and obsolete.

Modified:
  head/ObsoleteFiles.inc
  head/gnu/lib/Makefile
  head/share/mk/bsd.libnames.mk

Modified: head/ObsoleteFiles.inc
==
--- head/ObsoleteFiles.inc  Thu Nov 24 19:02:04 2011(r227950)
+++ head/ObsoleteFiles.inc  Thu Nov 24 20:31:06 2011(r227951)
@@ -38,6 +38,17 @@
 #   xargs -n1 | sort | uniq -d;
 # done
 
+# 2025: libodialog removed
+OLD_FILES+=usr/lib/libodialog.a
+OLD_FILES+=usr/lib/libodialog.so
+OLD_LIBS+=usr/lib/libodialog.so.7
+OLD_FILES+=usr/lib/libodialog_p.a
+.if ${TARGET_ARCH} == "amd64" || ${TARGET_ARCH} == "powerpc64"
+OLD_FILES+=usr/lib32/libodialog.a
+OLD_FILES+=usr/lib32/libodialog.so
+OLD_LIBS+=usr/lib32/libodialog.so.7
+OLD_FILES+=usr/lib32/libodialog_p.a
+.endif
 # 20110930: sysinstall removed
 OLD_FILES+=usr/sbin/sysinstall
 OLD_FILES+=usr/share/man/man8/sysinstall.8.gz

Modified: head/gnu/lib/Makefile
==
--- head/gnu/lib/Makefile   Thu Nov 24 19:02:04 2011(r227950)
+++ head/gnu/lib/Makefile   Thu Nov 24 20:31:06 2011(r227951)
@@ -2,8 +2,7 @@
 
 .include 
 
-SUBDIR= csu libgcc libgcov libdialog libgomp libodialog libregex libreadline \
-   libssp
+SUBDIR= csu libgcc libgcov libdialog libgomp libregex libreadline libssp
 
 # libsupc++ uses libstdc++ headers, although 'make includes' should
 # have taken care of that already.

Modified: head/share/mk/bsd.libnames.mk
==
--- head/share/mk/bsd.libnames.mk   Thu Nov 24 19:02:04 2011
(r227950)
+++ head/share/mk/bsd.libnames.mk   Thu Nov 24 20:31:06 2011
(r227951)
@@ -100,7 +100,6 @@ LIBNCURSESW?=   ${DESTDIR}${LIBDIR}/libncu
 LIBNETGRAPH?=  ${DESTDIR}${LIBDIR}/libnetgraph.a
 LIBNGATM?= ${DESTDIR}${LIBDIR}/libngatm.a
 LIBNVPAIR?=${DESTDIR}${LIBDIR}/libnvpair.a
-LIBODIALOG?=   ${DESTDIR}${LIBDIR}/libodialog.a
 LIBOPIE?=  ${DESTDIR}${LIBDIR}/libopie.a
 
 # The static PAM library doesn't know its secondary dependencies,
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227953 - head/gnu/lib/libodialog

2011-11-24 Thread Max Khon
Author: fjoe
Date: Thu Nov 24 20:34:50 2011
New Revision: 227953
URL: http://svn.freebsd.org/changeset/base/227953

Log:
  libodialog: Remove no longer used library.

Deleted:
  head/gnu/lib/libodialog/
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r227962 - head/usr.sbin/sade

2011-11-24 Thread Max Khon
Author: fjoe
Date: Fri Nov 25 07:10:59 2011
New Revision: 227962
URL: http://svn.freebsd.org/changeset/base/227962

Log:
  Remove DEBUG_FLAGS.

Modified:
  head/usr.sbin/sade/Makefile

Modified: head/usr.sbin/sade/Makefile
==
--- head/usr.sbin/sade/Makefile Fri Nov 25 04:03:37 2011(r227961)
+++ head/usr.sbin/sade/Makefile Fri Nov 25 07:10:59 2011(r227962)
@@ -20,7 +20,6 @@ WARNS?=   3
 CFLAGS+= -DPC98
 .endif
 CFLAGS+= -I${.CURDIR}/../../contrib/dialog -I.
-DEBUG_FLAGS= -O0 -g
 
 DPADD= ${LIBDIALOG} ${LIBNCURSESW} ${LIBM} ${LIBUTIL} ${LIBDISK}
 LDADD= -ldialog -lncursesw -lm -lutil -ldisk
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r227953 - head/gnu/lib/libodialog

2011-11-25 Thread Max Khon
Doug,

On Fri, Nov 25, 2011 at 4:12 AM, Doug Barton  wrote:

> Author: fjoe
> > Date: Thu Nov 24 20:34:50 2011
> > New Revision: 227953
> > URL: http://svn.freebsd.org/changeset/base/227953
> >
> > Log:
> >   libodialog: Remove no longer used library.
> >
> > Deleted:
> >   head/gnu/lib/libodialog/
>
> I'm curious ... I asked Nathan to please bring back sysinstall until we
> have a suitable replacement for the post-install configuration role that
> it plays. Is your change here going to make that more difficult?


I did not know about that. If we ever would want to resurrect it
1) I would insist on removing (or merging in) sade functionality
2) I would remove all installation and scripting functionality, leaving
only post-configuration
3) Given 1) and 2) I can help porting it to the new libdialog

Aside from that, I'd better implement new post-install configuration tool.

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228065 - head/lib/libpam/modules/pam_unix

2011-11-28 Thread Max Khon
Author: fjoe
Date: Mon Nov 28 14:01:17 2011
New Revision: 228065
URL: http://svn.freebsd.org/changeset/base/228065

Log:
  .include  instead of 
  
  The former allows common settings from ../Makefile.inc to be used.

Modified:
  head/lib/libpam/modules/pam_unix/Makefile

Modified: head/lib/libpam/modules/pam_unix/Makefile
==
--- head/lib/libpam/modules/pam_unix/Makefile   Mon Nov 28 13:42:59 2011
(r228064)
+++ head/lib/libpam/modules/pam_unix/Makefile   Mon Nov 28 14:01:17 2011
(r228065)
@@ -34,9 +34,7 @@
 #
 # $FreeBSD$
 
-NO_PROFILE=
-NO_INSTALLLIB=
-.include 
+.include 
 
 LIB=   pam_unix
 SRCS=  pam_unix.c
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228066 - head/tools

2011-11-28 Thread Max Khon
Author: fjoe
Date: Mon Nov 28 14:03:36 2011
New Revision: 228066
URL: http://svn.freebsd.org/changeset/base/228066

Log:
  Add a comment that shows how to limit the build to the specific list of 
arches.

Modified:
  head/tools/tinder.sh

Modified: head/tools/tinder.sh
==
--- head/tools/tinder.shMon Nov 28 14:01:17 2011(r228065)
+++ head/tools/tinder.shMon Nov 28 14:03:36 2011(r228066)
@@ -37,6 +37,8 @@
 # sh tools/tinder.sh gnu/lib/libdialog usr.sbin/sade NO_CLEAN=yes
 #  # build libdialog and sade for all architectures
 #  # without making clean
+# sh tools/tinder.sh gnu/lib/libdialog usr.sbin/sade TARGETS="amd64 i386"
+#  # build libdialog and sade only for amd64 and 
i386
 #
 
 if [ $# -eq 0 ]; then
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228074 - head/rescue/rescue

2011-11-28 Thread Max Khon
Author: fjoe
Date: Mon Nov 28 15:44:04 2011
New Revision: 228074
URL: http://svn.freebsd.org/changeset/base/228074

Log:
  -lpthread is required by -lzfs so should be later in LIBS list.
  
  There were no "undefined symbol pthread_xxx" errors during the link before
  this fix only because of STATIC_LIB_REQUIRE() declarations in
  lib/libthr/thread/thr_init.c.

Modified:
  head/rescue/rescue/Makefile

Modified: head/rescue/rescue/Makefile
==
--- head/rescue/rescue/Makefile Mon Nov 28 15:09:31 2011(r228073)
+++ head/rescue/rescue/Makefile Mon Nov 28 15:44:04 2011(r228074)
@@ -123,7 +123,7 @@ CRUNCH_LIBS+= -lalias -lcam -lcurses -ld
 CRUNCH_LIBS+= -lipx
 .endif
 .if ${MK_ZFS} != "no"
-CRUNCH_LIBS+= -lavl -lnvpair -lpthread -lzfs -luutil -lumem
+CRUNCH_LIBS+= -lavl -lnvpair -lzfs -lpthread -luutil -lumem
 .endif
 CRUNCH_LIBS+= -lgeom -lbsdxml -ljail -lkiconv -lmd -lreadline -lsbuf -lufs -lz
 
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228112 - head/rescue/rescue

2011-11-28 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 03:27:09 2011
New Revision: 228112
URL: http://svn.freebsd.org/changeset/base/228112

Log:
  -lreadline is not required anymore.

Modified:
  head/rescue/rescue/Makefile

Modified: head/rescue/rescue/Makefile
==
--- head/rescue/rescue/Makefile Tue Nov 29 02:07:07 2011(r228111)
+++ head/rescue/rescue/Makefile Tue Nov 29 03:27:09 2011(r228112)
@@ -125,7 +125,7 @@ CRUNCH_LIBS+= -lipx
 .if ${MK_ZFS} != "no"
 CRUNCH_LIBS+= -lavl -lnvpair -lzfs -lpthread -luutil -lumem
 .endif
-CRUNCH_LIBS+= -lgeom -lbsdxml -ljail -lkiconv -lmd -lreadline -lsbuf -lufs -lz
+CRUNCH_LIBS+= -lgeom -lbsdxml -ljail -lkiconv -lmd -lsbuf -lufs -lz
 
 .if ${MACHINE_CPUARCH} == "i386"
 CRUNCH_PROGS_sbin+= bsdlabel sconfig fdisk
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228113 - in head/kerberos5: usr.bin/kadmin usr.sbin/ktutil

2011-11-28 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 03:49:03 2011
New Revision: 228113
URL: http://svn.freebsd.org/changeset/base/228113

Log:
  Link with -ledit instead of -lreadline.

Modified:
  head/kerberos5/usr.bin/kadmin/Makefile
  head/kerberos5/usr.sbin/ktutil/Makefile

Modified: head/kerberos5/usr.bin/kadmin/Makefile
==
--- head/kerberos5/usr.bin/kadmin/Makefile  Tue Nov 29 03:27:09 2011
(r228112)
+++ head/kerberos5/usr.bin/kadmin/Makefile  Tue Nov 29 03:49:03 2011
(r228113)
@@ -29,11 +29,11 @@ CFLAGS+=-I${KRB5DIR}/lib/asn1 -I${KRB5DI
 DPADD= ${LIBKADM5CLNT} ${LIBKADM5SRV} ${LIBHDB} ${LIBKRB5} ${LIBHX509} \
${LIBSL} ${LIBROKEN} ${LIBVERS} ${LIBASN1} \
${LIBCRYPTO} ${LIBCRYPT} ${LIBCOM_ERR} \
-   ${LIBREADLINE} ${LIBNCURSES} ${LDAPDPADD}
+   ${LIBEDIT} ${LIBNCURSES} ${LDAPDPADD}
 LDADD= -lkadm5clnt -lkadm5srv -lhdb -lkrb5 -lhx509 \
${LIBSL} -lroken ${LIBVERS} -lasn1 \
-lcrypto -lcrypt -lcom_err \
-   -lreadline -lncurses ${LDAPLDADD}
+   -ledit -lncurses ${LDAPLDADD}
 LDFLAGS=${LDAPLDFLAGS}
 
 .include 

Modified: head/kerberos5/usr.sbin/ktutil/Makefile
==
--- head/kerberos5/usr.sbin/ktutil/Makefile Tue Nov 29 03:27:09 2011
(r228112)
+++ head/kerberos5/usr.sbin/ktutil/Makefile Tue Nov 29 03:49:03 2011
(r228113)
@@ -18,10 +18,10 @@ SRCS=   add.c \
 CFLAGS+=-I${KRB5DIR}/lib/roken -I${KRB5DIR}/lib/sl -I.
 DPADD= ${LIBKADM5CLNT} ${LIBKRB5} ${LIBHX509} ${LIBSL} ${LIBROKEN} ${LIBVERS} \
${LIBASN1} ${LIBCRYPTO} ${LIBCRYPT} ${LIBCOM_ERR} \
-   ${LIBREADLINE} ${LIBNCURSES}
+   ${LIBEDIT} ${LIBNCURSES}
 LDADD= -lkadm5clnt -lkrb5 -lhx509 ${LIBSL} -lroken ${LIBVERS} \
-lasn1 -lcrypto -lcrypt -lcom_err \
-   -lreadline -lncurses
+   -ledit -lncurses
 
 .include 
 
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228114 - head/lib/libedit/edit/readline

2011-11-28 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 04:50:57 2011
New Revision: 228114
URL: http://svn.freebsd.org/changeset/base/228114

Log:
  - Hide _rl_qsort_string_compare() that should be private to libreadline()
  implementation.
  - Add symlink /usr/include/edit/readline/tilde.h -> readline.h
  
  All this makes it possible to build and link gdb with -ledit.

Modified:
  head/lib/libedit/edit/readline/Makefile
  head/lib/libedit/edit/readline/readline.h

Modified: head/lib/libedit/edit/readline/Makefile
==
--- head/lib/libedit/edit/readline/Makefile Tue Nov 29 03:49:03 2011
(r228113)
+++ head/lib/libedit/edit/readline/Makefile Tue Nov 29 04:50:57 2011
(r228114)
@@ -2,6 +2,7 @@
 # $FreeBSD$
 
 INCS=  readline.h history.h
+SYMLINKS=  readline.h ${INCLUDEDIR}/edit/readline/tilde.h
 
 INCSDIR= ${INCLUDEDIR}/edit/readline
 

Modified: head/lib/libedit/edit/readline/readline.h
==
--- head/lib/libedit/edit/readline/readline.h   Tue Nov 29 03:49:03 2011
(r228113)
+++ head/lib/libedit/edit/readline/readline.h   Tue Nov 29 04:50:57 2011
(r228114)
@@ -200,7 +200,6 @@ void rl_get_screen_size(int *, int *);
 voidrl_set_screen_size(int, int);
 char   *rl_filename_completion_function (const char *, int);
 int _rl_abort_internal(void);
-int _rl_qsort_string_compare(char **, char **);
 char  **rl_completion_matches(const char *, rl_compentry_func_t *);
 voidrl_forced_update_display(void);
 int rl_set_prompt(const char *);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228116 - head/lib/libedit/edit/readline

2011-11-28 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 06:21:01 2011
New Revision: 228116
URL: http://svn.freebsd.org/changeset/base/228116

Log:
  Use INCSLINKS so that "make installincludes" installs links during buildworld.

Modified:
  head/lib/libedit/edit/readline/Makefile

Modified: head/lib/libedit/edit/readline/Makefile
==
--- head/lib/libedit/edit/readline/Makefile Tue Nov 29 06:12:19 2011
(r228115)
+++ head/lib/libedit/edit/readline/Makefile Tue Nov 29 06:21:01 2011
(r228116)
@@ -2,7 +2,7 @@
 # $FreeBSD$
 
 INCS=  readline.h history.h
-SYMLINKS=  readline.h ${INCLUDEDIR}/edit/readline/tilde.h
+INCSLINKS= readline.h ${INCSDIR}/tilde.h
 
 INCSDIR= ${INCLUDEDIR}/edit/readline
 
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228120 - head/share/mk

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 08:20:23 2011
New Revision: 228120
URL: http://svn.freebsd.org/changeset/base/228120

Log:
  Allow NO_FOO to override WITH_FOO that could be specified in /etc/src.conf.
  
  This is required to override knobs (e.g. WITH_PROFILE) during buildworld
  stages in Makefile.inc1 (otherwise the build is stopped due to both WITH_FOO
  and WITHOUT_FOO defined).

Modified:
  head/share/mk/bsd.own.mk

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkTue Nov 29 08:16:14 2011(r228119)
+++ head/share/mk/bsd.own.mkTue Nov 29 08:20:23 2011(r228120)
@@ -207,6 +207,9 @@ COMPRESS_EXT?=  .gz
 MAN \
 PROFILE
 .if defined(NO_${var})
+.if defined(WITH_${var})
+.undef WITH_${var}
+.endif
 WITHOUT_${var}=
 .endif
 .endfor
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228121 - head/sys/conf

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 08:21:54 2011
New Revision: 228121
URL: http://svn.freebsd.org/changeset/base/228121

Log:
  Fix typo in comments (conversation -> conversion).

Modified:
  head/sys/conf/kern.mk

Modified: head/sys/conf/kern.mk
==
--- head/sys/conf/kern.mk   Tue Nov 29 08:20:23 2011(r228120)
+++ head/sys/conf/kern.mk   Tue Nov 29 08:21:54 2011(r228121)
@@ -128,7 +128,7 @@ CFLAGS+=-fstack-protector
 .endif
 
 #
-# Enable CTF conversation on request
+# Enable CTF conversion on request
 #
 .if defined(WITH_CTF)
 .undef NO_CTF
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228124 - in head: share/mk sys/conf

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 08:38:47 2011
New Revision: 228124
URL: http://svn.freebsd.org/changeset/base/228124

Log:
  Conditionalize ctfconvert/ctfmerge runs on make level (.if/.endif) instead
  of executing a shell on every object or executable/library file.
  
  This shaves off more than 30,000 shell invocations during buildworld.

Modified:
  head/share/mk/bsd.lib.mk
  head/share/mk/bsd.prog.mk
  head/share/mk/sys.mk
  head/sys/conf/kern.post.mk
  head/sys/conf/kern.pre.mk
  head/sys/conf/kmod.mk

Modified: head/share/mk/bsd.lib.mk
==
--- head/share/mk/bsd.lib.mkTue Nov 29 08:33:40 2011(r228123)
+++ head/share/mk/bsd.lib.mkTue Nov 29 08:38:47 2011(r228124)
@@ -75,15 +75,15 @@ PO_FLAG=-pg
 
 .c.po:
${CC} ${PO_FLAG} ${STATIC_CFLAGS} ${PO_CFLAGS} -c ${.IMPSRC} -o 
${.TARGET}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .c.So:
${CC} ${PICFLAG} -DPIC ${SHARED_CFLAGS} ${CFLAGS} -c ${.IMPSRC} -o 
${.TARGET}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .cc.o:
${CXX} ${STATIC_CXXFLAGS} ${CXXFLAGS} -c ${.IMPSRC} -o ${.TARGET}
@@ -96,47 +96,47 @@ PO_FLAG=-pg
 
 .f.po:
${FC} -pg ${FFLAGS} -o ${.TARGET} -c ${.IMPSRC}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .f.So:
${FC} ${PICFLAG} -DPIC ${FFLAGS} -o ${.TARGET} -c ${.IMPSRC}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .s.po .s.So:
${AS} ${AFLAGS} -o ${.TARGET} ${.IMPSRC}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .asm.po:
${CC} -x assembler-with-cpp -DPROF ${PO_CFLAGS} ${ACFLAGS} \
-c ${.IMPSRC} -o ${.TARGET}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .asm.So:
${CC} -x assembler-with-cpp ${PICFLAG} -DPIC ${CFLAGS} ${ACFLAGS} \
-c ${.IMPSRC} -o ${.TARGET}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .S.po:
${CC} -DPROF ${PO_CFLAGS} ${ACFLAGS} -c ${.IMPSRC} -o ${.TARGET}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 .S.So:
${CC} ${PICFLAG} -DPIC ${CFLAGS} ${ACFLAGS} -c ${.IMPSRC} -o ${.TARGET}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+.if !defined(NO_CTF)
+   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.endif
 
 all: objwarn
 
@@ -211,9 +211,9 @@ ${SHLIB_NAME}: ${SOBJS}
-o ${.TARGET} -Wl,-soname,${SONAME} \
`NM='${NM}' lorder ${SOBJS} | tsort -q` ${LDADD}
 .endif
-   @[ -z "${CTFMERGE}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${SOBJS} && \
-   ${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${SOBJS})
+.if !defined(NO_CTF)
+   ${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${SOBJS}
+.endif
 .endif
 
 .if defined(INSTALL_PIC_ARCHIVE) && defined(LIB) && !empty(LIB) && 
${MK_TOOLCHAIN} != "no"

Modified: head/share/mk/bsd.prog.mk
==
--- head/share/mk/bsd.prog.mk   Tue Nov 29 08:33:40 2011(r228123)
+++ head/share/mk/bsd.prog.mk   Tue Nov 29 08:38:47 2011(r228124)
@@ -60,9 +60,9 @@ ${PROG}: ${OBJS}
 .else
${CC} ${CFLAGS} ${LDFLAGS} -o ${.TARGET} ${OBJS} ${LDADD}
 .endif
-   @[ -z "${CTFMERGE}" -o -n "${NO_CTF}" ] |

Re: svn commit: r228114 - head/lib/libedit/edit/readline

2011-11-29 Thread Max Khon
David,

On Tue, Nov 29, 2011 at 6:28 PM, David Chisnall wrote:

>  All this makes it possible to build and link gdb with -ledit.
>
> When I had a quick look a couple of days ago, kadmin and [k]gdb were the
> only things that linked against readline in base.  With them now linking
> against libedit, can we remove readline from base?


Yes, I am working on it. Stay tuned.

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228137 - in head: share/mk sys/conf

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 16:34:44 2011
New Revision: 228137
URL: http://svn.freebsd.org/changeset/base/228137

Log:
  - fix WITH_CTF when specified in /etc/src.conf [1]
  - CTFCONVERT_CMD=... is a hack (should be defined to empty string instead):
  make(1) should be taught to ignore empty commands silently in compat mode
  (as it does in !compat mode, GNU make also silently ignores empty commands)
  and to skip printing empty commands in !compat mode
  - config(8) should generate ${NORMAL_CTFCONVERT} invocation without '@':
  this will allow to simplify kern.pre.mk even more and lessen the number
  of shell invocations during kernel build when CTF is turned off
  - WITH_CTF can now be converted to usual MK_CTF=yes/no infrastructure
  
  Pointy hat to:fjoe [1]

Modified:
  head/share/mk/bsd.lib.mk
  head/share/mk/bsd.own.mk
  head/share/mk/bsd.prog.mk
  head/share/mk/sys.mk
  head/sys/conf/kern.mk
  head/sys/conf/kern.post.mk
  head/sys/conf/kern.pre.mk
  head/sys/conf/kmod.mk

Modified: head/share/mk/bsd.lib.mk
==
--- head/share/mk/bsd.lib.mkTue Nov 29 15:24:19 2011(r228136)
+++ head/share/mk/bsd.lib.mkTue Nov 29 16:34:44 2011(r228137)
@@ -33,15 +33,10 @@ CFLAGS+= -DNDEBUG
 NO_WERROR=
 .endif
 
-# Enable CTF conversion on request.
-.if defined(WITH_CTF)
-.undef NO_CTF
-.endif
-
 .if defined(DEBUG_FLAGS)
 CFLAGS+= ${DEBUG_FLAGS}
 
-.if !defined(NO_CTF) && (${DEBUG_FLAGS:M-g} != "")
+.if defined(WITH_CTF) && (${DEBUG_FLAGS:M-g} != "")
 CTFFLAGS+= -g
 .endif
 .endif
@@ -69,21 +64,15 @@ PO_FLAG=-pg
 
 .c.o:
${CC} ${STATIC_CFLAGS} ${CFLAGS} -c ${.IMPSRC} -o ${.TARGET}
-   @[ -z "${CTFCONVERT}" -o -n "${NO_CTF}" ] || \
-   (${ECHO} ${CTFCONVERT} ${CTFFLAGS} ${.TARGET} && \
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET})
+   ${CTFCONVERT_CMD}
 
 .c.po:
${CC} ${PO_FLAG} ${STATIC_CFLAGS} ${PO_CFLAGS} -c ${.IMPSRC} -o 
${.TARGET}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .c.So:
${CC} ${PICFLAG} -DPIC ${SHARED_CFLAGS} ${CFLAGS} -c ${.IMPSRC} -o 
${.TARGET}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .cc.o:
${CXX} ${STATIC_CXXFLAGS} ${CXXFLAGS} -c ${.IMPSRC} -o ${.TARGET}
@@ -96,47 +85,33 @@ PO_FLAG=-pg
 
 .f.po:
${FC} -pg ${FFLAGS} -o ${.TARGET} -c ${.IMPSRC}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .f.So:
${FC} ${PICFLAG} -DPIC ${FFLAGS} -o ${.TARGET} -c ${.IMPSRC}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .s.po .s.So:
${AS} ${AFLAGS} -o ${.TARGET} ${.IMPSRC}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .asm.po:
${CC} -x assembler-with-cpp -DPROF ${PO_CFLAGS} ${ACFLAGS} \
-c ${.IMPSRC} -o ${.TARGET}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .asm.So:
${CC} -x assembler-with-cpp ${PICFLAG} -DPIC ${CFLAGS} ${ACFLAGS} \
-c ${.IMPSRC} -o ${.TARGET}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .S.po:
${CC} -DPROF ${PO_CFLAGS} ${ACFLAGS} -c ${.IMPSRC} -o ${.TARGET}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 .S.So:
${CC} ${PICFLAG} -DPIC ${CFLAGS} ${ACFLAGS} -c ${.IMPSRC} -o ${.TARGET}
-.if !defined(NO_CTF)
-   ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.endif
+   ${CTFCONVERT_CMD}
 
 all: objwarn
 
@@ -211,7 +186,7 @@ ${SHLIB_NAME}: ${SOBJS}
-o ${.TARGET} -Wl,-soname,${SONAME} \
`NM='${NM}' lorder ${SOBJS} | tsort -q` ${LDADD}
 .endif
-.if !defined(NO_CTF)
+.if defined(WITH_CTF)
${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${SOBJS}
 .endif
 .endif

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkTue Nov 29 15:24:19 2011(r228136)
+++ head/share/mk/bsd.own.mkTue Nov 29 16:34:44 2011(r228137)
@@ -189,6 +189,12 @@ STRIP?=-s
 COMPRESS_CMD?= gzip -cn
 COMPRESS_EXT?= .gz
 
+.if defined(WITH_CTF)
+CTFCONVERT_CMD=${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.else
+CTFCONVERT_CMD=...
+.endif
+
 .if !defined(_WITHOUT_SRCCONF)
 #
 # Define MK_* variables (which are either "yes" or "no") for users

Modified: head/share/mk/bsd.prog.mk
==
--- head/share/mk/bsd.prog.mk   Tue Nov 29 15:24:19 2011(r228136)
+++ head/share/mk/bsd.prog.mk   Tue Nov 29 16:34:44 2011(r228137)
@@ -15,16 +15,11 @@ CFLAGS+= -DNDEBUG
 NO_WERROR=
 .endif
 
-# Enable CTF conversion on 

Re: svn commit: r228124 - in head: share/mk sys/conf

2011-11-29 Thread Max Khon
Alexander,

On Tue, Nov 29, 2011 at 9:00 PM, Alexander Kabaev  wrote:

> Author: fjoe
> > Date: Tue Nov 29 08:38:47 2011
> > New Revision: 228124
> > URL: http://svn.freebsd.org/changeset/base/228124
> >
> > Log:
> >   Conditionalize ctfconvert/ctfmerge runs on make level (.if/.endif)
> > instead of executing a shell on every object or executable/library
> > file.
> >   This shaves off more than 30,000 shell invocations during
> > buildworld.
>
> Have you looked as to _why_ it was done through shell invocation in the
> first place before removing it? You now have broken CTF support in
> cases where WITH_CTF is set not on make command line or in environment,
> but say in each individual Makefile.


I tested all the use-cases mentioned in the original commit but it looks
like I specified the location of modified share/mk incorrectly when I did
WITH_CTF userland tests.

Should be fixed now. Sorry for the breakage.

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228140 - head/sys/conf

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 18:52:02 2011
New Revision: 228140
URL: http://svn.freebsd.org/changeset/base/228140

Log:
  Do not echo ${CTFCONVERT} command: config(8) will be fixed to not
  emit '@' for ${NORMAL_CTFCONVERT} invocation.

Modified:
  head/sys/conf/kern.pre.mk

Modified: head/sys/conf/kern.pre.mk
==
--- head/sys/conf/kern.pre.mk   Tue Nov 29 18:23:08 2011(r228139)
+++ head/sys/conf/kern.pre.mk   Tue Nov 29 18:52:02 2011(r228140)
@@ -124,8 +124,7 @@ NORMAL_M= ${AWK} -f $S/tools/makeobjops.
  ${CC} -c ${CFLAGS} ${WERROR} ${PROF} ${.PREFIX}.c
 
 .if defined(WITH_CTF)
-CTFCONVERT_CMD=${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-NORMAL_CTFCONVERT= ${ECHO} ${CTFCONVERT_CMD} && ${CTFCONVERT_CMD}
+NORMAL_CTFCONVERT= ${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
 .else
 NORMAL_CTFCONVERT= :
 .endif
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228141 - head/sys/conf

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 19:13:01 2011
New Revision: 228141
URL: http://svn.freebsd.org/changeset/base/228141

Log:
  Call CTFMERGE only when WITH_CTF is defined.

Modified:
  head/sys/conf/kmod.mk

Modified: head/sys/conf/kmod.mk
==
--- head/sys/conf/kmod.mk   Tue Nov 29 18:52:02 2011(r228140)
+++ head/sys/conf/kmod.mk   Tue Nov 29 19:13:01 2011(r228141)
@@ -201,7 +201,7 @@ ${KMOD}.kld: ${OBJS}
 ${FULLPROG}: ${OBJS}
 .endif
${LD} ${LDFLAGS} -r -d -o ${.TARGET} ${OBJS}
-.if !defined(NO_CTF)
+.if defined(WITH_CTF)
${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${OBJS}
 .endif
 .if defined(EXPORT_SYMS)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228143 - in head: . share/mk tools/build/options

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 19:46:17 2011
New Revision: 228143
URL: http://svn.freebsd.org/changeset/base/228143

Log:
  Turn off profiled libs build by default.
  Can be enabled back using WITH_PROFILE=yes in /etc/src.conf

Added:
  head/tools/build/options/WITH_PROFILE
 - copied, changed from r227900, head/tools/build/options/WITHOUT_PROFILE
Deleted:
  head/tools/build/options/WITHOUT_PROFILE
Modified:
  head/Makefile.inc1
  head/share/mk/bsd.own.mk

Modified: head/Makefile.inc1
==
--- head/Makefile.inc1  Tue Nov 29 19:45:58 2011(r228142)
+++ head/Makefile.inc1  Tue Nov 29 19:46:17 2011(r228143)
@@ -239,7 +239,7 @@ BMAKE=  MAKEOBJDIRPREFIX=${WORLDTMP} \
BOOTSTRAPPING=${OSRELDATE} \
SSP_CFLAGS= \
-DWITHOUT_HTML -DWITHOUT_INFO -DNO_LINT -DWITHOUT_MAN \
-   -DNO_PIC -DWITHOUT_PROFILE -DNO_SHARED \
+   -DNO_PIC -DNO_PROFILE -DNO_SHARED \
-DNO_CPU_CFLAGS -DNO_WARNS -DNO_CTF
 
 # build-tools stage
@@ -438,7 +438,7 @@ _libraries:
@echo "--"
${_+_}cd ${.CURDIR}; \
${WMAKE} -DNO_FSCHG -DWITHOUT_HTML -DWITHOUT_INFO -DNO_LINT \
-   -DWITHOUT_MAN -DWITHOUT_PROFILE libraries
+   -DWITHOUT_MAN -DNO_PROFILE libraries
 _depend:
@echo
@echo "--"
@@ -1321,8 +1321,8 @@ ${_lib}__PL: .PHONY
cd ${.CURDIR}/${_lib}; \
${MAKE} DIRPRFX=${_lib}/ obj; \
${MAKE} DIRPRFX=${_lib}/ depend; \
-   ${MAKE} -DWITHOUT_PROFILE -DNO_PIC DIRPRFX=${_lib}/ all; \
-   ${MAKE} -DWITHOUT_PROFILE -DNO_PIC DIRPRFX=${_lib}/ install
+   ${MAKE} -DNO_PROFILE -DNO_PIC DIRPRFX=${_lib}/ all; \
+   ${MAKE} -DNO_PROFILE -DNO_PIC DIRPRFX=${_lib}/ install
 .endif
 .endfor
 
@@ -1555,7 +1555,7 @@ XDEV_CPUTYPE?=${TARGET_CPUTYPE}
 .endif
 
 NOFUN=-DNO_FSCHG -DWITHOUT_HTML -DWITHOUT_INFO -DNO_LINT \
-   -DWITHOUT_MAN -DWITHOUT_NLS -DWITHOUT_PROFILE \
+   -DWITHOUT_MAN -DWITHOUT_NLS -DNO_PROFILE \
-DWITHOUT_KERBEROS -DWITHOUT_RESCUE -DNO_WARNS \
TARGET=${XDEV} TARGET_ARCH=${XDEV_ARCH} \
CPUTYPE=${XDEV_CPUTYPE}

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkTue Nov 29 19:45:58 2011(r228142)
+++ head/share/mk/bsd.own.mkTue Nov 29 19:46:17 2011(r228143)
@@ -388,7 +388,6 @@ __DEFAULT_YES_OPTIONS = \
 PMC \
 PORTSNAP \
 PPP \
-PROFILE \
 QUOTAS \
 RCMDS \
 RCS \
@@ -423,7 +422,8 @@ __DEFAULT_NO_OPTIONS = \
 ICONV \
 IDEA \
 LIBCPLUSPLUS \
-OFED
+OFED \
+PROFILE
 
 #
 # Default behaviour of some options depends on the architecture.  Unfortunately

Copied and modified: head/tools/build/options/WITH_PROFILE (from r227900, 
head/tools/build/options/WITHOUT_PROFILE)
==
--- head/tools/build/options/WITHOUT_PROFILEWed Nov 23 18:15:49 2011
(r227900, copy source)
+++ head/tools/build/options/WITH_PROFILE   Tue Nov 29 19:46:17 2011
(r228143)
@@ -1,2 +1,2 @@
 .\" $FreeBSD$
-Set to avoid compiling profiled libraries.
+Set to build profiled libraries.
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228147 - head/share/mk

2011-11-29 Thread Max Khon
Author: fjoe
Date: Tue Nov 29 20:06:27 2011
New Revision: 228147
URL: http://svn.freebsd.org/changeset/base/228147

Log:
  CTFCONVERT_CMD=... hack does not work sufficiently well in parallel builds.
  
  Put a bandaid until make(1) is taught to ignore empty commands.

Modified:
  head/share/mk/bsd.own.mk

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkTue Nov 29 19:49:11 2011(r228146)
+++ head/share/mk/bsd.own.mkTue Nov 29 20:06:27 2011(r228147)
@@ -192,7 +192,7 @@ COMPRESS_EXT?=  .gz
 .if defined(WITH_CTF)
 CTFCONVERT_CMD=${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
 .else
-CTFCONVERT_CMD=...
+CTFCONVERT_CMD=@:
 .endif
 
 .if !defined(_WITHOUT_SRCCONF)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228149 - in head/tools/regression/usr.bin/make/execution: . ellipsis empty plus

2011-11-29 Thread Max Khon
Author: fjoe
Date: Wed Nov 30 05:49:17 2011
New Revision: 228149
URL: http://svn.freebsd.org/changeset/base/228149

Log:
  Add three execution tests for make(1):
  - plus: execute "+command" when run with -jX -n
  - ellipsis: ellipsis ("...") from variable
  - empty: empty command (from variable)
  
  Currently make(1) fails all three tests:
  - plus: segmentation fault due to incorrect command list handling
  - ellipsis: works in compat mode but fails in job (-jX) mode
  - empty:
  - compat mode: prints error message
- job mode: works but prints empty string

Added:
  head/tools/regression/usr.bin/make/execution/
  head/tools/regression/usr.bin/make/execution/ellipsis/
  head/tools/regression/usr.bin/make/execution/ellipsis/Makefile   (contents, 
props changed)
  head/tools/regression/usr.bin/make/execution/ellipsis/expected.status.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/ellipsis/expected.stderr.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/ellipsis/expected.stdout.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/ellipsis/test.t   (contents, 
props changed)
  head/tools/regression/usr.bin/make/execution/empty/
  head/tools/regression/usr.bin/make/execution/empty/Makefile   (contents, 
props changed)
  head/tools/regression/usr.bin/make/execution/empty/expected.status.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/empty/expected.stderr.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/empty/expected.stdout.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/empty/test.t   (contents, props 
changed)
  head/tools/regression/usr.bin/make/execution/plus/
  head/tools/regression/usr.bin/make/execution/plus/Makefile   (contents, props 
changed)
  head/tools/regression/usr.bin/make/execution/plus/expected.status.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/plus/expected.stderr.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/plus/expected.stdout.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/plus/test.t   (contents, props 
changed)

Added: head/tools/regression/usr.bin/make/execution/ellipsis/Makefile
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/ellipsis/Makefile  Wed Nov 
30 05:49:17 2011(r228149)
@@ -0,0 +1,21 @@
+# $FreeBSD$
+
+ELLIPSIS=  ...
+
+check-ellipsis:
+   @${MAKE} -f ${MAKEFILE} do-$@
+   @${MAKE} -f ${MAKEFILE} -j2 do-$@
+   @${MAKE} -f ${MAKEFILE} -j2 -B do-$@
+
+do-check-ellipsis: do-check-ellipsis-1 do-check-ellipsis-2
+.ORDER: do-check-ellipsis-1 do-check-ellipsis-2
+
+do-check-ellipsis-1:
+   @echo before $@
+   ${ELLIPSIS}
+   @echo after $@
+
+do-check-ellipsis-2:
+   @echo before $@
+   ${ELLIPSIS}
+   @echo after $@

Added: head/tools/regression/usr.bin/make/execution/ellipsis/expected.status.1
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/ellipsis/expected.status.1 
Wed Nov 30 05:49:17 2011(r228149)
@@ -0,0 +1 @@
+0

Added: head/tools/regression/usr.bin/make/execution/ellipsis/expected.stderr.1
==

Added: head/tools/regression/usr.bin/make/execution/ellipsis/expected.stdout.1
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/ellipsis/expected.stdout.1 
Wed Nov 30 05:49:17 2011(r228149)
@@ -0,0 +1,12 @@
+before do-check-ellipsis-1
+before do-check-ellipsis-2
+after do-check-ellipsis-1
+after do-check-ellipsis-2
+before do-check-ellipsis-1
+before do-check-ellipsis-2
+after do-check-ellipsis-1
+after do-check-ellipsis-2
+before do-check-ellipsis-1
+before do-check-ellipsis-2
+after do-check-ellipsis-1
+after do-check-ellipsis-2

Added: head/tools/regression/usr.bin/make/execution/ellipsis/test.t
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/ellipsis/test.tWed Nov 
30 05:49:17 2011(r228149)
@@ -0,0 +1,15 @@
+#!/bin/sh
+
+# $FreeBSD$
+
+cd `dirname $0`
+. ../../common.sh
+
+# Description
+DESC="Ellipsis command from variable"
+
+# Run
+TEST_N=1
+TEST_1=
+
+eval_cmd $*

Added: head/tools/regression/usr.bin/make/execution/empty/Makefile
==
--- /dev/null   00:00:00 1970   (empty, 

svn commit: r228153 - head/usr.sbin/config

2011-11-30 Thread Max Khon
Author: fjoe
Date: Wed Nov 30 13:33:09 2011
New Revision: 228153
URL: http://svn.freebsd.org/changeset/base/228153

Log:
  Generate ${NORMAL_CTFCONVERT} invocation without '@' modifier:
  - ${NORMAL_CC} is also invoked without '@'
  - Userland CTF support was changed previously to echo ctfconvert invocations 
too

Modified:
  head/usr.sbin/config/mkmakefile.c

Modified: head/usr.sbin/config/mkmakefile.c
==
--- head/usr.sbin/config/mkmakefile.c   Wed Nov 30 12:47:36 2011
(r228152)
+++ head/usr.sbin/config/mkmakefile.c   Wed Nov 30 13:33:09 2011
(r228153)
@@ -774,7 +774,7 @@ do_rules(FILE *f)
fprintf(f, "\t%s\n", compilewith);
 
if (!(ftp->f_flags & NO_OBJ))
-   fprintf(f, "\t@${NORMAL_CTFCONVERT}\n\n");
+   fprintf(f, "\t${NORMAL_CTFCONVERT}\n\n");
else
fprintf(f, "\n");
}
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r228143 - in head: . share/mk tools/build/options

2011-11-30 Thread Max Khon
John,

On Wed, Nov 30, 2011 at 3:36 AM, John Baldwin  wrote:

> Author: fjoe
> > Date: Tue Nov 29 19:46:17 2011
> > New Revision: 228143
> > URL: http://svn.freebsd.org/changeset/base/228143
> >
> > Log:
> >   Turn off profiled libs build by default.
> >   Can be enabled back using WITH_PROFILE=yes in /etc/src.conf
>
> Hmm, was discussed anywhere?  (I haven't seen it if so, but am a bit
> behind on
> a few lists from last week still.)


It was discussed in -current. Probably should have been discussed somewhere
else.


> Also, it seems you are hacking on several
> build-related things currently, is there a larger project you are working
> on
> to reduce build world times or some such?
>

Yes, I want to reduce build world times.

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228157 - head/usr.bin/make

2011-11-30 Thread Max Khon
Author: fjoe
Date: Wed Nov 30 18:07:38 2011
New Revision: 228157
URL: http://svn.freebsd.org/changeset/base/228157

Log:
  - Fix segmentation fault when running "+command" when run with -jX -n due
  to Compat_RunCommand() being called with `cmd' that is not on the 
node->commands
  list
  - Make ellipsis ("..." command) handling consistent: check for "..." command
  in job make after variables expansion to match compat make behavior
  - Fix empty command handling (after variables expansion and @+- modifiers
  are processed): now empty commands are ignored in compat make and are not
  printed in job make case
  - Bump MAKE_VERSION to 5-2011-11-30-0

Modified:
  head/usr.bin/make/Makefile
  head/usr.bin/make/job.c

Modified: head/usr.bin/make/Makefile
==
--- head/usr.bin/make/Makefile  Wed Nov 30 17:39:00 2011(r228156)
+++ head/usr.bin/make/Makefile  Wed Nov 30 18:07:38 2011(r228157)
@@ -10,7 +10,9 @@ SRCS= arch.c buf.c cond.c dir.c for.c ha
 
 NO_SHARED?=YES
 
-CFLAGS+=-DMAKE_VERSION=\"5200408120\"
+# Version has the RMMDDX format, where R is from RELENG_
+CFLAGS+=-DMAKE_VERSION=\"520300\"
+
 # There is no obvious performance improvement currently.
 # CFLAGS+=-DUSE_KQUEUE
 

Modified: head/usr.bin/make/job.c
==
--- head/usr.bin/make/job.c Wed Nov 30 17:39:00 2011(r228156)
+++ head/usr.bin/make/job.c Wed Nov 30 18:07:38 2011(r228157)
@@ -381,7 +381,7 @@ static int JobStart(GNode *, int, Job *)
 static void JobDoOutput(Job *, Boolean);
 static void JobInterrupt(int, int);
 static void JobRestartJobs(void);
-static int Compat_RunCommand(char *, struct GNode *);
+static int Compat_RunCommand(LstNode *, struct GNode *);
 
 static GNode   *curTarg = NULL;
 static GNode   *ENDNode;
@@ -647,7 +647,7 @@ JobPassSig(int signo)
  * numCommands is incremented if the command is actually printed.
  */
 static int
-JobPrintCommand(char *cmd, Job *job)
+JobPrintCommand(LstNode *cmdNode, Job *job)
 {
Boolean noSpecials; /* true if we shouldn't worry about
 * inserting special commands into
@@ -658,40 +658,30 @@ JobPrintCommand(char *cmd, Job *job)
 * off before printing the command
 * and need to turn it back on */
const char *cmdTemplate;/* Template to use when printing the command */
-   char*cmdStart;  /* Start of expanded command */
-   LstNode *cmdNode;   /* Node for replacing the command */
+   char*cmd;   /* Expanded command */
 
noSpecials = (noExecute && !(job->node->type & OP_MAKE));
 
-   if (strcmp(cmd, "...") == 0) {
-   job->node->type |= OP_SAVE_CMDS;
-   if ((job->flags & JOB_IGNDOTS) == 0) {
-   job->tailCmds =
-   Lst_Succ(Lst_Member(&job->node->commands, cmd));
-   return (1);
-   }
-   return (0);
-   }
-
 #defineDBPRINTF(fmt, arg)  \
DEBUGF(JOB, (fmt, arg));\
fprintf(job->cmdFILE, fmt, arg);\
fflush(job->cmdFILE);
 
-   numCommands += 1;
-
/*
 * For debugging, we replace each command with the result of expanding
 * the variables in the command.
 */
-   cmdNode = Lst_Member(&job->node->commands, cmd);
-
-   cmd = Buf_Peel(Var_Subst(cmd, job->node, FALSE));
-   cmdStart = cmd;
-
-   Lst_Replace(cmdNode, cmdStart);
-
-   cmdTemplate = "%s\n";
+   cmd = Buf_Peel(Var_Subst(Lst_Datum(cmdNode), job->node, FALSE));
+   if (strcmp(cmd, "...") == 0) {
+   free(cmd);
+   job->node->type |= OP_SAVE_CMDS;
+   if ((job->flags & JOB_IGNDOTS) == 0) {
+   job->tailCmds = Lst_Succ(cmdNode);
+   return (1);
+   }
+   return (0);
+   }
+   Lst_Replace(cmdNode, cmd);
 
/*
 * Check for leading @', -' or +'s to control echoing, error checking,
@@ -715,7 +705,7 @@ JobPrintCommand(char *cmd, Job *job)
 * but this one needs to be - use compat mode
 * just for it.
 */
-   Compat_RunCommand(cmd, job->node);
+   Compat_RunCommand(cmdNode, job->node);
return (0);
}
break;
@@ -726,6 +716,16 @@ JobPrintCommand(char *cmd, Job *job)
while (isspace((unsigned char)*cmd))
cmd++;
 
+   /*
+* Ignore empty commands
+*/
+   if (*cmd == '\0') {
+   return (0);
+   }
+
+   cmdTemplate = "%s\n";
+   numC

svn commit: r228158 - in head: . share/mk sys/conf

2011-11-30 Thread Max Khon
Author: fjoe
Date: Wed Nov 30 18:11:49 2011
New Revision: 228158
URL: http://svn.freebsd.org/changeset/base/228158

Log:
  - CTF knob is now implemented using common scheme: MK_CTF=yes/no is
  defined based on WITH/WITHOUT_CTF settings, default is WITHOUT_CTF,
  NO_CTF overrides WITH_CTF (used by Makefile.inc1)
  - CTFCONVERT_CMD/NORMAL_CTFCONVERT are now defined to empty string
  if make(1) can handle empty commands

Modified:
  head/Makefile.inc1
  head/share/mk/bsd.lib.mk
  head/share/mk/bsd.own.mk
  head/share/mk/bsd.prog.mk
  head/sys/conf/kern.post.mk
  head/sys/conf/kern.pre.mk
  head/sys/conf/kmod.mk

Modified: head/Makefile.inc1
==
--- head/Makefile.inc1  Wed Nov 30 18:07:38 2011(r228157)
+++ head/Makefile.inc1  Wed Nov 30 18:11:49 2011(r228158)
@@ -59,8 +59,6 @@ SUBDIR+=games
 .endif
 .if ${MK_CDDL} != "no"
 SUBDIR+=cddl
-.else
-NO_CTF=1
 .endif
 SUBDIR+=gnu include
 .if ${MK_KERBEROS} != "no"

Modified: head/share/mk/bsd.lib.mk
==
--- head/share/mk/bsd.lib.mkWed Nov 30 18:07:38 2011(r228157)
+++ head/share/mk/bsd.lib.mkWed Nov 30 18:11:49 2011(r228158)
@@ -36,7 +36,7 @@ NO_WERROR=
 .if defined(DEBUG_FLAGS)
 CFLAGS+= ${DEBUG_FLAGS}
 
-.if defined(WITH_CTF) && (${DEBUG_FLAGS:M-g} != "")
+.if ${MK_CTF} != "no" && ${DEBUG_FLAGS:M-g} != ""
 CTFFLAGS+= -g
 .endif
 .endif
@@ -186,7 +186,7 @@ ${SHLIB_NAME}: ${SOBJS}
-o ${.TARGET} -Wl,-soname,${SONAME} \
`NM='${NM}' lorder ${SOBJS} | tsort -q` ${LDADD}
 .endif
-.if defined(WITH_CTF)
+.if ${MK_CTF} != "no"
${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${SOBJS}
 .endif
 .endif

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkWed Nov 30 18:07:38 2011(r228157)
+++ head/share/mk/bsd.own.mkWed Nov 30 18:11:49 2011(r228158)
@@ -189,12 +189,6 @@ STRIP?=-s
 COMPRESS_CMD?= gzip -cn
 COMPRESS_EXT?= .gz
 
-.if defined(WITH_CTF)
-CTFCONVERT_CMD=${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
-.else
-CTFCONVERT_CMD=@:
-.endif
-
 .if !defined(_WITHOUT_SRCCONF)
 #
 # Define MK_* variables (which are either "yes" or "no") for users
@@ -209,6 +203,7 @@ CTFCONVERT_CMD= @:
 # regardless of user's setting).
 #
 .for var in \
+CTF \
 INSTALLLIB \
 MAN \
 PROFILE
@@ -418,6 +413,7 @@ __DEFAULT_NO_OPTIONS = \
 BIND_LIBS \
 BIND_SIGCHASE \
 BIND_XML \
+CTF \
 HESIOD \
 ICONV \
 IDEA \
@@ -516,6 +512,7 @@ MK_BIND_ETC:=   no
 
 .if ${MK_CDDL} == "no"
 MK_ZFS:=   no
+MK_CTF:=   no
 .endif
 
 .if ${MK_CRYPT} == "no"
@@ -616,6 +613,14 @@ MK_${vv:H}:=   ${MK_${vv:T}}
 .endif
 .endfor
 
+.if ${MK_CTF} != "no"
+CTFCONVERT_CMD=${CTFCONVERT} ${CTFFLAGS} ${.TARGET}
+.elif ${MAKE_VERSION} >= 520300
+CTFCONVERT_CMD=
+.else
+CTFCONVERT_CMD=@:
+.endif 
+
 .endif # !_WITHOUT_SRCCONF
 
 .endif # !target()

Modified: head/share/mk/bsd.prog.mk
==
--- head/share/mk/bsd.prog.mk   Wed Nov 30 18:07:38 2011(r228157)
+++ head/share/mk/bsd.prog.mk   Wed Nov 30 18:11:49 2011(r228158)
@@ -19,7 +19,7 @@ NO_WERROR=
 CFLAGS+=${DEBUG_FLAGS}
 CXXFLAGS+=${DEBUG_FLAGS}
 
-.if defined(WITH_CTF) && (${DEBUG_FLAGS:M-g} != "")
+.if ${MK_CTF} != "no" && ${DEBUG_FLAGS:M-g} != ""
 CTFFLAGS+= -g
 .endif
 .endif
@@ -55,7 +55,7 @@ ${PROG}: ${OBJS}
 .else
${CC} ${CFLAGS} ${LDFLAGS} -o ${.TARGET} ${OBJS} ${LDADD}
 .endif
-.if defined(WITH_CTF)
+.if ${MK_CTF} != "no"
${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${OBJS}
 .endif
 
@@ -85,7 +85,7 @@ ${PROG}: ${OBJS}
 .else
${CC} ${CFLAGS} ${LDFLAGS} -o ${.TARGET} ${OBJS} ${LDADD}
 .endif
-.if defined(WITH_CTF)
+.if ${MK_CTF} != "no"
${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${OBJS}
 .endif
 .endif

Modified: head/sys/conf/kern.post.mk
==
--- head/sys/conf/kern.post.mk  Wed Nov 30 18:07:38 2011(r228157)
+++ head/sys/conf/kern.post.mk  Wed Nov 30 18:11:49 2011(r228158)
@@ -94,7 +94,7 @@ ${FULLKERNEL}: ${SYSTEM_DEP} vers.o
@rm -f ${.TARGET}
@echo linking ${.TARGET}
${SYSTEM_LD}
-.if defined(WITH_CTF)
+.if ${MK_CTF} != "no"
${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${SYSTEM_OBJS} vers.o
 .endif
 .if !defined(DEBUG)
@@ -249,7 +249,7 @@ kernel-reinstall:
 
 config.o env.o hints.o vers.o vnode_if.o:
${NORMAL_C}
-   @${NORMAL_CTFCONVERT}
+   ${NORMAL_CTFCONVERT}
 
 config.ln env.ln hints.ln vers.ln vnode_if.ln:
${NORMAL_LINT}

Modified: head/sys/conf/kern.pre.mk
==
--- head/sys/conf/kern.pre.mk   Wed Nov 30 18:07:38 2011(r

svn commit: r228159 - head/tools/build/options

2011-11-30 Thread Max Khon
Author: fjoe
Date: Wed Nov 30 18:22:44 2011
New Revision: 228159
URL: http://svn.freebsd.org/changeset/base/228159

Log:
  Add WITH_CTF description.

Added:
  head/tools/build/options/WITH_CTF   (contents, props changed)

Added: head/tools/build/options/WITH_CTF
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/build/options/WITH_CTF   Wed Nov 30 18:22:44 2011
(r228159)
@@ -0,0 +1,4 @@
+.\" $FreeBSD$
+Set to compile with CTF (Compact C Type Format) data.
+CTF data encapsulates a reduced form of debugging information
+similar to DWARF and the venerable stabs and is required for DTrace.
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228160 - head/share/man/man5

2011-11-30 Thread Max Khon
Author: fjoe
Date: Wed Nov 30 18:24:43 2011
New Revision: 228160
URL: http://svn.freebsd.org/changeset/base/228160

Log:
  Regen.

Modified:
  head/share/man/man5/src.conf.5

Modified: head/share/man/man5/src.conf.5
==
--- head/share/man/man5/src.conf.5  Wed Nov 30 18:22:44 2011
(r228159)
+++ head/share/man/man5/src.conf.5  Wed Nov 30 18:24:43 2011
(r228160)
@@ -1,7 +1,7 @@
 .\" DO NOT EDIT-- this file is automatically generated.
 .\" from FreeBSD: head/tools/build/options/makeman 221733 2011-05-10 13:01:11Z 
ru
 .\" $FreeBSD$
-.Dd November 28, 2011
+.Dd December 1, 2011
 .Dt SRC.CONF 5
 .Os
 .Sh NAME
@@ -258,6 +258,8 @@ When set, it also enforces the following
 .Pp
 .Bl -item -compact
 .It
+.Va WITHOUT_CTF
+.It
 .Va WITHOUT_ZFS
 .El
 .It Va WITHOUT_CLANG
@@ -300,6 +302,11 @@ When set, the following options are also
 .Va WITH_GSSAPI
 is set explicitly)
 .El
+.It Va WITH_CTF
+.\" $FreeBSD$
+Set to compile with CTF (Compact C Type Format) data.
+CTF data encapsulates a reduced form of debugging information
+similar to DWARF and the venerable stabs and is required for DTrace.
 .It Va WITHOUT_CTM
 .\" from FreeBSD: head/tools/build/options/WITHOUT_CTM 183242 2008-09-21 
22:02:26Z sam
 Set to not build
@@ -813,9 +820,9 @@ and related files.
 Set to not build
 .Xr ppp 8
 and related programs.
-.It Va WITHOUT_PROFILE
-.\" from FreeBSD: head/tools/build/options/WITHOUT_PROFILE 156932 2006-03-21 
07:50:50Z ru
-Set to avoid compiling profiled libraries.
+.It Va WITH_PROFILE
+.\" from FreeBSD: head/tools/build/options/WITH_PROFILE 228143 2011-11-29 
19:46:17Z fjoe
+Set to build profiled libraries.
 .It Va WITHOUT_QUOTAS
 .\" from FreeBSD: head/tools/build/options/WITHOUT_QUOTAS 183242 2008-09-21 
22:02:26Z sam
 Set to not build
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r228157 - head/usr.bin/make

2011-11-30 Thread Max Khon
Alexander,

On Thu, Dec 1, 2011 at 7:16 AM, Alexander Best  wrote:

> Author: fjoe
> > Date: Wed Nov 30 18:07:38 2011
> > New Revision: 228157
> > URL: http://svn.freebsd.org/changeset/base/228157
> >
> > Log:
> >   - Fix segmentation fault when running "+command" when run with -jX -n
> due
> >   to Compat_RunCommand() being called with `cmd' that is not on the
> node->commands
> >   list
> >   - Make ellipsis ("..." command) handling consistent: check for "..."
> command
> >   in job make after variables expansion to match compat make behavior
> >   - Fix empty command handling (after variables expansion and @+-
> modifiers
> >   are processed): now empty commands are ignored in compat make and are
> not
> >   printed in job make case
> >   - Bump MAKE_VERSION to 5-2011-11-30-0
>
> it would also be nice, if at some point, somebody could dive into the code
> to
> see why 'make buildkernel' will let clang produce coloured output, but
> 'make -j(N>1) buildkernel' doesn't (and why adding a -B switch to that
> command
> fixes it).
>

This one is simple: job make (-jX) runs commands with stdin/stdout/stderr
redirected to pipes.
-B turns on compat mode for job make.

This can be demonstrated by running make with -jX or -jX -B with the
following Makefile:
--- cut here ---
all:
@if [ -t 1 ]; then echo "stdout is a tty"; else echo "stdout is not
a tty"; fi
--- cut here ---

If you really want to see colored output in -jX case you should teach clang
to output ANSI color sequences not only in isatty(1) case, but also
when MAKE_JOBS_FIFO environment variable is present (it is set when make
runs in job mode).

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r228157 - head/usr.bin/make

2011-11-30 Thread Max Khon
Alexander,

On Thu, Dec 1, 2011 at 11:05 AM, Max Khon  wrote:

it would also be nice, if at some point, somebody could dive into the code
>> to
>> see why 'make buildkernel' will let clang produce coloured output, but
>> 'make -j(N>1) buildkernel' doesn't (and why adding a -B switch to that
>> command
>> fixes it).
>>
>
> This one is simple: job make (-jX) runs commands with stdin/stdout/stderr
> redirected to pipes.
> -B turns on compat mode for job make.
>
> This can be demonstrated by running make with -jX or -jX -B with the
> following Makefile:
> --- cut here ---
> all:
> @if [ -t 1 ]; then echo "stdout is a tty"; else echo "stdout is
> not a tty"; fi
> --- cut here ---
>
> If you really want to see colored output in -jX case you should teach
> clang to output ANSI color sequences not only in isatty(1) case, but also
> when MAKE_JOBS_FIFO environment variable is present (it is set when make
> runs in job mode).
>

This will not work for the cases when make(1) is itself redirected.
Something like attached patch should work, but it blocks sometimes in
"ttyout" state for some reason (needs more work).

Max
Index: usr.bin/make/Makefile
===
--- usr.bin/make/Makefile	(revision 228157)
+++ usr.bin/make/Makefile	(working copy)
@@ -7,6 +7,8 @@
 SRCS=	arch.c buf.c cond.c dir.c for.c hash.c hash_tables.c job.c	\
 	lst.c main.c make.c parse.c proc.c shell.c str.c suff.c targ.c	\
 	util.c var.c
+DPADD=	${LIBUTIL}
+LDADD=	-lutil
 
 NO_SHARED?=	YES
 
Index: usr.bin/make/job.c
===
--- usr.bin/make/job.c	(revision 228157)
+++ usr.bin/make/job.c	(working copy)
@@ -115,6 +115,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -1798,8 +1799,13 @@
 		if (usePipes) {
 			int fd[2];
 
-			if (pipe(fd) == -1)
-Punt("Cannot create pipe: %s", strerror(errno));
+			if (isatty(1)) {
+if (openpty(fd, fd + 1, NULL, NULL, NULL) == -1)
+	Punt("Cannot open pty: %s", strerror(errno));
+			} else {
+if (pipe(fd) == -1)
+	Punt("Cannot create pipe: %s", strerror(errno));
+			}
 			job->inPipe = fd[0];
 			job->outPipe = fd[1];
 			fcntl(job->inPipe, F_SETFD, 1);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Re: svn commit: r228157 - head/usr.bin/make

2011-11-30 Thread Max Khon
Alexander,

On Thu, Dec 1, 2011 at 12:03 PM, Max Khon  wrote:

it would also be nice, if at some point, somebody could dive into the code
>>> to
>>> see why 'make buildkernel' will let clang produce coloured output, but
>>> 'make -j(N>1) buildkernel' doesn't (and why adding a -B switch to that
>>> command
>>> fixes it).
>>>
>>
>> This one is simple: job make (-jX) runs commands with stdin/stdout/stderr
>> redirected to pipes.
>> -B turns on compat mode for job make.
>>
>> This can be demonstrated by running make with -jX or -jX -B with the
>> following Makefile:
>> --- cut here ---
>> all:
>> @if [ -t 1 ]; then echo "stdout is a tty"; else echo "stdout is
>> not a tty"; fi
>> --- cut here ---
>>
>> If you really want to see colored output in -jX case you should teach
>> clang to output ANSI color sequences not only in isatty(1) case, but also
>> when MAKE_JOBS_FIFO environment variable is present (it is set when make
>> runs in job mode).
>>
>
> This will not work for the cases when make(1) is itself redirected.
> Something like attached patch should work, but it blocks sometimes in
> "ttyout" state for some reason (needs more work).
>

It hangs when tty_drain() is called when make(1) closes slave pty. Looks
like a race condition there when tty_wait() is called from tty_drain().

If I exchange pty master and slave hangs disappear. So attached patch works
for me.
It may add performance penalty but I think that the impact will be
negligible.

Max
Index: usr.bin/make/Makefile
===
--- usr.bin/make/Makefile	(revision 228157)
+++ usr.bin/make/Makefile	(working copy)
@@ -7,6 +7,8 @@
 SRCS=	arch.c buf.c cond.c dir.c for.c hash.c hash_tables.c job.c	\
 	lst.c main.c make.c parse.c proc.c shell.c str.c suff.c targ.c	\
 	util.c var.c
+DPADD=	${LIBUTIL}
+LDADD=	-lutil
 
 NO_SHARED?=	YES
 
Index: usr.bin/make/job.c
===
--- usr.bin/make/job.c	(revision 228157)
+++ usr.bin/make/job.c	(working copy)
@@ -115,6 +115,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -1798,8 +1799,13 @@
 		if (usePipes) {
 			int fd[2];
 
-			if (pipe(fd) == -1)
-Punt("Cannot create pipe: %s", strerror(errno));
+			if (isatty(1)) {
+if (openpty(fd + 1, fd + 0, NULL, NULL, NULL) == -1)
+	Punt("Cannot open pty: %s", strerror(errno));
+			} else {
+if (pipe(fd) == -1)
+	Punt("Cannot create pipe: %s", strerror(errno));
+			}
 			job->inPipe = fd[0];
 			job->outPipe = fd[1];
 			fcntl(job->inPipe, F_SETFD, 1);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

svn commit: r228175 - head/usr.sbin/sade

2011-12-01 Thread Max Khon
Author: fjoe
Date: Thu Dec  1 09:01:51 2011
New Revision: 228175
URL: http://svn.freebsd.org/changeset/base/228175

Log:
  Fix dialog autosizing: dlg_count_columns() does not handle NL characters.

Modified:
  head/usr.sbin/sade/misc.c

Modified: head/usr.sbin/sade/misc.c
==
--- head/usr.sbin/sade/misc.c   Thu Dec  1 07:41:30 2011(r228174)
+++ head/usr.sbin/sade/misc.c   Thu Dec  1 09:01:51 2011(r228175)
@@ -233,6 +233,23 @@ xdialog_count_rows(const char *p)
return rows ? rows : 1;
 }
 
+static int
+xdialog_count_columns(const char *p)
+{
+   int len;
+   int max_len = 0;
+   const char *q;
+
+   for (; (q = strchr(p, '\n')) != NULL; p = q + 1) {
+   len = q - p;
+   max_len = MAX(max_len, len);
+   }
+
+   len = strlen(p);
+   max_len = MAX(max_len, len);
+   return max_len;
+}
+
 int
 xdialog_menu(const char *title, const char *cprompt, int height, int width,
 int menu_height, int item_no, dialogMenuItem *ditems)
@@ -270,7 +287,7 @@ xdialog_menu(const char *title, const ch
tag_x = MAX(tag_x, l + k + 2);
}
}
-   width = MAX(dlg_count_columns(cprompt), title != NULL ? 
dlg_count_columns(title) : 0);
+   width = MAX(xdialog_count_columns(cprompt), title != NULL ? 
xdialog_count_columns(title) : 0);
width = MAX(width, tag_x + 4) + 4;
}
width = MAX(width, 24);
@@ -345,7 +362,7 @@ xdialog_radiolist(const char *title, con
check_x = MAX(check_x, l + k + 6);
}
}
-   width = MAX(dlg_count_columns(cprompt), title != NULL ? 
dlg_count_columns(title) : 0);
+   width = MAX(xdialog_count_columns(cprompt), title != NULL ? 
xdialog_count_columns(title) : 0);
width = MAX(width, check_x + 4) + 4;
}
width = MAX(width, 24);
@@ -397,8 +414,8 @@ xdialog_msgbox(const char *title, const 
 
/* calculate width */
if (width < 0) {
-   width = title != NULL ? dlg_count_columns(title) : 0;
-   width = MAX(width, dlg_count_columns(cprompt)) + 4;
+   width = title != NULL ? xdialog_count_columns(title) : 0;
+   width = MAX(width, xdialog_count_columns(cprompt)) + 4;
}
if (pauseopt)
width = MAX(width, 10);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228176 - head/usr.sbin/tzsetup

2011-12-01 Thread Max Khon
Author: fjoe
Date: Thu Dec  1 09:02:57 2011
New Revision: 228176
URL: http://svn.freebsd.org/changeset/base/228176

Log:
  Sync xdialog_menu() implementation with sade.

Modified:
  head/usr.sbin/tzsetup/tzsetup.c

Modified: head/usr.sbin/tzsetup/tzsetup.c
==
--- head/usr.sbin/tzsetup/tzsetup.c Thu Dec  1 09:01:51 2011
(r228175)
+++ head/usr.sbin/tzsetup/tzsetup.c Thu Dec  1 09:02:57 2011
(r228176)
@@ -73,6 +73,38 @@ typedef struct dialogMenuItem {
 } dialogMenuItem;
 
 static int
+xdialog_count_rows(const char *p)
+{
+   int rows = 0;
+
+   while ((p = strchr(p, '\n')) != NULL) {
+   p++;
+   if (*p == '\0')
+   break;
+   rows++;
+   }
+
+   return rows ? rows : 1;
+}
+
+static int
+xdialog_count_columns(const char *p)
+{
+   int len;
+   int max_len = 0;
+   const char *q;
+
+   for (; (q = strchr(p, '\n')) != NULL; p = q + 1) {
+   len = q - p;
+   max_len = MAX(max_len, len);
+   }
+
+   len = strlen(p);
+   max_len = MAX(max_len, len);
+   return max_len;
+}
+
+static int
 xdialog_menu(const char *title, const char *cprompt, int height, int width,
 int menu_height, int item_no, dialogMenuItem *ditems)
 {
@@ -90,6 +122,12 @@ xdialog_menu(const char *title, const ch
listitems[i].text = ditems[i].title;
}
 
+   /* calculate height */
+   if (height < 0)
+   height = xdialog_count_rows(cprompt) + menu_height + 4 + 2;
+   if (height > LINES)
+   height = LINES;
+
/* calculate width */
if (width < 0) {
int tag_x = 0;
@@ -103,7 +141,7 @@ xdialog_menu(const char *title, const ch
tag_x = MAX(tag_x, l + k + 2);
}
}
-   width = MAX(dlg_count_columns(cprompt), title != NULL ? 
dlg_count_columns(title) : 0);
+   width = MAX(xdialog_count_columns(cprompt), title != NULL ? 
xdialog_count_columns(title) : 0);
width = MAX(width, tag_x + 4) + 4;
}
width = MAX(width, 24);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228196 - in head: share/mk tools/build/options

2011-12-02 Thread Max Khon
Author: fjoe
Date: Fri Dec  2 09:09:54 2011
New Revision: 228196
URL: http://svn.freebsd.org/changeset/base/228196

Log:
  Revert to conservative defaults. WITH_PROFILE is a default now.
  
  NO_PROFILE overrides in Makefile.inc1 and bsd.own.m are left intact
  so that the reversal to the WITHOUT_PROFILE will be easier in future.

Added:
  head/tools/build/options/WITHOUT_PROFILE
 - copied, changed from r228148, head/tools/build/options/WITH_PROFILE
Deleted:
  head/tools/build/options/WITH_PROFILE
Modified:
  head/share/mk/bsd.own.mk

Modified: head/share/mk/bsd.own.mk
==
--- head/share/mk/bsd.own.mkFri Dec  2 02:19:03 2011(r228195)
+++ head/share/mk/bsd.own.mkFri Dec  2 09:09:54 2011(r228196)
@@ -383,6 +383,7 @@ __DEFAULT_YES_OPTIONS = \
 PMC \
 PORTSNAP \
 PPP \
+PROFILE \
 QUOTAS \
 RCMDS \
 RCS \
@@ -418,8 +419,7 @@ __DEFAULT_NO_OPTIONS = \
 ICONV \
 IDEA \
 LIBCPLUSPLUS \
-OFED \
-PROFILE
+OFED
 
 #
 # Default behaviour of some options depends on the architecture.  Unfortunately

Copied and modified: head/tools/build/options/WITHOUT_PROFILE (from r228148, 
head/tools/build/options/WITH_PROFILE)
==
--- head/tools/build/options/WITH_PROFILE   Tue Nov 29 21:28:48 2011
(r228148, copy source)
+++ head/tools/build/options/WITHOUT_PROFILEFri Dec  2 09:09:54 2011
(r228196)
@@ -1,2 +1,2 @@
 .\" $FreeBSD$
-Set to build profiled libraries.
+Set to avoid compiling profiled libraries.
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228197 - head/share/man/man5

2011-12-02 Thread Max Khon
Author: fjoe
Date: Fri Dec  2 09:28:19 2011
New Revision: 228197
URL: http://svn.freebsd.org/changeset/base/228197

Log:
  Regen.

Modified:
  head/share/man/man5/src.conf.5

Modified: head/share/man/man5/src.conf.5
==
--- head/share/man/man5/src.conf.5  Fri Dec  2 09:09:54 2011
(r228196)
+++ head/share/man/man5/src.conf.5  Fri Dec  2 09:28:19 2011
(r228197)
@@ -1,7 +1,7 @@
 .\" DO NOT EDIT-- this file is automatically generated.
 .\" from FreeBSD: head/tools/build/options/makeman 221733 2011-05-10 13:01:11Z 
ru
 .\" $FreeBSD$
-.Dd December 1, 2011
+.Dd December 2, 2011
 .Dt SRC.CONF 5
 .Os
 .Sh NAME
@@ -303,7 +303,7 @@ When set, the following options are also
 is set explicitly)
 .El
 .It Va WITH_CTF
-.\" $FreeBSD$
+.\" from FreeBSD: head/tools/build/options/WITH_CTF 228159 2011-11-30 
18:22:44Z fjoe
 Set to compile with CTF (Compact C Type Format) data.
 CTF data encapsulates a reduced form of debugging information
 similar to DWARF and the venerable stabs and is required for DTrace.
@@ -820,9 +820,9 @@ and related files.
 Set to not build
 .Xr ppp 8
 and related programs.
-.It Va WITH_PROFILE
+.It Va WITHOUT_PROFILE
 .\" from FreeBSD: head/tools/build/options/WITH_PROFILE 228143 2011-11-29 
19:46:17Z fjoe
-Set to build profiled libraries.
+Set to avoid compiling profiled libraries.
 .It Va WITHOUT_QUOTAS
 .\" from FreeBSD: head/tools/build/options/WITHOUT_QUOTAS 183242 2008-09-21 
22:02:26Z sam
 Set to not build
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r228196 - in head: share/mk tools/build/options

2011-12-03 Thread Max Khon
Bruce,

On Fri, Dec 2, 2011 at 9:41 PM, Bruce Evans  wrote:

>> Log:
>>  Revert to conservative defaults. WITH_PROFILE is a default now.
>>
>>  NO_PROFILE overrides in Makefile.inc1 and bsd.own.m are left intact
>>  so that the reversal to the WITHOUT_PROFILE will be easier in future.
>
>
> Thanks.  Not building profiled libraries might have been best in 1993
> when makeworld took hours (4-10?).  When I optimized makeworld in 1998,
> it still took more than 1 hour but I removing profiling from it was
> intentionally left out (except for bootstrapping).  It would have saved
> only a few minutes.  Now, a whole makeworld should take a few minutes
> and the few extra seconds for profiling libraries doesn't matter.

The real goal was to have reasonable defaults. The definition of
reasonable differs. The definition for some of us is: if I use it it
should be the default. My definition is: the system needs minimum
tweaks for the majority of users.

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r228153 - head/usr.sbin/config

2011-12-03 Thread Max Khon
David,

On Fri, Dec 2, 2011 at 4:40 PM, David O'Brien  wrote:

>>   Generate ${NORMAL_CTFCONVERT} invocation without '@' modifier:
>>   - ${NORMAL_CC} is also invoked without '@'
>>   - Userland CTF support was changed previously to echo ctfconvert
>>   invocations too
>
> Thank you for your work on cleaning up the CTF part of the build.
> Enabling WITH_CTF has been too different (and conflicting) from our
> other knobs.
> I also had local patches to not hide ctfconvert so one could easily see
> when they didn't have WITH_CTF enabled in the right magical incantations.

You're welcome.

> Will you be MFC'ing the CTF build work?  Please. :-)

Frankly, I do see reasons for the MFC (and I do not want to respond to
people who will be unhappy with this MFC because they do not see
reasons for the MFC too).

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228311 - head/sys/conf

2011-12-06 Thread Max Khon
Author: fjoe
Date: Tue Dec  6 18:01:09 2011
New Revision: 228311
URL: http://svn.freebsd.org/changeset/base/228311

Log:
  MK_CTF is not defined when kmod.mk is used with old bsd.own.mk.

Modified:
  head/sys/conf/kmod.mk

Modified: head/sys/conf/kmod.mk
==
--- head/sys/conf/kmod.mk   Tue Dec  6 17:54:59 2011(r228310)
+++ head/sys/conf/kmod.mk   Tue Dec  6 18:01:09 2011(r228311)
@@ -201,7 +201,7 @@ ${KMOD}.kld: ${OBJS}
 ${FULLPROG}: ${OBJS}
 .endif
${LD} ${LDFLAGS} -r -d -o ${.TARGET} ${OBJS}
-.if ${MK_CTF} != "no"
+.if defined(MK_CTF) && ${MK_CTF} != "no"
${CTFMERGE} ${CTFFLAGS} -o ${.TARGET} ${OBJS}
 .endif
 .if defined(EXPORT_SYMS)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r228158 - in head: . share/mk sys/conf

2011-12-06 Thread Max Khon
Gary, Hans,

On Tue, Dec 6, 2011 at 4:06 PM, Gary Jennejohn wrote:

> I see regressions when building kernel modules from /usr/ports:
> >
> > On Monday 05 December 2011 14:39:56 Robert Huff wrote:
> > > Hello:
> > >   When trying to update I get:
> > >
> > > ===>  Building for cuse4bsd-kmod-0.1.21_2
> > > make -f
> > >
> /data/port-work/usr/ports/multimedia/cuse4bsd-kmod/work/cuse4bsd-kmod-0.1.
> > > 21/Makefile.lib HAVE_DEBUG=YES all Warning: Object directory not
> changed
> > > from original
> > >
> /data/port-work/usr/ports/multimedia/cuse4bsd-kmod/work/cuse4bsd-kmod-0.1.
> > > 21 make -f
> > >
> /data/port-work/usr/ports/multimedia/cuse4bsd-kmod/work/cuse4bsd-kmod-0.1.
> > > 21/Makefile.kmod all "/sys/conf/kmod.mk", line 204: Malformed
> conditional
> > > (${MK_CTF} != "no") "/sys/conf/kmod.mk", line 206: if-less endif
> > > make: fatal errors encountered -- cannot continue
> > > *** Error code 1
> >
>
> I encountered this problem yesterday.  My quick hack was to put
> .if defined(MK_CTF) .endif around the offending lines, similar to
> EXPORT_SYMS.
>

This should be fixed now (rev. 228311)

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228521 - head/usr.bin/make

2011-12-14 Thread Max Khon
Author: fjoe
Date: Thu Dec 15 03:13:23 2011
New Revision: 228521
URL: http://svn.freebsd.org/changeset/base/228521

Log:
  job make: if stdout is a tty create a pty when running a command.

Modified:
  head/usr.bin/make/Makefile
  head/usr.bin/make/job.c

Modified: head/usr.bin/make/Makefile
==
--- head/usr.bin/make/Makefile  Thu Dec 15 02:26:53 2011(r228520)
+++ head/usr.bin/make/Makefile  Thu Dec 15 03:13:23 2011(r228521)
@@ -7,6 +7,8 @@ CFLAGS+=-I${.CURDIR}
 SRCS=  arch.c buf.c cond.c dir.c for.c hash.c hash_tables.c job.c  \
lst.c main.c make.c parse.c proc.c shell.c str.c suff.c targ.c  \
util.c var.c
+DPADD= ${LIBUTIL}
+LDADD= -lutil
 
 NO_SHARED?=YES
 

Modified: head/usr.bin/make/job.c
==
--- head/usr.bin/make/job.c Thu Dec 15 02:26:53 2011(r228520)
+++ head/usr.bin/make/job.c Thu Dec 15 03:13:23 2011(r228521)
@@ -115,6 +115,7 @@ __FBSDID("$FreeBSD$");
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -1798,8 +1799,13 @@ JobStart(GNode *gn, int flags, Job *prev
if (usePipes) {
int fd[2];
 
-   if (pipe(fd) == -1)
-   Punt("Cannot create pipe: %s", strerror(errno));
+   if (isatty(1)) {
+   if (openpty(fd + 1, fd + 0, NULL, NULL, NULL) 
== -1)
+   Punt("Cannot open pty: %s", 
strerror(errno));
+   } else {
+   if (pipe(fd) == -1)
+   Punt("Cannot create pipe: %s", 
strerror(errno));
+   }
job->inPipe = fd[0];
job->outPipe = fd[1];
fcntl(job->inPipe, F_SETFD, 1);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228523 - head/tools/regression/usr.bin/make/execution/joberr

2011-12-14 Thread Max Khon
Author: fjoe
Date: Thu Dec 15 06:01:06 2011
New Revision: 228523
URL: http://svn.freebsd.org/changeset/base/228523

Log:
  Add job error output test.
  
  make(1) with openpty() currently fails this test: there's a race condition
  and error output is sometimes lost.

Added:
  head/tools/regression/usr.bin/make/execution/joberr/
  head/tools/regression/usr.bin/make/execution/joberr/Makefile   (contents, 
props changed)
  head/tools/regression/usr.bin/make/execution/joberr/expected.status.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/joberr/expected.stderr.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/joberr/expected.stdout.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/execution/joberr/test.t   (contents, props 
changed)

Added: head/tools/regression/usr.bin/make/execution/joberr/Makefile
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/joberr/MakefileThu Dec 
15 06:01:06 2011(r228523)
@@ -0,0 +1,10 @@
+# $FreeBSD$
+
+check-jobherr:
+.for _t in 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9
+   @-${MAKE} -f ${MAKEFILE} -j2 do-$@
+.endfor
+
+do-check-joberr:
+   @echo "Error: build failed"
+   @exit 1

Added: head/tools/regression/usr.bin/make/execution/joberr/expected.status.1
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/joberr/expected.status.1   
Thu Dec 15 06:01:06 2011(r228523)
@@ -0,0 +1 @@
+0

Added: head/tools/regression/usr.bin/make/execution/joberr/expected.stderr.1
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/joberr/expected.stderr.1   
Thu Dec 15 06:01:06 2011(r228523)
@@ -0,0 +1,30 @@
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error
+1 error

Added: head/tools/regression/usr.bin/make/execution/joberr/expected.stdout.1
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/execution/joberr/expected.stdout.1   
Thu Dec 15 06:01:06 2011(r228523)
@@ -0,0 +1,90 @@
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+Error: build failed
+*** Error code 1
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)
+*** Error code 2 (ignored)

Added: head/tools/regression/usr.bin/make/execution/joberr/test.t
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools

svn commit: r228524 - head/tools/regression/usr.bin/make/execution/joberr

2011-12-14 Thread Max Khon
Author: fjoe
Date: Thu Dec 15 06:12:43 2011
New Revision: 228524
URL: http://svn.freebsd.org/changeset/base/228524

Log:
  Fix last-minute typo.

Modified:
  head/tools/regression/usr.bin/make/execution/joberr/Makefile

Modified: head/tools/regression/usr.bin/make/execution/joberr/Makefile
==
--- head/tools/regression/usr.bin/make/execution/joberr/MakefileThu Dec 
15 06:01:06 2011(r228523)
+++ head/tools/regression/usr.bin/make/execution/joberr/MakefileThu Dec 
15 06:12:43 2011(r228524)
@@ -1,6 +1,6 @@
 # $FreeBSD$
 
-check-jobherr:
+check-joberr:
 .for _t in 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9
@-${MAKE} -f ${MAKEFILE} -j2 do-$@
 .endfor
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r228525 - head/usr.bin/make

2011-12-14 Thread Max Khon
Author: fjoe
Date: Thu Dec 15 06:25:19 2011
New Revision: 228525
URL: http://svn.freebsd.org/changeset/base/228525

Log:
  Revert r228521: sometimes job output is lost
  (see tools/regression/usr.bin/make/execution/joberr test).
  
  openpty(fd + 0, fd + 1,...) version does not have this problem but
  it sometimes enters an infinite sleep in "ttywait" state in tty_drain()
  when make(1) closes slave pty.

Modified:
  head/usr.bin/make/Makefile
  head/usr.bin/make/job.c

Modified: head/usr.bin/make/Makefile
==
--- head/usr.bin/make/Makefile  Thu Dec 15 06:12:43 2011(r228524)
+++ head/usr.bin/make/Makefile  Thu Dec 15 06:25:19 2011(r228525)
@@ -7,8 +7,6 @@ CFLAGS+=-I${.CURDIR}
 SRCS=  arch.c buf.c cond.c dir.c for.c hash.c hash_tables.c job.c  \
lst.c main.c make.c parse.c proc.c shell.c str.c suff.c targ.c  \
util.c var.c
-DPADD= ${LIBUTIL}
-LDADD= -lutil
 
 NO_SHARED?=YES
 

Modified: head/usr.bin/make/job.c
==
--- head/usr.bin/make/job.c Thu Dec 15 06:12:43 2011(r228524)
+++ head/usr.bin/make/job.c Thu Dec 15 06:25:19 2011(r228525)
@@ -115,7 +115,6 @@ __FBSDID("$FreeBSD$");
 #include 
 #include 
 #include 
-#include 
 #include 
 #include 
 #include 
@@ -1799,13 +1798,8 @@ JobStart(GNode *gn, int flags, Job *prev
if (usePipes) {
int fd[2];
 
-   if (isatty(1)) {
-   if (openpty(fd + 1, fd + 0, NULL, NULL, NULL) 
== -1)
-   Punt("Cannot open pty: %s", 
strerror(errno));
-   } else {
-   if (pipe(fd) == -1)
-   Punt("Cannot create pipe: %s", 
strerror(errno));
-   }
+   if (pipe(fd) == -1)
+   Punt("Cannot create pipe: %s", strerror(errno));
job->inPipe = fd[0];
job->outPipe = fd[1];
fcntl(job->inPipe, F_SETFD, 1);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r238933 - head/sys/dev/puc

2012-07-30 Thread Max Khon
Author: fjoe
Date: Tue Jul 31 05:23:23 2012
New Revision: 238933
URL: http://svn.freebsd.org/changeset/base/238933

Log:
  - Change back "d_ofs" to int8_t to not pessimize padding and size of "struct 
puc_cfg".
  - Use "puc_config_moxa" for Moxa boards that need d_ofs greater than 0x7f
  
  Prodded by:   marcel@, gavin@
  MFC after:3 days

Modified:
  head/sys/dev/puc/puc_cfg.h
  head/sys/dev/puc/pucdata.c

Modified: head/sys/dev/puc/puc_cfg.h
==
--- head/sys/dev/puc/puc_cfg.h  Tue Jul 31 04:09:27 2012(r238932)
+++ head/sys/dev/puc/puc_cfg.h  Tue Jul 31 05:23:23 2012(r238933)
@@ -79,7 +79,7 @@ struct puc_cfg {
int8_t  ports;
int8_t  rid;/* Rid of first port */
int8_t  d_rid;  /* Delta rid of next ports */
-   int16_t d_ofs;  /* Delta offset of next ports */
+   int8_t  d_ofs;  /* Delta offset of next ports */
puc_config_f*config_function;
 };
 

Modified: head/sys/dev/puc/pucdata.c
==
--- head/sys/dev/puc/pucdata.c  Tue Jul 31 04:09:27 2012(r238932)
+++ head/sys/dev/puc/pucdata.c  Tue Jul 31 05:23:23 2012(r238933)
@@ -510,13 +510,15 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1024, 0x, 0,
"Moxa Technologies, Smartio CP-102E/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1025, 0x, 0,
"Moxa Technologies, Smartio CP-102EL/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200,
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1040, 0x, 0,
@@ -572,7 +574,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1182, 0x, 0,
"Moxa Technologies, Smartio CP-118EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1680, 0x, 0,
@@ -596,7 +599,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1683, 0x, 0,
"Moxa Technologies, Smartio CP-168EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x13a8, 0x0152, 0x, 0,
@@ -1159,7 +1163,12 @@ puc_config_moxa(struct puc_softc *sc, en
 intptr_t *res)
 {
if (cmd == PUC_CFG_GET_OFS) {
-   *res = ((port == 3) ? 7 : port) * 0x200;
+   const struct puc_cfg *cfg = sc->sc_cfg;
+
+   if (port == 3 && (cfg->device == 0x1045 || cfg->device == 
0x1144))
+   port = 7;
+   *res = port * 0x200;
+
return 0;
}
return (ENXIO);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r238775 - stable/8/sys/dev/puc

2012-07-30 Thread Max Khon
John,

On Mon, Jul 30, 2012 at 10:32 PM, John Baldwin  wrote:

> On Wednesday, July 25, 2012 1:42:57 pm Max Khon wrote:
>> Author: fjoe
>> Date: Wed Jul 25 17:42:57 2012
>> New Revision: 238775
>> URL: http://svn.freebsd.org/changeset/base/238775
>>
>> Log:
>>   MFC: r227457, r237350, r237357
>>
>>   Add support for the following Moxa PCIe multiport serial boards:
>>   - CP102E
>>   - CP102EL
>>   - CP104EL-A
>>   - CP104JU
>>   - CP114EL
>>   - CP118EL-A
>>   - CP168EL-A
>
> You ignored the comments from Marcel when you committed this to HEAD it seems.
> You need to revert d_ofs back to a int8_t and add a config_function for these
> adapters instead that use an offset of 0x200.

Well, I did not ignore comments from Marcel - his e-mail got lost in
my inbox. Sorry about that.

I was notified about the issue off the list several days ago and fixed it today.

Max
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r239006 - stable/8/sys/dev/puc

2012-08-03 Thread Max Khon
Author: fjoe
Date: Fri Aug  3 11:22:09 2012
New Revision: 239006
URL: http://svn.freebsd.org/changeset/base/239006

Log:
  MFC: r238933
  
  - Change back "d_ofs" to int8_t to not pessimize padding and size of "struct 
puc_cfg"
  - Use "puc_config_moxa" for Moxa boards that need d_ofs greater than 0x7f

Modified:
  stable/8/sys/dev/puc/puc_cfg.h
  stable/8/sys/dev/puc/pucdata.c
Directory Properties:
  stable/8/   (props changed)
  stable/8/sys/   (props changed)
  stable/8/sys/dev/   (props changed)

Modified: stable/8/sys/dev/puc/puc_cfg.h
==
--- stable/8/sys/dev/puc/puc_cfg.h  Fri Aug  3 09:08:20 2012
(r239005)
+++ stable/8/sys/dev/puc/puc_cfg.h  Fri Aug  3 11:22:09 2012
(r239006)
@@ -79,7 +79,7 @@ struct puc_cfg {
int8_t  ports;
int8_t  rid;/* Rid of first port */
int8_t  d_rid;  /* Delta rid of next ports */
-   int16_t d_ofs;  /* Delta offset of next ports */
+   int8_t  d_ofs;  /* Delta offset of next ports */
puc_config_f*config_function;
 };
 

Modified: stable/8/sys/dev/puc/pucdata.c
==
--- stable/8/sys/dev/puc/pucdata.c  Fri Aug  3 09:08:20 2012
(r239005)
+++ stable/8/sys/dev/puc/pucdata.c  Fri Aug  3 11:22:09 2012
(r239006)
@@ -510,13 +510,15 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1024, 0x, 0,
"Moxa Technologies, Smartio CP-102E/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1025, 0x, 0,
"Moxa Technologies, Smartio CP-102EL/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200,
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1040, 0x, 0,
@@ -572,7 +574,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1182, 0x, 0,
"Moxa Technologies, Smartio CP-118EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1680, 0x, 0,
@@ -596,7 +599,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1683, 0x, 0,
"Moxa Technologies, Smartio CP-168EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x13a8, 0x0152, 0x, 0,
@@ -1153,7 +1157,12 @@ puc_config_moxa(struct puc_softc *sc, en
 intptr_t *res)
 {
if (cmd == PUC_CFG_GET_OFS) {
-   *res = ((port == 3) ? 7 : port) * 0x200;
+   const struct puc_cfg *cfg = sc->sc_cfg;
+
+   if (port == 3 && (cfg->device == 0x1045 || cfg->device == 
0x1144))
+   port = 7;
+   *res = port * 0x200;
+
return 0;
}
return (ENXIO);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r239783 - stable/9/sys/dev/puc

2012-08-28 Thread Max Khon
Author: fjoe
Date: Tue Aug 28 17:08:47 2012
New Revision: 239783
URL: http://svn.freebsd.org/changeset/base/239783

Log:
  MFC: r238933
  
  - Change back "d_ofs" to int8_t to not pessimize padding and size of "struct 
puc_cfg".
  - Use "puc_config_moxa" for Moxa boards that need d_ofs greater than 0x7f
  
  Approved by:  re (kib)

Modified:
  stable/9/sys/dev/puc/puc_cfg.h
  stable/9/sys/dev/puc/pucdata.c
Directory Properties:
  stable/9/   (props changed)
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/puc/puc_cfg.h
==
--- stable/9/sys/dev/puc/puc_cfg.h  Tue Aug 28 15:45:37 2012
(r239782)
+++ stable/9/sys/dev/puc/puc_cfg.h  Tue Aug 28 17:08:47 2012
(r239783)
@@ -79,7 +79,7 @@ struct puc_cfg {
int8_t  ports;
int8_t  rid;/* Rid of first port */
int8_t  d_rid;  /* Delta rid of next ports */
-   int16_t d_ofs;  /* Delta offset of next ports */
+   int8_t  d_ofs;  /* Delta offset of next ports */
puc_config_f*config_function;
 };
 

Modified: stable/9/sys/dev/puc/pucdata.c
==
--- stable/9/sys/dev/puc/pucdata.c  Tue Aug 28 15:45:37 2012
(r239782)
+++ stable/9/sys/dev/puc/pucdata.c  Tue Aug 28 17:08:47 2012
(r239783)
@@ -510,13 +510,15 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1024, 0x, 0,
"Moxa Technologies, Smartio CP-102E/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1025, 0x, 0,
"Moxa Technologies, Smartio CP-102EL/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200,
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1040, 0x, 0,
@@ -572,7 +574,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1182, 0x, 0,
"Moxa Technologies, Smartio CP-118EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1680, 0x, 0,
@@ -596,7 +599,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1683, 0x, 0,
"Moxa Technologies, Smartio CP-168EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x13a8, 0x0152, 0x, 0,
@@ -1153,7 +1157,12 @@ puc_config_moxa(struct puc_softc *sc, en
 intptr_t *res)
 {
if (cmd == PUC_CFG_GET_OFS) {
-   *res = ((port == 3) ? 7 : port) * 0x200;
+   const struct puc_cfg *cfg = sc->sc_cfg;
+
+   if (port == 3 && (cfg->device == 0x1045 || cfg->device == 
0x1144))
+   port = 7;
+   *res = port * 0x200;
+
return 0;
}
return (ENXIO);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r239784 - releng/9.1/sys/dev/puc

2012-08-28 Thread Max Khon
Author: fjoe
Date: Tue Aug 28 17:09:34 2012
New Revision: 239784
URL: http://svn.freebsd.org/changeset/base/239784

Log:
  MFC: r238933
  
  - Change back "d_ofs" to int8_t to not pessimize padding and size of "struct 
puc_cfg".
  - Use "puc_config_moxa" for Moxa boards that need d_ofs greater than 0x7f
  
  Approved by:  re (kib)

Modified:
  releng/9.1/sys/dev/puc/puc_cfg.h
  releng/9.1/sys/dev/puc/pucdata.c
Directory Properties:
  releng/9.1/   (props changed)
  releng/9.1/sys/   (props changed)
  releng/9.1/sys/dev/   (props changed)

Modified: releng/9.1/sys/dev/puc/puc_cfg.h
==
--- releng/9.1/sys/dev/puc/puc_cfg.hTue Aug 28 17:08:47 2012
(r239783)
+++ releng/9.1/sys/dev/puc/puc_cfg.hTue Aug 28 17:09:34 2012
(r239784)
@@ -79,7 +79,7 @@ struct puc_cfg {
int8_t  ports;
int8_t  rid;/* Rid of first port */
int8_t  d_rid;  /* Delta rid of next ports */
-   int16_t d_ofs;  /* Delta offset of next ports */
+   int8_t  d_ofs;  /* Delta offset of next ports */
puc_config_f*config_function;
 };
 

Modified: releng/9.1/sys/dev/puc/pucdata.c
==
--- releng/9.1/sys/dev/puc/pucdata.cTue Aug 28 17:08:47 2012
(r239783)
+++ releng/9.1/sys/dev/puc/pucdata.cTue Aug 28 17:09:34 2012
(r239784)
@@ -510,13 +510,15 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1024, 0x, 0,
"Moxa Technologies, Smartio CP-102E/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1025, 0x, 0,
"Moxa Technologies, Smartio CP-102EL/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_2S, 0x14, 0, 0x200,
+   PUC_PORT_2S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1040, 0x, 0,
@@ -572,7 +574,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1182, 0x, 0,
"Moxa Technologies, Smartio CP-118EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x1393, 0x1680, 0x, 0,
@@ -596,7 +599,8 @@ const struct puc_cfg puc_pci_devices[] =
{   0x1393, 0x1683, 0x, 0,
"Moxa Technologies, Smartio CP-168EL-A/PCIe",
DEFAULT_RCLK * 8,
-   PUC_PORT_8S, 0x14, 0, 0x200,
+   PUC_PORT_8S, 0x14, 0, -1,
+   .config_function = puc_config_moxa
},
 
{   0x13a8, 0x0152, 0x, 0,
@@ -1153,7 +1157,12 @@ puc_config_moxa(struct puc_softc *sc, en
 intptr_t *res)
 {
if (cmd == PUC_CFG_GET_OFS) {
-   *res = ((port == 3) ? 7 : port) * 0x200;
+   const struct puc_cfg *cfg = sc->sc_cfg;
+
+   if (port == 3 && (cfg->device == 0x1045 || cfg->device == 
0x1144))
+   port = 7;
+   *res = port * 0x200;
+
return 0;
}
return (ENXIO);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r211094 - head/games/fortune/datfiles

2010-08-09 Thread Max Khon
Author: fjoe
Date: Mon Aug  9 09:26:17 2010
New Revision: 211094
URL: http://svn.freebsd.org/changeset/base/211094

Log:
  "whereis netscape" gives empty output for quite a long time already.
  Suggest to use "whereis firefox" instead.

Modified:
  head/games/fortune/datfiles/freebsd-tips

Modified: head/games/fortune/datfiles/freebsd-tips
==
--- head/games/fortune/datfiles/freebsd-tipsMon Aug  9 08:41:29 2010
(r211093)
+++ head/games/fortune/datfiles/freebsd-tipsMon Aug  9 09:26:17 2010
(r211094)
@@ -476,6 +476,6 @@ You can use "whereis" to search standard
 directories for the specified programs. This can be particularly handy
 when you are trying to find where in the ports tree an application is.
 
-Try "whereis netscape" and "whereis whereis".
+Try "whereis firefox" and "whereis whereis".
-- Konstantinos Konstantinidis 
 %
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r241002 - head/sys/contrib/ipfilter/netinet

2012-09-27 Thread Max Khon
Author: fjoe
Date: Thu Sep 27 18:15:01 2012
New Revision: 241002
URL: http://svn.freebsd.org/changeset/base/241002

Log:
  Fix pseudo checksum calculation.
  
  This fixes ipfilter w/ network controllers that implement only
  partial rx csum offloading.
  
  PR:   106438
  Obtained from:upstream
  MFC after:1 week

Modified:
  head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c

Modified: head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c
==
--- head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c  Thu Sep 27 17:48:04 
2012(r241001)
+++ head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c  Thu Sep 27 18:15:01 
2012(r241002)
@@ -1357,7 +1357,9 @@ fr_info_t *fin;
else
sum = in_pseudo(ip->ip_src.s_addr, ip->ip_dst.s_addr,
htonl(m->m_pkthdr.csum_data +
-   fin->fin_ip->ip_len + fin->fin_p));
+   fin->fin_ip->ip_len -
+   (fin->fin_ip->ip_hl << 2) +
+   fin->fin_p));
sum ^= 0x;
if (sum != 0) {
fin->fin_flx |= FI_BAD;
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r241192 - in stable/9: . gnu/usr.bin/gdb gnu/usr.bin/gdb/arch/amd64 gnu/usr.bin/gdb/arch/arm gnu/usr.bin/gdb/arch/i386 gnu/usr.bin/gdb/arch/ia64 gnu/usr.bin/gdb/arch/mips gnu/usr.bin/gd...

2012-10-04 Thread Max Khon
Author: fjoe
Date: Thu Oct  4 09:49:53 2012
New Revision: 241192
URL: http://svn.freebsd.org/changeset/base/241192

Log:
  MFC: r241002
  
  Fix pseudo checksum calculation.
  
  This fixes ipfilter w/ network controllers that implement only
  partial rx csum offloading.
  
  PR:   106438
  Obtained from:upstream

Modified:
  stable/9/Makefile.inc1
  stable/9/gnu/usr.bin/gdb/Makefile.inc
  stable/9/gnu/usr.bin/gdb/arch/amd64/Makefile
  stable/9/gnu/usr.bin/gdb/arch/amd64/init.c
  stable/9/gnu/usr.bin/gdb/arch/arm/Makefile
  stable/9/gnu/usr.bin/gdb/arch/arm/init.c
  stable/9/gnu/usr.bin/gdb/arch/i386/Makefile
  stable/9/gnu/usr.bin/gdb/arch/i386/init.c
  stable/9/gnu/usr.bin/gdb/arch/ia64/Makefile
  stable/9/gnu/usr.bin/gdb/arch/ia64/init.c
  stable/9/gnu/usr.bin/gdb/arch/mips/Makefile
  stable/9/gnu/usr.bin/gdb/arch/mips/init.c
  stable/9/gnu/usr.bin/gdb/arch/powerpc/Makefile
  stable/9/gnu/usr.bin/gdb/arch/powerpc/init.c
  stable/9/gnu/usr.bin/gdb/arch/powerpc64/Makefile
  stable/9/gnu/usr.bin/gdb/arch/powerpc64/init.c
  stable/9/kerberos5/usr.bin/kadmin/Makefile
  stable/9/kerberos5/usr.sbin/ktutil/Makefile
  stable/9/rescue/rescue/Makefile
  stable/9/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c
  stable/9/sys/fs/ext2fs/ext2_bmap.c
  stable/9/sys/fs/ext2fs/ext2_extern.h
  stable/9/sys/fs/ext2fs/ext2_vnops.c
  stable/9/usr.bin/grep/util.c
Directory Properties:
  stable/9/   (props changed)
  stable/9/sys/   (props changed)

Modified: stable/9/Makefile.inc1
==
--- stable/9/Makefile.inc1  Thu Oct  4 09:02:38 2012(r241191)
+++ stable/9/Makefile.inc1  Thu Oct  4 09:49:53 2012(r241192)
@@ -172,7 +172,11 @@ OBJTREE=   ${MAKEOBJDIRPREFIX}/${TARGET}.$
 .endif
 WORLDTMP=  ${OBJTREE}${.CURDIR}/tmp
 # /usr/games added for fortune which depend on strfile
+.if defined(NO_WORLDTMP)
+BPATH= ${PATH}
+.else
 BPATH= 
${WORLDTMP}/legacy/usr/sbin:${WORLDTMP}/legacy/usr/bin:${WORLDTMP}/legacy/usr/games
+.endif
 XPATH= ${WORLDTMP}/usr/sbin:${WORLDTMP}/usr/bin:${WORLDTMP}/usr/games
 STRICTTMPPATH= ${BPATH}:${XPATH}
 TMPPATH=   ${STRICTTMPPATH}:${PATH}

Modified: stable/9/gnu/usr.bin/gdb/Makefile.inc
==
--- stable/9/gnu/usr.bin/gdb/Makefile.inc   Thu Oct  4 09:02:38 2012
(r241191)
+++ stable/9/gnu/usr.bin/gdb/Makefile.inc   Thu Oct  4 09:49:53 2012
(r241192)
@@ -45,6 +45,7 @@ CFLAGS+= -I${CNTRB_GDB}/gdb/config
 CFLAGS+= -I${CNTRB_BU}/include
 CFLAGS+= -I${CNTRB_GDB}/include
 CFLAGS+= -I${CNTRB_BU}/bfd
+DEBUG_FLAGS=   -g -O0
 
 GENSRCS+= nm.h tm.h
 

Modified: stable/9/gnu/usr.bin/gdb/arch/amd64/Makefile
==
--- stable/9/gnu/usr.bin/gdb/arch/amd64/MakefileThu Oct  4 09:02:38 
2012(r241191)
+++ stable/9/gnu/usr.bin/gdb/arch/amd64/MakefileThu Oct  4 09:49:53 
2012(r241192)
@@ -5,7 +5,7 @@ GENSRCS+= xm.h
 LIBSRCS+= fbsd-proc.c fbsd-threads.c gcore.c
 LIBSRCS+= amd64-nat.c amd64bsd-nat.c amd64fbsd-nat.c
 .endif
-LIBSRCS+= solib.c solib-svr4.c
+LIBSRCS+= solib.c solib-svr4.c solib-legacy.c
 LIBSRCS+= amd64-tdep.c amd64fbsd-tdep.c i386-tdep.c i386bsd-tdep.c \
i386fbsd-tdep-fixed.c i387-tdep.c
 

Modified: stable/9/gnu/usr.bin/gdb/arch/amd64/init.c
==
--- stable/9/gnu/usr.bin/gdb/arch/amd64/init.c  Thu Oct  4 09:02:38 2012
(r241191)
+++ stable/9/gnu/usr.bin/gdb/arch/amd64/init.c  Thu Oct  4 09:49:53 2012
(r241192)
@@ -12,6 +12,7 @@ extern initialize_file_ftype _initialize
 extern initialize_file_ftype _initialize_corelow;
 extern initialize_file_ftype _initialize_solib;
 extern initialize_file_ftype _initialize_svr4_solib;
+extern initialize_file_ftype _initialize_svr4_lm;
 extern initialize_file_ftype _initialize_ser_hardwire;
 extern initialize_file_ftype _initialize_ser_pipe;
 extern initialize_file_ftype _initialize_ser_tcp;
@@ -126,6 +127,7 @@ initialize_all_files (void)
   _initialize_corelow ();
   _initialize_solib ();
   _initialize_svr4_solib ();
+  _initialize_svr4_lm ();
   _initialize_ser_hardwire ();
   _initialize_ser_pipe ();
   _initialize_ser_tcp ();

Modified: stable/9/gnu/usr.bin/gdb/arch/arm/Makefile
==
--- stable/9/gnu/usr.bin/gdb/arch/arm/Makefile  Thu Oct  4 09:02:38 2012
(r241191)
+++ stable/9/gnu/usr.bin/gdb/arch/arm/Makefile  Thu Oct  4 09:49:53 2012
(r241192)
@@ -2,7 +2,7 @@
 
 GENSRCS+= xm.h
 LIBSRCS+= armfbsd-nat.c
-LIBSRCS+= arm-tdep.c armfbsd-tdep.c solib.c solib-svr4.c
+LIBSRCS+= arm-tdep.c armfbsd-tdep.c solib.c solib-svr4.c solib-legacy.c
 .if !defined(GDB_CROSS_DEBUGGER)
 LIBSRCS+= fbsd-threads.c
 .endif

Modified: stable/9/gnu/usr.bin/gdb/arch/arm/init.c
===

svn commit: r241193 - in stable/9: . gnu/usr.bin/gdb gnu/usr.bin/gdb/arch/amd64 gnu/usr.bin/gdb/arch/arm gnu/usr.bin/gdb/arch/i386 gnu/usr.bin/gdb/arch/ia64 gnu/usr.bin/gdb/arch/mips gnu/usr.bin/gd...

2012-10-04 Thread Max Khon
Author: fjoe
Date: Thu Oct  4 09:53:59 2012
New Revision: 241193
URL: http://svn.freebsd.org/changeset/base/241193

Log:
  Revert changes that ocassionally got into r241192.
  
  Pointy hat to:me

Modified:
  stable/9/Makefile.inc1
  stable/9/gnu/usr.bin/gdb/Makefile.inc
  stable/9/gnu/usr.bin/gdb/arch/amd64/Makefile
  stable/9/gnu/usr.bin/gdb/arch/amd64/init.c
  stable/9/gnu/usr.bin/gdb/arch/arm/Makefile
  stable/9/gnu/usr.bin/gdb/arch/arm/init.c
  stable/9/gnu/usr.bin/gdb/arch/i386/Makefile
  stable/9/gnu/usr.bin/gdb/arch/i386/init.c
  stable/9/gnu/usr.bin/gdb/arch/ia64/Makefile
  stable/9/gnu/usr.bin/gdb/arch/ia64/init.c
  stable/9/gnu/usr.bin/gdb/arch/mips/Makefile
  stable/9/gnu/usr.bin/gdb/arch/mips/init.c
  stable/9/gnu/usr.bin/gdb/arch/powerpc/Makefile
  stable/9/gnu/usr.bin/gdb/arch/powerpc/init.c
  stable/9/gnu/usr.bin/gdb/arch/powerpc64/Makefile
  stable/9/gnu/usr.bin/gdb/arch/powerpc64/init.c
  stable/9/kerberos5/usr.bin/kadmin/Makefile
  stable/9/kerberos5/usr.sbin/ktutil/Makefile
  stable/9/rescue/rescue/Makefile
  stable/9/sys/fs/ext2fs/ext2_bmap.c
  stable/9/sys/fs/ext2fs/ext2_extern.h
  stable/9/sys/fs/ext2fs/ext2_vnops.c
  stable/9/usr.bin/grep/util.c

Modified: stable/9/Makefile.inc1
==
--- stable/9/Makefile.inc1  Thu Oct  4 09:49:53 2012(r241192)
+++ stable/9/Makefile.inc1  Thu Oct  4 09:53:59 2012(r241193)
@@ -172,11 +172,7 @@ OBJTREE=   ${MAKEOBJDIRPREFIX}/${TARGET}.$
 .endif
 WORLDTMP=  ${OBJTREE}${.CURDIR}/tmp
 # /usr/games added for fortune which depend on strfile
-.if defined(NO_WORLDTMP)
-BPATH= ${PATH}
-.else
 BPATH= 
${WORLDTMP}/legacy/usr/sbin:${WORLDTMP}/legacy/usr/bin:${WORLDTMP}/legacy/usr/games
-.endif
 XPATH= ${WORLDTMP}/usr/sbin:${WORLDTMP}/usr/bin:${WORLDTMP}/usr/games
 STRICTTMPPATH= ${BPATH}:${XPATH}
 TMPPATH=   ${STRICTTMPPATH}:${PATH}

Modified: stable/9/gnu/usr.bin/gdb/Makefile.inc
==
--- stable/9/gnu/usr.bin/gdb/Makefile.inc   Thu Oct  4 09:49:53 2012
(r241192)
+++ stable/9/gnu/usr.bin/gdb/Makefile.inc   Thu Oct  4 09:53:59 2012
(r241193)
@@ -45,7 +45,6 @@ CFLAGS+= -I${CNTRB_GDB}/gdb/config
 CFLAGS+= -I${CNTRB_BU}/include
 CFLAGS+= -I${CNTRB_GDB}/include
 CFLAGS+= -I${CNTRB_BU}/bfd
-DEBUG_FLAGS=   -g -O0
 
 GENSRCS+= nm.h tm.h
 

Modified: stable/9/gnu/usr.bin/gdb/arch/amd64/Makefile
==
--- stable/9/gnu/usr.bin/gdb/arch/amd64/MakefileThu Oct  4 09:49:53 
2012(r241192)
+++ stable/9/gnu/usr.bin/gdb/arch/amd64/MakefileThu Oct  4 09:53:59 
2012(r241193)
@@ -5,7 +5,7 @@ GENSRCS+= xm.h
 LIBSRCS+= fbsd-proc.c fbsd-threads.c gcore.c
 LIBSRCS+= amd64-nat.c amd64bsd-nat.c amd64fbsd-nat.c
 .endif
-LIBSRCS+= solib.c solib-svr4.c solib-legacy.c
+LIBSRCS+= solib.c solib-svr4.c
 LIBSRCS+= amd64-tdep.c amd64fbsd-tdep.c i386-tdep.c i386bsd-tdep.c \
i386fbsd-tdep-fixed.c i387-tdep.c
 

Modified: stable/9/gnu/usr.bin/gdb/arch/amd64/init.c
==
--- stable/9/gnu/usr.bin/gdb/arch/amd64/init.c  Thu Oct  4 09:49:53 2012
(r241192)
+++ stable/9/gnu/usr.bin/gdb/arch/amd64/init.c  Thu Oct  4 09:53:59 2012
(r241193)
@@ -12,7 +12,6 @@ extern initialize_file_ftype _initialize
 extern initialize_file_ftype _initialize_corelow;
 extern initialize_file_ftype _initialize_solib;
 extern initialize_file_ftype _initialize_svr4_solib;
-extern initialize_file_ftype _initialize_svr4_lm;
 extern initialize_file_ftype _initialize_ser_hardwire;
 extern initialize_file_ftype _initialize_ser_pipe;
 extern initialize_file_ftype _initialize_ser_tcp;
@@ -127,7 +126,6 @@ initialize_all_files (void)
   _initialize_corelow ();
   _initialize_solib ();
   _initialize_svr4_solib ();
-  _initialize_svr4_lm ();
   _initialize_ser_hardwire ();
   _initialize_ser_pipe ();
   _initialize_ser_tcp ();

Modified: stable/9/gnu/usr.bin/gdb/arch/arm/Makefile
==
--- stable/9/gnu/usr.bin/gdb/arch/arm/Makefile  Thu Oct  4 09:49:53 2012
(r241192)
+++ stable/9/gnu/usr.bin/gdb/arch/arm/Makefile  Thu Oct  4 09:53:59 2012
(r241193)
@@ -2,7 +2,7 @@
 
 GENSRCS+= xm.h
 LIBSRCS+= armfbsd-nat.c
-LIBSRCS+= arm-tdep.c armfbsd-tdep.c solib.c solib-svr4.c solib-legacy.c
+LIBSRCS+= arm-tdep.c armfbsd-tdep.c solib.c solib-svr4.c
 .if !defined(GDB_CROSS_DEBUGGER)
 LIBSRCS+= fbsd-threads.c
 .endif

Modified: stable/9/gnu/usr.bin/gdb/arch/arm/init.c
==
--- stable/9/gnu/usr.bin/gdb/arch/arm/init.cThu Oct  4 09:49:53 2012
(r241192)
+++ stable/9/gnu/usr.bin/gdb/arch/arm/init.cThu Oct  4 09:53:59 2012
(r241193)
@@ -10,7 +10,6 @

svn commit: r217576 - stable/8

2011-01-19 Thread Max Khon
Author: fjoe
Date: Wed Jan 19 09:59:54 2011
New Revision: 217576
URL: http://svn.freebsd.org/changeset/base/217576

Log:
  Fix buildworld on pre-8.2.
  
  PR:   153947

Modified:
  stable/8/Makefile.inc1

Modified: stable/8/Makefile.inc1
==
--- stable/8/Makefile.inc1  Wed Jan 19 09:47:20 2011(r217575)
+++ stable/8/Makefile.inc1  Wed Jan 19 09:59:54 2011(r217576)
@@ -926,6 +926,10 @@ _groff=gnu/usr.bin/groff/tmac
 _ar=   usr.bin/ar
 .endif
 
+.if ${BOOTSTRAPPING} < 802000
+_lex=  usr.bin/lex
+.endif
+
 .if ${BOOTSTRAPPING} < 800013
 _mklocale= usr.bin/mklocale
 .endif
@@ -951,6 +955,7 @@ bootstrap-tools:
 ${_gperf} \
 ${_groff} \
 ${_ar} \
+${_lex} \
 usr.bin/lorder \
 usr.bin/makewhatis \
 ${_mklocale} \
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r198197 - head/usr.bin/make

2009-10-18 Thread Max Khon
Author: fjoe
Date: Sun Oct 18 11:26:09 2009
New Revision: 198197
URL: http://svn.freebsd.org/changeset/base/198197

Log:
  Exit with non-zero error code in case of errors
  when make is run with -k and not parallel (compat) make is run.

Modified:
  head/usr.bin/make/job.c

Modified: head/usr.bin/make/job.c
==
--- head/usr.bin/make/job.c Sun Oct 18 11:23:56 2009(r198196)
+++ head/usr.bin/make/job.c Sun Oct 18 11:26:09 2009(r198197)
@@ -3380,6 +3380,8 @@ Compat_Run(Lst *targs)
printf("`%s' not remade because of errors.\n",
gn->name);
makeErrors++;
+   } else if (gn->made == ERROR) {
+   makeErrors++;
}
}
 
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r198199 - head/usr.bin/make

2009-10-18 Thread Max Khon
Author: fjoe
Date: Sun Oct 18 11:28:31 2009
New Revision: 198199
URL: http://svn.freebsd.org/changeset/base/198199

Log:
  Reset UPTODATE gnodes after remaking makefiles when make
  is not going to be restarted: such nodes could be marked UPTODATE
  without doing rebuild due to remakingMakefiles being TRUE.

Modified:
  head/usr.bin/make/main.c

Modified: head/usr.bin/make/main.c
==
--- head/usr.bin/make/main.cSun Oct 18 11:27:34 2009(r198198)
+++ head/usr.bin/make/main.cSun Oct 18 11:28:31 2009(r198199)
@@ -706,6 +706,7 @@ Main_AddSourceMakefile(const char *name)
 static void
 Remake_Makefiles(void)
 {
+   Lst cleanup;
LstNode *ln;
int error_cnt = 0;
int remade_cnt = 0;
@@ -716,6 +717,7 @@ Remake_Makefiles(void)
Fatal("Failed to change directory to %s.", curdir);
}
 
+   Lst_Init(&cleanup);
LST_FOREACH(ln, &source_makefiles) {
LstNode *ln2;
struct GNode *gn;
@@ -791,21 +793,7 @@ Remake_Makefiles(void)
gn->name);
error_cnt++;
} else if (gn->made == UPTODATE) {
-   Lst examine;
-
-   Lst_Init(&examine);
-   Lst_EnQueue(&examine, gn);
-   while (!Lst_IsEmpty(&examine)) {
-   LstNode *eln;
-   GNode *egn = Lst_DeQueue(&examine);
-
-   egn->make = FALSE;
-   LST_FOREACH(eln, &egn->children) {
-   GNode *cgn = Lst_Datum(eln);
-
-   Lst_EnQueue(&examine, cgn);
-   }
-   }
+   Lst_EnQueue(&cleanup, gn);
}
}
 
@@ -827,6 +815,24 @@ Remake_Makefiles(void)
}
}
 
+   while (!Lst_IsEmpty(&cleanup)) {
+   GNode *gn = Lst_DeQueue(&cleanup);
+
+   gn->unmade = 0;
+   gn->make = FALSE;
+   gn->made = UNMADE;
+   gn->childMade = FALSE;
+   gn->mtime = gn->cmtime = 0;
+   gn->cmtime_gn = NULL;
+
+   LST_FOREACH(ln, &gn->children) {
+   GNode *cgn = Lst_Datum(ln);
+
+   gn->unmade++;
+   Lst_EnQueue(&cleanup, cgn);
+   }
+   }
+
if (curdir != objdir) {
if (chdir(objdir) < 0)
Fatal("Failed to change directory to %s.", objdir);
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r198277 - head/sys/tools

2009-10-20 Thread Max Khon
Author: fjoe
Date: Tue Oct 20 11:54:06 2009
New Revision: 198277
URL: http://svn.freebsd.org/changeset/base/198277

Log:
  Allow KMOD with hypens and dots.
  
  MFC after:1 week

Modified:
  head/sys/tools/fw_stub.awk

Modified: head/sys/tools/fw_stub.awk
==
--- head/sys/tools/fw_stub.awk  Tue Oct 20 11:52:39 2009(r198276)
+++ head/sys/tools/fw_stub.awk  Tue Oct 20 11:54:06 2009(r198277)
@@ -118,6 +118,8 @@ if (!num_files || !opt_m)
 
 cfilename = opt_c;
 ctmpfilename = cfilename ".tmp";
+modname = opt_m;
+gsub(/[-\.]/, "_", modname);
 
 printc("#include \
 #include \
@@ -139,7 +141,7 @@ for (file_i = 0; file_i < num_files; fil
 }
 
 printc("\nstatic int\n"\
-opt_m "_fw_modevent(module_t mod, int type, void *unused)\
+modname "_fw_modevent(module_t mod, int type, void *unused)\
 {\
const struct firmware *fp, *parent;\
int error;\
@@ -206,14 +208,14 @@ printc("\t\treturn (error);\
return (EINVAL);\
 }\
 \
-static moduledata_t " opt_m "_fw_mod = {\
-\"" opt_m "_fw\",\
-" opt_m "_fw_modevent,\
+static moduledata_t " modname "_fw_mod = {\
+\"" modname "_fw\",\
+" modname "_fw_modevent,\
 0\
 };\
-DECLARE_MODULE(" opt_m "_fw, " opt_m "_fw_mod, SI_SUB_DRIVERS, 
SI_ORDER_FIRST);\
-MODULE_VERSION(" opt_m "_fw, 1);\
-MODULE_DEPEND(" opt_m "_fw, firmware, 1, 1, 1);\
+DECLARE_MODULE(" modname "_fw, " modname "_fw_mod, SI_SUB_DRIVERS, 
SI_ORDER_FIRST);\
+MODULE_VERSION(" modname "_fw, 1);\
+MODULE_DEPEND(" modname "_fw, firmware, 1, 1, 1);\
 ");
 
 if (opt_c)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r198783 - stable/8/sys/tools

2009-11-02 Thread Max Khon
Author: fjoe
Date: Mon Nov  2 09:47:15 2009
New Revision: 198783
URL: http://svn.freebsd.org/changeset/base/198783

Log:
  MFC: Allow KMOD with hypens and dots.

Modified:
  stable/8/sys/tools/fw_stub.awk

Modified: stable/8/sys/tools/fw_stub.awk
==
--- stable/8/sys/tools/fw_stub.awk  Mon Nov  2 08:31:00 2009
(r198782)
+++ stable/8/sys/tools/fw_stub.awk  Mon Nov  2 09:47:15 2009
(r198783)
@@ -118,6 +118,8 @@ if (!num_files || !opt_m)
 
 cfilename = opt_c;
 ctmpfilename = cfilename ".tmp";
+modname = opt_m;
+gsub(/[-\.]/, "_", modname);
 
 printc("#include \
 #include \
@@ -139,7 +141,7 @@ for (file_i = 0; file_i < num_files; fil
 }
 
 printc("\nstatic int\n"\
-opt_m "_fw_modevent(module_t mod, int type, void *unused)\
+modname "_fw_modevent(module_t mod, int type, void *unused)\
 {\
const struct firmware *fp, *parent;\
int error;\
@@ -206,14 +208,14 @@ printc("\t\treturn (error);\
return (EINVAL);\
 }\
 \
-static moduledata_t " opt_m "_fw_mod = {\
-\"" opt_m "_fw\",\
-" opt_m "_fw_modevent,\
+static moduledata_t " modname "_fw_mod = {\
+\"" modname "_fw\",\
+" modname "_fw_modevent,\
 0\
 };\
-DECLARE_MODULE(" opt_m "_fw, " opt_m "_fw_mod, SI_SUB_DRIVERS, 
SI_ORDER_FIRST);\
-MODULE_VERSION(" opt_m "_fw, 1);\
-MODULE_DEPEND(" opt_m "_fw, firmware, 1, 1, 1);\
+DECLARE_MODULE(" modname "_fw, " modname "_fw_mod, SI_SUB_DRIVERS, 
SI_ORDER_FIRST);\
+MODULE_VERSION(" modname "_fw, 1);\
+MODULE_DEPEND(" modname "_fw, firmware, 1, 1, 1);\
 ");
 
 if (opt_c)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r198784 - stable/7/sys/tools

2009-11-02 Thread Max Khon
Author: fjoe
Date: Mon Nov  2 09:47:41 2009
New Revision: 198784
URL: http://svn.freebsd.org/changeset/base/198784

Log:
  MFC: Allow KMOD with hypens and dots.

Modified:
  stable/7/sys/tools/fw_stub.awk

Modified: stable/7/sys/tools/fw_stub.awk
==
--- stable/7/sys/tools/fw_stub.awk  Mon Nov  2 09:47:15 2009
(r198783)
+++ stable/7/sys/tools/fw_stub.awk  Mon Nov  2 09:47:41 2009
(r198784)
@@ -118,6 +118,8 @@ if (!num_files || !opt_m)
 
 cfilename = opt_c;
 ctmpfilename = cfilename ".tmp";
+modname = opt_m;
+gsub(/[-\.]/, "_", modname);
 
 printc("#include \
 #include \
@@ -139,7 +141,7 @@ for (file_i = 0; file_i < num_files; fil
 }
 
 printc("\nstatic int\n"\
-opt_m "_fw_modevent(module_t mod, int type, void *unused)\
+modname "_fw_modevent(module_t mod, int type, void *unused)\
 {\
const struct firmware *fp, *parent;\
int error;\
@@ -206,14 +208,14 @@ printc("\t\treturn (error);\
return (EINVAL);\
 }\
 \
-static moduledata_t " opt_m "_fw_mod = {\
-\"" opt_m "_fw\",\
-" opt_m "_fw_modevent,\
+static moduledata_t " modname "_fw_mod = {\
+\"" modname "_fw\",\
+" modname "_fw_modevent,\
 0\
 };\
-DECLARE_MODULE(" opt_m "_fw, " opt_m "_fw_mod, SI_SUB_DRIVERS, 
SI_ORDER_FIRST);\
-MODULE_VERSION(" opt_m "_fw, 1);\
-MODULE_DEPEND(" opt_m "_fw, firmware, 1, 1, 1);\
+DECLARE_MODULE(" modname "_fw, " modname "_fw_mod, SI_SUB_DRIVERS, 
SI_ORDER_FIRST);\
+MODULE_VERSION(" modname "_fw, 1);\
+MODULE_DEPEND(" modname "_fw, firmware, 1, 1, 1);\
 ");
 
 if (opt_c)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r201924 - head/sys/netgraph

2010-01-09 Thread Max Khon
Author: fjoe
Date: Sat Jan  9 19:03:48 2010
New Revision: 201924
URL: http://svn.freebsd.org/changeset/base/201924

Log:
  Send link state change control messages to "orphans" hook as well.
  
  MFC after:1 week

Modified:
  head/sys/netgraph/ng_ether.c

Modified: head/sys/netgraph/ng_ether.c
==
--- head/sys/netgraph/ng_ether.cSat Jan  9 19:02:33 2010
(r201923)
+++ head/sys/netgraph/ng_ether.cSat Jan  9 19:03:48 2010
(r201924)
@@ -359,9 +359,6 @@ ng_ether_link_state(struct ifnet *ifp, i
struct ng_mesg *msg;
int cmd, dummy_error = 0;
 
-   if (priv->lower == NULL)
-return;
-
if (state == LINK_STATE_UP)
cmd = NGM_LINK_IS_UP;
else if (state == LINK_STATE_DOWN)
@@ -369,9 +366,16 @@ ng_ether_link_state(struct ifnet *ifp, i
else
return;
 
-   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
-   if (msg != NULL)
-   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->lower, 0);
+   if (priv->lower != NULL) {
+   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
+   if (msg != NULL)
+   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->lower, 
0);
+   }
+   if (priv->orphan != NULL) {
+   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
+   if (msg != NULL)
+   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->orphan, 
0);
+   }
 }
 
 /**
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r202911 - stable/8/sys/netgraph

2010-01-24 Thread Max Khon
Author: fjoe
Date: Sun Jan 24 08:59:33 2010
New Revision: 202911
URL: http://svn.freebsd.org/changeset/base/202911

Log:
  MFC: Send link state change control messages to "orphans" hook as well.

Modified:
  stable/8/sys/netgraph/ng_ether.c

Modified: stable/8/sys/netgraph/ng_ether.c
==
--- stable/8/sys/netgraph/ng_ether.cSun Jan 24 05:23:08 2010
(r202910)
+++ stable/8/sys/netgraph/ng_ether.cSun Jan 24 08:59:33 2010
(r202911)
@@ -359,9 +359,6 @@ ng_ether_link_state(struct ifnet *ifp, i
struct ng_mesg *msg;
int cmd, dummy_error = 0;
 
-   if (priv->lower == NULL)
-return;
-
if (state == LINK_STATE_UP)
cmd = NGM_LINK_IS_UP;
else if (state == LINK_STATE_DOWN)
@@ -369,9 +366,16 @@ ng_ether_link_state(struct ifnet *ifp, i
else
return;
 
-   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
-   if (msg != NULL)
-   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->lower, 0);
+   if (priv->lower != NULL) {
+   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
+   if (msg != NULL)
+   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->lower, 
0);
+   }
+   if (priv->orphan != NULL) {
+   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
+   if (msg != NULL)
+   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->orphan, 
0);
+   }
 }
 
 /**
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r202932 - stable/7/sys/netgraph

2010-01-24 Thread Max Khon
Author: fjoe
Date: Sun Jan 24 14:58:49 2010
New Revision: 202932
URL: http://svn.freebsd.org/changeset/base/202932

Log:
  MFC rev. 201924: Send link state change control messages
  to "orphans" hook as well.
  
  Approved by:  re (kib)

Modified:
  stable/7/sys/netgraph/ng_ether.c
Directory Properties:
  stable/7/sys/   (props changed)
  stable/7/sys/cddl/contrib/opensolaris/   (props changed)
  stable/7/sys/contrib/dev/acpica/   (props changed)
  stable/7/sys/contrib/pf/   (props changed)

Modified: stable/7/sys/netgraph/ng_ether.c
==
--- stable/7/sys/netgraph/ng_ether.cSun Jan 24 14:57:54 2010
(r202931)
+++ stable/7/sys/netgraph/ng_ether.cSun Jan 24 14:58:49 2010
(r202932)
@@ -343,9 +343,6 @@ ng_ether_link_state(struct ifnet *ifp, i
struct ng_mesg *msg;
int cmd, dummy_error = 0;
 
-   if (priv->lower == NULL)
-return;
-
if (state == LINK_STATE_UP)
cmd = NGM_LINK_IS_UP;
else if (state == LINK_STATE_DOWN)
@@ -353,9 +350,16 @@ ng_ether_link_state(struct ifnet *ifp, i
else
return;
 
-   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
-   if (msg != NULL)
-   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->lower, 0);
+   if (priv->lower != NULL) {
+   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
+   if (msg != NULL)
+   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->lower, 
0);
+   }
+   if (priv->orphan != NULL) {
+   NG_MKMESSAGE(msg, NGM_FLOW_COOKIE, cmd, 0, M_NOWAIT);
+   if (msg != NULL)
+   NG_SEND_MSG_HOOK(dummy_error, node, msg, priv->orphan, 
0);
+   }
 }
 
 /**
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r202936 - in stable/8/sys: . amd64/include/xen cddl/contrib/opensolaris contrib/dev/acpica contrib/pf dev/xen/xenpci

2010-01-24 Thread Max Khon
Author: fjoe
Date: Sun Jan 24 16:58:20 2010
New Revision: 202936
URL: http://svn.freebsd.org/changeset/base/202936

Log:
  Record rev. 201924 MFC.

Modified:
Directory Properties:
  stable/8/sys/   (props changed)
  stable/8/sys/amd64/include/xen/   (props changed)
  stable/8/sys/cddl/contrib/opensolaris/   (props changed)
  stable/8/sys/contrib/dev/acpica/   (props changed)
  stable/8/sys/contrib/pf/   (props changed)
  stable/8/sys/dev/xen/xenpci/   (props changed)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r202938 - in stable/8/sys: . amd64/include/xen cddl/contrib/opensolaris contrib/dev/acpica contrib/pf dev/xen/xenpci

2010-01-24 Thread Max Khon
Author: fjoe
Date: Sun Jan 24 17:00:21 2010
New Revision: 202938
URL: http://svn.freebsd.org/changeset/base/202938

Log:
  Record rev. 198277 MFC.

Modified:
Directory Properties:
  stable/8/sys/   (props changed)
  stable/8/sys/amd64/include/xen/   (props changed)
  stable/8/sys/cddl/contrib/opensolaris/   (props changed)
  stable/8/sys/contrib/dev/acpica/   (props changed)
  stable/8/sys/contrib/pf/   (props changed)
  stable/8/sys/dev/xen/xenpci/   (props changed)
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r186279 - head/usr.bin/make

2008-12-18 Thread Max Khon
Author: fjoe
Date: Thu Dec 18 12:16:51 2008
New Revision: 186279
URL: http://svn.freebsd.org/changeset/base/186279

Log:
  Exit with error code 2 when run with -k (continue if errors) and build failed.

Modified:
  head/usr.bin/make/globals.h
  head/usr.bin/make/job.c
  head/usr.bin/make/job.h
  head/usr.bin/make/main.c
  head/usr.bin/make/make.c

Modified: head/usr.bin/make/globals.h
==
--- head/usr.bin/make/globals.h Thu Dec 18 12:01:53 2008(r186278)
+++ head/usr.bin/make/globals.h Thu Dec 18 12:16:51 2008(r186279)
@@ -70,6 +70,7 @@ extern struct Path parseIncPath;
 extern struct Path sysIncPath;
 
 extern int jobLimit;   /* -j argument: maximum number of jobs */
+extern int makeErrors; /* Number of targets not remade due to errors */
 extern Boolean jobsRunning;/* True if jobs are running */
 extern Boolean compatMake; /* True if we are make compatible */
 extern Boolean ignoreErrors;   /* True if should ignore all errors */

Modified: head/usr.bin/make/job.c
==
--- head/usr.bin/make/job.c Thu Dec 18 12:01:53 2008(r186278)
+++ head/usr.bin/make/job.c Thu Dec 18 12:16:51 2008(r186279)
@@ -263,7 +263,6 @@ TAILQ_HEAD(JobList, Job);
 /*
  * error handling variables
  */
-static int errors = 0; /* number of errors reported */
 static int aborting = 0;   /* why is the make aborting? */
 #defineABORT_ERROR 1   /* Because of an error */
 #defineABORT_INTERRUPT 2   /* Because it was interrupted */
@@ -849,7 +848,7 @@ JobClose(Job *job)
  *
  * If we got an error and are aborting (aborting == ABORT_ERROR) and
  * the job list is now empty, we are done for the day.
- * If we recognized an error (errors !=0), we set the aborting flag
+ * If we recognized an error (makeErrors !=0), we set the aborting flag
  * to ABORT_ERROR so no more jobs will be started.
  */
 static void
@@ -1124,7 +1123,7 @@ JobFinish(Job *job, int *status)
free(job);
 
} else if (*status != 0) {
-   errors += 1;
+   makeErrors++;
free(job);
}
 
@@ -1133,7 +1132,7 @@ JobFinish(Job *job, int *status)
/*
 * Set aborting if any error.
 */
-   if (errors && !keepgoing && aborting != ABORT_INTERRUPT) {
+   if (makeErrors && !keepgoing && aborting != ABORT_INTERRUPT) {
/*
 * If we found any errors in this batch of children and the -k
 * flag wasn't given, we set the aborting flag so no more jobs
@@ -1146,7 +1145,7 @@ JobFinish(Job *job, int *status)
/*
 * If we are aborting and the job table is now empty, we finish.
 */
-   Finish(errors);
+   Finish(makeErrors);
}
 }
 
@@ -2347,7 +2346,7 @@ Job_Init(int maxproc)
nJobs = 0;
 
aborting = 0;
-   errors = 0;
+   makeErrors = 0;
 
lastNode = NULL;
 
@@ -2539,14 +2538,14 @@ JobInterrupt(int runINTERRUPT, int signo
  * attached to the .END target.
  *
  * Results:
- * Number of errors reported.
+ * None.
  */
-int
+void
 Job_Finish(void)
 {
 
if (postCommands != NULL && !Lst_IsEmpty(&postCommands->commands)) {
-   if (errors) {
+   if (makeErrors) {
Error("Errors reported so .END ignored");
} else {
JobStart(postCommands, JOB_SPECIAL | JOB_IGNDOTS, NULL);
@@ -2563,7 +2562,6 @@ Job_Finish(void)
if (fifoMaster)
unlink(fifoName);
}
-   return (errors);
 }
 
 /**
@@ -3327,7 +3325,6 @@ void
 Compat_Run(Lst *targs)
 {
GNode   *gn = NULL; /* Current root target */
-   int error_cnt;  /* Number of targets not remade due to 
errors */
LstNode *ln;
 
Compat_InstallSignalHandlers();
@@ -3360,7 +3357,7 @@ Compat_Run(Lst *targs)
 *  ABORTED   gn was not remade because one of its inferiors
 *could not be made due to errors.
 */
-   error_cnt = 0;
+   makeErrors = 0;
while (!Lst_IsEmpty(targs)) {
gn = Lst_DeQueue(targs);
Compat_Make(gn, gn);
@@ -3370,18 +3367,17 @@ Compat_Run(Lst *targs)
} else if (gn->made == ABORTED) {
printf("`%s' not remade because of errors.\n",
gn->name);
-   error_cnt += 1;
+   makeErrors++;
}
}
 
/*
 * If the user has defined a .END target, run its commands.
 */
-   if (error_cnt == 0) {
+   if (makeErrors == 0) {
LST_FOREACH(ln, &ENDNode->commands) {
if (Compat_RunCommand

svn commit: r190821 - head/usr.bin/make

2009-04-07 Thread Max Khon
Author: fjoe
Date: Tue Apr  7 19:49:38 2009
New Revision: 190821
URL: http://svn.freebsd.org/changeset/base/190821

Log:
  Avoid infinite loops when remaking makefiles not only
  for Makefile targets but also for targets they depend on.

Modified:
  head/usr.bin/make/globals.h
  head/usr.bin/make/main.c
  head/usr.bin/make/make.1
  head/usr.bin/make/make.c

Modified: head/usr.bin/make/globals.h
==
--- head/usr.bin/make/globals.h Tue Apr  7 19:44:06 2009(r190820)
+++ head/usr.bin/make/globals.h Tue Apr  7 19:49:38 2009(r190821)
@@ -81,6 +81,7 @@ extern BooleannoExecute;  /* True if sho
 extern Boolean allPrecious;/* True if every target is precious */
 extern Boolean is_posix;   /* .POSIX target seen */
 extern Boolean mfAutoDeps; /* .MAKEFILEDEPS target seen */
+extern Boolean remakingMakefiles; /* True if remaking makefiles is in progress 
*/
 
 /* True if should continue on unaffected portions of the graph
  * when have an error in one portion */

Modified: head/usr.bin/make/main.c
==
--- head/usr.bin/make/main.cTue Apr  7 19:44:06 2009(r190820)
+++ head/usr.bin/make/main.cTue Apr  7 19:49:38 2009(r190821)
@@ -124,6 +124,7 @@ Lst create = Lst_Initializer(create);
 BooleanallPrecious;/* .PRECIOUS given on line by itself */
 Booleanis_posix;   /* .POSIX target seen */
 BooleanmfAutoDeps; /* .MAKEFILEDEPS target seen */
+BooleanremakingMakefiles; /* True if remaking makefiles is in 
progress */
 BooleanbeSilent;   /* -s flag */
 BooleanbeVerbose;  /* -v flag */
 BooleanbeQuiet;/* -Q flag */
@@ -732,41 +733,6 @@ Remake_Makefiles(void)
Suff_FindDeps(gn);
 
/*
-* ! dependencies as well as
-* dependencies with .FORCE, .EXEC and .PHONY attributes
-* are skipped to prevent infinite loops
-*/
-   if (gn->type & (OP_FORCE | OP_EXEC | OP_PHONY)) {
-   DEBUGF(MAKE, ("skipping (force, exec or phony).\n",
-   gn->name));
-   continue;
-   }
-
-   /*
-* Skip :: targets that have commands and no children
-* because such targets are always out-of-date
-*/
-   if ((gn->type & OP_DOUBLEDEP) &&
-   !Lst_IsEmpty(&gn->commands) &&
-   Lst_IsEmpty(&gn->children)) {
-   DEBUGF(MAKE, ("skipping (doubledep, no sources "
-   "and has commands).\n"));
-   continue;
-   }
-
-   /*
-* Skip targets without sources and without commands
-*/
-   if (Lst_IsEmpty(&gn->commands) &&
-   Lst_IsEmpty(&gn->children)) {
-   DEBUGF(MAKE,
-   ("skipping (no sources and no commands).\n"));
-   continue;
-   }
-
-   DEBUGF(MAKE, ("\n"));
-
-   /*
 * -t, -q and -n has no effect unless the makefile is
 * specified as one of the targets explicitly in the
 * command line
@@ -787,7 +753,9 @@ Remake_Makefiles(void)
 * Check and remake the makefile
 */
mtime = Dir_MTime(gn);
+   remakingMakefiles = TRUE;
Compat_Make(gn, gn);
+   remakingMakefiles = FALSE;
 
/*
 * Restore -t, -q and -n behaviour

Modified: head/usr.bin/make/make.1
==
--- head/usr.bin/make/make.1Tue Apr  7 19:44:06 2009(r190820)
+++ head/usr.bin/make/make.1Tue Apr  7 19:49:38 2009(r190821)
@@ -1629,7 +1629,7 @@ To prevent infinite loops the following 
 .Bl -bullet
 .It
 .Ic ::
-targets that have no prerequisites but have commands
+targets that have no prerequisites
 .It
 .Ic !
 targets

Modified: head/usr.bin/make/make.c
==
--- head/usr.bin/make/make.cTue Apr  7 19:44:06 2009(r190820)
+++ head/usr.bin/make/make.cTue Apr  7 19:49:38 2009(r190821)
@@ -211,11 +211,15 @@ Make_OODate(GNode *gn)
} else {
DEBUGF(MAKE, (".EXEC node..."));
}
-   oodate = TRUE;
 
-   } else if ((gn->mtime < gn->cmtime) ||
-   ((gn->cmtime == 0) &&
-   ((gn->mtime==0) || (gn->type & OP_DOUBLEDEP {
+   if (remakingMakefiles) {
+   DEBUGF(MAKE, ("skipping (remaking

svn commit: r188001 - head/usr.bin/make

2009-02-01 Thread Max Khon
Author: fjoe
Date: Mon Feb  2 06:25:57 2009
New Revision: 188001
URL: http://svn.freebsd.org/changeset/base/188001

Log:
  Remove duplicate OPTFLAGS definition.

Modified:
  head/usr.bin/make/main.c

Modified: head/usr.bin/make/main.c
==
--- head/usr.bin/make/main.cMon Feb  2 04:59:00 2009(r188000)
+++ head/usr.bin/make/main.cMon Feb  2 06:25:57 2009(r188001)
@@ -371,7 +371,6 @@ MainParseArgs(int argc, char **argv)
 rearg:
optind = 1; /* since we're called more than once */
optreset = 1;
-#define OPTFLAGS "ABC:D:E:I:PSV:Xd:ef:ij:km:nQpqrstvx:"
 #define OPTFLAGS "ABC:D:d:E:ef:I:ij:km:nPpQqrSstV:vXx:"
for (;;) {
if ((optind < argc) && strcmp(argv[optind], "--") == 0) {
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r188392 - head/sys/pci

2009-02-09 Thread Max Khon
Author: fjoe
Date: Mon Feb  9 16:53:37 2009
New Revision: 188392
URL: http://svn.freebsd.org/changeset/base/188392

Log:
  Destroy TX tag outside of loop scope.
  
  Found with:   Coverity Prevent(tm)
  CID:  3886

Modified:
  head/sys/pci/if_rl.c

Modified: head/sys/pci/if_rl.c
==
--- head/sys/pci/if_rl.cMon Feb  9 15:55:21 2009(r188391)
+++ head/sys/pci/if_rl.cMon Feb  9 16:53:37 2009(r188392)
@@ -1151,9 +1151,9 @@ rl_dma_free(struct rl_softc *sc)
sc->rl_cdata.rl_tx_dmamap[i]);
sc->rl_cdata.rl_tx_dmamap[i] = NULL;
}
+   }
bus_dma_tag_destroy(sc->rl_cdata.rl_tx_tag);
sc->rl_cdata.rl_tx_tag = NULL;
-   }
}
 
if (sc->rl_parent_tag != NULL) {
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r188393 - head/sys/dev/firewire

2009-02-09 Thread Max Khon
Author: fjoe
Date: Mon Feb  9 16:57:07 2009
New Revision: 188393
URL: http://svn.freebsd.org/changeset/base/188393

Log:
  Remove unused variable.
  
  Found with:   Coverity Prevent(tm)
  CID:  3693

Modified:
  head/sys/dev/firewire/sbp.c

Modified: head/sys/dev/firewire/sbp.c
==
--- head/sys/dev/firewire/sbp.c Mon Feb  9 16:53:37 2009(r188392)
+++ head/sys/dev/firewire/sbp.c Mon Feb  9 16:57:07 2009(r188393)
@@ -330,12 +330,11 @@ static char *orb_status1_serial_bus_erro
 static void
 sbp_identify(driver_t *driver, device_t parent)
 {
-   device_t child;
 SBP_DEBUG(0)
printf("sbp_identify\n");
 END_DEBUG
 
-   child = BUS_ADD_CHILD(parent, 0, "sbp", device_get_unit(parent));
+   BUS_ADD_CHILD(parent, 0, "sbp", device_get_unit(parent));
 }
 
 /*
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r188394 - head/sys/dev/firewire

2009-02-09 Thread Max Khon
Author: fjoe
Date: Mon Feb  9 16:58:18 2009
New Revision: 188394
URL: http://svn.freebsd.org/changeset/base/188394

Log:
  Do not self-initialize a variable.
  
  Found with:   Coverity Prevent(tm)
  CID:  3864, 3865

Modified:
  head/sys/dev/firewire/if_fwe.c
  head/sys/dev/firewire/if_fwip.c

Modified: head/sys/dev/firewire/if_fwe.c
==
--- head/sys/dev/firewire/if_fwe.c  Mon Feb  9 16:57:07 2009
(r188393)
+++ head/sys/dev/firewire/if_fwe.c  Mon Feb  9 16:58:18 2009
(r188394)
@@ -445,7 +445,7 @@ fwe_ioctl(struct ifnet *ifp, u_long cmd,
 #ifdef DEVICE_POLLING
{
struct ifreq *ifr = (struct ifreq *) data;
-   struct firewire_comm *fc = fc = fwe->fd.fc;
+   struct firewire_comm *fc = fwe->fd.fc;
 
if (ifr->ifr_reqcap & IFCAP_POLLING &&
!(ifp->if_capenable & IFCAP_POLLING)) {

Modified: head/sys/dev/firewire/if_fwip.c
==
--- head/sys/dev/firewire/if_fwip.c Mon Feb  9 16:57:07 2009
(r188393)
+++ head/sys/dev/firewire/if_fwip.c Mon Feb  9 16:58:18 2009
(r188394)
@@ -427,7 +427,7 @@ fwip_ioctl(struct ifnet *ifp, u_long cmd
 #ifdef DEVICE_POLLING
{
struct ifreq *ifr = (struct ifreq *) data;
-   struct firewire_comm *fc = fc = fwip->fd.fc;
+   struct firewire_comm *fc = fwip->fd.fc;
 
if (ifr->ifr_reqcap & IFCAP_POLLING &&
!(ifp->if_capenable & IFCAP_POLLING)) {
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r188395 - head/sys/cam

2009-02-09 Thread Max Khon
Author: fjoe
Date: Mon Feb  9 17:02:54 2009
New Revision: 188395
URL: http://svn.freebsd.org/changeset/base/188395

Log:
  cam_periph_alloc: fix "invalid periph name" error condition
  
  Found with:   Coverity Prevent(tm)
  CID:  130

Modified:
  head/sys/cam/cam_periph.c

Modified: head/sys/cam/cam_periph.c
==
--- head/sys/cam/cam_periph.c   Mon Feb  9 16:58:18 2009(r188394)
+++ head/sys/cam/cam_periph.c   Mon Feb  9 17:02:54 2009(r188395)
@@ -171,7 +171,7 @@ cam_periph_alloc(periph_ctor_t *periph_c
break;
}
xpt_unlock_buses();
-   if (p_drv == NULL) {
+   if (*p_drv == NULL) {
printf("cam_periph_alloc: invalid periph name '%s'\n", name);
return (CAM_REQ_INVALID);
}
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r188396 - head/sys/dev/de

2009-02-09 Thread Max Khon
Author: fjoe
Date: Mon Feb  9 17:07:29 2009
New Revision: 188396
URL: http://svn.freebsd.org/changeset/base/188396

Log:
  tulip_busdma_cleanup: pass correct vaddr (2nd arg) to bus_dmamem_free()
  
  Found with:   Coverity Prevent(tm)
  CID:  1998

Modified:
  head/sys/dev/de/if_de.c

Modified: head/sys/dev/de/if_de.c
==
--- head/sys/dev/de/if_de.c Mon Feb  9 17:02:54 2009(r188395)
+++ head/sys/dev/de/if_de.c Mon Feb  9 17:07:29 2009(r188396)
@@ -4561,7 +4561,7 @@ tulip_busdma_cleanup(tulip_softc_t * con
sc->tulip_setup_dma_addr = 0;
 }
 if (sc->tulip_setupbuf != NULL) {
-   bus_dmamem_free(sc->tulip_setup_tag, sc->tulip_setupdata,
+   bus_dmamem_free(sc->tulip_setup_tag, sc->tulip_setupbuf,
sc->tulip_setup_map);
sc->tulip_setup_map = NULL;
sc->tulip_setupbuf = NULL;
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r231543 - head/sys/netgraph

2012-02-11 Thread Max Khon
Author: fjoe
Date: Sun Feb 12 05:14:12 2012
New Revision: 231543
URL: http://svn.freebsd.org/changeset/base/231543

Log:
  - Use fixed-width integer types.
  - Prefer to use C99 stdint types.
  
  This fixes ng_cisco on 64-bit architectures.
  
  MFC after:1 week

Modified:
  head/sys/netgraph/ng_cisco.c
  head/sys/netgraph/ng_cisco.h

Modified: head/sys/netgraph/ng_cisco.c
==
--- head/sys/netgraph/ng_cisco.cSun Feb 12 05:01:49 2012
(r231542)
+++ head/sys/netgraph/ng_cisco.cSun Feb 12 05:14:12 2012
(r231543)
@@ -75,33 +75,33 @@
 #define KEEPALIVE_SECS 10
 
 struct cisco_header {
-   u_char  address;
-   u_char  control;
-   u_short protocol;
-};
+   uint8_t  address;
+   uint8_t  control;
+   uint16_t protocol;
+} __packed;
 
 #defineCISCO_HEADER_LENsizeof (struct cisco_header)
 
 struct cisco_packet {
-   u_long  type;
-   u_long  par1;
-   u_long  par2;
-   u_short rel;
-   u_short time0;
-   u_short time1;
-};
+   uint32_t type;
+   uint32_t par1;
+   uint32_t par2;
+   uint16_t rel;
+   uint16_t time0;
+   uint16_t time1;
+} __packed;
 
 #defineCISCO_PACKET_LEN (sizeof(struct cisco_packet))
 
 struct protoent {
hook_p  hook;   /* the hook for this proto */
-   u_short af; /* address family, -1 = downstream */
+   uint16_t af;/* address family, -1 = downstream */
 };
 
 struct cisco_priv {
-   u_long  local_seq;
-   u_long  remote_seq;
-   u_long  seqRetries; /* how many times we've been here throwing out
+   uint32_t local_seq;
+   uint32_t remote_seq;
+   uint32_t seqRetries;/* how many times we've been here throwing out
 * the same sequence number without ack */
node_p  node;
struct callout handle;
@@ -271,7 +271,7 @@ cisco_rcvmsg(node_p node, item_p item, h
pos = sprintf(arg,
  "keepalive period: %d sec; ", KEEPALIVE_SECS);
pos += sprintf(arg + pos,
- "unacknowledged keepalives: %ld", sc->seqRetries);
+ "unacknowledged keepalives: %d", sc->seqRetries);
resp->header.arglen = pos + 1;
break;
}
@@ -603,7 +603,7 @@ cisco_send(sc_p sc, int type, long par1,
struct cisco_packet *ch;
struct mbuf *m;
struct timeval time;
-   u_long  t;
+   uint32_t t;
int error = 0;
 
getmicrouptime(&time);
@@ -626,8 +626,8 @@ cisco_send(sc_p sc, int type, long par1,
ch->par1 = htonl(par1);
ch->par2 = htonl(par2);
ch->rel = -1;
-   ch->time0 = htons((u_short) (t >> 16));
-   ch->time1 = htons((u_short) t);
+   ch->time0 = htons((uint16_t) (t >> 16));
+   ch->time1 = htons((uint16_t) t);
 
NG_SEND_DATA_ONLY(error, sc->downstream.hook, m);
return (error);

Modified: head/sys/netgraph/ng_cisco.h
==
--- head/sys/netgraph/ng_cisco.hSun Feb 12 05:01:49 2012
(r231542)
+++ head/sys/netgraph/ng_cisco.hSun Feb 12 05:14:12 2012
(r231543)
@@ -76,8 +76,8 @@ struct ng_cisco_ipaddr {
 }
 
 struct ng_cisco_stats {
-   u_int32_t   seqRetries; /* # unack'd retries */
-   u_int32_t   keepAlivePeriod;/* in seconds */
+   uint32_t   seqRetries;  /* # unack'd retries */
+   uint32_t   keepAlivePeriod; /* in seconds */
 };
 
 /* Keep this in sync with the above structure definition */
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r231544 - in head: tools/regression/usr.bin/make/execution/joberr usr.bin/make

2012-02-11 Thread Max Khon
Author: fjoe
Date: Sun Feb 12 05:34:01 2012
New Revision: 231544
URL: http://svn.freebsd.org/changeset/base/231544

Log:
  Include target names in diagnostic output.
  
  Submitted by: Garrett Cooper

Modified:
  head/tools/regression/usr.bin/make/execution/joberr/expected.stdout.1
  head/usr.bin/make/job.c

Modified: head/tools/regression/usr.bin/make/execution/joberr/expected.stdout.1
==
--- head/tools/regression/usr.bin/make/execution/joberr/expected.stdout.1   
Sun Feb 12 05:14:12 2012(r231543)
+++ head/tools/regression/usr.bin/make/execution/joberr/expected.stdout.1   
Sun Feb 12 05:34:01 2012(r231544)
@@ -1,90 +1,90 @@
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
+*** [do-check-joberr] Error code 1
 Error: build failed
-*** Error code 1
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
-*** Error code 2 (ignored)
+*** [do-check-joberr] Error code 1
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-joberr] Error code 2 (ignored)
+*** [check-job

svn commit: r231752 - stable/9/sys/netgraph

2012-02-15 Thread Max Khon
Author: fjoe
Date: Wed Feb 15 10:08:13 2012
New Revision: 231752
URL: http://svn.freebsd.org/changeset/base/231752

Log:
  MFC: rev. 231543
  
  - Use fixed-width integer types.
  - Prefer to use C99 stdint types.
  
  This fixes ng_cisco on 64-bit architectures.

Modified:
  stable/9/sys/netgraph/ng_cisco.c
  stable/9/sys/netgraph/ng_cisco.h
Directory Properties:
  stable/9/sys/   (props changed)

Modified: stable/9/sys/netgraph/ng_cisco.c
==
--- stable/9/sys/netgraph/ng_cisco.cWed Feb 15 10:02:19 2012
(r231751)
+++ stable/9/sys/netgraph/ng_cisco.cWed Feb 15 10:08:13 2012
(r231752)
@@ -75,33 +75,33 @@
 #define KEEPALIVE_SECS 10
 
 struct cisco_header {
-   u_char  address;
-   u_char  control;
-   u_short protocol;
-};
+   uint8_t  address;
+   uint8_t  control;
+   uint16_t protocol;
+} __packed;
 
 #defineCISCO_HEADER_LENsizeof (struct cisco_header)
 
 struct cisco_packet {
-   u_long  type;
-   u_long  par1;
-   u_long  par2;
-   u_short rel;
-   u_short time0;
-   u_short time1;
-};
+   uint32_t type;
+   uint32_t par1;
+   uint32_t par2;
+   uint16_t rel;
+   uint16_t time0;
+   uint16_t time1;
+} __packed;
 
 #defineCISCO_PACKET_LEN (sizeof(struct cisco_packet))
 
 struct protoent {
hook_p  hook;   /* the hook for this proto */
-   u_short af; /* address family, -1 = downstream */
+   uint16_t af;/* address family, -1 = downstream */
 };
 
 struct cisco_priv {
-   u_long  local_seq;
-   u_long  remote_seq;
-   u_long  seqRetries; /* how many times we've been here throwing out
+   uint32_t local_seq;
+   uint32_t remote_seq;
+   uint32_t seqRetries;/* how many times we've been here throwing out
 * the same sequence number without ack */
node_p  node;
struct callout handle;
@@ -271,7 +271,7 @@ cisco_rcvmsg(node_p node, item_p item, h
pos = sprintf(arg,
  "keepalive period: %d sec; ", KEEPALIVE_SECS);
pos += sprintf(arg + pos,
- "unacknowledged keepalives: %ld", sc->seqRetries);
+ "unacknowledged keepalives: %d", sc->seqRetries);
resp->header.arglen = pos + 1;
break;
}
@@ -602,7 +602,7 @@ cisco_send(sc_p sc, int type, long par1,
struct cisco_packet *ch;
struct mbuf *m;
struct timeval time;
-   u_long  t;
+   uint32_t t;
int error = 0;
 
getmicrouptime(&time);
@@ -625,8 +625,8 @@ cisco_send(sc_p sc, int type, long par1,
ch->par1 = htonl(par1);
ch->par2 = htonl(par2);
ch->rel = -1;
-   ch->time0 = htons((u_short) (t >> 16));
-   ch->time1 = htons((u_short) t);
+   ch->time0 = htons((uint16_t) (t >> 16));
+   ch->time1 = htons((uint16_t) t);
 
NG_SEND_DATA_ONLY(error, sc->downstream.hook, m);
return (error);

Modified: stable/9/sys/netgraph/ng_cisco.h
==
--- stable/9/sys/netgraph/ng_cisco.hWed Feb 15 10:02:19 2012
(r231751)
+++ stable/9/sys/netgraph/ng_cisco.hWed Feb 15 10:08:13 2012
(r231752)
@@ -76,8 +76,8 @@ struct ng_cisco_ipaddr {
 }
 
 struct ng_cisco_stats {
-   u_int32_t   seqRetries; /* # unack'd retries */
-   u_int32_t   keepAlivePeriod;/* in seconds */
+   uint32_t   seqRetries;  /* # unack'd retries */
+   uint32_t   keepAlivePeriod; /* in seconds */
 };
 
 /* Keep this in sync with the above structure definition */
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r231753 - stable/8/sys/netgraph

2012-02-15 Thread Max Khon
Author: fjoe
Date: Wed Feb 15 10:15:15 2012
New Revision: 231753
URL: http://svn.freebsd.org/changeset/base/231753

Log:
  MFC r231543:
  
  - Use fixed-width integer types.
  - Prefer to use C99 stdint types.
  
  This fixes ng_cisco on 64-bit architectures.

Modified:
  stable/8/sys/netgraph/ng_cisco.c
  stable/8/sys/netgraph/ng_cisco.h
Directory Properties:
  stable/8/sys/   (props changed)

Modified: stable/8/sys/netgraph/ng_cisco.c
==
--- stable/8/sys/netgraph/ng_cisco.cWed Feb 15 10:08:13 2012
(r231752)
+++ stable/8/sys/netgraph/ng_cisco.cWed Feb 15 10:15:15 2012
(r231753)
@@ -75,33 +75,33 @@
 #define KEEPALIVE_SECS 10
 
 struct cisco_header {
-   u_char  address;
-   u_char  control;
-   u_short protocol;
-};
+   uint8_t  address;
+   uint8_t  control;
+   uint16_t protocol;
+} __packed;
 
 #defineCISCO_HEADER_LENsizeof (struct cisco_header)
 
 struct cisco_packet {
-   u_long  type;
-   u_long  par1;
-   u_long  par2;
-   u_short rel;
-   u_short time0;
-   u_short time1;
-};
+   uint32_t type;
+   uint32_t par1;
+   uint32_t par2;
+   uint16_t rel;
+   uint16_t time0;
+   uint16_t time1;
+} __packed;
 
 #defineCISCO_PACKET_LEN (sizeof(struct cisco_packet))
 
 struct protoent {
hook_p  hook;   /* the hook for this proto */
-   u_short af; /* address family, -1 = downstream */
+   uint16_t af;/* address family, -1 = downstream */
 };
 
 struct cisco_priv {
-   u_long  local_seq;
-   u_long  remote_seq;
-   u_long  seqRetries; /* how many times we've been here throwing out
+   uint32_t local_seq;
+   uint32_t remote_seq;
+   uint32_t seqRetries;/* how many times we've been here throwing out
 * the same sequence number without ack */
node_p  node;
struct callout handle;
@@ -273,7 +273,7 @@ cisco_rcvmsg(node_p node, item_p item, h
pos = sprintf(arg,
  "keepalive period: %d sec; ", KEEPALIVE_SECS);
pos += sprintf(arg + pos,
- "unacknowledged keepalives: %ld", sc->seqRetries);
+ "unacknowledged keepalives: %d", sc->seqRetries);
resp->header.arglen = pos + 1;
break;
}
@@ -604,7 +604,7 @@ cisco_send(sc_p sc, int type, long par1,
struct cisco_packet *ch;
struct mbuf *m;
struct timeval time;
-   u_long  t;
+   uint32_t t;
int error = 0;
 
getmicrouptime(&time);
@@ -627,8 +627,8 @@ cisco_send(sc_p sc, int type, long par1,
ch->par1 = htonl(par1);
ch->par2 = htonl(par2);
ch->rel = -1;
-   ch->time0 = htons((u_short) (t >> 16));
-   ch->time1 = htons((u_short) t);
+   ch->time0 = htons((uint16_t) (t >> 16));
+   ch->time1 = htons((uint16_t) t);
 
NG_SEND_DATA_ONLY(error, sc->downstream.hook, m);
return (error);

Modified: stable/8/sys/netgraph/ng_cisco.h
==
--- stable/8/sys/netgraph/ng_cisco.hWed Feb 15 10:08:13 2012
(r231752)
+++ stable/8/sys/netgraph/ng_cisco.hWed Feb 15 10:15:15 2012
(r231753)
@@ -76,8 +76,8 @@ struct ng_cisco_ipaddr {
 }
 
 struct ng_cisco_stats {
-   u_int32_t   seqRetries; /* # unack'd retries */
-   u_int32_t   keepAlivePeriod;/* in seconds */
+   uint32_t   seqRetries;  /* # unack'd retries */
+   uint32_t   keepAlivePeriod; /* in seconds */
 };
 
 /* Keep this in sync with the above structure definition */
___
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


  1   2   >