Re: Failed to compile current kernel with llvm/clang

2012-02-19 Thread Dimitry Andric
On 2012-02-20 02:54, Jia-Shiun Li wrote:
> I am trying to build world and kernel with llvm according to
> instructions on wiki:
> 
> http://wiki.freebsd.org/BuildingFreeBSDWithClang
> 
> buildworld is fine, but when building GENERIC kernel it failed on hpt27xx:
> 
> ===> hpt27xx (all)
> /usr/src/sys/modules/hpt27xx/../../dev/hpt27xx/osm_bsd.c:1180:25:
> error: format string is not a string literal (potentially insecure)
> [-Werror,-Wformat-security]
> S_IRUSR | S_IWUSR, driver_name);
>^~~
> @/dev/hpt27xx/hpt27xx_config.h:46:21: note: expanded from:
> #define driver_name hpt27xx_driver_name
> ^~~
> 
> I cannot find symbol hpt27xx_driver_name in that directory. Is it
> expanded from some macros I am not aware of?

This is one of the very few warnings left in the kernel.  Userland
already compiles without any fatal warning.

Because the hpt driver is submitted by the vendor, the warning you are
encountering can't easily be fixed.  It will hopefully be fixed on the
next code drop from the vendor.

In the meantime, I will put a band-aid in the Makefile.
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"


Re: Failed to compile current kernel with llvm/clang

2012-02-19 Thread matt
On 02/19/12 17:54, Jia-Shiun Li wrote:
> Hi all,
>
> I am trying to build world and kernel with llvm according to
> instructions on wiki:
>
> http://wiki.freebsd.org/BuildingFreeBSDWithClang
>
> buildworld is fine, but when building GENERIC kernel it failed on hpt27xx:
>
> ===> hpt27xx (all)
> /usr/src/sys/modules/hpt27xx/../../dev/hpt27xx/osm_bsd.c:1180:25:
> error: format string is not a string literal (potentially insecure)
> [-Werror,-Wformat-security]
> S_IRUSR | S_IWUSR, driver_name);
>^~~
> @/dev/hpt27xx/hpt27xx_config.h:46:21: note: expanded from:
> #define driver_name hpt27xx_driver_name
> ^~~
>
> I cannot find symbol hpt27xx_driver_name in that directory. Is it
> expanded from some macros I am not aware of?
>
>
> BTW clang does generate much friendly and useful warnings so far as I saw.
>
>
> Regards,
> Jia-Shiun
> ___
> freebsd-current@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"
You have the
WERROR=
NO_WERROR=

lines in /etc/make.conf?

Matt

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


Failed to compile current kernel with llvm/clang

2012-02-19 Thread Jia-Shiun Li
Hi all,

I am trying to build world and kernel with llvm according to
instructions on wiki:

http://wiki.freebsd.org/BuildingFreeBSDWithClang

buildworld is fine, but when building GENERIC kernel it failed on hpt27xx:

===> hpt27xx (all)
/usr/src/sys/modules/hpt27xx/../../dev/hpt27xx/osm_bsd.c:1180:25:
error: format string is not a string literal (potentially insecure)
[-Werror,-Wformat-security]
    S_IRUSR | S_IWUSR, driver_name);
   ^~~
@/dev/hpt27xx/hpt27xx_config.h:46:21: note: expanded from:
#define driver_name hpt27xx_driver_name
    ^~~

I cannot find symbol hpt27xx_driver_name in that directory. Is it
expanded from some macros I am not aware of?


BTW clang does generate much friendly and useful warnings so far as I saw.


Regards,
Jia-Shiun
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"


Re: Problems with 9.0 PowerPC images from main ftp site

2012-02-19 Thread Nathan Whitehorn

On 02/19/12 04:28, Super Bisquit wrote:

I decided to try burning an image from Index of
ftp://ftp.freebsd.org/pub/FreeBSD/releases/powerpc/powerpc/ISO-IMAGES/9.0/

And here are the following errors:

1.) The usb image doesn't boot. From following Whitehorn and others on
the FreeBSD PowerPC mailing list, one needs to have the bootloader
dd'ed to a HFS formatted partition of 1MB.


The USB image includes this. Some Powermacs don't have the ability to 
boot from USB, however.



A README with basic instructions and/or references to forum and
mailing list posts needs to be available for all non i386/amd64
systems.

2) The boot-only and  disc-1 images stop at
sc0: Unknown<16 virtual consoles, flags=0x300>


What hardware were you using? The ISOs work fine on at least the systems 
I've tested on. There was a problem with the ATA driver on a few 
machines that could supposedly be solved by setting the loader tunable 
hint.ata.X.mode to turn off DMA (see 
http://www.freebsd.org/cgi/query-pr.cgi?pr=164123).

-Nathan
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"


Re: devd based AUTOMOUNTER

2012-02-19 Thread vermaden
Hi,

sorry for late response, but I currently have quite a
lot 'weekend activities' that are definitely not near
a computer ;)



written by Gleb Kurtsou ... 
>> __state_lock() {
>>   while [ -f ${STATE}.lock ]; do sleep 0.5; done
>>   :> ${STATE}.lock
>> }
>
> Why not keep it stateless, unmounting by hand would kill integrity.

The state file is needed for automatic umount of fuse-based mounts,
the device that script would be triggered will be /dev/da0 for example
while the 'virtual' device that was actually used for the mount was
/dev/fuse0, that is why I need to track what is mounted and how.

These functions are mostly to keep that 'state file' integral,
to prevent two various event writing at the file simultanously,
which would probable killed the information integrity.

Unmounting the device does not kill integrity at all, I also thought
about that, You may plug the device, do whatever You want, event
change the filesystem there and mount it again, but when You umount
it and unplug it, then the script will umount that device (yes
pointless here at this moment) and remove the entry from state file.

> Usage of `file` is a big security concern.
Can You explain why?

> I think geom config and list of mounted file systems should be
> sufficient for collecting information at runtime. Although it may not be
> that easy for disks without partitioning.
geom config?

I also make use of the list of mounted filesystems, but the problem
with fuse-based devices remain as I stated earlier.

> I'm using a python script for mounting/unmounting removable disks,
> but having similar tool written in sh would be great.
> 
> https://github.com/glk/mmount

Thanks for sharing, maybe I will find some ideas there ;)



written by Uffe Jakobsen ...

> Nice,
Thanks.

> Instead of requiring modification to /etc/devd.conf why not just
> put a "plugin" conf-file in /etc/devd/ - well even better put in
> /usr/local/etc/devd/ - that way your devd.conf modifications are
> not lost upon patching/updating base os.
Great advice, I will do that in the next 'release' ;)

> There is an existing port called "automounter" by Dominic Fandrey
> which is much similar to your work.
Yes but its amd based.

> His "automounter" also works with disk labels
> (as found in /dev/ufs/ and /dev/msdosfs/ etc)
The labels are only 'an addition' to appearing device nodes at /dev,
Yes it would be nice to see that /dev/msdosfs/BACKUP is mounted
instead of /dev/da0s1, but I will have to make a lot of additional
check to see if that new label is part of that or other device etc.

> You should consider make a port out of your work.
I probably will try to do that, but I think it stil needs polish ;)



written by Lars Engels ...

> And please don't hardcode polish locales in mount_msdosfs :-)

I already changed that, I also plan to create simple config file,
it was just not 'the most important thing' that I was dealing
with at that time, its easy to move options into variables,
its harder to make script work the way it should ;)

But thanks for suggestion, good point.



written by Joe Culler ...

> Thanks for working on it. It works great for me, thanks!
Thanks, good to know ;)

> I have a question, could you export a fusefs-exfat or
> fusefs-ntfs share directory over NFS? I can't get it work.
> If you have a solution, please let me know, thanks.
I will try to do that tomorrow ane let You know.



Regards,
vermaden
































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